openmetadata: allow to define service nodeport

Author: Andrea Scarpino

charts/openmetadata/Chart.yaml

@@ -1,7 +1,6 @@
 apiVersion: v2
 name: openmetadata
 description: A Helm chart for OpenMetadata on Kubernetes
-
 # A chart can be either an 'application' or a 'library' chart.
 #
 # Application charts are a collection of templates that can be packaged into versioned archives
@@ -11,24 +10,19 @@ description: A Helm chart for OpenMetadata on Kubernetes
 # a dependency of application charts to inject those utilities and functions into the rendering
 # pipeline. Library charts do not define any templates and therefore cannot be deployed.
 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: 1.12.1
-
+version: 1.12.2
 # 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
 # follow Semantic Versioning. They should reflect the version the application is using.
 # It is recommended to use it with quotes.
 appVersion: "1.12.1"
-
 home: https://open-metadata.org/
-
 sources:
   - https://github.com/open-metadata/OpenMetadata
   - https://github.com/open-metadata/openmetadata-helm-charts
-
 keywords:
   - metadata
   - data-science
@@ -49,13 +43,10 @@ keywords:
   - dataquality
   - bigdataanalytics
   - datadiscovery
-
 maintainers:
   - name: OpenMetadata
     email: support@open-metadata.org
-
 icon: https://open-metadata.org/assets/favicon.png
-
 annotations:
   artifacthub.io/images: |
     - name: openmetadata-server

charts/openmetadata/README.md

@@ -299,8 +299,10 @@ helm install openmetadata open-metadata/openmetadata --values <<path-to-values-f
 | testConnection.resources | object | `{}` |
 | securityContext | object | `{}` |
 | service.adminPort | string | `8586` |
+| service.adminNodePort | int | `` |
 | service.annotations | object | `{}` |
 | service.port | int | `8585` |
+| service.nodePort | int | `` |
 | service.type | string | `"ClusterIP"` |
 | serviceAccount.annotations | object | `{}` |
 | serviceAccount.create | bool | `true` |

charts/openmetadata/templates/service.yaml

@@ -13,10 +13,16 @@ spec:
   ports:
     - port: {{ .Values.service.port }}
       targetPort: http
+      {{- if and (eq "NodePort" .Values.service.type) .Values.service.nodePort }}
+      nodePort: {{ .Values.service.nodePort }}
+      {{- end }}
       protocol: TCP
       name: http
     - port: {{ .Values.service.adminPort }}
       targetPort: http-admin
+      {{- if and (eq "NodePort" .Values.service.type) .Values.service.adminNodePort }}
+      nodePort: {{ .Values.service.adminNodePort }}
+      {{- end }}
       protocol: TCP
       name: http-admin
   selector:

charts/openmetadata/values.yaml

@@ -485,7 +485,9 @@ securityContext: {}
 service:
   type: ClusterIP
   port: 8585
+  # nodePort: 32085
   adminPort: 8586
+  # adminNodePort: 32086
   annotations: {}
 
 # Service monitor for Prometheus metrics