Skip to content

chore: bump ADC image tag to 0.27.1#300

Closed
AlinsRan wants to merge 1 commit into
mainfrom
chore/bump-adc-0.27.1
Closed

chore: bump ADC image tag to 0.27.1#300
AlinsRan wants to merge 1 commit into
mainfrom
chore/bump-adc-0.27.1

Conversation

@AlinsRan

@AlinsRan AlinsRan commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

What

Bump the ADC image tag used by the ingress-controller chart from 0.26.0 to 0.27.1.

Changes

  • charts/ingress-controller/values.yaml: adc.image.tag 0.26.0 -> 0.27.1
  • charts/ingress-controller/README.md: regenerate the adc.image.tag default
  • charts/ingress-controller/Chart.yaml: bump chart version 0.1.24 -> 0.1.25

Summary by CodeRabbit

  • Chores
    • Updated ingress controller Helm chart to version 0.1.25
    • Upgraded ADC component to version 0.27.1

- Bump ADC image tag: 0.26.0 -> 0.27.1
- Bump ingress-controller chart version: 0.1.24 -> 0.1.25
@coderabbitai

coderabbitai Bot commented Jun 23, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

The Helm chart ingress-controller version is bumped from 0.1.24 to 0.1.25. The ADC image tag default is updated from 0.26.0 to 0.27.1 in values.yaml, with the corresponding README documentation updated to match.

Changes

ingress-controller Chart Version Bump and ADC Image Tag Update

Layer / File(s) Summary
Chart version and ADC image tag update
charts/ingress-controller/Chart.yaml, charts/ingress-controller/values.yaml, charts/ingress-controller/README.md
Chart version incremented to 0.1.25; adc.image.tag updated from 0.26.0 to 0.27.1 in values and README.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • api7/api7-helm-chart#290: Previous chart version bump for charts/ingress-controller also updating Chart.yaml version and adc.image.tag default, directly preceding this PR's changes.

Suggested reviewers

  • Baoyuantop

Important

Pre-merge checks failed

Please resolve all errors before merging. Addressing warnings is optional.

❌ Failed checks (1 error)

Check name Status Explanation Resolution
Security Check ❌ Error README.md badge shows Chart version 0.1.24 but Chart.yaml has 0.1.25; documentation is inconsistent with actual chart version. Update README.md line 3 badge from "Version-0.1.24" to "Version-0.1.25" to match the updated Chart.yaml version.
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main objective of the pull request: bumping the ADC image tag to 0.27.1.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
E2e Test Quality Review ✅ Passed This PR only updates Helm chart configuration files (Chart.yaml, values.yaml, README.md) to bump the ADC image tag. It contains no test code, test files, or E2E tests. The custom check for "E2E Tes...
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/bump-adc-0.27.1

Comment @coderabbitai help to get the list of available commands.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
charts/ingress-controller/README.md (1)

3-3: 📐 Maintainability & Code Quality | 🟠 Major | ⚡ Quick win

README.md was not regenerated with helm-docs per coding guidelines.

Line 3 shows a stale Version badge (0.1.24) that does not match the updated Chart.yaml version (0.1.25). The README.md must be regenerated using helm-docs --chart-search-root=charts to keep the autogenerated header badges and metadata in sync with Chart.yaml. This will ensure the helm-docs CI check passes.

🔧 How to fix

Run the following command from the repository root to regenerate the README:

helm-docs --chart-search-root=charts

This will update the Version badge on line 3 to 0.1.25 and ensure all autogenerated documentation is current.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@charts/ingress-controller/README.md` at line 3, The README.md file in the
ingress-controller chart contains stale version badges that do not match the
current Chart.yaml version. Regenerate the README.md file by running the
helm-docs command with the appropriate chart search root from the repository
root directory to ensure all autogenerated headers, version badges, and metadata
are synchronized with the current Chart.yaml. This will update the stale Version
badge from 0.1.24 to 0.1.25 and ensure the helm-docs CI check passes.

Source: Coding guidelines

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Outside diff comments:
In `@charts/ingress-controller/README.md`:
- Line 3: The README.md file in the ingress-controller chart contains stale
version badges that do not match the current Chart.yaml version. Regenerate the
README.md file by running the helm-docs command with the appropriate chart
search root from the repository root directory to ensure all autogenerated
headers, version badges, and metadata are synchronized with the current
Chart.yaml. This will update the stale Version badge from 0.1.24 to 0.1.25 and
ensure the helm-docs CI check passes.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 6948a967-7975-441b-a511-c9d944f6c007

📥 Commits

Reviewing files that changed from the base of the PR and between 40b9c46 and ee32efd.

📒 Files selected for processing (3)
  • charts/ingress-controller/Chart.yaml
  • charts/ingress-controller/README.md
  • charts/ingress-controller/values.yaml

@AlinsRan AlinsRan closed this Jun 23, 2026
@AlinsRan AlinsRan deleted the chore/bump-adc-0.27.1 branch June 23, 2026 10:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant