fix(bakeManifest): helm --set option (#441)

adds an option to toggle between `--set` and `--set-string`. we recently
had to revert a commit which changed `--set-string` to `--set` and
introduced a breaking change. this keeps the default behavior of
`--set-string` but allows users to choose which option they want to use.

rosco-manifests/src/main/java/com/netflix/spinnaker/rosco/manifests/helm/HelmBakeManifestRequest.java

@@ -11,4 +11,5 @@
 public class HelmBakeManifestRequest extends BakeManifestRequest {
   String namespace;
   List<Artifact> inputArtifacts;
+  boolean rawOverrides;
 }

rosco-manifests/src/main/java/com/netflix/spinnaker/rosco/manifests/helm/HelmTemplateUtils.java

@@ -73,7 +73,8 @@ public BakeRecipe buildBakeRecipe(BakeManifestEnvironment env, HelmBakeManifestR
       for (Map.Entry<String, Object> entry : overrides.entrySet()) {
         overrideList.add(entry.getKey() + "=" + entry.getValue().toString());
       }
-      command.add("--set-string");
+      String overrideOption = request.isRawOverrides() ? "--set" : "--set-string";
+      command.add(overrideOption);
       command.add(overrideList.stream().collect(Collectors.joining(",")));
     }