chore: upgrade api7 and gateway to v3.9.17 - #325
Merged
Conversation
Version bumps: - api7ee3 chart 3.9.2 -> 3.9.3, appVersion 3.9.16 -> 3.9.17 - gateway chart 3.9.7 -> 3.9.8, appVersion 3.9.16 -> 3.9.17 - dashboard / dp_manager / file_server / developer_portal image tags -> v3.9.17, gateway image tag -> 3.9.17 Structural changes ported from the 3.9.17 sources: - database.max_lifetime is now set to 1h for the dashboard, dp_manager and developer_portal configurations, matching the control plane's new default (control-plane #2850). It was previously a commented-out 60s example, so a chart-deployed control plane kept pooled connections indefinitely. - dashboard_configuration.console gains hybrid_mode, sentry and a commented custom_menu_group example. /api/fe-config is now served by the Go binary and driven by these keys (control-plane #2842), so they have to be reachable from the chart. - nginx.stream.realIpFrom exposes the gateway's new nginx_config.stream.real_ip_from (gateway #1985), rendered into the configmap only when non-empty. Mirrors what the 3.10 line does. READMEs regenerated with helm-docs.
|
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: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
nic-6443
approved these changes
Jul 28, 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.
3.9.17 chart release. Targets
release/3.9because 3.9 is no longer the latest line —mainis on 3.10.Version bumps
api7ee3gatewayImage tags:
dashboard,dp_manager,file_server,developer_portal→v3.9.17; gateway →3.9.17.Chart patches follow this line's own counter, so they stay below the 3.10 line's numbers and
--version '~3.9'keeps working.Structural changes
Three, all ported from the 3.9.17 sources and matching what the 3.10 line already carries:
database.max_lifetime: 1hfor thedashboard,dp_manageranddeveloper_portalconfigurations. The control plane now bounds pooled connection lifetime by default (api7/api7ee-3-control-plane#2850); the chart previously carried only a commented-out# max_lifetime: 60s, so a chart-deployed control plane reused pooled connections indefinitely and could keep talking to a primary demoted by a failover.dashboard_configuration.consolegainshybrid_mode,sentry, and a commentedcustom_menu_groupexample./api/fe-configis now served by the Go binary and driven by these keys (api7/api7ee-3-control-plane#2842), so they need to be settable from the chart. The existingconsole.addris extended in place rather than re-declared.nginx.stream.realIpFromexposes the gateway's newnginx_config.stream.real_ip_from(api7/api7-ee-3-gateway#1985), rendered into the configmap only when non-empty. Defaults to[], so a default render is unchanged.Verification
values.yamlparsed with a duplicate-key-rejecting loader — no duplicate keys, andconsole.addrsurvives alongside the three new keys.helm templatewithnginx.stream.realIpFromunset renders no streamreal_ip_from; with it set to two CIDRs it renders both, quoted, inside thestream:block.config.yamlpassesapisix initinapi7-ee-3-gateway:3.9.17, which validates it against the real config schema.helm-docs.Release notes: api7/docs#1950 (EN), api7/docs.apiseven.com#359 (CN).