Cleanup helm chart options for flytepropeller and prometheus

Signed-off-by: Jason Parraga <[email protected]>
flyteorg · Jul 9, 2024 · 87fb65f · 87fb65f
1 parent b7e6959
commit 87fb65f
Show file tree

Hide file tree

Showing 8 changed files with 12 additions and 25 deletions.
diff --git a/charts/flyte-core/README.md b/charts/flyte-core/README.md
@@ -246,8 +246,6 @@ helm install gateway bitnami/contour -n flyte
 | flytepropeller.podLabels | object | `{}` | Labels for Flytepropeller pods |
 | flytepropeller.priorityClassName | string | `""` | Sets priorityClassName for propeller pod(s). |
 | flytepropeller.prometheus.enabled | bool | `false` |  |
-| flytepropeller.prometheus.path | string | `"/metrics"` |  |
-| flytepropeller.prometheus.port | int | `10254` |  |
 | flytepropeller.replicaCount | int | `1` | Replicas count for Flytepropeller deployment |
 | flytepropeller.resources | object | `{"limits":{"cpu":"200m","ephemeral-storage":"100Mi","memory":"200Mi"},"requests":{"cpu":"10m","ephemeral-storage":"50Mi","memory":"100Mi"}}` | Default resources requests and limits for Flytepropeller deployment |
 | flytepropeller.securityContext | object | `{"fsGroup":65534,"fsGroupChangePolicy":"Always","runAsUser":1001}` | Sets securityContext for flytepropeller pod(s). |
@@ -323,8 +321,6 @@ helm install gateway bitnami/contour -n flyte
 | webhook.podLabels | object | `{}` | Labels for webhook pods |
 | webhook.priorityClassName | string | `""` | Sets priorityClassName for webhook pod |
 | webhook.prometheus.enabled | bool | `false` |  |
-| webhook.prometheus.path | string | `"/metrics"` |  |
-| webhook.prometheus.port | int | `10254` |  |
 | webhook.resources.requests.cpu | string | `"200m"` |  |
 | webhook.resources.requests.ephemeral-storage | string | `"500Mi"` |  |
 | webhook.resources.requests.memory | string | `"500Mi"` |  |

diff --git a/charts/flyte-core/templates/propeller/deployment.yaml b/charts/flyte-core/templates/propeller/deployment.yaml
@@ -25,8 +25,8 @@ spec:
         {{- with .Values.flytepropeller.podAnnotations }}
         {{- toYaml . | nindent 8 }}
         {{- end }}
-        prometheus.io/path: {{ .Values.flytepropeller.prometheus.path | quote }}
-        prometheus.io/port: {{ .Values.flytepropeller.prometheus.port | quote }}
+        prometheus.io/path: "/metrics"
+        prometheus.io/port: {{ index .Values.configmap.core.propeller "prof-port" | quote }}
         {{- with .Values.flytepropeller.prometheus.enabled }}
         prometheus.io/scrape: "true"
         {{- end }}
@@ -83,11 +83,6 @@ spec:
         {{- end }}
         ports:
         - containerPort: {{ index .Values.configmap.core.propeller "prof-port" }}
-        {{- if .Values.flytepropeller.prometheus.enabled }}
-        - containerPort: {{ .Values.flytepropeller.prometheus.port }}
-          name: debug
-          protocol: TCP
-        {{- end }}
         resources: {{- toYaml .Values.flytepropeller.resources | nindent 10 }}
         volumeMounts:
         - name: config-volume

diff --git a/charts/flyte-core/templates/propeller/service.yaml b/charts/flyte-core/templates/propeller/service.yaml
@@ -10,7 +10,7 @@ spec:
   ports:
     - name: http-metrics
       protocol: TCP
-      port: 10254
+      port: {{ index .Values.configmap.core.propeller "prof-port" }}
     {{- with .Values.flytepropeller.service.additionalPorts -}}
     {{ tpl (toYaml .) $ | nindent 4 }}
     {{- end }}

diff --git a/charts/flyte-core/templates/propeller/webhook.yaml b/charts/flyte-core/templates/propeller/webhook.yaml
@@ -34,8 +34,8 @@ spec:
         {{- with .Values.webhook.podAnnotations }}
         {{- toYaml . | nindent 8 }}
         {{- end }}
-        prometheus.io/path: {{ .Values.webhook.prometheus.path | quote }}
-        prometheus.io/port: {{ .Values.webhook.prometheus.port | quote }}
+        prometheus.io/path: "/metrics"
+        prometheus.io/port: {{ index .Values.configmap.core.propeller "prof-port" | quote }}
         {{- with .Values.webhook.prometheus.enabled }}
         prometheus.io/scrape: "true"
         {{- end }}
@@ -108,7 +108,7 @@ spec:
           ports:
           - containerPort: 9443
           {{- if .Values.webhook.prometheus.enabled }}
-          - containerPort: {{ .Values.webhook.prometheus.port }}
+          - containerPort: {{ index .Values.configmap.core.propeller "prof-port" }}
             name: debug
             protocol: TCP
           {{- end }}

diff --git a/charts/flyte-core/values.yaml b/charts/flyte-core/values.yaml
@@ -386,8 +386,6 @@ flytepropeller:
 
   prometheus:
     enabled: false
-    path: "/metrics"
-    port: 10254
 
 #
 # FLYTECONSOLE SETTINGS
@@ -541,8 +539,6 @@ webhook:
 
   prometheus:
     enabled: false
-    path: "/metrics"
-    port: 10254
 
 # ------------------------------------------------
 #

diff --git a/docker/sandbox-bundled/manifests/complete-agent.yaml b/docker/sandbox-bundled/manifests/complete-agent.yaml
@@ -816,7 +816,7 @@ type: Opaque
 ---
 apiVersion: v1
 data:
-  haSharedSecret: VmlJYW1VZUVnRmdPRlpBMg==
+  haSharedSecret: T1d2RGxNWExCeWJkbnprRg==
   proxyPassword: ""
   proxyUsername: ""
 kind: Secret
@@ -1413,7 +1413,7 @@ spec:
     metadata:
       annotations:
         checksum/config: 8f50e768255a87f078ba8b9879a0c174c3e045ffb46ac8723d2eedbe293c8d81
-        checksum/secret: 7b3511fcf4990e8af9648ce0e2a906ec48f528c2f284b412f12d74fc4b656700
+        checksum/secret: 5d62986957bf66295a76c79d0a92a13ca161af9d459c260559092423a2776cb7
       labels:
         app: docker-registry
         release: flyte-sandbox

diff --git a/docker/sandbox-bundled/manifests/complete.yaml b/docker/sandbox-bundled/manifests/complete.yaml
@@ -796,7 +796,7 @@ type: Opaque
 ---
 apiVersion: v1
 data:
-  haSharedSecret: ZFpwYnNuWEVvWDlDSk9pSA==
+  haSharedSecret: M1N4cVljNDg4TWwwWHZpUQ==
   proxyPassword: ""
   proxyUsername: ""
 kind: Secret
@@ -1360,7 +1360,7 @@ spec:
     metadata:
       annotations:
         checksum/config: 8f50e768255a87f078ba8b9879a0c174c3e045ffb46ac8723d2eedbe293c8d81
-        checksum/secret: 0ded6dcac5c615b4ba549828986d3503b2d57fe98d120b17347047a62397f1d4
+        checksum/secret: d1729f091a14714c4346e39265ead043cd94b80a0f3c2896580c542856c90378
       labels:
         app: docker-registry
         release: flyte-sandbox

diff --git a/docker/sandbox-bundled/manifests/dev.yaml b/docker/sandbox-bundled/manifests/dev.yaml
@@ -499,7 +499,7 @@ metadata:
 ---
 apiVersion: v1
 data:
-  haSharedSecret: ZXkzazVaelQ5RllQR3BUUg==
+  haSharedSecret: WkZzZllBMjhCTTVOdU1rYQ==
   proxyPassword: ""
   proxyUsername: ""
 kind: Secret
@@ -934,7 +934,7 @@ spec:
     metadata:
       annotations:
         checksum/config: 8f50e768255a87f078ba8b9879a0c174c3e045ffb46ac8723d2eedbe293c8d81
-        checksum/secret: 6dc14ff0b0570f1d991e696b23fb2bdc5c2e0013af403ba73f35b63c20936650
+        checksum/secret: 1eb092c356fc247e78a38b60b8b2949a490151ac44b9e63894fcffc7378d36eb
       labels:
         app: docker-registry
         release: flyte-sandbox