Add missing crd defaults (#588)

# Description *Please add a description here. This will become the commit message of the merge request later.* Co-authored-by: Malte Sander <[email protected]>
stackabletech · May 12, 2023 · 6065e79 · 6065e79
1 parent 3821770
commit 6065e79
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -8,6 +8,7 @@ All notable changes to this project will be documented in this file.
 
 - Generate OLM bundle for Release 23.4.0 ([#585]).
 - Fixed upgrade test on Openshift ([#585]).
+- Missing CRD defaults for `status.conditions` field ([#588]).
 
 ### Changed
 
@@ -18,6 +19,7 @@ All notable changes to this project will be documented in this file.
 [#583]: https://github.com/stackabletech/kafka-operator/pull/583
 [#584]: https://github.com/stackabletech/kafka-operator/pull/584
 [#585]: https://github.com/stackabletech/kafka-operator/pull/585
+[#588]: https://github.com/stackabletech/kafka-operator/pull/588
 
 ## [23.4.0] - 2023-04-17
 

diff --git a/deploy/helm/kafka-operator/crds/crds.yaml b/deploy/helm/kafka-operator/crds/crds.yaml
@@ -1515,6 +1515,7 @@ spec:
               nullable: true
               properties:
                 conditions:
+                  default: []
                   items:
                     properties:
                       lastTransitionTime:
@@ -1556,8 +1557,6 @@ spec:
                       - type
                     type: object
                   type: array
-              required:
-                - conditions
               type: object
           required:
             - spec

diff --git a/rust/crd/src/lib.rs b/rust/crd/src/lib.rs
@@ -404,6 +404,7 @@ impl Configuration for KafkaConfigFragment {
 #[derive(Clone, Default, Debug, Deserialize, Eq, JsonSchema, PartialEq, Serialize)]
 #[serde(rename_all = "camelCase")]
 pub struct KafkaClusterStatus {
+    #[serde(default)]
     pub conditions: Vec<ClusterCondition>,
 }