add serde cache defaults

stackabletech · Feb 26, 2025 · f9f8c7e · f9f8c7e
1 parent 499d02c
commit f9f8c7e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/deploy/helm/superset-operator/crds/crds.yaml b/deploy/helm/superset-operator/crds/crds.yaml
@@ -82,6 +82,9 @@ spec:
                           description: Configure the OPA stacklet [discovery ConfigMap](https://docs.stackable.tech/home/nightly/concepts/service_discovery) and the name of the Rego package containing your authorization rules. Consult the [OPA authorization documentation](https://docs.stackable.tech/home/nightly/concepts/opa) to learn how to deploy Rego authorization rules with OPA.
                           properties:
                             cache:
+                              default:
+                                entryTimeToLive: 30s
+                                maxEntries: 10000
                               description: Configuration for an Superset internal cache for calls to OPA
                               properties:
                                 entryTimeToLive:
@@ -103,7 +106,6 @@ spec:
                               nullable: true
                               type: string
                           required:
-                            - cache
                             - configMapName
                           type: object
                       required:

diff --git a/rust/crd/src/lib.rs b/rust/crd/src/lib.rs
@@ -276,6 +276,7 @@ pub struct SupersetOpaRoleMappingConfig {
     pub opa: OpaConfig,
 
     /// Configuration for an Superset internal cache for calls to OPA
+    #[serde(default)]
     pub cache: UserInformationCache,
 }