Added option for httpHostPort and transportHostPort

Signed-off-by: Sebastian Manthey <sbnmanthey@gmail.com>
opensearch-project · Mar 13, 2023 · 25b746b · 25b746b
1 parent 6f61d16
commit 25b746b
Show file tree

Hide file tree

Showing 5 changed files with 20 additions and 1 deletion.
diff --git a/charts/opensearch/CHANGELOG.md b/charts/opensearch/CHANGELOG.md
@@ -12,6 +12,15 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Fixed
 ### Security
 ---
+## [1.18.1]
+### Added
+- Added option for `httpHostPort` and `transportHostPort`. Backport, see [Issue #335](https://github.com/opensearch-project/helm-charts/issues/335)
+### Changed
+### Deprecated
+### Removed
+### Fixed
+### Security
+---
 ## [1.18.0]
 ### Added
 ### Changed

diff --git a/charts/opensearch/Chart.yaml b/charts/opensearch/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: 1.18.0
+version: 1.18.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

diff --git a/charts/opensearch/README.md b/charts/opensearch/README.md
@@ -53,6 +53,7 @@ helm uninstall my-release
 | `fullnameOverride`                 | Overrides the `clusterName` and `nodeGroup` when used in the naming of resources. This should only be used when using a single `nodeGroup`, otherwise you will have name conflicts                                                                        | `""`                                            |
 | `hostAliases`                      | Configurable [hostAliases][]                                                                                                                                                                                                                              | `[]`                                            |
 | `httpPort`                         | The http port that Kubernetes will use for the healthchecks and the service. If you change this you will also need to set `http.port` in `extraEnvs`                                                                                                    | `9200`                                          |
+| `httpHostPort`                     | Expose another http-port as hostPort. Refer to documentation for more information and requirements about using hostPorts. | `""`                                          |
 | `image.pullPolicy`                 | The Kubernetes [imagePullPolicy][] value                                                                                                                                                                                                                  | `IfNotPresent`                                  |
 | `imagePullSecrets`                 | Configuration for [imagePullSecrets][] so that you can use a private registry for your image                                                                                                                                                              | `[]`                                            |
 | `image.tag`                        | The OpenSearch Docker image tag                                                                                                                                                                                                                        | `1.0.0`                               |
@@ -103,6 +104,7 @@ helm uninstall my-release
 | `tolerations`                      | Configurable [tolerations][]                                                                                                                                                                                                                              | `[]`                                            |
 | `topologySpreadConstraints`        | Configuration for pod [topologySpreadConstraints][]                                                                                                                                                                                                       | `[]`                                            |
 | `transportPort`                    | The transport port that Kubernetes will use for the service. If you change this you will also need to set transport port configuration in `extraEnvs`                                                                                                 | `9300`                                          |
+| `transportHostPort`                | Expose another transport port as hostPort. Refer to documentation for more information and requirements about using hostPorts. | `""`                                          |
 | `updateStrategy`                   | The [updateStrategy][] for the StatefulSet. By default Kubernetes will wait for the cluster to be green after upgrading each pod. Setting this to `OnDelete` will allow you to manually delete each pod during upgrades                                   | `RollingUpdate`                                 |
 | `volumeClaimTemplate`              | Configuration for the [volumeClaimTemplate for StatefulSets][]. You will want to adjust the storage (default `30Gi` ) and the `storageClassName` if you are using a different storage class                                                               | see [values.yaml][]                             |
 | `extraObjects`                     | Array of extra K8s manifests to deploy                                                                                                                                                                                                                    | list `[]`                                       |                                   |

diff --git a/charts/opensearch/templates/statefulset.yaml b/charts/opensearch/templates/statefulset.yaml
@@ -339,8 +339,14 @@ spec:
         ports:
         - name: http
           containerPort: {{ .Values.httpPort }}
+        {{- if .Values.httpHostPort }}
+          hostPort: {{ .Values.httpHostPort }}
+        {{- end }}
         - name: transport
           containerPort: {{ .Values.transportPort }}
+        {{- if .Values.transportHostPort }}
+          hostPort: {{ .Values.transportHostPort }}
+        {{- end }}
         resources:
 {{ toYaml .Values.resources | indent 10 }}
         env:

diff --git a/charts/opensearch/values.yaml b/charts/opensearch/values.yaml
@@ -259,6 +259,8 @@ enableServiceLinks: true
 protocol: https
 httpPort: 9200
 transportPort: 9300
+httpHostPort: ""
+transportHostPort: ""
 
 service:
   labels: {}