diff --git a/api/builtins/PatchStrategicMergeTransformer.go b/api/builtins/PatchStrategicMergeTransformer.go index e94b8c4a2..77ebbc8ae 100644 --- a/api/builtins/PatchStrategicMergeTransformer.go +++ b/api/builtins/PatchStrategicMergeTransformer.go @@ -60,11 +60,10 @@ func (p *PatchStrategicMergeTransformerPlugin) Config( "patch appears to be empty; files=%v, Patch=%s", p.Paths, p.Patches) } // Merge the patches, looking for conflicts. - m, err := h.ResmapFactory().ConflatePatches(p.loadedPatches) + _, err = h.ResmapFactory().ConflatePatches(p.loadedPatches) if err != nil { return err } - p.loadedPatches = m.Resources() return nil } diff --git a/api/krusty/patchdelete_test.go b/api/krusty/patchdelete_test.go index e0245c125..148f50d1b 100644 --- a/api/krusty/patchdelete_test.go +++ b/api/krusty/patchdelete_test.go @@ -69,28 +69,6 @@ spec: image: helloworld name: whatever ` - // Allow expected variable to be unused - _ = expected - - // Currently, kustomize inserts $patch: delete elements into the resulting resources - erroneousActual := ` -apiVersion: apps/v1 -kind: Deployment -metadata: - name: whatever -spec: - template: - spec: - containers: - - env: - - $patch: delete - name: NOT_EXISTING_FOR_REMOVAL - - name: EXISTING - value: EXISTING_VALUE - image: helloworld - name: whatever -` - m := th.Run(".", th.MakeDefaultOptions()) - th.AssertActualEqualsExpected(m, erroneousActual) + th.AssertActualEqualsExpected(m, expected) }