From cfbfd92c7bfbc39f69be3ed68a4f9c6e564a60b8 Mon Sep 17 00:00:00 2001
From: jvoravong <jvoravong@splunk.com>
Date: Wed, 17 Apr 2024 12:22:14 -0600
Subject: [PATCH] temp: add simple feature gate fix

---
 .../rendered_manifests/operator/deployment.yaml |  1 +
 helm-charts/splunk-otel-collector/values.yaml   | 17 +++++++++++------
 2 files changed, 12 insertions(+), 6 deletions(-)

diff --git a/examples/enable-operator-and-auto-instrumentation/rendered_manifests/operator/deployment.yaml b/examples/enable-operator-and-auto-instrumentation/rendered_manifests/operator/deployment.yaml
index a4171e9512..01e470eb66 100644
--- a/examples/enable-operator-and-auto-instrumentation/rendered_manifests/operator/deployment.yaml
+++ b/examples/enable-operator-and-auto-instrumentation/rendered_manifests/operator/deployment.yaml
@@ -34,6 +34,7 @@ spec:
             - --health-probe-addr=:8081
             - --webhook-port=9443
             - --collector-image=otel/opentelemetry-collector-contrib:0.95.0
+            - --feature-gates=-operator.autoinstrumentation.apache-httpd,-operator.autoinstrumentation.go,-operator.autoinstrumentation.nginx,-operator.autoinstrumentation.python
           command:
             - /manager
           env:
diff --git a/helm-charts/splunk-otel-collector/values.yaml b/helm-charts/splunk-otel-collector/values.yaml
index a8306ab67b..4dbf903610 100644
--- a/helm-charts/splunk-otel-collector/values.yaml
+++ b/helm-charts/splunk-otel-collector/values.yaml
@@ -1194,6 +1194,7 @@ operator:
       #   - repository: Specifies the Docker image repository.
       #   - tag: Indicates the Docker image tag.
       #   - env: (Optional) Allows you to add any additional environment variables.
+      # Auto-instrumentation for the various languages that are distributed by Splunk are enabled by default.
       java:
         repository: ghcr.io/signalfx/splunk-otel-java/splunk-otel-java
         tag: v1.31.0
@@ -1206,21 +1207,25 @@ operator:
         env:
           - name: OTEL_DOTNET_AUTO_PLUGINS
             value: Splunk.OpenTelemetry.AutoInstrumentation.Plugin, Splunk.OpenTelemetry.AutoInstrumentation
-      # Instrumentation for go is disabled by default, enable it with `operator.manager.featureGates=+operator.autoinstrumentation.go”`
+      # Auto-instrumentation for the various languages distributed by OpenTelemetry natively (below) are disabled by default.
+      # Enable any language by changing its corresponding feature gate from "-" to "+" under "operator.manager.featureGates".
+      apache-httpd:
+        repository: ghcr.io/open-telemetry/opentelemetry-operator/autoinstrumentation-apache-httpd
+        tag: 1.0.4
       go:
         repository: ghcr.io/open-telemetry/opentelemetry-operator/autoinstrumentation-go
         tag: v0.10.1-alpha
-      apache-httpd:
+      nginx:
         repository: ghcr.io/open-telemetry/opentelemetry-operator/autoinstrumentation-apache-httpd
         tag: 1.0.4
       python:
         repository: ghcr.io/open-telemetry/opentelemetry-operator/autoinstrumentation-python
         tag: 0.44b0
-      # Instrumentation for nginx is disabled by default, enable it with `operator.manager.featureGates=+operator.autoinstrumentation.nginx”`
-      nginx:
-        repository: ghcr.io/open-telemetry/opentelemetry-operator/autoinstrumentation-apache-httpd
-        tag: 1.0.4
       # Auto-instrumentation Libraries (End)
+  manager:
+    # Configure feature gates here. Prefix with "+" to enable or "-" to disable a feature.
+    # TODO: See if there is a better way to set feature gates per language
+    featureGates: "-operator.autoinstrumentation.apache-httpd,-operator.autoinstrumentation.go,-operator.autoinstrumentation.nginx,-operator.autoinstrumentation.python"
   admissionWebhooks:
     certManager:
       # Annotate the certificate and issuer to ensure they are created after the cert-manager CRDs have been installed.