Skip to content

Commit 7c90a0a

Browse files
Calebjhkd7lxl
andauthored
Allow for the nginx service port and container port to be different (#589)
* Allow for the nginx service port and container port to be different Signed-off-by: chorst <chorst@infoblox.com> * Update changelog Signed-off-by: chorst <chorst@infoblox.com> --------- Signed-off-by: chorst <chorst@infoblox.com> Co-authored-by: Tom Hayward <thayward@infoblox.com>
1 parent f9194fb commit 7c90a0a

5 files changed

Lines changed: 6 additions & 2 deletions

File tree

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44

55
* [FEATURE] Add a parquet converter. #579
66
* [ENHANCEMENT] Avoid scheduling queriers and ingesters on the same node #583
7+
* [ENHANCEMENT] Allow for the nginx service port and container port to be different #589
78
* [DEPENDENCY] Update quay.io/cortexproject/cortex Docker tag to v1.20.1 #577
89

910
## 3.0.0 / 2025-11-27

README.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -488,6 +488,7 @@ Kubernetes: `^1.19.0-0`
488488
| nginx.&ZeroWidthSpace;securityContext | object | `{}` | |
489489
| nginx.&ZeroWidthSpace;service.&ZeroWidthSpace;annotations | object | `{}` | |
490490
| nginx.&ZeroWidthSpace;service.&ZeroWidthSpace;labels | object | `{}` | |
491+
| nginx.&ZeroWidthSpace;service.&ZeroWidthSpace;port | string | `""` | Replaces default port value from nginx.http_listen_port when set |
491492
| nginx.&ZeroWidthSpace;service.&ZeroWidthSpace;type | string | `"ClusterIP"` | |
492493
| nginx.&ZeroWidthSpace;serviceAccount.&ZeroWidthSpace;name | string | `""` | "" disables the individual serviceAccount and uses the global serviceAccount for that component |
493494
| nginx.&ZeroWidthSpace;startupProbe.&ZeroWidthSpace;failureThreshold | int | `10` | |

templates/nginx/nginx-ingress.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ spec:
3434
service:
3535
name: {{ include "cortex.nginxFullname" $ }}
3636
port:
37-
number: {{ $.Values.nginx.http_listen_port }}
37+
number: {{ $.Values.nginx.service.port | default $.Values.nginx.http_listen_port }}
3838
{{- end }}
3939
{{- end }}
4040
{{- end }}

templates/nginx/nginx-svc.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ metadata:
1414
spec:
1515
type: {{ .Values.nginx.service.type }}
1616
ports:
17-
- port: {{ .Values.nginx.http_listen_port }}
17+
- port: {{ .Values.nginx.service.port | default .Values.nginx.http_listen_port }}
1818
protocol: TCP
1919
name: http-metrics
2020
targetPort: http-metrics

values.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1342,6 +1342,8 @@ nginx:
13421342
type: ClusterIP
13431343
annotations: {}
13441344
labels: {}
1345+
# -- Replaces default port value from nginx.http_listen_port when set
1346+
port: ""
13451347

13461348
serviceAccount:
13471349
# -- "" disables the individual serviceAccount and uses the global serviceAccount for that component

0 commit comments

Comments
 (0)