fix(kubernetes): Fix Optional ofNullable (#3265)

The usages of Optional.of should be Optional.ofNullable.
spinnaker · Oct 29, 2019 · e383297 · e383297
1 parent 2136674
commit e383297
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/...rc/main/groovy/com/netflix/spinnaker/orca/bakery/tasks/manifests/BakeManifestContext.java b/...rc/main/groovy/com/netflix/spinnaker/orca/bakery/tasks/manifests/BakeManifestContext.java
@@ -51,12 +51,12 @@ public BakeManifestContext(
       @Nullable @JsonProperty("inputArtifact") CreateBakeManifestTask.InputArtifact inputArtifact,
       @Nullable @JsonProperty("kustomizeFilePath") String kustomizeFilePath,
       @JsonProperty("rawOverrides") Boolean rawOverrides) {
-    this.inputArtifacts = Optional.of(inputArtifacts).orElse(new ArrayList<>());
+    this.inputArtifacts = Optional.ofNullable(inputArtifacts).orElse(new ArrayList<>());
     // Kustomize stage configs provide a single input artifact
     if (this.inputArtifacts.isEmpty() && inputArtifact != null) {
       this.inputArtifacts.add(inputArtifact);
     }
-    this.expectedArtifacts = Optional.of(expectedArtifacts).orElse(new ArrayList<>());
+    this.expectedArtifacts = Optional.ofNullable(expectedArtifacts).orElse(new ArrayList<>());
     this.overrides = overrides;
     this.evaluateOverrideExpressions = evaluateOverrideExpressions;
     this.templateRenderer = templateRenderer;