disable installing CRDs by default

signalfx · Jan 10, 2025 · fd5dc03 · fd5dc03
1 parent 7e467b4
commit fd5dc03
Show file tree

Hide file tree

Showing 6 changed files with 13 additions and 10 deletions.
diff --git a/functional_tests/testdata/values/aks_test_values.yaml.tmpl b/functional_tests/testdata/values/aks_test_values.yaml.tmpl
@@ -67,7 +67,8 @@ readinessProbe:
   initialDelaySeconds: 60
 livenessProbe:
   initialDelaySeconds: 60
-
+operatorcrds:
+  install: true
 operator:
   enabled: true
   admissionWebhooks:

diff --git a/functional_tests/testdata/values/autopilot_test_values.yaml.tmpl b/functional_tests/testdata/values/autopilot_test_values.yaml.tmpl
@@ -59,7 +59,8 @@ clusterName: test
 environment: test
 cloudProvider: gcp
 distribution: gke/autopilot
-
+operatorcrds:
+  install: true
 operator:
   enabled: true
   admissionWebhooks:

diff --git a/functional_tests/testdata/values/test_values.yaml.tmpl b/functional_tests/testdata/values/test_values.yaml.tmpl
@@ -69,6 +69,8 @@ extraAttributes:
      value: "customvalue2"
 
 environment: dev
+operatorcrds:
+  install: true
 operator:
   enabled: true
 targetAllocator:

diff --git a/helm-charts/splunk-otel-collector/Chart.yaml b/helm-charts/splunk-otel-collector/Chart.yaml
@@ -28,9 +28,8 @@ dependencies:
     repository: https://charts.jetstack.io
     condition: certmanager.enabled
   - name: opentelemetry-operator-crds
-    version: "0.0.0"
+    version: "0.0.1"
     alias: operatorcrds
-    # TODO: See if we can improve this conditional more
     condition: operatorcrds.install
   - name: opentelemetry-operator
     version: 0.71.2

diff --git a/helm-charts/splunk-otel-collector/charts/opentelemetry-operator-crds/Chart.yaml b/helm-charts/splunk-otel-collector/charts/opentelemetry-operator-crds/Chart.yaml
@@ -1,3 +1,3 @@
 apiVersion: v2
 name: opentelemetry-operator-crds
-version: 0.0.0
+version: 0.0.1
diff --git a/helm-charts/splunk-otel-collector/values.yaml b/helm-charts/splunk-otel-collector/values.yaml
@@ -1174,6 +1174,11 @@ service:
 # Full list of Helm value configurations: https://artifacthub.io/packages/helm/opentelemetry-helm/opentelemetry-operator?modal=values
 ################################################################################
 
+# Should the CRDs be installed by a chart's crd/ directory or by using templates.
+operatorcrds:
+  # Should the CRDs be installed
+  install: false
+
 operator:
   enabled: false
   # This is disabled by default in favor of using `operatorcrds.install=true`, as doing so creates
@@ -1200,11 +1205,6 @@ operator:
     collectorImage:
       repository: quay.io/signalfx/splunk-otel-collector
 
-# Should the CRDs be installed by a chart's crd/ directory or by using templates.
-operatorcrds:
-  # Should the CRDs be installed
-  install: true
-
 # The default Splunk Instrumentation object deployed when operator.enabled=true.
 # For more details see:
 # - Splunk Documentation: https://docs.splunk.com/observability/en/gdi/opentelemetry/automatic-discovery/k8s/k8s-backend.html#optional-configure-the-instrumentation