clusterresourceset: Allow strategy to be changed from "" to ApplyOnce.

Naadir Jeewa · Naadir Jeewa · commit 03f186910b77 · 2021-04-09T16:43:10.000+01:00
Signed-off-by: Naadir Jeewa &lt;jeewan@vmware.com&gt;
diff --git a/exp/addons/api/v1alpha3/clusterresourceset_webhook.go b/exp/addons/api/v1alpha3/clusterresourceset_webhook.go
@@ -87,7 +87,7 @@ func (m *ClusterResourceSet) validate(old *ClusterResourceSet) error {
 		)
 	}
 
-	if old != nil && old.Spec.Strategy != m.Spec.Strategy {
+	if old != nil && old.Spec.Strategy != "" && old.Spec.Strategy != m.Spec.Strategy {
 		allErrs = append(
 			allErrs,
 			field.Invalid(field.NewPath("spec", "strategy"), m.Spec.Strategy, "field is immutable"),

Original file line number	Diff line number	Diff line change
`@@ -87,7 +87,7 @@ func (m ClusterResourceSet) validate(old ClusterResourceSet) error {`
`87`	`87`	`)`
`88`	`88`	`}`
`89`	`89`
`90`		`- if old != nil && old.Spec.Strategy != m.Spec.Strategy {`
	`90`	`+ if old != nil && old.Spec.Strategy != "" && old.Spec.Strategy != m.Spec.Strategy {`
`91`	`91`	`allErrs = append(`
`92`	`92`	`allErrs,`
`93`	`93`	`field.Invalid(field.NewPath("spec", "strategy"), m.Spec.Strategy, "field is immutable"),`