AlinsRan opened a new pull request, #1027: URL: https://github.com/apache/apisix-helm-chart/pull/1027
The APISIX chart pins ingress controller chart 1.3.0, which explicitly renders `listener_port_match_mode: auto`. That overrides the controller 2.2.0 default of `off`. When a Gateway listener uses port 80 but the Service forwards to APISIX on port 9080, listener-port predicates can prevent routes from matching. Set `ingress-controller.config.listenerPortMatchMode` to `off` in the umbrella chart. Explicit `auto` and `explicit` overrides remain supported. The change works with the currently published dependency, without waiting for a controller chart release. Chart version is bumped to 2.18.3 and the upgrade behavior is documented. Related to #1001 and #1002. The latter updates the standalone controller chart; this PR covers the umbrella chart independently. Validation: - Added a CI regression test using the published controller dependency for the default and explicit `off`, `auto`, and `explicit` values. The test fails on the current master default and passes with this change. - `helm lint charts/apisix` passes. - `ct lint --validate-maintainers=false --charts charts/apisix --skip-helm-dependencies` passes using the CI image `quay.io/helmpack/chart-testing:v3.13.0`, after building dependencies. - `helm-docs v1.11.0` regeneration is idempotent; shell syntax, workflow YAML parsing, and `git diff --check` pass. No live-cluster routing test was run for this values-only change. -- 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]
