Merge pull request #2694 from monopole/switchToApiV0_5_0

Switch to kustomize/api v0.5.0
Switch kustomize to api/v0.5.0
2026-06-17 03:48:17 +00:00 · 2020-07-04 21:22:19 -07:00 · 2020-07-04 19:45:45 -07:00 · 2020-07-04 17:11:56 -07:00 · 2020-07-04 16:21:01 -07:00
46 changed files with 295 additions and 142 deletions
--- a/10
+++ b/10
@@ -14,8 +14,8 @@ all: verify-kustomize
 verify-kustomize: \
 	lint-kustomize \
 	test-unit-kustomize-all \
-	test-examples-kustomize-against-HEAD \
-	test-examples-kustomize-against-3.6.1
+	test-examples-kustomize-against-HEAD
+#	test-examples-kustomize-against-3.7.0

 # The following target referenced by a file in
 # https://github.com/kubernetes/test-infra/tree/master/config/jobs/kubernetes-sigs/kustomize
@@ -24,9 +24,9 @@ prow-presubmit-check: \
 	lint-kustomize \
 	test-unit-kustomize-all \
 	test-examples-kustomize-against-HEAD \
-	test-examples-kustomize-against-3.6.1 \
 	test-unit-cmd-all \
 	test-go-mod
+#	test-examples-kustomize-against-3.7.0 \

 .PHONY: verify-kustomize-e2e
 verify-kustomize-e2e: test-examples-e2e-kustomize
@@ -233,10 +233,10 @@ test-examples-kustomize-against-HEAD: $(MYGOBIN)/kustomize $(MYGOBIN)/mdrip
 	./hack/testExamplesAgainstKustomize.sh HEAD

 .PHONY:
-test-examples-kustomize-against-3.6.1: $(MYGOBIN)/mdrip
+test-examples-kustomize-against-3.7.0: $(MYGOBIN)/mdrip
 	( \
 		set -e; \
-		tag=v3.6.1; \
+		tag=v3.7.0; \
 		/bin/rm -f $(MYGOBIN)/kustomize; \
 		echo "Installing kustomize $$tag."; \
 		GO111MODULE=on go get sigs.k8s.io/kustomize/kustomize/v3@$${tag}; \
--- a/api/builtins/NamespaceTransformer.go
+++ b/api/builtins/NamespaceTransformer.go
@@ -5,6 +5,7 @@ package builtins

 import (
 	"fmt"
+	"strings"

 	"sigs.k8s.io/kustomize/api/filters/namespace"
 	"sigs.k8s.io/kustomize/api/resid"
@@ -31,6 +32,11 @@ func (p *NamespaceTransformerPlugin) Config(
 	_ *resmap.PluginHelpers, c []byte) (err error) {
 	p.Namespace = ""
 	p.FieldSpecs = nil
+	if !strings.Contains(string(c), "yamlSupport") {
+		// If not explicitly denied,
+		// activate kyaml-based transformation.
+		p.YAMLSupport = true
+	}
 	return yaml.Unmarshal(c, p)
 }

--- a/api/builtins/PatchJson6902Transformer.go
+++ b/api/builtins/PatchJson6902Transformer.go
@@ -5,6 +5,7 @@ package builtins

 import (
 	"fmt"
+	"strings"

 	jsonpatch "github.com/evanphx/json-patch"
 	"github.com/pkg/errors"
@@ -34,6 +35,11 @@ func (p *PatchJson6902TransformerPlugin) Config(
 	if err != nil {
 		return err
 	}
+	if !strings.Contains(string(c), "yamlSupport") {
+		// If not explicitly denied,
+		// activate kyaml-based transformation.
+		p.YAMLSupport = true
+	}
 	if p.Target.Name == "" {
 		return fmt.Errorf("must specify the target name")
 	}
--- a/api/builtins/PatchStrategicMergeTransformer.go
+++ b/api/builtins/PatchStrategicMergeTransformer.go
@@ -5,6 +5,7 @@ package builtins

 import (
 	"fmt"
+	"strings"

 	"sigs.k8s.io/kustomize/api/filters/patchstrategicmerge"
 	"sigs.k8s.io/kustomize/api/resmap"
@@ -30,6 +31,11 @@ func (p *PatchStrategicMergeTransformerPlugin) Config(
 	if err != nil {
 		return err
 	}
+	if !strings.Contains(string(c), "yamlSupport") {
+		// If not explicitly denied,
+		// activate kyaml-based transformation.
+		p.YAMLSupport = true
+	}
 	if len(p.Paths) == 0 && p.Patches == "" {
 		return fmt.Errorf("empty file path and empty patch content")
 	}
@@ -75,17 +81,6 @@ func (p *PatchStrategicMergeTransformerPlugin) Transform(m resmap.ResMap) error
 		}
 		if !p.YAMLSupport {
 			err = target.Patch(patch.Copy())
-			if err != nil {
-				return err
-			}
-			// remove the resource from resmap
-			// when the patch is to $patch: delete that target
-			if len(target.Map()) == 0 {
-				err = m.Remove(target.CurId())
-				if err != nil {
-					return err
-				}
-			}
 		} else {
 			patchCopy := patch.DeepCopy()
 			patchCopy.SetName(target.GetName())
@@ -99,6 +94,19 @@ func (p *PatchStrategicMergeTransformerPlugin) Transform(m resmap.ResMap) error
 				Patch: node,
 			}, target)
 		}
+		if err != nil {
+			return err
+		}
+		if len(target.Map()) == 0 {
+			// This means all fields have been removed from the object.
+			// This can happen if a patch required deletion of the
+			// entire resource (not just a part of it).  This means
+			// the overall resmap must shrink by one.
+			err = m.Remove(target.CurId())
+			if err != nil {
+				return err
+			}
+		}
 	}
 	return nil
 }
--- a/api/builtins/PatchTransformer.go
+++ b/api/builtins/PatchTransformer.go
@@ -34,6 +34,11 @@ func (p *PatchTransformerPlugin) Config(
 	if err != nil {
 		return err
 	}
+	if !strings.Contains(string(c), "yamlSupport") {
+		// If not explicitly denied,
+		// activate kyaml-based transformation.
+		p.YAMLSupport = true
+	}
 	p.Patch = strings.TrimSpace(p.Patch)
 	if p.Patch == "" && p.Path == "" {
 		return fmt.Errorf(
--- a/api/filters/patchstrategicmerge/patchstrategicmerge.go
+++ b/api/filters/patchstrategicmerge/patchstrategicmerge.go
@@ -1,3 +1,6 @@
+// Copyright 2019 The Kubernetes Authors.
+// SPDX-License-Identifier: Apache-2.0
+
 package patchstrategicmerge

 import (
@@ -13,9 +16,13 @@ type Filter struct {
 var _ kio.Filter = Filter{}

 func (pf Filter) Filter(nodes []*yaml.RNode) ([]*yaml.RNode, error) {
-	return kio.FilterAll(yaml.FilterFunc(pf.run)).Filter(nodes)
-}
-
-func (pf Filter) run(node *yaml.RNode) (*yaml.RNode, error) {
-	return merge2.Merge(pf.Patch, node)
+	var result []*yaml.RNode
+	for i := range nodes {
+		r, err := merge2.Merge(pf.Patch, nodes[i])
+		if err != nil {
+			return nil, err
+		}
+		result = append(result, r)
+	}
+	return result, nil
 }
--- a/api/krusty/baseandoverlaymedium_test.go
+++ b/api/krusty/baseandoverlaymedium_test.go
@@ -225,13 +225,13 @@ spec:
    spec:
      containers:
      - env:
+        - name: foo
+          value: bar
        - name: FOO
          valueFrom:
            configMapKeyRef:
              key: somekey
              name: test-infra-app-env-ffmd9b969m
-        - name: foo
-          value: bar
        image: nginx:1.8.0
        name: nginx
        ports:
--- a/api/krusty/customconfig_test.go
+++ b/api/krusty/customconfig_test.go
@@ -59,6 +59,7 @@ spec:
  location: SW
 `)
 	th.WriteF("/app/base/animalPark.yaml", `
+apiVersion: foo
 kind: AnimalPark
 metadata:
  name: sandiego
@@ -93,6 +94,7 @@ varReference:
 `)
 	m := th.Run("/app/base", th.MakeDefaultOptions())
 	th.AssertActualEqualsExpected(m, `
+apiVersion: foo
 kind: AnimalPark
 metadata:
  labels:
@@ -161,6 +163,7 @@ varReference:
 `)
 	m := th.Run("/app/base", th.MakeDefaultOptions())
 	th.AssertActualEqualsExpected(m, `
+apiVersion: foo
 kind: AnimalPark
 metadata:
  labels:
@@ -212,14 +215,17 @@ func TestFixedBug605_BaseCustomizationAvailableInOverlay(t *testing.T) {
 nameReference:
 - kind: Gorilla
  fieldSpecs:
-  - kind: AnimalPark
+  - apiVersion: foo
+    kind: AnimalPark
    path: spec/gorillaRef/name
 - kind: Giraffe
  fieldSpecs:
-  - kind: AnimalPark
+  - apiVersion: foo
+    kind: AnimalPark
    path: spec/giraffeRef/name
 varReference:
 - path: spec/food
+  apiVersion: foo
  kind: AnimalPark
 `)
 	th.WriteK("/app/overlay", `
@@ -242,6 +248,7 @@ spec:
 `)
 	// The following replaces the gorillaRef in the AnimalPark.
 	th.WriteF("/app/overlay/animalPark.yaml", `
+apiVersion: foo
 kind: AnimalPark
 metadata:
  name: sandiego
@@ -251,6 +258,7 @@ spec:
 `)
 	m := th.Run("/app/overlay", th.MakeDefaultOptions())
 	th.AssertActualEqualsExpected(m, `
+apiVersion: foo
 kind: AnimalPark
 metadata:
  labels:
--- a/api/krusty/extendedpatch_test.go
+++ b/api/krusty/extendedpatch_test.go
@@ -156,8 +156,7 @@ spec:
        - mountPath: /tmp/ps
          name: busybox-persistent-storage
      volumes:
-      - emptyDir: {}
-        name: busybox-persistent-storage
+      - name: busybox-persistent-storage
      - configMap:
          name: configmap-in-base
        name: configmap-in-base
@@ -233,8 +232,7 @@ spec:
        - mountPath: /tmp/ps
          name: nginx-persistent-storage
      volumes:
-      - emptyDir: {}
-        name: nginx-persistent-storage
+      - name: nginx-persistent-storage
      - configMap:
          name: configmap-in-base
        name: configmap-in-base
@@ -260,8 +258,7 @@ spec:
        - mountPath: /tmp/ps
          name: busybox-persistent-storage
      volumes:
-      - emptyDir: {}
-        name: busybox-persistent-storage
+      - name: busybox-persistent-storage
      - configMap:
          name: configmap-in-base
        name: configmap-in-base
@@ -335,8 +332,7 @@ spec:
        - mountPath: /tmp/ps
          name: nginx-persistent-storage
      volumes:
-      - emptyDir: {}
-        name: nginx-persistent-storage
+      - name: nginx-persistent-storage
      - configMap:
          name: configmap-in-base
        name: configmap-in-base
@@ -463,8 +459,7 @@ spec:
        - mountPath: /tmp/ps
          name: busybox-persistent-storage
      volumes:
-      - emptyDir: {}
-        name: busybox-persistent-storage
+      - name: busybox-persistent-storage
      - configMap:
          name: configmap-in-base
        name: configmap-in-base
@@ -564,8 +559,7 @@ spec:
        - mountPath: /tmp/ps
          name: busybox-persistent-storage
      volumes:
-      - emptyDir: {}
-        name: busybox-persistent-storage
+      - name: busybox-persistent-storage
      - configMap:
          name: configmap-in-base
        name: configmap-in-base
@@ -667,8 +661,7 @@ spec:
        - mountPath: /tmp/ps
          name: busybox-persistent-storage
      volumes:
-      - emptyDir: {}
-        name: busybox-persistent-storage
+      - name: busybox-persistent-storage
      - configMap:
          name: configmap-in-base
        name: configmap-in-base
@@ -769,8 +762,7 @@ spec:
        - mountPath: /tmp/ps
          name: busybox-persistent-storage
      volumes:
-      - emptyDir: {}
-        name: busybox-persistent-storage
+      - name: busybox-persistent-storage
      - configMap:
          name: configmap-in-base
        name: configmap-in-base
@@ -965,8 +957,7 @@ spec:
        - mountPath: /tmp/ps
          name: busybox-persistent-storage
      volumes:
-      - emptyDir: {}
-        name: busybox-persistent-storage
+      - name: busybox-persistent-storage
      - configMap:
          name: configmap-in-base
        name: configmap-in-base
@@ -1180,8 +1171,7 @@ spec:
        - mountPath: /tmp/ps
          name: busybox-persistent-storage
      volumes:
-      - emptyDir: {}
-        name: busybox-persistent-storage
+      - name: busybox-persistent-storage
      - configMap:
          name: configmap-in-base
        name: configmap-in-base
--- a/api/krusty/generatormergeandreplace_test.go
+++ b/api/krusty/generatormergeandreplace_test.go
@@ -150,8 +150,6 @@ spec:

 func makeBaseWithGenerators(th kusttest_test.Harness) {
 	th.WriteK("/app", `
-apiVersion: kustomize.config.k8s.io/v1beta1
-kind: Kustomization
 namePrefix: team-foo-
 commonLabels:
  app: mynginx
@@ -325,8 +323,6 @@ spec:
        name: configmap-in-overlay
 `)
 	th.WriteK("/overlay", `
-apiVersion: kustomize.config.k8s.io/v1beta1
-kind: Kustomization
 namePrefix: staging-
 commonLabels:
  env: staging
@@ -389,12 +385,12 @@ spec:
      - gcePersistentDisk:
          pdName: nginx-persistent-storage
        name: nginx-persistent-storage
-      - configMap:
-          name: staging-configmap-in-overlay-k7cbc75tg8
-        name: configmap-in-overlay
      - configMap:
          name: staging-team-foo-configmap-in-base-gh9d7t85gb
        name: configmap-in-base
+      - configMap:
+          name: staging-configmap-in-overlay-k7cbc75tg8
+        name: configmap-in-overlay
 ---
 apiVersion: v1
 kind: Service
--- a/api/krusty/inlinepatch_test.go
+++ b/api/krusty/inlinepatch_test.go
@@ -79,8 +79,7 @@ spec:
        - mountPath: /tmp/ps
          name: nginx-persistent-storage
      volumes:
-      - emptyDir: {}
-        name: nginx-persistent-storage
+      - name: nginx-persistent-storage
      - configMap:
          name: configmap-in-base
        name: configmap-in-base
@@ -223,8 +222,7 @@ spec:
        - mountPath: /tmp/ps
          name: nginx-persistent-storage
      volumes:
-      - emptyDir: {}
-        name: nginx-persistent-storage
+      - name: nginx-persistent-storage
      - configMap:
          name: configmap-in-base
        name: configmap-in-base
--- a/api/krusty/multiplepatch_test.go
+++ b/api/krusty/multiplepatch_test.go
@@ -146,12 +146,12 @@ spec:
      - gcePersistentDisk:
          pdName: nginx-persistent-storage
        name: nginx-persistent-storage
-      - configMap:
-          name: a-configmap-in-overlay-ffm9hf78mc
-        name: configmap-in-overlay
      - configMap:
          name: a-b-configmap-in-base-fm96mhk4dt
        name: configmap-in-base
+      - configMap:
+          name: a-configmap-in-overlay-ffm9hf78mc
+        name: configmap-in-overlay
 ---
 apiVersion: v1
 kind: Service
@@ -190,8 +190,6 @@ metadata:

 func makeCommonFileForMultiplePatchTest(th kusttest_test.Harness) {
 	th.WriteK("/app/base", `
-apiVersion: kustomize.config.k8s.io/v1beta1
-kind: Kustomization
 namePrefix: team-foo-
 commonLabels:
  app: mynginx
@@ -249,8 +247,6 @@ spec:
    app: nginx
 `)
 	th.WriteK("/app/overlay/staging", `
-apiVersion: kustomize.config.k8s.io/v1beta1
-kind: Kustomization
 namePrefix: staging-
 commonLabels:
  env: staging
@@ -355,12 +351,12 @@ spec:
      - gcePersistentDisk:
          pdName: nginx-persistent-storage
        name: nginx-persistent-storage
-      - configMap:
-          name: staging-configmap-in-overlay-k7cbc75tg8
-        name: configmap-in-overlay
      - configMap:
          name: staging-team-foo-configmap-in-base-g7k6gt2889
        name: configmap-in-base
+      - configMap:
+          name: staging-configmap-in-overlay-k7cbc75tg8
+        name: configmap-in-overlay
 ---
 apiVersion: v1
 kind: Service
@@ -544,8 +540,7 @@ spec:
        - mountPath: /tmp/ps
          name: nginx-persistent-storage
      volumes:
-      - emptyDir: {}
-        name: nginx-persistent-storage
+      - name: nginx-persistent-storage
      - configMap:
          name: staging-team-foo-configmap-in-base-g7k6gt2889
        name: configmap-in-base
--- a/examples/patchMultipleObjects.md
+++ b/examples/patchMultipleObjects.md
@@ -162,16 +162,16 @@ spec:
        old-label: old-value
    spec:
      containers:
-      - args:
-        - proxy
-        - sidecar
-        image: docker.io/istio/proxyv2
-        name: istio-proxy
      - args:
        - one
        - two
        image: nginx
        name: nginx
+      - args:
+        - proxy
+        - sidecar
+        image: docker.io/istio/proxyv2
+        name: istio-proxy
 ---
 apiVersion: apps/v1
 kind: Deployment
@@ -184,13 +184,13 @@ spec:
        key: value
    spec:
      containers:
+      - image: busybox
+        name: busybox
      - args:
        - proxy
        - sidecar
        image: docker.io/istio/proxyv2
        name: istio-proxy
-      - image: busybox
-        name: busybox
 EOF
 ```

--- a/kustomize/go.mod
+++ b/kustomize/go.mod
@@ -7,7 +7,7 @@ require (
 	github.com/spf13/cobra v1.0.0
 	github.com/spf13/pflag v1.0.5
 	k8s.io/client-go v0.17.3
-	sigs.k8s.io/kustomize/api v0.4.2
+	sigs.k8s.io/kustomize/api v0.5.0
 	sigs.k8s.io/kustomize/cmd/config v0.2.0
 	sigs.k8s.io/yaml v1.2.0
 )
--- a/kustomize/go.sum
+++ b/kustomize/go.sum
@@ -750,8 +750,8 @@ sigs.k8s.io/controller-runtime v0.4.0 h1:wATM6/m+3w8lj8FXNaO6Fs/rq/vqoOjO1Q116Z9
 sigs.k8s.io/controller-runtime v0.4.0/go.mod h1:ApC79lpY3PHW9xj/w9pj+lYkLgwAAUZwfXkME1Lajns=
 sigs.k8s.io/kustomize v2.0.3+incompatible h1:JUufWFNlI44MdtnjUqVnvh29rR37PQFzPbLXqhyOyX0=
 sigs.k8s.io/kustomize v2.0.3+incompatible/go.mod h1:MkjgH3RdOWrievjo6c9T245dYlB5QeXV4WCbnt/PEpU=
-sigs.k8s.io/kustomize/api v0.4.2 h1:ETPJPgfpWYXxXZw8n4EzuoIzmb8vSEwNx4Lr4/E6hGs=
-sigs.k8s.io/kustomize/api v0.4.2/go.mod h1:OND6MoJxF5BoS3IiAqDjPKdI7c+oTSaxuXKbljPbwNU=
+sigs.k8s.io/kustomize/api v0.5.0 h1:LT/Yxm6J8M9BJoLxfYIwGWamvuGSPQl7ELQXdlOR/fY=
+sigs.k8s.io/kustomize/api v0.5.0/go.mod h1:OND6MoJxF5BoS3IiAqDjPKdI7c+oTSaxuXKbljPbwNU=
 sigs.k8s.io/kustomize/cmd/config v0.2.0 h1:VNAWKb1JLl7dFjMAD5MwdZpGjreN3qL63C8DKtsUYck=
 sigs.k8s.io/kustomize/cmd/config v0.2.0/go.mod h1:oXzY7QJS6JlmWgusEjra2O3cW7GSIICXa59/3DvjBfE=
 sigs.k8s.io/kustomize/kyaml v0.1.4/go.mod h1:461i94nj0h0ylJ6w83jLkR4SqqVhn1iY6fjD0JSTQeE=
--- a/kyaml/yaml/merge2/map_test.go
+++ b/kyaml/yaml/merge2/map_test.go
@@ -63,6 +63,34 @@ spec:
 `,
 	},

+	{description: `strategic merge patch delete 4`,
+		source: `
+apiVersion: apps/v1
+metadata:
+  name: myDeploy
+kind: Deployment
+$patch: delete
+`,
+		dest: `
+apiVersion: apps/v1
+metadata:
+  name: myDeploy
+kind: Deployment
+spec:
+  replica: 2
+  template:
+    metadata:
+      labels:
+        old-label: old-value
+    spec:
+      containers:
+      - name: nginx
+        image: nginx
+`,
+		expected: `
+`,
+	},
+
 	{description: `strategic merge patch replace 1`,
 		source: `
 kind: Deployment
--- a/plugin/builtin/annotationstransformer/go.mod
+++ b/plugin/builtin/annotationstransformer/go.mod
@@ -3,9 +3,9 @@ module sigs.k8s.io/kustomize/plugin/builtin/annotationstransformer
 go 1.14

 require (
-	sigs.k8s.io/kustomize/api v0.4.2
+	sigs.k8s.io/kustomize/api v0.5.0
 	sigs.k8s.io/kustomize/kyaml v0.3.4
 	sigs.k8s.io/yaml v1.2.0
 )

-replace sigs.k8s.io/kustomize/api v0.4.2 => ../../../api
+replace sigs.k8s.io/kustomize/api v0.5.0 => ../../../api
--- a/plugin/builtin/configmapgenerator/go.mod
+++ b/plugin/builtin/configmapgenerator/go.mod
@@ -3,8 +3,8 @@ module sigs.k8s.io/kustomize/plugin/builtin/configmapgenerator
 go 1.14

 require (
-	sigs.k8s.io/kustomize/api v0.4.2
+	sigs.k8s.io/kustomize/api v0.5.0
 	sigs.k8s.io/yaml v1.2.0
 )

-replace sigs.k8s.io/kustomize/api v0.4.2 => ../../../api
+replace sigs.k8s.io/kustomize/api v0.5.0 => ../../../api
--- a/plugin/builtin/hashtransformer/go.mod
+++ b/plugin/builtin/hashtransformer/go.mod
@@ -2,6 +2,6 @@ module sigs.k8s.io/kustomize/plugin/builtin/hashtransformer

 go 1.14

-require sigs.k8s.io/kustomize/api v0.4.2
+require sigs.k8s.io/kustomize/api v0.5.0

-replace sigs.k8s.io/kustomize/api v0.4.2 => ../../../api
+replace sigs.k8s.io/kustomize/api v0.5.0 => ../../../api
--- a/plugin/builtin/imagetagtransformer/go.mod
+++ b/plugin/builtin/imagetagtransformer/go.mod
@@ -3,8 +3,8 @@ module sigs.k8s.io/kustomize/plugin/builtin/imagetagtransformer
 go 1.14

 require (
-	sigs.k8s.io/kustomize/api v0.4.2
+	sigs.k8s.io/kustomize/api v0.5.0
 	sigs.k8s.io/yaml v1.2.0
 )

-replace sigs.k8s.io/kustomize/api v0.4.2 => ../../../api
+replace sigs.k8s.io/kustomize/api v0.5.0 => ../../../api
--- a/plugin/builtin/labeltransformer/go.mod
+++ b/plugin/builtin/labeltransformer/go.mod
@@ -3,9 +3,9 @@ module sigs.k8s.io/kustomize/plugin/builtin/labeltransformer
 go 1.14

 require (
-	sigs.k8s.io/kustomize/api v0.4.2
+	sigs.k8s.io/kustomize/api v0.5.0
 	sigs.k8s.io/kustomize/kyaml v0.3.4
 	sigs.k8s.io/yaml v1.2.0
 )

-replace sigs.k8s.io/kustomize/api v0.4.2 => ../../../api
+replace sigs.k8s.io/kustomize/api v0.5.0 => ../../../api
--- a/plugin/builtin/legacyordertransformer/go.mod
+++ b/plugin/builtin/legacyordertransformer/go.mod
@@ -4,7 +4,7 @@ go 1.14

 require (
 	github.com/pkg/errors v0.8.1
-	sigs.k8s.io/kustomize/api v0.4.2
+	sigs.k8s.io/kustomize/api v0.5.0
 )

-replace sigs.k8s.io/kustomize/api v0.4.2 => ../../../api
+replace sigs.k8s.io/kustomize/api v0.5.0 => ../../../api
--- a/plugin/builtin/namespacetransformer/NamespaceTransformer.go
+++ b/plugin/builtin/namespacetransformer/NamespaceTransformer.go
@@ -6,6 +6,7 @@ package main

 import (
 	"fmt"
+	"strings"

 	"sigs.k8s.io/kustomize/api/filters/namespace"
 	"sigs.k8s.io/kustomize/api/resid"
@@ -35,6 +36,11 @@ func (p *plugin) Config(
 	_ *resmap.PluginHelpers, c []byte) (err error) {
 	p.Namespace = ""
 	p.FieldSpecs = nil
+	if !strings.Contains(string(c), "yamlSupport") {
+		// If not explicitly denied,
+		// activate kyaml-based transformation.
+		p.YAMLSupport = true
+	}
 	return yaml.Unmarshal(c, p)
 }

--- a/plugin/builtin/namespacetransformer/go.mod
+++ b/plugin/builtin/namespacetransformer/go.mod
@@ -3,9 +3,9 @@ module sigs.k8s.io/kustomize/plugin/builtin/namespacetransformer
 go 1.14

 require (
-	sigs.k8s.io/kustomize/api v0.4.2
+	sigs.k8s.io/kustomize/api v0.5.0
 	sigs.k8s.io/kustomize/kyaml v0.3.4
 	sigs.k8s.io/yaml v1.2.0
 )

-replace sigs.k8s.io/kustomize/api v0.4.2 => ../../../api
+replace sigs.k8s.io/kustomize/api v0.5.0 => ../../../api
--- a/plugin/builtin/patchjson6902transformer/PatchJson6902Transformer.go
+++ b/plugin/builtin/patchjson6902transformer/PatchJson6902Transformer.go
@@ -6,6 +6,7 @@ package main

 import (
 	"fmt"
+	"strings"

 	jsonpatch "github.com/evanphx/json-patch"
 	"github.com/pkg/errors"
@@ -38,6 +39,11 @@ func (p *plugin) Config(
 	if err != nil {
 		return err
 	}
+	if !strings.Contains(string(c), "yamlSupport") {
+		// If not explicitly denied,
+		// activate kyaml-based transformation.
+		p.YAMLSupport = true
+	}
 	if p.Target.Name == "" {
 		return fmt.Errorf("must specify the target name")
 	}
--- a/plugin/builtin/patchjson6902transformer/go.mod
+++ b/plugin/builtin/patchjson6902transformer/go.mod
@@ -5,9 +5,9 @@ go 1.14
 require (
 	github.com/evanphx/json-patch v4.5.0+incompatible
 	github.com/pkg/errors v0.8.1
-	sigs.k8s.io/kustomize/api v0.4.2
+	sigs.k8s.io/kustomize/api v0.5.0
 	sigs.k8s.io/kustomize/kyaml v0.3.4
 	sigs.k8s.io/yaml v1.2.0
 )

-replace sigs.k8s.io/kustomize/api v0.4.2 => ../../../api
+replace sigs.k8s.io/kustomize/api v0.5.0 => ../../../api
--- a/plugin/builtin/patchstrategicmergetransformer/PatchStrategicMergeTransformer.go
+++ b/plugin/builtin/patchstrategicmergetransformer/PatchStrategicMergeTransformer.go
@@ -6,6 +6,7 @@ package main

 import (
 	"fmt"
+	"strings"

 	"sigs.k8s.io/kustomize/api/filters/patchstrategicmerge"
 	"sigs.k8s.io/kustomize/api/resmap"
@@ -34,6 +35,11 @@ func (p *plugin) Config(
 	if err != nil {
 		return err
 	}
+	if !strings.Contains(string(c), "yamlSupport") {
+		// If not explicitly denied,
+		// activate kyaml-based transformation.
+		p.YAMLSupport = true
+	}
 	if len(p.Paths) == 0 && p.Patches == "" {
 		return fmt.Errorf("empty file path and empty patch content")
 	}
@@ -79,17 +85,6 @@ func (p *plugin) Transform(m resmap.ResMap) error {
 		}
 		if !p.YAMLSupport {
 			err = target.Patch(patch.Copy())
-			if err != nil {
-				return err
-			}
-			// remove the resource from resmap
-			// when the patch is to $patch: delete that target
-			if len(target.Map()) == 0 {
-				err = m.Remove(target.CurId())
-				if err != nil {
-					return err
-				}
-			}
 		} else {
 			patchCopy := patch.DeepCopy()
 			patchCopy.SetName(target.GetName())
@@ -103,6 +98,19 @@ func (p *plugin) Transform(m resmap.ResMap) error {
 				Patch: node,
 			}, target)
 		}
+		if err != nil {
+			return err
+		}
+		if len(target.Map()) == 0 {
+			// This means all fields have been removed from the object.
+			// This can happen if a patch required deletion of the
+			// entire resource (not just a part of it).  This means
+			// the overall resmap must shrink by one.
+			err = m.Remove(target.CurId())
+			if err != nil {
+				return err
+			}
+		}
 	}
 	return nil
 }
--- a/plugin/builtin/patchstrategicmergetransformer/PatchStrategicMergeTransformer_test.go
+++ b/plugin/builtin/patchstrategicmergetransformer/PatchStrategicMergeTransformer_test.go
@@ -1208,9 +1208,89 @@ func TestMultipleNamespaces(t *testing.T) {
 	}
 }

-// We don't run this for the kyaml implementation since we don't support
-// the strategic merge patch directives (yet).
-func TestPatchStrategicMergeTransformerPatchDelete(t *testing.T) {
+func TestPatchStrategicMergeTransformerPatchDelete1(t *testing.T) {
+	th := kusttest_test.MakeEnhancedHarness(t).
+		PrepBuiltin("PatchStrategicMergeTransformer")
+	defer th.Reset()
+
+	th.WriteF("patch.yaml", `
+apiVersion: apps/v1
+metadata:
+  name: myDeploy
+kind: Deployment
+spec:
+  replica: 2
+  template:
+    $patch: delete
+    metadata:
+      labels:
+        old-label: old-value
+    spec:
+      containers:
+      - name: nginx
+        image: nginx
+`)
+
+	rm := th.LoadAndRunTransformer(`
+apiVersion: builtin
+kind: PatchStrategicMergeTransformer
+metadata:
+  name: notImportantHere
+paths:
+- patch.yaml
+`, target)
+
+	th.AssertActualEqualsExpected(rm, `
+apiVersion: apps/v1
+kind: Deployment
+metadata:
+  name: myDeploy
+spec:
+  replica: 2
+`)
+}
+
+func TestPatchStrategicMergeTransformerPatchDelete2(t *testing.T) {
+	th := kusttest_test.MakeEnhancedHarness(t).
+		PrepBuiltin("PatchStrategicMergeTransformer")
+	defer th.Reset()
+
+	th.WriteF("patch.yaml", `
+apiVersion: apps/v1
+metadata:
+  name: myDeploy
+kind: Deployment
+spec:
+  $patch: delete
+  replica: 2
+  template:
+    metadata:
+      labels:
+        old-label: old-value
+    spec:
+      containers:
+      - name: nginx
+        image: nginx
+`)
+
+	rm := th.LoadAndRunTransformer(`
+apiVersion: builtin
+kind: PatchStrategicMergeTransformer
+metadata:
+  name: notImportantHere
+paths:
+- patch.yaml
+`, target)
+
+	th.AssertActualEqualsExpected(rm, `
+apiVersion: apps/v1
+kind: Deployment
+metadata:
+  name: myDeploy
+`)
+}
+
+func TestPatchStrategicMergeTransformerPatchDelete3(t *testing.T) {
 	th := kusttest_test.MakeEnhancedHarness(t).
 		PrepBuiltin("PatchStrategicMergeTransformer")
 	defer th.Reset()
@@ -1234,3 +1314,4 @@ paths:

 	th.AssertActualEqualsExpected(rm, ``)
 }
+
--- a/plugin/builtin/patchstrategicmergetransformer/go.mod
+++ b/plugin/builtin/patchstrategicmergetransformer/go.mod
@@ -3,9 +3,9 @@ module sigs.k8s.io/kustomize/plugin/builtin/patchstrategicmergetransformer
 go 1.14

 require (
-	sigs.k8s.io/kustomize/api v0.4.2
+	sigs.k8s.io/kustomize/api v0.5.0
 	sigs.k8s.io/kustomize/kyaml v0.3.4
 	sigs.k8s.io/yaml v1.2.0
 )

-replace sigs.k8s.io/kustomize/api v0.4.2 => ../../../api
+replace sigs.k8s.io/kustomize/api v0.5.0 => ../../../api
--- a/plugin/builtin/patchtransformer/PatchTransformer.go
+++ b/plugin/builtin/patchtransformer/PatchTransformer.go
@@ -38,6 +38,11 @@ func (p *plugin) Config(
 	if err != nil {
 		return err
 	}
+	if !strings.Contains(string(c), "yamlSupport") {
+		// If not explicitly denied,
+		// activate kyaml-based transformation.
+		p.YAMLSupport = true
+	}
 	p.Patch = strings.TrimSpace(p.Patch)
 	if p.Patch == "" && p.Path == "" {
 		return fmt.Errorf(
--- a/plugin/builtin/patchtransformer/go.mod
+++ b/plugin/builtin/patchtransformer/go.mod
@@ -5,9 +5,9 @@ go 1.14
 require (
 	github.com/evanphx/json-patch v4.5.0+incompatible
 	github.com/pkg/errors v0.8.1
-	sigs.k8s.io/kustomize/api v0.4.2
+	sigs.k8s.io/kustomize/api v0.5.0
 	sigs.k8s.io/kustomize/kyaml v0.3.4
 	sigs.k8s.io/yaml v1.2.0
 )

-replace sigs.k8s.io/kustomize/api v0.4.2 => ../../../api
+replace sigs.k8s.io/kustomize/api v0.5.0 => ../../../api
--- a/plugin/builtin/prefixsuffixtransformer/go.mod
+++ b/plugin/builtin/prefixsuffixtransformer/go.mod
@@ -3,8 +3,8 @@ module sigs.k8s.io/kustomize/plugin/builtin/prefixsuffixtransformer
 go 1.14

 require (
-	sigs.k8s.io/kustomize/api v0.4.2
+	sigs.k8s.io/kustomize/api v0.5.0
 	sigs.k8s.io/yaml v1.2.0
 )

-replace sigs.k8s.io/kustomize/api v0.4.2 => ../../../api
+replace sigs.k8s.io/kustomize/api v0.5.0 => ../../../api
--- a/plugin/builtin/replicacounttransformer/go.mod
+++ b/plugin/builtin/replicacounttransformer/go.mod
@@ -3,8 +3,8 @@ module sigs.k8s.io/kustomize/plugin/builtin/replicacounttransformer
 go 1.14

 require (
-	sigs.k8s.io/kustomize/api v0.4.2
+	sigs.k8s.io/kustomize/api v0.5.0
 	sigs.k8s.io/yaml v1.2.0
 )

-replace sigs.k8s.io/kustomize/api v0.4.2 => ../../../api
+replace sigs.k8s.io/kustomize/api v0.5.0 => ../../../api
--- a/plugin/builtin/secretgenerator/go.mod
+++ b/plugin/builtin/secretgenerator/go.mod
@@ -3,8 +3,8 @@ module sigs.k8s.io/kustomize/plugin/builtin/secretgenerator
 go 1.14

 require (
-	sigs.k8s.io/kustomize/api v0.4.2
+	sigs.k8s.io/kustomize/api v0.5.0
 	sigs.k8s.io/yaml v1.2.0
 )

-replace sigs.k8s.io/kustomize/api v0.4.2 => ../../../api
+replace sigs.k8s.io/kustomize/api v0.5.0 => ../../../api
--- a/plugin/builtin/valueaddtransformer/go.mod
+++ b/plugin/builtin/valueaddtransformer/go.mod
@@ -3,9 +3,9 @@ module sigs.k8s.io/kustomize/plugin/someteam.example.com/v1/valueaddtransformer
 go 1.14

 require (
-	sigs.k8s.io/kustomize/api v0.4.2
+	sigs.k8s.io/kustomize/api v0.5.0
 	sigs.k8s.io/kustomize/kyaml v0.3.4
 	sigs.k8s.io/yaml v1.2.0
 )

-replace sigs.k8s.io/kustomize/api v0.4.2 => ../../../api
+replace sigs.k8s.io/kustomize/api v0.5.0 => ../../../api
--- a/plugin/someteam.example.com/v1/bashedconfigmap/go.mod
+++ b/plugin/someteam.example.com/v1/bashedconfigmap/go.mod
@@ -2,6 +2,6 @@ module sigs.k8s.io/kustomize/plugin/someteam.example.com/v1/bashedconfigmap

 go 1.14

-require sigs.k8s.io/kustomize/api v0.4.2
+require sigs.k8s.io/kustomize/api v0.5.0

-replace sigs.k8s.io/kustomize/api v0.4.2 => ../../../../api
+replace sigs.k8s.io/kustomize/api v0.5.0 => ../../../../api
--- a/plugin/someteam.example.com/v1/chartinflator/go.mod
+++ b/plugin/someteam.example.com/v1/chartinflator/go.mod
@@ -2,6 +2,6 @@ module sigs.k8s.io/kustomize/plugin/someteam.example.com/v1/chartinflator

 go 1.14

-require sigs.k8s.io/kustomize/api v0.4.2
+require sigs.k8s.io/kustomize/api v0.5.0

-replace sigs.k8s.io/kustomize/api v0.4.2 => ../../../../api
+replace sigs.k8s.io/kustomize/api v0.5.0 => ../../../../api
--- a/plugin/someteam.example.com/v1/dateprefixer/go.mod
+++ b/plugin/someteam.example.com/v1/dateprefixer/go.mod
@@ -4,8 +4,8 @@ go 1.14

 require (
 	github.com/pkg/errors v0.8.1
-	sigs.k8s.io/kustomize/api v0.4.2
+	sigs.k8s.io/kustomize/api v0.5.0
 	sigs.k8s.io/yaml v1.2.0
 )

-replace sigs.k8s.io/kustomize/api v0.4.2 => ../../../../api
+replace sigs.k8s.io/kustomize/api v0.5.0 => ../../../../api
--- a/plugin/someteam.example.com/v1/gogetter/go.mod
+++ b/plugin/someteam.example.com/v1/gogetter/go.mod
@@ -2,6 +2,6 @@ module sigs.k8s.io/kustomize/plugin/someteam.example.com/v1/gogetter

 go 1.14

-require sigs.k8s.io/kustomize/api v0.4.2
+require sigs.k8s.io/kustomize/api v0.5.0

-replace sigs.k8s.io/kustomize/api v0.4.2 => ../../../../api
+replace sigs.k8s.io/kustomize/api v0.5.0 => ../../../../api
--- a/plugin/someteam.example.com/v1/printpluginenv/go.mod
+++ b/plugin/someteam.example.com/v1/printpluginenv/go.mod
@@ -2,6 +2,6 @@ module sigs.k8s.io/kustomize/plugin/someteam.example.com/v1/printworkdir

 go 1.14

-require sigs.k8s.io/kustomize/api v0.4.2
+require sigs.k8s.io/kustomize/api v0.5.0

-replace sigs.k8s.io/kustomize/api v0.4.2 => ../../../../api
+replace sigs.k8s.io/kustomize/api v0.5.0 => ../../../../api
--- a/plugin/someteam.example.com/v1/replacementtransformer/go.mod
+++ b/plugin/someteam.example.com/v1/replacementtransformer/go.mod
@@ -3,8 +3,8 @@ module sigs.k8s.io/kustomize/plugin/someteam.example.com/v1/replacementtransform
 go 1.14

 require (
-	sigs.k8s.io/kustomize/api v0.4.2
+	sigs.k8s.io/kustomize/api v0.5.0
 	sigs.k8s.io/yaml v1.2.0
 )

-replace sigs.k8s.io/kustomize/api v0.4.2 => ../../../../api
+replace sigs.k8s.io/kustomize/api v0.5.0 => ../../../../api
--- a/plugin/someteam.example.com/v1/secretsfromdatabase/go.mod
+++ b/plugin/someteam.example.com/v1/secretsfromdatabase/go.mod
@@ -3,8 +3,8 @@ module sigs.k8s.io/kustomize/plugin/someteam.example.com/v1/secretsfromdatabase
 go 1.14

 require (
-	sigs.k8s.io/kustomize/api v0.4.2
+	sigs.k8s.io/kustomize/api v0.5.0
 	sigs.k8s.io/yaml v1.2.0
 )

-replace sigs.k8s.io/kustomize/api v0.4.2 => ../../../../api
+replace sigs.k8s.io/kustomize/api v0.5.0 => ../../../../api
--- a/plugin/someteam.example.com/v1/sedtransformer/go.mod
+++ b/plugin/someteam.example.com/v1/sedtransformer/go.mod
@@ -2,6 +2,6 @@ module sigs.k8s.io/kustomize/plugin/someteam.example.com/v1/sedtransformer

 go 1.14

-require sigs.k8s.io/kustomize/api v0.4.2
+require sigs.k8s.io/kustomize/api v0.5.0

-replace sigs.k8s.io/kustomize/api v0.4.2 => ../../../../api
+replace sigs.k8s.io/kustomize/api v0.5.0 => ../../../../api
--- a/plugin/someteam.example.com/v1/someservicegenerator/go.mod
+++ b/plugin/someteam.example.com/v1/someservicegenerator/go.mod
@@ -3,8 +3,8 @@ module sigs.k8s.io/kustomize/plugin/someteam.example.com/v1/someservicegenerator
 go 1.14

 require (
-	sigs.k8s.io/kustomize/api v0.4.2
+	sigs.k8s.io/kustomize/api v0.5.0
 	sigs.k8s.io/yaml v1.2.0
 )

-replace sigs.k8s.io/kustomize/api v0.4.2 => ../../../../api
+replace sigs.k8s.io/kustomize/api v0.5.0 => ../../../../api
--- a/plugin/someteam.example.com/v1/stringprefixer/go.mod
+++ b/plugin/someteam.example.com/v1/stringprefixer/go.mod
@@ -4,8 +4,8 @@ go 1.14

 require (
 	github.com/pkg/errors v0.8.1
-	sigs.k8s.io/kustomize/api v0.4.2
+	sigs.k8s.io/kustomize/api v0.5.0
 	sigs.k8s.io/yaml v1.2.0
 )

-replace sigs.k8s.io/kustomize/api v0.4.2 => ../../../../api
+replace sigs.k8s.io/kustomize/api v0.5.0 => ../../../../api
--- a/plugin/someteam.example.com/v1/validator/go.mod
+++ b/plugin/someteam.example.com/v1/validator/go.mod
@@ -2,6 +2,6 @@ module sigs.k8s.io/kustomize/plugin/someteam.example.com/v1/validator

 go 1.14

-require sigs.k8s.io/kustomize/api v0.4.2
+require sigs.k8s.io/kustomize/api v0.5.0

-replace sigs.k8s.io/kustomize/api v0.4.2 => ../../../../api
+replace sigs.k8s.io/kustomize/api v0.5.0 => ../../../../api
Author	SHA1	Message	Date
Jeff Regan	6a50372dd5	Merge pull request #2694 from monopole/switchToApiV0_5_0 Switch to kustomize/api v0.5.0	2020-07-04 21:22:19 -07:00
jregan	5a0228629f	Switch kustomize to api/v0.5.0	2020-07-04 19:45:45 -07:00
Jeff Regan	def00220ce	Merge pull request #2668 from monopole/doesItWork Switch namespace and patch transformers to kyaml.	2020-07-04 17:11:56 -07:00
jregan	d3a7335bbc	Switch namespace and patch transformers to kyaml.	2020-07-04 16:21:01 -07:00