somaz94 opened a new pull request, #40785:
URL: https://github.com/apache/superset/pull/40785

   ### SUMMARY
   
   Adds an optional Gateway API `HTTPRoute` to the `superset` chart as an 
alternative to the existing `Ingress`, for clusters standardizing on the 
[Gateway API](https://gateway-api.sigs.k8s.io/). The route is **disabled by 
default** (`httproute.enabled: false`), so this is a fully additive, 
backward-compatible change — no existing deployment is affected, and CI needs 
no Gateway API CRDs.
   
   The template mirrors the existing `ingress.yaml`:
   
   - The primary rule backs onto the Superset service (`{{ template 
"superset.fullname" . }}`, port `service.port`).
   - When `supersetWebsockets.enabled` is set, a second rule routes the 
websockets path (`supersetWebsockets.ingress.path`, default `/ws`) to the `-ws` 
service (port `supersetWebsockets.service.port`), matching the Ingress behavior.
   - Labels (`app`/`chart`/`release`/`heritage` + `extraLabels`) and the 
`tpl`-evaluated `hostnames` follow the chart's existing conventions.
   
   Note that HTTPRoute `backendRefs` reference a numeric service port (the 
Gateway API uses port numbers, not the named ports that Ingress accepts), so 
the rules render `service.port` / `supersetWebsockets.service.port` directly.
   
   New `httproute` values: `enabled`, `apiVersion` (default 
`gateway.networking.k8s.io/v1`, overridable to `v1beta1`), `parentRefs`, 
`hostnames`, `matches` (default `PathPrefix /`), `filters`, `additionalRules`, 
`labels`, `annotations`.
   
   `Chart.yaml` version is bumped `0.15.5` → `0.16.0` (additive feature → 
minor, per the chart's [versioning 
policy](https://github.com/apache/superset/blob/master/helm/superset/README.md#versioning)),
 and `README.md` is regenerated with helm-docs `v1.14.2`.
   
   ### BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
   
   N/A — chart-only change, no UI.
   
   ### TESTING INSTRUCTIONS
   
   `ct lint --charts helm/superset` passes (helm lint + yamllint against 
`lintconf.yaml` + version-increment check).
   
   `helm template` verified across scenarios:
   
   - Default (`httproute.enabled: false`) → no `HTTPRoute` rendered (full-chart 
render confirms 0).
   - Enabled with `parentRefs` + `hostnames`.
   - `apiVersion` override to `gateway.networking.k8s.io/v1beta1`.
   - `supersetWebsockets.enabled: true` → second rule routes `/ws` to the `-ws` 
service on port `8080`.
   - `filters` + templated `additionalRules` + `labels`/`annotations`.
   
   ### ADDITIONAL INFORMATION
   
   - [ ] Has associated issue:
   - [ ] Required feature flags:
   - [ ] Changes UI
   - [ ] Includes DB Migration (follow approval process in 
[SIP-59](https://github.com/apache/superset/issues/13351))
     - [ ] Migration is atomic, supports rollback & is backwards-compatible
     - [ ] Confirm DB migration upgrade and downgrade tested
     - [ ] Runtime estimates and downtime expectations provided
   - [x] Introduces new feature or API
   - [ ] Removes existing feature or API
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to