Merge pull request #934 from truenas/ix-chart-nodeportmin

NAS-119472 / 23.10 / Remove `min` for ports in UI, but validate with template

charts/ix-chart/2212.0.1/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: 2212.0.0
+version: 2212.0.1
 
 # 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

charts/ix-chart/2212.0.1/questions.yaml

@@ -77,7 +77,6 @@ questions:
           description: "Specify port to be used for Portal access"
           schema:
             type: int
-            min: 9000
             max: 65535
             default: 15000
 

charts/ix-chart/2212.0.1/templates/_networking.tpl

@@ -21,3 +21,10 @@ Get configuration for host network
 {{- print "true" -}}
 {{- end -}}
 {{- end -}}
+
+{{/* Validate portal port */}}
+{{- if .Values.enableUIPortal }}
+  {{- if and (not .Values.hostNetwork) (lt .Values.portalDetails.port 9000) }}
+    {{- fail (printf "Port (%d) is too low. Minimum allowed port is 9000." .Values.portalDetails.port) }}
+  {{- end }}
+{{- end }}

test/ix-chart/2212.0.1/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: 2212.0.0
+version: 2212.0.1
 
 # 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/2212.0.1/questions.yaml

@@ -427,7 +427,6 @@ questions:
                 schema:
                   type: int
                   required: true
-                  min: 9000
                   max: 65535
               - variable: protocol
                 label: "Protocol"

test/ix-chart/2212.0.1/templates/_networking.tpl

@@ -21,3 +21,10 @@ Get configuration for host network
 {{- print "true" -}}
 {{- end -}}
 {{- end -}}
+
+{{/* Validate portal port */}}
+{{- if .Values.enableUIPortal }}
+  {{- if and (not .Values.hostNetwork) (lt .Values.portalDetails.port 9000) }}
+    {{- fail (printf "Port (%d) is too low. Minimum allowed port is 9000." .Values.portalDetails.port) }}
+  {{- end }}
+{{- end }}