fix(crd-generator): Sort required properties to ensure deterministic …

…output
fabric8io · Apr 25, 2024 · ac2960d · ac2960d
1 parent f6c56d5
commit ac2960d
Show file tree

Hide file tree

Showing 3 changed files with 19 additions and 15 deletions.
diff --git a/crd-generator/api/src/main/java/io/fabric8/crd/generator/AbstractJsonSchema.java b/crd-generator/api/src/main/java/io/fabric8/crd/generator/AbstractJsonSchema.java
@@ -400,7 +400,11 @@ private T internalFromImpl(TypeDef definition, LinkedHashMap<String, String> vis
         .collect(Collectors.toList());
 
     swaps.throwIfUnmatchedSwaps();
-    return build(builder, required, validationRules, preserveUnknownFields);
+
+    List<String> sortedRequiredProperties = required.stream().sorted()
+      .collect(Collectors.toList());
+
+    return build(builder, sortedRequiredProperties, validationRules, preserveUnknownFields);
   }
 
   private Map<String, Method> indexPotentialAccessors(TypeDef definition) {

diff --git a/crd-generator/api/src/test/resources/k8svalidations.samples.fabric8.io-v1.yml b/crd-generator/api/src/test/resources/k8svalidations.samples.fabric8.io-v1.yml
@@ -123,18 +123,18 @@ spec:
                 x-kubernetes-validations:
                 - rule: self.startsWith('simple-')
             required:
-            - minReplicas
-            - onAttributeAndGetter
-            - replicas
             - deepLevel1
-            - onAttributeAndClass
-            - simple
+            - maxReplicas
+            - minReplicas
             - multiple
+            - namePrefix
             - onAbstractClass
+            - onAttributeAndClass
+            - onAttributeAndGetter
             - onGetter
-            - maxReplicas
             - priority
-            - namePrefix
+            - replicas
+            - simple
             type: object
             x-kubernetes-validations:
             - fieldPath: .replicas

diff --git a/crd-generator/api/src/test/resources/k8svalidations.samples.fabric8.io-v1beta1.yml b/crd-generator/api/src/test/resources/k8svalidations.samples.fabric8.io-v1beta1.yml
@@ -123,18 +123,18 @@ spec:
               x-kubernetes-validations:
               - rule: self.startsWith('simple-')
           required:
-          - minReplicas
-          - onAttributeAndGetter
-          - replicas
           - deepLevel1
-          - onAttributeAndClass
-          - simple
+          - maxReplicas
+          - minReplicas
           - multiple
+          - namePrefix
           - onAbstractClass
+          - onAttributeAndClass
+          - onAttributeAndGetter
           - onGetter
-          - maxReplicas
           - priority
-          - namePrefix
+          - replicas
+          - simple
           type: object
           x-kubernetes-validations:
           - fieldPath: .replicas