Fix Helm values names

Author: marcleblanc2

charts/sourcegraph/templates/otel-collector/otel-agent.DaemonSet.yaml

@@ -71,13 +71,13 @@ spec:
         ports:
           - name: grpc-otlp
             containerPort: 4317
-            hostPort: {{ toYaml .Values.openTelemetry.agent.hostPorts.grpc-otlp }}
+            hostPort: {{ toYaml .Values.openTelemetry.agent.hostPorts.grpcOtlp }}
           - name: http-otlp
             containerPort: 4318
-            hostPort: {{ toYaml .Values.openTelemetry.agent.hostPorts.http-otlp }}
+            hostPort: {{ toYaml .Values.openTelemetry.agent.hostPorts.httpOtlp }}
           - name: http-zpages
             containerPort: 55679
-            hostPort: {{ toYaml .Values.openTelemetry.agent.hostPorts.http-zpages }}
+            hostPort: {{ toYaml .Values.openTelemetry.agent.hostPorts.httpZpages }}
         volumeMounts:
           - name: config
             mountPath: /etc/otel-agent

charts/sourcegraph/values.yaml

@@ -85,7 +85,8 @@ sourcegraph:
 #   # Toggle deployment of applications on/off. Applies to databases and third-party applications
 #   enabled: true
 
-alpine: # Used in init containers
+alpine:
+  # Used in init containers
   image:
     # -- Docker image tag for the `alpine` image
     defaultTag: 6.9.0@sha256:47ddeb68cfdb767949b4e7f6e2306fa20a1ee5a2619efbe56bf06cb5a62e7729
@@ -167,7 +168,7 @@ codeInsightsDB:
     # -- Sets codeinsights-db port
     port: "5432"
     # -- Sets codeinsights-db SSL mode
-    sslmode: "disable"  # set to "require" to enable SSL
+    sslmode: "disable" # set to "require" to enable SSL
   # -- Environment variables for the `codeinsights-db` container
   env: {}
   # -- Name of existing ConfigMap for `codeinsights-db`. It must contain a `postgresql.conf` key.
@@ -242,7 +243,7 @@ codeIntelDB:
     # -- Sets codeintel-db port
     port: "5432"
     # -- Sets codeintel-db SSL mode
-    sslmode: "disable"  # set to "require" to enable SSL
+    sslmode: "disable" # set to "require" to enable SSL
   # -- Name of existing ConfigMap for `codeintel-db`. It must contain a `postgresql.conf` key
   existingConfig: ""
   # -- Additional PostgreSQL configuration. This will override or extend our default configuration.
@@ -643,9 +644,9 @@ openTelemetry:
     # -- Resource requests & limits for the `otel-agent` container,
     # learn more from the [Kubernetes documentation](https://kubernetes.io/docs/concepts/configuration/manage-resources-containers/)
     hostPorts:
-      grpc-otlp: 4317
-      http-otlp: 4318
-      http-zpages: 55679
+      grpcOtlp: 4317
+      httpOtlp: 4318
+      httpZpages: 55679
     resources:
       limits:
         cpu: "500m"
@@ -730,9 +731,9 @@ pgsql:
     # -- Sets postgres port
     port: "5432"
     # -- Sets postgres SSL mode
-    sslmode: "disable"  # set to "require" to enable SSL
+    sslmode: "disable" # set to "require" to enable SSL
   # -- Name of existing ConfigMap for `pgsql`. It must contain a `postgresql.conf` key
-  existingConfig: ""  # Name of an existing configmap
+  existingConfig: "" # Name of an existing configmap
   # -- Additional PostgreSQL configuration. This will override or extend our default configuration.
   # Notes: This is expecting a multiline string.
   # Learn more from our [recommended PostgreSQL configuration](https://docs.sourcegraph.com/admin/config/postgres-conf) and [PostgreSQL documentation](https://www.postgresql.org/docs/12/config-setting.html)
@@ -1159,14 +1160,7 @@ jaeger:
   # -- Name used by resources. Does not affect service names or PVCs.
   name: "jaeger"
   # -- Default args passed to the `jaeger` binary
-  args:
-    [
-      "--memory.max-traces=20000",
-      "--sampling.strategies-file=/etc/jaeger/sampling_strategies.json",
-      "--collector.otlp.enabled",
-      "--collector.otlp.grpc.host-port=:4320",
-      "--collector.otlp.http.host-port=:4321",
-    ]
+  args: [ "--memory.max-traces=20000", "--sampling.strategies-file=/etc/jaeger/sampling_strategies.json", "--collector.otlp.enabled", "--collector.otlp.grpc.host-port=:4320", "--collector.otlp.http.host-port=:4321" ]
   # -- Security context for the `jaeger` container,
   # learn more from the [Kubernetes documentation](https://kubernetes.io/docs/tasks/configure-pod-container/security-context/#set-the-security-context-for-a-container)
   containerSecurityContext:
@@ -1233,8 +1227,7 @@ worker:
   # -- Scale worker horizontally by configuring additional replicas dedicated to specific jobs.
   # for each replica, configure the dedicated jobs to run on this replica.
   # learn more from https://sourcegraph.com/docs/admin/workers#3-split-jobs-and-scale-independently
-  replicas:
-    []
+  replicas: []
     # - jobs: []
     #   resources:
     #     limits: