fix: set-labels handle slice, change log

examples/set-labels-advanced/README.md

@@ -61,4 +61,4 @@ $ kpt fn render set-labels-advanced
 Check all resources have 2 labels: `color: orange` and `fruit: apple`. And the
 resource of kind `MyResource` also has these 2 labels in `spec.selector.labels`.
 
-[`set-labels`]: https://catalog.kpt.dev/set-labels/v0.1/
+[`set-labels`]: https://catalog.kpt.dev/set-labels/v0.1/

functions/go/set-labels/README.md

@@ -57,27 +57,9 @@ data:
 ```
 
 To use a `SetLabels` custom resource as the `functionConfig`, the desired labels
-must be specified in the `labels` field. Sometimes you have resources (
-especially custom resources) that have labels or selectors fields in fields
-other than the [defaults][commonlabels], you can specify such label fields using
-`additionalLabelFields`. It will be used jointly with the
-[defaults][commonlabels].
-
-`additionalLabelFields` has following fields:
-
-- `group`: Select the resources by API version group. Will select all groups if
-  omitted.
-- `version`: Select the resources by API version. Will select all versions if
-  omitted.
-- `kind`: Select the resources by resource kind. Will select all kinds if
-  omitted.
-- `path`: Specify the path to the field that the value needs to be updated. This
-  field is required.
-- `create`: If it's set to true, the field specified will be created if it
-  doesn't exist. Otherwise, the function will only update the existing field.
-
-To add 2 labels `color: orange` and `fruit: apple` to all built-in resources and
-the path `data.selector` in `MyOwnKind` resource, we use the
+must be specified in the `labels` field. 
+
+To add 2 labels `color: orange` and `fruit: apple` to all built-in resources, we use the
 following `functionConfig`:
 
 ```yaml
@@ -88,10 +70,6 @@ metadata:
 labels:
   color: orange
   fruit: apple
-additionalLabelFields:
-  - path: data/selector
-    kind: MyOwnKind
-    create: true
 ```
 
 <!--mdtogo-->

functions/go/set-labels/go.sum

@@ -41,8 +41,6 @@ github.com/BurntSushi/toml v0.3.1/go.mod h1:xHWCNGjB5oqiDr8zfno3MHue2Ht5sIBksp03
 github.com/BurntSushi/xgb v0.0.0-20160522181843-27f122750802/go.mod h1:IVnqGOEym/WlBOVXweHU+Q+/VP0lqqI8lqeDx9IjBqo=
 github.com/GoogleContainerTools/kpt-functions-sdk/go/api v0.0.0-20220720212527-133180134b93 h1:c1GhPzYzU2a3E+/2WB9OxoljK2pNYfsBF5Fz9GkdYXs=
 github.com/GoogleContainerTools/kpt-functions-sdk/go/api v0.0.0-20220720212527-133180134b93/go.mod h1:gkK43tTaPXFNASpbIbQImzhmt1hdcdin++kvzTblykc=
-github.com/GoogleContainerTools/kpt-functions-sdk/go/fn v0.0.0-20220608182028-78748f08d997 h1:yv4zLzpaPg0MJZWhNUlQxQNgqkV6qxL4xXVlaVoyMhg=
-github.com/GoogleContainerTools/kpt-functions-sdk/go/fn v0.0.0-20220608182028-78748f08d997/go.mod h1:5m1pYKzRjlYG/Oi//vcJ1pCHsW8b+gD+wAPXwxz/4jc=
 github.com/GoogleContainerTools/kpt-functions-sdk/go/fn v0.0.0-20220723081830-33aee7fe8a2e h1:a4rfmCZpbG78rCqOZYDB9HV2avrx5lZaytUVDpj8MqE=
 github.com/GoogleContainerTools/kpt-functions-sdk/go/fn v0.0.0-20220723081830-33aee7fe8a2e/go.mod h1:Kfyd06y9XFV1QXUI1L7OI5xxINyK0Xq+7BcXcnv28c4=
 github.com/NYTimes/gziphandler v0.0.0-20170623195520-56545f4a5d46/go.mod h1:3wb06e3pkSAbeQ52E9H9iFoQsEEwGN64994WTCIhntQ=

functions/go/set-labels/transformer/common.go

@@ -15,8 +15,8 @@
 package transformer
 
 const (
-	FnConfigKind       = "SetLabels"
-	fnDeprecateField   = "additionalLabelFields"
+	FnConfigKind     = "SetLabels"
+	fnDeprecateField = "additionalLabelFields"
 )
 
 // generate common label paths
@@ -46,31 +46,6 @@ var CommonSpecs = FieldSpecs{
 		FieldPath:          FieldPath{"spec", "template", "metadata", "labels"},
 		CreateIfNotPresent: true,
 	},
-	FieldSpec{
-		Gvk:                GVK{"apps", "", "Deployment"},
-		FieldPath:          FieldPath{"spec", "template", "spec", "affinity", "podAffinity", "preferredDuringSchedulingIgnoredDuringExecution", "podAffinityTerm", "labelSelector", "matchLabels"},
-		CreateIfNotPresent: false,
-	},
-	FieldSpec{
-		Gvk:                GVK{"apps", "", "Deployment"},
-		FieldPath:          FieldPath{"spec", "template", "spec", "affinity", "podAffinity", "requiredDuringSchedulingIgnoredDuringExecution", "labelSelector", "matchLabels"},
-		CreateIfNotPresent: false,
-	},
-	FieldSpec{
-		Gvk:                GVK{"apps", "", "Deployment"},
-		FieldPath:          FieldPath{"spec", "template", "spec", "affinity", "podAntiAffinity", "preferredDuringSchedulingIgnoredDuringExecution", "podAffinityTerm", "labelSelector", "matchLabels"},
-		CreateIfNotPresent: false,
-	},
-	FieldSpec{
-		Gvk:                GVK{"apps", "", "Deployment"},
-		FieldPath:          FieldPath{"spec", "template", "spec", "affinity", "podAntiAffinity", "requiredDuringSchedulingIgnoredDuringExecution", "labelSelector", "matchLabels"},
-		CreateIfNotPresent: false,
-	},
-	FieldSpec{
-		Gvk:                GVK{"apps", "", "Deployment"},
-		FieldPath:          FieldPath{"spec", "template", "spec", "topologySpreadConstraints", "labelSelector", "matchLabels"},
-		CreateIfNotPresent: false,
-	},
 	FieldSpec{
 		Gvk:                GVK{"", "", "ReplicaSet"},
 		FieldPath:          FieldPath{"spec", "selector", "matchLabels"},
@@ -101,31 +76,6 @@ var CommonSpecs = FieldSpecs{
 		FieldPath:          FieldPath{"spec", "template", "metadata", "labels"},
 		CreateIfNotPresent: true,
 	},
-	FieldSpec{
-		Gvk:                GVK{"apps", "", "StatefulSet"},
-		FieldPath:          FieldPath{"spec", "template", "spec", "affinity", "podAffinity", "preferredDuringSchedulingIgnoredDuringExecution", "podAffinityTerm", "labelSelector", "matchLabels"},
-		CreateIfNotPresent: false,
-	},
-	FieldSpec{
-		Gvk:                GVK{"apps", "", "StatefulSet"},
-		FieldPath:          FieldPath{"spec", "template", "spec", "affinity", "podAffinity", "requiredDuringSchedulingIgnoredDuringExecution", "labelSelector", "matchLabels"},
-		CreateIfNotPresent: false,
-	},
-	FieldSpec{
-		Gvk:                GVK{"apps", "", "StatefulSet"},
-		FieldPath:          FieldPath{"spec", "template", "spec", "affinity", "podAntiAffinity", "preferredDuringSchedulingIgnoredDuringExecution", "podAffinityTerm", "labelSelector", "matchLabels"},
-		CreateIfNotPresent: false,
-	},
-	FieldSpec{
-		Gvk:                GVK{"apps", "", "StatefulSet"},
-		FieldPath:          FieldPath{"spec", "template", "spec", "affinity", "podAntiAffinity", "requiredDuringSchedulingIgnoredDuringExecution", "labelSelector", "matchLabels"},
-		CreateIfNotPresent: false,
-	},
-	FieldSpec{
-		Gvk:                GVK{"apps", "", "StatefulSet"},
-		FieldPath:          FieldPath{"spec", "template", "spec", "topologySpreadConstraints", "labelSelector", "matchLabels"},
-		CreateIfNotPresent: false,
-	},
 	FieldSpec{
 		Gvk:                GVK{"batch", "", "Job"},
 		FieldPath:          FieldPath{"spec", "selector", "matchLabels"},
@@ -161,14 +111,4 @@ var CommonSpecs = FieldSpecs{
 		FieldPath:          FieldPath{"spec", "podSelector", "matchLabels"},
 		CreateIfNotPresent: false,
 	},
-	FieldSpec{
-		Gvk:                GVK{"networking.k8s.io", "", "NetworkPolicy"},
-		FieldPath:          FieldPath{"spec", "ingress", "from", "podSelector", "matchLabels"},
-		CreateIfNotPresent: false,
-	},
-	FieldSpec{
-		Gvk:                GVK{"networking.k8s.io", "", "NetworkPolicy"},
-		FieldPath:          FieldPath{"spec", "egress", "to", "podSelector", "matchLabels"},
-		CreateIfNotPresent: false,
-	},
 }