Switch from disabled to enabled

chart-tests/application/ci/test-disabled-service-values.yaml

@@ -0,0 +1,6 @@
+service:
+  enabled: false
+readinessProbe:
+  enabled: false
+livenessProbe:
+  enabled: false

charts/application/README.md

@@ -24,14 +24,14 @@ Generic application chart with common requirements of a typical workload.
 | autoscaling.minReplicaCount | int | `1` |  |
 | autoscaling.maxReplicaCount | int | `1` |  |
 | autoscaling.averageUtilization.cpu | int | `80` |  |
-| livenessProbe.disabled | bool | `false` |  |
+| livenessProbe.enabled | bool | `true` |  |
 | livenessProbe.path | string | `"/.well-known/live"` |  |
 | livenessProbe.cmd | list | `[]` |  |
 | livenessProbe.initialDelaySeconds | int | `10` |  |
 | livenessProbe.periodSeconds | int | `10` |  |
 | livenessProbe.failureThreshold | int | `3` |  |
 | livenessProbe.timeoutSeconds | int | `5` |  |
-| readinessProbe.disabled | bool | `false` |  |
+| readinessProbe.enabled | bool | `true` |  |
 | readinessProbe.path | string | `"/.well-known/ready"` |  |
 | readinessProbe.cmd | list | `[]` |  |
 | readinessProbe.initialDelaySeconds | int | `10` |  |
@@ -51,7 +51,7 @@ Generic application chart with common requirements of a typical workload.
 | image.repositoryProvider | string | `"generic"` |  |
 | image.imageAutomationNamespace | string | `nil` |  |
 | image.pullSecrets | list | `[]` |  |
-| service.disabled | bool | `false` |  |
+| service.enabled | bool | `true` |  |
 | service.port | int | `80` |  |
 | service.timeout | string | `"120s"` |  |
 | service.annotations | object | `{}` |  |

charts/application/templates/_podTemplate.tpl

@@ -230,7 +230,7 @@ spec:
           containerPort: {{ $ap.containerPort }}
           protocol: {{ $ap.protocol }}
         {{- end }}
-      {{- if not .Values.livenessProbe.disabled }}
+      {{- if .Values.livenessProbe.enabled }}
       livenessProbe:
         {{- if .Values.livenessProbe.cmd }}
         exec:
@@ -248,7 +248,7 @@ spec:
         failureThreshold: {{ .Values.livenessProbe.failureThreshold }}
         timeoutSeconds: {{ .Values.livenessProbe.timeoutSeconds }}
       {{- end }}
-      {{- if not .Values.readinessProbe.disabled }}
+      {{- if .Values.readinessProbe.enabled }}
       readinessProbe:
         {{- if .Values.readinessProbe.cmd }}
         exec:

charts/application/templates/k8s-service.yaml

@@ -1,4 +1,4 @@
-{{- if and (not .Values.service.disabled) (not (and .Values.canary.enabled .Values.istio.enabled)) }}
+{{- if and .Values.service.enabled (not (and .Values.canary.enabled .Values.istio.enabled)) }}
 apiVersion: v1
 kind: Service
 metadata:

charts/application/values.yaml

@@ -21,8 +21,8 @@ autoscaling:
 
 # Check if the container is running
 livenessProbe:
-  # Disable the liveness probe
-  disabled: false
+  # Allows to disable the liveness probe
+  enabled: true
   path: /.well-known/live
   # Use a command to check liveness from inside the container, for example ['curl', '-f', 'http://localhost/health/live'].
   # This will take precedence over the given `path` value, if provided.
@@ -38,8 +38,8 @@ livenessProbe:
 
 # Check if the container are able to service requests
 readinessProbe:
-  # Disable the readiness probe
-  disabled: false
+  # Allows to disable the readiness probe
+  enabled: true
   path: /.well-known/ready
   # Use a command to check readiness from inside the container, for example ['curl', '-f', 'http://localhost/health/ready'].
   # This will take precedence over the given `path` value, if provided.
@@ -86,7 +86,7 @@ image:
   pullSecrets: []
 
 service:
-  disabled: false
+  enabled: true
   port: 80
   timeout: 120s
   annotations: {}