Merge pull request #26 from truenas/NAS-108986

Deployments only support "Always" Restart Policy

test/ix-chart/2012.0.0/Chart.yaml → test/ix-chart/2101.0.0/Chart.yaml

@@ -15,7 +15,7 @@ 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: 2012.0.0
+version: 2101.0.0
 
 # This is the version number of the application being deployed. This version number should be
 # incremented each time you make changes to the application. Versions are not expected to

test/ix-chart/2012.0.0/README.md → test/ix-chart/2101.0.0/README.md

@@ -22,7 +22,6 @@ their default values.
 | `image.tag`                    | The image tag to pull from                                                                                      | `latest`                                                             |
 | `image.pullPolicy`             | Image pull policy                                                                                               | `IfNotPresent`                                                       |
 | `updateStrategy`               | Upgrade Policy                                                                                                  | `RollingUpdate`                                                      |
-| `restartPolicy`                | Restart Policy for containers in workload                                                                       | `Always`                                                             |
 | `jobRestartPolicy`             | Restart Policy for job type workload ( only applicable if `workloadType` is `Job`/`CronJob`                     | `OnFailure`                                                          |
 | `containerCommand`             | Commands to execute inside container overriding image CMD default                                               | `null`                                                               |
 | `containerArgs`                | Specify arguments for container command                                                                         | `null`                                                               |

test/ix-chart/2012.0.0/questions.yaml → test/ix-chart/2101.0.0/questions.yaml

@@ -99,21 +99,6 @@ questions:
           description: "Kill existing pods before creating new ones"
 
   # Restart Policy
-  - variable: restartPolicy
-    description: "Restart Policy for workload"
-    label: "Restart Policy"
-    group: "Restart Policy"
-    schema:
-      type: string
-      show_if: [["workloadType", "=", "Deployment"]]
-      default: "Always"
-      enum:
-        - value: "Always"
-          description: "Always restart containers in a pod if they exit"
-        - value: "OnFailure"
-          description: "Only restart containers if they exit with a failure"
-        - value: "Never"
-          description: "Never restart containers if they exit"
   - variable: jobRestartPolicy
     description: "Restart Policy for workload"
     label: "Restart Policy"

test/ix-chart/2012.0.0/templates/_workload.tpl → test/ix-chart/2101.0.0/templates/_workload.tpl

@@ -39,7 +39,7 @@ Get Restart policy based on workload type
 */}}
 {{- define "restartPolicy" -}}
 {{- if eq (include "workloadIsDeployment" .) "true" }}
-{{- printf "%s" .Values.restartPolicy }}
+{{- print "Always" }}
 {{- else }}
 {{- printf "%s" .Values.jobRestartPolicy }}
 {{- end }}

test/ix-chart/2012.0.0/test_values.yaml → test/ix-chart/2101.0.0/test_values.yaml

@@ -7,7 +7,6 @@ image:
   tag: latest
 
 # Restart / Update policy
-restartPolicy: Always
 updateStrategy: RollingUpdate
 
 # Container CMD / entrypoint

test/ix-chart/2012.0.0/values.yaml → test/ix-chart/2101.0.0/values.yaml

@@ -7,7 +7,6 @@ image:
   tag: "latest"
 
 # Restart / Update policy
-restartPolicy: Always
 updateStrategy: RollingUpdate
 
 # Container CMD / entrypoint