Remove duplicated 'shell quoting' for path (#846)

pulumi · Oct 16, 2019 · 7f66549 · 7f66549
1 parent 715de59
commit 7f66549
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -6,6 +6,10 @@
 - v1.15.x
 - v1.14.x
 
+### Improvements
+
+-   Fix a bug that causes helm crash when referencing 'scoped packages' that start with '@'. (https://github.com/pulumi/pulumi-kubernetes/pull/846)
+
 ## 1.2.1 (October 10, 2019)
 
 ### Supported Kubernetes versions

diff --git a/pkg/gen/nodejs-templates/helm/v2/helm.ts b/pkg/gen/nodejs-templates/helm/v2/helm.ts
@@ -171,7 +171,7 @@ export class Chart extends yaml.CollectionComponentResource {
                         nodepath.join(chartDir.name, fetchedChartName, "values.yaml")
                     );
                 } else {
-                    chart = path.quotePath(cfg.path);
+                    chart = cfg.path;
                     defaultValues = path.quotePath(nodepath.join(chart, "values.yaml"));
                 }
 

diff --git a/sdk/nodejs/helm/v2/helm.ts b/sdk/nodejs/helm/v2/helm.ts
@@ -171,7 +171,7 @@ export class Chart extends yaml.CollectionComponentResource {
                         nodepath.join(chartDir.name, fetchedChartName, "values.yaml")
                     );
                 } else {
-                    chart = path.quotePath(cfg.path);
+                    chart = cfg.path;
                     defaultValues = path.quotePath(nodepath.join(chart, "values.yaml"));
                 }