restore the default of disabling license key checks for now

but allow it to be overriden for customers who want to use license keys

Author: doy-materialize

misc/helm-charts/operator/templates/deployment.yaml

@@ -57,6 +57,9 @@ spec:
         {{- if .Values.operator.args.enableInternalStatementLogging }}
         - "--enable-internal-statement-logging"
         {{- end }}
+        {{- if not .Values.operator.args.enableLicenseKeyChecks }}
+        - "--disable-license-key-checks"
+        {{- end }}
 
         {{/* AWS Configuration */}}
         {{- if eq .Values.operator.cloudProvider.type "aws" }}

misc/helm-charts/operator/tests/deployment_test.yaml

@@ -64,21 +64,21 @@ tests:
     storage.storageClass.name: ""
   asserts:
   - matchRegex:
-      path: spec.template.spec.containers[0].args[12]      # Index of the environmentd-cluster-replica-sizes argument
+      path: spec.template.spec.containers[0].args[13]      # Index of the environmentd-cluster-replica-sizes argument
       pattern: disk_limit":"0"
   - matchRegex:
-      path: spec.template.spec.containers[0].args[12]
+      path: spec.template.spec.containers[0].args[13]
       pattern: is_cc":true
 
 - it: should have a cluster with disk limit to 1552MiB when storage class is configured
   set:
     storage.storageClass.name: "my-storage-class"
   asserts:
   - matchRegex:
-      path: spec.template.spec.containers[0].args[12]
+      path: spec.template.spec.containers[0].args[13]
       pattern: disk_limit":"1552MiB"
   - matchRegex:
-      path: spec.template.spec.containers[0].args[12]
+      path: spec.template.spec.containers[0].args[13]
       pattern: is_cc":true
 
 - it: should configure for AWS provider correctly