Merge pull request #13 from truenas/ix-chart-fixes

Make sure hostports are not used when update strategy is rolling update

test/ix-chart/2012.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: 2010.0.2
+version: 2012.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/templates/_workload.tpl

@@ -96,6 +96,9 @@ Deployment Spec
 */}}
 {{- define "deploymentSpec" }}
 strategy:
+  {{- if and (eq .Values.updateStrategy "RollingUpdate") .Values.hostPortsList }}
+  {{- fail "RollingUpdate is not allowed when host ports are specified" }}
+  {{- end }}
   type: {{ .Values.updateStrategy }}
 selector:
   matchLabels: