This is an automated email from the ASF dual-hosted git repository.
yzheng pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/polaris.git
The following commit(s) were added to refs/heads/main by this push:
new 952a34c45 Ingress paths missing pathType in values.yaml (#3721)
952a34c45 is described below
commit 952a34c45581607991ece882620668525752f06b
Author: Yong Zheng <[email protected]>
AuthorDate: Tue Feb 10 12:07:49 2026 -0600
Ingress paths missing pathType in values.yaml (#3721)
---
helm/polaris/README.md | 2 +-
helm/polaris/tests/ingress_test.yaml | 29 ++++++++++++++++++++++++-----
helm/polaris/values.yaml | 4 +++-
site/content/in-dev/unreleased/helm.md | 2 +-
4 files changed, 29 insertions(+), 8 deletions(-)
diff --git a/helm/polaris/README.md b/helm/polaris/README.md
index c9bf272d8..b5530ed1e 100644
--- a/helm/polaris/README.md
+++ b/helm/polaris/README.md
@@ -331,7 +331,7 @@ ct install --namespace polaris --charts ./helm/polaris
| ingress.annotations | object | `{}` | Annotations to add to the ingress. |
| ingress.className | string | `""` | Specifies the ingressClassName; leave
empty if you don't want to customize it |
| ingress.enabled | bool | `false` | Specifies whether an ingress should be
created. |
-| ingress.hosts | list | `[{"host":"chart-example.local","paths":[]}]` | A
list of host paths used to configure the ingress. |
+| ingress.hosts | list |
`[{"host":"chart-example.local","paths":[{"path":"/","pathType":"Prefix"}]}]` |
A list of host paths used to configure the ingress. |
| ingress.tls | list | `[]` | A list of TLS certificates; each entry has a
list of hosts in the certificate, along with the secret name used to terminate
TLS traffic on port 443. |
| livenessProbe | object |
`{"failureThreshold":3,"initialDelaySeconds":5,"periodSeconds":10,"successThreshold":1,"terminationGracePeriodSeconds":30,"timeoutSeconds":10}`
| Configures the liveness probe for polaris pods. |
| livenessProbe.failureThreshold | int | `3` | Minimum consecutive failures
for the probe to be considered failed after having succeeded. Minimum value is
1. |
diff --git a/helm/polaris/tests/ingress_test.yaml
b/helm/polaris/tests/ingress_test.yaml
index d9613bf54..9af164c29 100644
--- a/helm/polaris/tests/ingress_test.yaml
+++ b/helm/polaris/tests/ingress_test.yaml
@@ -151,15 +151,34 @@ tests:
- "chart-example2.local"
secretName: secret1
- # spec.rules (with ingress enabled)
- - it: should set ingress TLS
+ # spec.rules (with ingress enabled - default values)
+ - it: should set ingress with default values
+ set:
+ ingress.enabled: true
+ asserts:
+ - equal:
+ path: spec.rules
+ value:
+ - host: chart-example.local
+ http:
+ paths:
+ - path: /
+ pathType: Prefix
+ backend:
+ service:
+ name: polaris-release
+ port:
+ number: 8181
+
+ # spec.rules (with ingress enabled - custom paths)
+ - it: should set ingress with custom paths
set:
ingress.enabled: true
ingress.hosts:
- host: chart-example.local
paths:
- - path: /
- pathType: Prefix
+ - path: /api
+ pathType: Prefix
asserts:
- equal:
path: spec.rules
@@ -167,7 +186,7 @@ tests:
- host: chart-example.local
http:
paths:
- - path: /
+ - path: /api
pathType: Prefix
backend:
service:
diff --git a/helm/polaris/values.yaml b/helm/polaris/values.yaml
index 80954d87d..41d351c33 100644
--- a/helm/polaris/values.yaml
+++ b/helm/polaris/values.yaml
@@ -233,7 +233,9 @@ ingress:
# -- A list of host paths used to configure the ingress.
hosts:
- host: chart-example.local
- paths: []
+ paths:
+ - path: /
+ pathType: Prefix
# -- A list of TLS certificates; each entry has a list of hosts in the
certificate,
# along with the secret name used to terminate TLS traffic on port 443.
tls: []
diff --git a/site/content/in-dev/unreleased/helm.md
b/site/content/in-dev/unreleased/helm.md
index e3ef87461..ebdd1b7ed 100644
--- a/site/content/in-dev/unreleased/helm.md
+++ b/site/content/in-dev/unreleased/helm.md
@@ -318,7 +318,7 @@ ct install --namespace polaris --charts ./helm/polaris
| ingress.annotations | object | `{}` | Annotations to add to the ingress. |
| ingress.className | string | `""` | Specifies the ingressClassName; leave
empty if you don't want to customize it |
| ingress.enabled | bool | `false` | Specifies whether an ingress should be
created. |
-| ingress.hosts | list | `[{"host":"chart-example.local","paths":[]}]` | A
list of host paths used to configure the ingress. |
+| ingress.hosts | list |
`[{"host":"chart-example.local","paths":[{"path":"/","pathType":"Prefix"}]}]` |
A list of host paths used to configure the ingress. |
| ingress.tls | list | `[]` | A list of TLS certificates; each entry has a
list of hosts in the certificate, along with the secret name used to terminate
TLS traffic on port 443. |
| livenessProbe | object |
`{"failureThreshold":3,"initialDelaySeconds":5,"periodSeconds":10,"successThreshold":1,"terminationGracePeriodSeconds":30,"timeoutSeconds":10}`
| Configures the liveness probe for polaris pods. |
| livenessProbe.failureThreshold | int | `3` | Minimum consecutive failures
for the probe to be considered failed after having succeeded. Minimum value is
1. |
