Commit 211640ef authored by Waqar Ahmed's avatar Waqar Ahmed

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

parent 08ab5911
...@@ -15,7 +15,7 @@ type: application ...@@ -15,7 +15,7 @@ type: application
# This is the chart version. This version number should be incremented each time you make changes # 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. # to the chart and its templates, including the app version.
# Versions are expected to follow Semantic Versioning (https://semver.org/) # 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 # 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 # incremented each time you make changes to the application. Versions are not expected to
......
...@@ -96,6 +96,9 @@ Deployment Spec ...@@ -96,6 +96,9 @@ Deployment Spec
*/}} */}}
{{- define "deploymentSpec" }} {{- define "deploymentSpec" }}
strategy: strategy:
{{- if and (eq .Values.updateStrategy "RollingUpdate") .Values.hostPortsList }}
{{- fail "RollingUpdate is not allowed when host ports are specified" }}
{{- end }}
type: {{ .Values.updateStrategy }} type: {{ .Values.updateStrategy }}
selector: selector:
matchLabels: matchLabels:
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment