feat: expose metrics port

Azure · Apr 18, 2023 · 14d7c97 · 14d7c97
1 parent 5f5049e
commit 14d7c97
Show file tree

Hide file tree

Showing 5 changed files with 21 additions and 2 deletions.
diff --git a/...g/charts/csi-secrets-store-provider-azure/templates/provider-azure-installer-windows.yaml b/...g/charts/csi-secrets-store-provider-azure/templates/provider-azure-installer-windows.yaml
@@ -25,7 +25,7 @@ spec:
 {{- if .Values.enableArcExtension }}
 {{- if .Values.arc.enableMonitoring }}
         prometheus.io/scrape: "true"
-        prometheus.io/port: "8898"
+        prometheus.io/port: "{{ .Values.providerMetricsAddr }}"
 {{- end }}
 {{- end }}
 {{- end }}
@@ -76,6 +76,10 @@ spec:
                 name: arc-proxy-config
           {{- end }}
           {{- end }}
+          ports:
+            - containerPort: {{ .Values.providerMetricsAddr }}
+              name: metrics
+              protocol: TCP          
           volumeMounts:
             - name: provider-vol
               mountPath: "C:\\provider"

diff --git a/...t_staging/charts/csi-secrets-store-provider-azure/templates/provider-azure-installer.yaml b/...t_staging/charts/csi-secrets-store-provider-azure/templates/provider-azure-installer.yaml
@@ -25,7 +25,7 @@ spec:
 {{- if .Values.enableArcExtension }}
 {{- if .Values.arc.enableMonitoring }}
         prometheus.io/scrape: "true"
-        prometheus.io/port: "8898"
+        prometheus.io/port: "{{ .Values.providerMetricsAddr }}"
 {{- end }}
 {{- end }}
 # this will ensure pods are rolled in next helm upgrade if cert files are updated.
@@ -76,6 +76,10 @@ spec:
             periodSeconds: 30
           resources:
 {{ toYaml .Values.linux.resources | indent 12 }}
+          ports:
+            - containerPort: {{ .Values.providerMetricsAddr }}
+              name: metrics
+              protocol: TCP
           securityContext:
           {{- if not .Values.linux.privileged }}
             allowPrivilegeEscalation: false

diff --git a/manifest_staging/charts/csi-secrets-store-provider-azure/values.yaml b/manifest_staging/charts/csi-secrets-store-provider-azure/values.yaml
@@ -176,3 +176,6 @@ constructPEMChain: true
 
 # Write cert and key in separate files. The individual files will be named as <secret-name>.crt and <secret-name>.key. These files will be created in addition to the single file.
 writeCertAndKeyInSeparateFiles: false
+
+# Port that servers metrics
+providerMetricsAddr: "8898"
diff --git a/manifest_staging/deployment/provider-azure-installer-windows.yaml b/manifest_staging/deployment/provider-azure-installer-windows.yaml
@@ -46,6 +46,10 @@ spec:
             limits:
               cpu: 100m
               memory: 200Mi
+          ports:
+            - containerPort: {{ .Values.providerMetricsAddr }}
+              name: metrics
+              protocol: TCP                 
           volumeMounts:
             - mountPath: "C:\\provider"
               name: providervol

diff --git a/manifest_staging/deployment/provider-azure-installer.yaml b/manifest_staging/deployment/provider-azure-installer.yaml
@@ -47,6 +47,10 @@ spec:
             limits:
               cpu: 50m
               memory: 100Mi
+          ports:
+            - containerPort: {{ .Values.providerMetricsAddr }}
+              name: metrics
+              protocol: TCP                 
           securityContext:
             allowPrivilegeEscalation: false
             readOnlyRootFilesystem: true