chore: upgrade api7 and gateway to v3.9.21 - #395
Merged
Merged
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Essentials Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
nic-6443
approved these changes
Sep 24, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Chart release for API7 EE v3.9.21 on the 3.9 maintenance line.
Version bumps
charts/api7charts/gatewayImage tags:
charts/api7/values.yaml:dashboard.image.tag,dp_manager.image.tag,developer_portal.image.tag,file_server.image.tag->v3.9.21charts/gateway/values.yaml:apisix.image.tag->3.9.21openapi-to-mcp sidecar 1.0.5 -> 1.0.6
charts/gateway/values.yamlopenapiToMcp.image.tagis bumped to1.0.6for two reasons:OPENAPI_TO_MCP_VERSION=1.0.6in EE v3.9.21'sVERSION.libssl3t64CVEs — CVE-2026-14456 and CVE-2026-45447 — that are present in both 1.0.4 and 1.0.5 and are fixed by 1.0.6's rebuilt base image. A version pin bump alone would not have cleared them; the base image had to be rebuilt.1.0.6 also supports the per-route
cache_enabled/cache_ttloptions sent by gateway #2261.Removed
dashboard_configuration.console.addrThe control plane dropped this key from its own chart values in control-plane #3011: on this line the dashboard binary serves the console SPA itself, so there is no Node process on
127.0.0.1:3000and the key is dead config. The rest of theconsole:block (hybrid_mode,sentry) is unchanged.Gateway shared dict
plugin-saml-auth-replay: no chart change neededGateway v3.9.21 adds
plugin-saml-auth-replay: 10mtonginx_config.http.lua_shared_dictinconf/config-default.yaml(saml-auth assertion replay). The chart carries its own explicitapisix.http.luaSharedDictmap, so the question was whether that map replaces the image default or is merged onto it.Verified against the release image rather than assumed — the chart's rendered ConfigMap
config.yaml(which does not containplugin-saml-auth-replay) was run throughapisix init:APISIX deep-merges the chart's partial map onto
config-default.yaml: the new dict is present in the generatednginx.confwithout any chart change, while chart-set entries still win (prometheus-metricsrenders at the chart's128m). The chart'sluaSharedDictlist is therefore an override list, not a replacement, which is also why it can omitplugin-ai-rate-limitingtoday and still work.Validation
helm lint charts/api7andhelm lint charts/gateway— both pass.helm templatefor both charts; the api7 dashboard ConfigMap rendersconsole: {hybrid_mode, sentry}with noaddr.openapiToMcp.enabled=truerenders the sidecar atapi7/openapi-to-mcp:1.0.6; unset renders no sidecar.apisix initin the 3.9.21 image (exit 0).make helm-docs; onlycharts/api7andcharts/gatewaychange.