fix: spec.preserveUnknownFields must be explicitly false to allow upgrades from apiextensions.k8s.io/v1beta1

hack/gen-crd-spec/main.go

@@ -82,6 +82,7 @@ func NewCustomResourceDefinition() []*extensionsobj.CustomResourceDefinition {
 		// by using the 'setValidationOverride' function in this file.
 		"crd:preserveUnknownFields=false",
 		"crd:crdVersions=v1",
+		"crd:maxDescLen=0",
 		"output:crd:stdout",
 	).Output()
 	if err != nil {
@@ -110,7 +111,6 @@ func NewCustomResourceDefinition() []*extensionsobj.CustomResourceDefinition {
 
 	for i := range objs {
 		obj := objs[i]
-		removeDescriptions(obj)
 		removeK8S118Fields(obj)
 		createMetadataValidation(obj)
 		crd := toCRD(obj)
@@ -228,18 +228,6 @@ func createMetadataValidation(un *unstructured.Unstructured) {
 	}
 }
 
-func removeDescriptions(un *unstructured.Unstructured) {
-	objVersions, _, _ := unstructured.NestedSlice(un.Object, "spec", "versions")
-
-	var validation []interface{}
-	for _, v := range objVersions {
-		removeFieldHelper(v.(map[string]interface{}), "description")
-		validation = append(validation, v)
-	}
-
-	unstructured.SetNestedSlice(un.Object, validation, "spec", "versions")
-}
-
 func removeFieldHelper(obj map[string]interface{}, fieldName string) {
 	for k, v := range obj {
 		if k == fieldName {
@@ -332,6 +320,11 @@ func main() {
 		err = json.Unmarshal(jsonBytes, &r.Object)
 		checkErr(err)
 
+		// Need to explicitly set spec.preserveUnknownFields to false, despite false being the
+		// default value in v1, in order to facilitate upgrades from apiextensions.k8s.io/v1beta1 v1.
+		// See https://github.com/argoproj/argo-rollouts/issues/1067
+		unstructured.SetNestedField(r.Object, false, "spec", "preserveUnknownFields")
+
 		// clean up crd yaml before marshalling
 		unstructured.RemoveNestedField(r.Object, "status")
 		unstructured.RemoveNestedField(r.Object, "metadata", "creationTimestamp")

manifests/crds/analysis-run-crd.yaml

@@ -13,6 +13,7 @@ spec:
     shortNames:
     - ar
     singular: analysisrun
+  preserveUnknownFields: false
   scope: Namespaced
   versions:
   - additionalPrinterColumns:

manifests/crds/analysis-template-crd.yaml

@@ -13,6 +13,7 @@ spec:
     shortNames:
     - at
     singular: analysistemplate
+  preserveUnknownFields: false
   scope: Namespaced
   versions:
   - name: v1alpha1

manifests/crds/cluster-analysis-template-crd.yaml

@@ -13,6 +13,7 @@ spec:
     shortNames:
     - cat
     singular: clusteranalysistemplate
+  preserveUnknownFields: false
   scope: Cluster
   versions:
   - name: v1alpha1

manifests/crds/experiment-crd.yaml

@@ -13,6 +13,7 @@ spec:
     shortNames:
     - exp
     singular: experiment
+  preserveUnknownFields: false
   scope: Namespaced
   versions:
   - additionalPrinterColumns:

manifests/crds/rollout-crd.yaml

@@ -13,6 +13,7 @@ spec:
     shortNames:
     - ro
     singular: rollout
+  preserveUnknownFields: false
   scope: Namespaced
   versions:
   - additionalPrinterColumns:

manifests/install.yaml

@@ -14,6 +14,7 @@ spec:
     shortNames:
     - ar
     singular: analysisrun
+  preserveUnknownFields: false
   scope: Namespaced
   versions:
   - additionalPrinterColumns:
@@ -2516,6 +2517,7 @@ spec:
     shortNames:
     - at
     singular: analysistemplate
+  preserveUnknownFields: false
   scope: Namespaced
   versions:
   - name: v1alpha1
@@ -4940,6 +4942,7 @@ spec:
     shortNames:
     - cat
     singular: clusteranalysistemplate
+  preserveUnknownFields: false
   scope: Cluster
   versions:
   - name: v1alpha1
@@ -7364,6 +7367,7 @@ spec:
     shortNames:
     - exp
     singular: experiment
+  preserveUnknownFields: false
   scope: Namespaced
   versions:
   - additionalPrinterColumns:
@@ -9698,6 +9702,7 @@ spec:
     shortNames:
     - ro
     singular: rollout
+  preserveUnknownFields: false
   scope: Namespaced
   versions:
   - additionalPrinterColumns:

manifests/namespace-install.yaml

@@ -14,6 +14,7 @@ spec:
     shortNames:
     - ar
     singular: analysisrun
+  preserveUnknownFields: false
   scope: Namespaced
   versions:
   - additionalPrinterColumns:
@@ -2516,6 +2517,7 @@ spec:
     shortNames:
     - at
     singular: analysistemplate
+  preserveUnknownFields: false
   scope: Namespaced
   versions:
   - name: v1alpha1
@@ -4940,6 +4942,7 @@ spec:
     shortNames:
     - cat
     singular: clusteranalysistemplate
+  preserveUnknownFields: false
   scope: Cluster
   versions:
   - name: v1alpha1
@@ -7364,6 +7367,7 @@ spec:
     shortNames:
     - exp
     singular: experiment
+  preserveUnknownFields: false
   scope: Namespaced
   versions:
   - additionalPrinterColumns:
@@ -9698,6 +9702,7 @@ spec:
     shortNames:
     - ro
     singular: rollout
+  preserveUnknownFields: false
   scope: Namespaced
   versions:
   - additionalPrinterColumns: