Split metrics/local service types in values (#47)

* Split metrics/local service types in values * Add external traffic policy option
Chia-Network · May 2, 2024 · 5fb5cd4 · 5fb5cd4
1 parent c938d6b
commit 5fb5cd4
Show file tree

Hide file tree

Showing 5 changed files with 13 additions and 3 deletions.
diff --git a/charts/chia-blockchain/Chart.yaml b/charts/chia-blockchain/Chart.yaml
@@ -15,4 +15,4 @@ type: application
 # This is the chart version. This version number should be incremented each time you make changes
 # to the chart and its templates, including the app version.
 # Versions are expected to follow Semantic Versioning (https://semver.org/)
-version: 0.6.1
+version: 0.7.0
diff --git a/charts/chia-blockchain/templates/service-local.yaml b/charts/chia-blockchain/templates/service-local.yaml
@@ -5,7 +5,7 @@ metadata:
   labels:
     {{- include "chia-blockchain.labels" . | nindent 4 }}
 spec:
-  type: {{ .Values.service.type }}
+  type: {{ .Values.localService.type }}
   internalTrafficPolicy: Local
   ports:
     - port: {{ .Values.chia.daemonPort }}

diff --git a/charts/chia-blockchain/templates/service-metrics.yaml b/charts/chia-blockchain/templates/service-metrics.yaml
@@ -9,7 +9,7 @@ metadata:
     {{- toYaml . | nindent 4 }}
     {{- end }}
 spec:
-  type: {{ .Values.service.type }}
+  type: {{ .Values.metricsService.type }}
   ports:
     - port: 9914
       targetPort: metrics

diff --git a/charts/chia-blockchain/templates/service.yaml b/charts/chia-blockchain/templates/service.yaml
@@ -6,6 +6,9 @@ metadata:
     {{- include "chia-blockchain.labels" . | nindent 4 }}
 spec:
   type: {{ .Values.service.type }}
+  {{- if .Values.service.externalTrafficPolicy }}
+  externalTrafficPolicy: {{ .Values.service.externalTrafficPolicy }}
+  {{- end }}
   ports:
     - port: {{ .Values.chia.daemonPort }}
       targetPort: daemon

diff --git a/charts/chia-blockchain/values.yaml b/charts/chia-blockchain/values.yaml
@@ -132,6 +132,13 @@ chiaExporter:
 
 service:
   type: ClusterIP
+  externalTrafficPolicy: ""
+
+localService:
+  type: ClusterIP
+
+metricsService:
+  type: ClusterIP
 
 resources: {}
   # We usually recommend not to specify default resources and to leave this as a conscious