chore: bump ADC image tag to 0.27.1#300
Conversation
- Bump ADC image tag: 0.26.0 -> 0.27.1 - Bump ingress-controller chart version: 0.1.24 -> 0.1.25
📝 WalkthroughWalkthroughThe Helm chart Changesingress-controller Chart Version Bump and ADC Image Tag Update
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Suggested reviewers
Important Pre-merge checks failedPlease resolve all errors before merging. Addressing warnings is optional. ❌ Failed checks (1 error)
✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
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 winREADME.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 usinghelm-docs --chart-search-root=chartsto 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=chartsThis will update the Version badge on line 3 to
0.1.25and 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
📒 Files selected for processing (3)
charts/ingress-controller/Chart.yamlcharts/ingress-controller/README.mdcharts/ingress-controller/values.yaml
What
Bump the ADC image tag used by the
ingress-controllerchart from0.26.0to0.27.1.Changes
charts/ingress-controller/values.yaml:adc.image.tag0.26.0->0.27.1charts/ingress-controller/README.md: regenerate theadc.image.tagdefaultcharts/ingress-controller/Chart.yaml: bump chartversion0.1.24->0.1.25Summary by CodeRabbit