[jarvis9443]

Releases the AISIX charts for 1.10.0: aisix-cp and aisix both go to version and appVersion 1.10.0, so the images resolve to docker.io/api7/aisix-cp-api:1.10.0, docker.io/api7/aisix-cp-dpm:1.10.0 and docker.io/api7/aisix:1.10.0.

charts/aisix-cp is synced from helm/aisix-cp at AISIX-Cloud v1.10.0. The only functional delta this release is in the managed-mode NOTES: the gateway docker run is now detached and named aisix-dp with --restart unless-stopped --stop-timeout 45, and it is followed by a /readyz wait so the operator can tell the gateway connected and loaded its configuration (and a pointer to docker logs aisix-dp if it doesn't). The public adaptations are preserved: docker.io registries, image tags defaulting to .Chart.AppVersion, a real gateway image in NOTES (api.dpImage, else docker.io/api7/aisix:<appVersion>) where the dev chart prints a placeholder, and the public README.

charts/aisix takes the held #425: config.observability.log_level now defaults to warn, matching the 1.10.0 gateway's own default. An installation that doesn't set it writes only WARN and above after upgrading, plus the access log, which config.observability.access_log alone controls. Set config.observability.log_level: info to keep the old output. The chart's config drift check matches config.reference.json at v1.10.0.

READMEs are regenerated. Merging publishes aisix-cp-1.10.0 and aisix-1.10.0.

🤖 Generated with Claude Code

Summary by CodeRabbit

• Updates
  • Updated the chart and application version references to 1.10.0.
  • Changed the default log level to warn; info is recommended when diagnosing issues. Access logs are written regardless of the configured log level.
• New Features
  • Managed-mode setup now runs the service in the background, restarts it after failures, publishes port 3000, and checks readiness. If it remains unready, setup points you to the service logs.

[coderabbitai]

No actionable comments were generated in the recent review.

Walkthrough

Both Helm charts move to version 1.10.0. The managed-mode Docker instructions add container restart, port, and readiness details. The aisix chart changes its documented log-level default to warn and updates the access-log description.

Changes: Chart release and configuration updates

Layer / File(s)	Summary
Chart version references charts/aisix-cp/Chart.yaml, charts/aisix-cp/README.md, charts/aisix/Chart.yaml, charts/aisix/README.md	Both chart versions and app versions change from 1.9.0 to 1.10.0. The aisix-cp installation examples and gateway image reference also use 1.10.0.
Managed-mode container instructions charts/aisix-cp/templates/NOTES.txt	The Docker command runs a named, detached container with restart behavior, a 45-second stop timeout, and port 3000 published. The instructions add /readyz polling and log guidance.
Observability defaults charts/aisix/values.yaml, charts/aisix/README.md	The documented default for config.observability.log_level changes from info to warn. The comment recommends info while diagnosing, and the access-log description is updated.

Estimated code review effort: 2 (Simple) | ~10 minutes

Suggested reviewers: nic-6443

Merge Risk: Low · up to ff747

This release bumps both charts to 1.10.0, makes warn the default log level for the aisix chart, and improves the managed-mode Docker instructions. Confirm that the 1.10.0 images are published and that CI passes before merging, because the config boot check could not pull the aisix 1.10.0 image.

Pre-merge checks: 6

Check name	Status	Explanation
Description Check		Check skipped - CodeRabbit’s high-level summary is enabled.
Title check		The title "release(aisix): 1.10.0" names the primary change: a release bump of the aisix charts to version 1.10.0. The changeset updates the version and appVersion of the aisix-cp and aisix charts, th…
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.
E2e Test Quality Review		The check is not applicable to this pull request. The check reviews E2E and other test code. The reviewed range (refs/coderabbit/pre-merge-check/base to head) changes six files, and none is a test…
Security Check		The PR changes six files: version bumps in charts/aisix-cp and charts/aisix (Chart.yaml and README badges), a log_level default change from info to warn in charts/aisix/values.yaml and its…

Full details: Title check

Explanation

The title "release(aisix): 1.10.0" names the primary change: a release bump of the aisix charts to version 1.10.0. The changeset updates the version and appVersion of the aisix-cp and aisix charts, their READMEs, and the aisix log_level default, which fits a 1.10.0 release. The title omits the notable behavior change (log_level default moving from info to warn) and the managed-mode NOTES change, but it is not misleading and does not need to cover every detail.

Full details: E2e Test Quality Review

Explanation

The check is not applicable to this pull request. The check reviews E2E and other test code. The reviewed range (refs/coderabbit/pre-merge-check/base to head) changes six files, and none is a test, spec, E2E, or CI file: - charts/aisix-cp/Chart.yaml - charts/aisix-cp/README.md - charts/aisix-cp/templates/NOTES.txt - charts/aisix/Chart.yaml - charts/aisix/README.md - charts/aisix/values.yaml The changes are a version bump from 1.9.0 to 1.10.0, README regeneration, a managed-mode NOTES text change, and a config.observability.log_level default change from info to warn. The check's failure conditions (test completeness, scenario coverage, readability of tests, test dependencies, assertion readability, mock use, and test-linked code quality) do not apply to chart metadata, documentation, and template text. The check does not state a failure condition for a PR that adds no tests. Missing test coverage for the default change is therefore not a failing condition under this check. The PR scope is consistent with its stated release goal, so the PR scope criterion does not show unrelated changes.

Full details: Security Check

Explanation

The PR changes six files: version bumps in charts/aisix-cp and charts/aisix (Chart.yaml and README badges), a log_level default change from info to warn in charts/aisix/values.yaml and its README, an access-log doc reword, and a managed-mode NOTES.txt change. I read the full diff for each code file. Category review: 1. Sensitive data exposure in logs and responses. The changed NOTES.txt lines do not add any secret to a log or response. The AISIX_MANAGED__CP_KEY_PEM and AISIX_MANAGED__CP_CA_PEM lines are unchanged context and hold placeholders ('<client-key-pem>', '<ca-cert-pem>'). The new docker logs aisix-dp hint only points operators to existing container logs. The warn default lowers log verbosity and does not add sensitive output. 2. Secrets stored unencrypted. No database, model, or migration code changed. 3. Authorization bypass. No handler or permission code changed. 4. Cross-resource ownership. No API code changed. 5. TLS and cryptographic configuration. No TLS flag or verification logic changed. 6. Shared resource safety. No entity or cascade logic changed. 7. Secret reference resolution. No $env:// or $secret:// handling changed. Observation outside the listed categories: the NOTES change adds -p 3000:3000, which publishes the gateway port on the Docker host. The check does not list port exposure as a failure condition, so it does not cause a FAIL. Operators who deploy this command should be aware that the port is reachable from the host network. The log_level change in values.yaml also has no secret-exposure effect. No finding matches an explicit failure condition in the check.

Finishing Touches

Generate unit tests (beta)

• Commit to this branch
• Create a new PR

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Recent review info

Run configuration

• Source mode: Auto
• Repository: Unavailable
• Branch: Unavailable
• File: No configuration file from this repository applied
• Commit: Unavailable
• Layers: Organization UI
• Review profile: CHILL
• Plan: Essentials
• Run ID: 20ee478f-25c3-43f3-bbe9-544e9d166543

Commits

Reviewing files that changed from the base of the PR and between 93820bf and ff7470a.

Files selected for processing: 6

• charts/aisix-cp/Chart.yaml
• charts/aisix-cp/README.md
• charts/aisix-cp/templates/NOTES.txt
• charts/aisix/Chart.yaml
• charts/aisix/README.md
• charts/aisix/values.yaml

Included review availability: This review used your included allowance. 0 included reviews remain after this review. Your included PR review attempts over the past 7 days set your current allowance at 1 review per hour.

[nic-6443]

Residual diff against the control-plane chart at v1.10.0 is only the preserved public adaptations; CI is green on ff7470a.