fix: addressed first round of review feedback

Author: dervoeti

crates/stackable-operator/crds/DummyCluster.yaml

@@ -164,9 +164,55 @@ spec:
                 type: object
               hostName:
                 type: string
+              jsonConfigOverrides:
+                nullable: true
+                oneOf:
+                - required:
+                  - jsonMergePatch
+                - required:
+                  - jsonPatches
+                - required:
+                  - userProvided
+                properties:
+                  jsonMergePatch:
+                    description: |-
+                      Can be set to arbitrary YAML content, which is converted to JSON and used as
+                      [RFC 7396](https://datatracker.ietf.org/doc/html/rfc7396) JSON merge patch.
+                    x-kubernetes-preserve-unknown-fields: true
+                  jsonPatches:
+                    description: |-
+                      List of [RFC 6902](https://datatracker.ietf.org/doc/html/rfc6902) JSON patches.
+
+                      Can be used when more flexibility is needed, e.g. to only modify elements
+                      in a list based on a condition.
+
+                      A patch looks something like
+
+                      `{"op": "test", "path": "/0/name", "value": "Andrew"}`
+
+                      or
+
+                      `{"op": "add", "path": "/0/happy", "value": true}`
+                    items:
+                      type: string
+                    type: array
+                  userProvided:
+                    description: Override the entire config file with the specified String.
+                    type: string
+                type: object
               kerberosRealmName:
                 description: A validated kerberos realm name type, for use in CRDs.
                 type: string
+              keyValueConfigOverrides:
+                additionalProperties:
+                  type: string
+                description: |-
+                  Flat key-value overrides for `*.properties`, Hadoop XML, etc.
+
+                  This is backwards-compatible with the existing flat key-value YAML format
+                  used by `HashMap<String, String>`.
+                nullable: true
+                type: object
               nodes:
                 description: |-
                   This struct represents a role - e.g. HDFS datanodes or Trino workers. It has a key-value-map containing

crates/stackable-operator/src/config_overrides.rs

@@ -7,31 +7,11 @@
 
 use std::collections::{BTreeMap, HashMap};
 
-use schemars::{JsonSchema, Schema, json_schema};
+use schemars::JsonSchema;
 use serde::{Deserialize, Serialize};
 use snafu::{ResultExt, Snafu};
 
-/// Generates a JSON schema that accepts any JSON value.
-///
-/// Kubernetes CRDs do not support the `true` schema shorthand that
-/// `serde_json::Value` generates by default. Instead we emit a schema
-/// with `x-kubernetes-preserve-unknown-fields: true` which tells the
-/// API server to store the value as-is.
-fn arbitrary_json_value(_gen: &mut schemars::generate::SchemaGenerator) -> Schema {
-    json_schema!({
-        "x-kubernetes-preserve-unknown-fields": true,
-    })
-}
-
-/// Generates a JSON schema for a list of JSON patch operation strings (RFC 6902).
-fn json_patch_string_list(_gen: &mut schemars::generate::SchemaGenerator) -> Schema {
-    json_schema!({
-        "type": "array",
-        "items": {
-            "type": "string",
-        },
-    })
-}
+use crate::utils::crds::arbitrary_json_value;
 
 #[derive(Debug, Snafu)]
 pub enum Error {
@@ -122,7 +102,6 @@ pub enum JsonConfigOverrides {
     /// or
     ///
     /// `{"op": "add", "path": "/0/happy", "value": true}`
-    #[schemars(schema_with = "json_patch_string_list")]
     JsonPatches(Vec<String>),
 
     /// Override the entire config file with the specified String.

crates/stackable-operator/src/utils/crds.rs

@@ -15,6 +15,18 @@ pub fn raw_optional_object_schema(_: &mut schemars::generate::SchemaGenerator) -
     })
 }
 
+/// Generates a JSON schema that accepts any JSON value.
+///
+/// Kubernetes CRDs do not support the `true` schema shorthand that
+/// `serde_json::Value` generates by default. Instead we emit a schema
+/// with `x-kubernetes-preserve-unknown-fields: true` which tells the
+/// API server to store the value as-is.
+pub fn arbitrary_json_value(_: &mut schemars::generate::SchemaGenerator) -> Schema {
+    json_schema!({
+        "x-kubernetes-preserve-unknown-fields": true,
+    })
+}
+
 pub fn raw_object_list_schema(_: &mut schemars::generate::SchemaGenerator) -> Schema {
     json_schema!({
         "type": "array",

crates/xtask/src/crd/dummy.rs

@@ -54,6 +54,10 @@ pub mod versioned {
         #[serde(default)]
         pub object_overrides: ObjectOverrides,
 
+        json_config_overrides: Option<stackable_operator::config_overrides::JsonConfigOverrides>,
+        key_value_config_overrides:
+            Option<stackable_operator::config_overrides::KeyValueConfigOverrides>,
+
         // Already versioned
         client_authentication_details:
             stackable_operator::crd::authentication::core::v1alpha1::ClientAuthenticationDetails,