Adding sort to reorder webhookDescription based on webhookType

changelog/fragments/fix_order_of_generated_webhooks.yaml

@@ -0,0 +1,8 @@
+entries:
+  - description: >
+      Preserve the order of `webhookdefinitions` while generating bundles, 
+      by sorting `webhookDescriptions` in CSV generator.
+
+    kind: bugfix
+
+    breaking: false

internal/generate/clusterserviceversion/clusterserviceversion_updaters.go

@@ -336,6 +336,10 @@ func applyWebhooks(c *collector.Manifests, csv *operatorsv1alpha1.ClusterService
 			webhookDescriptions = append(webhookDescriptions, des...)
 		}
 	}
+	// Sorts the WebhookDescriptions based on natural order of webhookDescriptions Type
+	sort.Slice(webhookDescriptions, func(i, j int) bool {
+		return webhookDescriptions[i].Type < webhookDescriptions[j].Type
+	})
 	csv.Spec.WebhookDefinitions = webhookDescriptions
 }
 

internal/generate/testdata/clusterserviceversions/output/memcached-operator-multiVersion.yaml

@@ -208,13 +208,23 @@ spec:
     url: https://your.domain
   version: 0.0.1
   webhookdefinitions:
+  - admissionReviewVersions: null
+    containerPort: 443
+    conversionCRDs:
+    - memcacheds.cache.example.com
+    deploymentName: memcached-operator-controller-manager
+    generateName: cmemcacheds.kb.io
+    sideEffects: None
+    targetPort: 9443
+    type: ConversionWebhook
+    webhookPath: /convert
   - admissionReviewVersions:
     - v1
     - v1beta1
     containerPort: 443
     deploymentName: memcached-operator-controller-manager
     failurePolicy: Fail
-    generateName: vmemcached.kb.io
+    generateName: mmemcached.kb.io
     rules:
     - apiGroups:
       - cache.example.com
@@ -227,15 +237,15 @@ spec:
       - memcacheds
     sideEffects: None
     targetPort: 9443
-    type: ValidatingAdmissionWebhook
-    webhookPath: /validate-cache-example-com-v1alpha1-memcached
+    type: MutatingAdmissionWebhook
+    webhookPath: /mutate-cache-example-com-v1alpha1-memcached
   - admissionReviewVersions:
     - v1
     - v1beta1
     containerPort: 443
     deploymentName: memcached-operator-controller-manager
     failurePolicy: Fail
-    generateName: mmemcached.kb.io
+    generateName: vmemcached.kb.io
     rules:
     - apiGroups:
       - cache.example.com
@@ -248,15 +258,5 @@ spec:
       - memcacheds
     sideEffects: None
     targetPort: 9443
-    type: MutatingAdmissionWebhook
-    webhookPath: /mutate-cache-example-com-v1alpha1-memcached
-  - admissionReviewVersions: null
-    containerPort: 443
-    conversionCRDs:
-    - memcacheds.cache.example.com
-    deploymentName: memcached-operator-controller-manager
-    generateName: cmemcacheds.kb.io
-    sideEffects: None
-    targetPort: 9443
-    type: ConversionWebhook
-    webhookPath: /convert
+    type: ValidatingAdmissionWebhook
+    webhookPath: /validate-cache-example-com-v1alpha1-memcached

internal/generate/testdata/clusterserviceversions/output/memcached-operator.clusterserviceversion.yaml

@@ -157,12 +157,23 @@ spec:
     type: AllNamespaces
   version: 0.0.1
   webhookdefinitions:
+  - admissionReviewVersions:
+    - v1beta1
+    containerPort: 443
+    conversionCRDs:
+    - memcacheds.cache.example.com
+    deploymentName: memcached-operator-controller-manager
+    generateName: cmemcacheds.kb.io
+    sideEffects: None
+    targetPort: 9443
+    type: ConversionWebhook
+    webhookPath: /convert
   - admissionReviewVersions:
     - v1beta1
     containerPort: 443
     deploymentName: memcached-operator-controller-manager
     failurePolicy: Fail
-    generateName: vmemcached.kb.io
+    generateName: mmemcached.kb.io
     rules:
     - apiGroups:
       - cache.example.com
@@ -175,14 +186,14 @@ spec:
       - memcacheds
     sideEffects: None
     targetPort: 9443
-    type: ValidatingAdmissionWebhook
-    webhookPath: /validate-cache-my-domain-v1alpha1-memcached
+    type: MutatingAdmissionWebhook
+    webhookPath: /mutate-cache-my-domain-v1alpha1-memcached
   - admissionReviewVersions:
     - v1beta1
     containerPort: 443
     deploymentName: memcached-operator-controller-manager
     failurePolicy: Fail
-    generateName: mmemcached.kb.io
+    generateName: vmemcached.kb.io
     rules:
     - apiGroups:
       - cache.example.com
@@ -195,16 +206,5 @@ spec:
       - memcacheds
     sideEffects: None
     targetPort: 9443
-    type: MutatingAdmissionWebhook
-    webhookPath: /mutate-cache-my-domain-v1alpha1-memcached
-  - admissionReviewVersions:
-    - v1beta1
-    containerPort: 443
-    conversionCRDs:
-    - memcacheds.cache.example.com
-    deploymentName: memcached-operator-controller-manager
-    generateName: cmemcacheds.kb.io
-    sideEffects: None
-    targetPort: 9443
-    type: ConversionWebhook
-    webhookPath: /convert
+    type: ValidatingAdmissionWebhook
+    webhookPath: /validate-cache-my-domain-v1alpha1-memcached

internal/generate/testdata/clusterserviceversions/output/with-ui-metadata.clusterserviceversion.yaml

@@ -213,12 +213,23 @@ spec:
     url: https://your.domain
   version: 0.0.1
   webhookdefinitions:
+  - admissionReviewVersions:
+    - v1beta1
+    containerPort: 443
+    conversionCRDs:
+    - memcacheds.cache.example.com
+    deploymentName: memcached-operator-controller-manager
+    generateName: cmemcacheds.kb.io
+    sideEffects: None
+    targetPort: 9443
+    type: ConversionWebhook
+    webhookPath: /convert
   - admissionReviewVersions:
     - v1beta1
     containerPort: 443
     deploymentName: memcached-operator-controller-manager
     failurePolicy: Fail
-    generateName: vmemcached.kb.io
+    generateName: mmemcached.kb.io
     rules:
     - apiGroups:
       - cache.example.com
@@ -231,14 +242,14 @@ spec:
       - memcacheds
     sideEffects: None
     targetPort: 9443
-    type: ValidatingAdmissionWebhook
-    webhookPath: /validate-cache-my-domain-v1alpha1-memcached
+    type: MutatingAdmissionWebhook
+    webhookPath: /mutate-cache-my-domain-v1alpha1-memcached
   - admissionReviewVersions:
     - v1beta1
     containerPort: 443
     deploymentName: memcached-operator-controller-manager
     failurePolicy: Fail
-    generateName: mmemcached.kb.io
+    generateName: vmemcached.kb.io
     rules:
     - apiGroups:
       - cache.example.com
@@ -251,16 +262,5 @@ spec:
       - memcacheds
     sideEffects: None
     targetPort: 9443
-    type: MutatingAdmissionWebhook
-    webhookPath: /mutate-cache-my-domain-v1alpha1-memcached
-  - admissionReviewVersions:
-    - v1beta1
-    containerPort: 443
-    conversionCRDs:
-    - memcacheds.cache.example.com
-    deploymentName: memcached-operator-controller-manager
-    generateName: cmemcacheds.kb.io
-    sideEffects: None
-    targetPort: 9443
-    type: ConversionWebhook
-    webhookPath: /convert
+    type: ValidatingAdmissionWebhook
+    webhookPath: /validate-cache-my-domain-v1alpha1-memcached

testdata/go/v2/memcached-operator/bundle/manifests/memcached-operator.clusterserviceversion.yaml

@@ -197,7 +197,7 @@ spec:
     containerPort: 443
     deploymentName: memcached-operator-controller-manager
     failurePolicy: Fail
-    generateName: vmemcached.kb.io
+    generateName: mmemcached.kb.io
     rules:
     - apiGroups:
       - cache.example.com
@@ -210,14 +210,14 @@ spec:
       - memcacheds
     sideEffects: None
     targetPort: 9443
-    type: ValidatingAdmissionWebhook
-    webhookPath: /validate-cache-example-com-v1alpha1-memcached
+    type: MutatingAdmissionWebhook
+    webhookPath: /mutate-cache-example-com-v1alpha1-memcached
   - admissionReviewVersions:
     - v1beta1
     containerPort: 443
     deploymentName: memcached-operator-controller-manager
     failurePolicy: Fail
-    generateName: mmemcached.kb.io
+    generateName: vmemcached.kb.io
     rules:
     - apiGroups:
       - cache.example.com
@@ -230,5 +230,5 @@ spec:
       - memcacheds
     sideEffects: None
     targetPort: 9443
-    type: MutatingAdmissionWebhook
-    webhookPath: /mutate-cache-example-com-v1alpha1-memcached
+    type: ValidatingAdmissionWebhook
+    webhookPath: /validate-cache-example-com-v1alpha1-memcached

testdata/go/v3/memcached-operator/bundle/manifests/memcached-operator.clusterserviceversion.yaml

@@ -222,7 +222,7 @@ spec:
     containerPort: 443
     deploymentName: memcached-operator-controller-manager
     failurePolicy: Fail
-    generateName: vmemcached.kb.io
+    generateName: mmemcached.kb.io
     rules:
     - apiGroups:
       - cache.example.com
@@ -235,15 +235,15 @@ spec:
       - memcacheds
     sideEffects: None
     targetPort: 9443
-    type: ValidatingAdmissionWebhook
-    webhookPath: /validate-cache-example-com-v1alpha1-memcached
+    type: MutatingAdmissionWebhook
+    webhookPath: /mutate-cache-example-com-v1alpha1-memcached
   - admissionReviewVersions:
     - v1
     - v1beta1
     containerPort: 443
     deploymentName: memcached-operator-controller-manager
     failurePolicy: Fail
-    generateName: mmemcached.kb.io
+    generateName: vmemcached.kb.io
     rules:
     - apiGroups:
       - cache.example.com
@@ -256,5 +256,5 @@ spec:
       - memcacheds
     sideEffects: None
     targetPort: 9443
-    type: MutatingAdmissionWebhook
-    webhookPath: /mutate-cache-example-com-v1alpha1-memcached
+    type: ValidatingAdmissionWebhook
+    webhookPath: /validate-cache-example-com-v1alpha1-memcached