Merge pull request #85 from codecademy-engineering/DEVOPS-5022-option…

…al-service-object [DEVOPS-5022] Optional service object
codecademy-engineering · Dec 5, 2022 · 869f221 · 869f221
2 parents 79aa6e8 + 54a430d
commit 869f221
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 1 deletion.
diff --git a/charts/service/Chart.yaml b/charts/service/Chart.yaml
@@ -2,7 +2,7 @@ apiVersion: v2
 name: service
 description: A generic k8s service chart
 type: application
-version: 1.4.0
+version: 1.5.0
 maintainers:
   - email: [email protected]
     name: devops
diff --git a/charts/service/ci/custom-values.yaml b/charts/service/ci/custom-values.yaml
@@ -21,6 +21,9 @@ image:
     - '-g'
     - 'daemon off;'
 
+service:
+  enabled: false
+
 ingress:
   enabled: true
   annotations:

diff --git a/charts/service/templates/service.yaml b/charts/service/templates/service.yaml
@@ -1,3 +1,4 @@
+{{- if .Values.service.enabled -}}
 apiVersion: v1
 kind: Service
 metadata:
@@ -18,3 +19,4 @@ spec:
     {{- end }}
   selector:
     {{- include "service.selectorLabels" . | nindent 4 }}
+{{- end }}
diff --git a/charts/service/values.yaml b/charts/service/values.yaml
@@ -67,6 +67,7 @@ volumes: {}
 # Defaults to port 80 on service resources
 # DO NOT CHANGE, unless you absolutely need to
 service:
+  enabled: true
   type: ClusterIP
   port: 80