fix(base-cluster/docs): there is no 9.0.0 release for now...#1563
Conversation
WalkthroughThe changes update the migration section formatting in both the Changes
Possibly related PRs
Suggested labels
Poem
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Pull Request Overview
This PR removes the placeholder “### 8.x.x -> 9.0.0” section since no 9.0.0 release exists yet, replacing it with a horizontal rule for clarity.
- Replace version subheading with a markdown horizontal rule
- Apply change consistently in template and generated README
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| charts/base-cluster/README.md.gotmpl | Replaced unused “### 8.x.x -> 9.0.0” header |
| charts/base-cluster/README.md | Mirrored header removal in generated README |
Comments suppressed due to low confidence (2)
charts/base-cluster/README.md.gotmpl:347
- Indent the horizontal rule (
---) to match the surrounding markdown block so that formatting remains consistent.
---
charts/base-cluster/README.md:357
- Add the same indentation to this horizontal rule (
---) to keep README formatting aligned.
---
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (2)
charts/base-cluster/README.md.gotmpl (1)
345-349: Heading removal hurts navigation – prefer a proper sub-heading over a lone---Replacing the former
### 8.x.x -> 9.0.0heading with a horizontal rule removes the Markdown anchor that GitHub and most editors use for quick navigation. Anyone who bookmarked or deep-linked to that section (or who relies on the auto-generated TOC) will now hit a 404 / broken link.If the intent is simply to signal that 9.0.0 is not yet available, consider keeping a heading and marking it as “TBD” (or similar) instead of dropping the heading altogether:
---- +### 8.x.x –> 8.y.z / Next minor (TBD)This preserves structure and anchors while still clarifying that a 9.0.0 migration does not yet exist.
charts/base-cluster/README.md (1)
357-358: Retaining a heading improves linkability and TOC generationReplacing the semantic heading with a horizontal rule removes an anchor that other docs or users might deep-link to and drops the entry from any auto-generated table of contents.
If the intent is merely to hide the “9.0.0” version while still marking the migration boundary, consider a neutral heading instead of---, e.g.:---- +### 8.x.x → next major (TBD)That keeps navigation intact without promising a specific version.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
charts/base-cluster/README.md(1 hunks)charts/base-cluster/README.md.gotmpl(1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (4)
- GitHub Check: generateDiffCommentBody
- GitHub Check: check licenses
- GitHub Check: lint helm chart (base-cluster)
- GitHub Check: wait-for-checks
Pull Request is not mergeable
🤖 I have diffed this beep boop"/$namespace/$kind/$name.yaml" for normal resources
|
🤖 I have created a release *beep* *boop* --- ## [8.2.0](base-cluster-v8.1.0...base-cluster-v8.2.0) (2025-07-21) ### Features * **base-cluster/flux:** add alert about suspended resources ([#1540](#1540)) ([bb1555e](bb1555e)) * **base-cluster/monitoring:** lower cpu request for prometheus ([#1578](#1578)) ([4a83bf6](4a83bf6)) * **base-cluster/monitoring:** non-critical alerts aren't routed to on-call ([#1533](#1533)) ([0d080f4](0d080f4)) * **base-cluster/rbac:** adjust rbac stuff for OIDC accounts ([#1538](#1538)) ([3f9aa69](3f9aa69)) ### Bug Fixes * **base-cluster/docs:** there is no 9.0.0 release for now... ([#1563](#1563)) ([8e417fb](8e417fb)) ### Miscellaneous Chores * **base-cluster/dependencies:** update common docker tag to v1.5.0 ([#1520](#1520)) ([cb4a522](cb4a522)) * **base-cluster/dependencies:** update docker.io/bitnami/kubectl docker tag to v1.33.3-debian-12-r1 ([#1521](#1521)) ([4ed2a77](4ed2a77)) * **base-cluster/dependencies:** update docker.io/curlimages/curl docker tag to v8.13.0 ([#1523](#1523)) ([e451428](e451428)) * **base-cluster/dependencies:** update docker.io/curlimages/curl docker tag to v8.14.1 ([#1544](#1544)) ([02ba163](02ba163)) * **base-cluster/dependencies:** update docker.io/curlimages/curl docker tag to v8.15.0 ([#1585](#1585)) ([a672a67](a672a67)) * **base-cluster/dependencies:** update docker.io/fluxcd/flux-cli docker tag to v2.6.1 ([#1524](#1524)) ([956ad7e](956ad7e)) * **base-cluster/dependencies:** update docker.io/fluxcd/flux-cli docker tag to v2.6.4 ([#1536](#1536)) ([32a69cc](32a69cc)) * **base-cluster/dependencies:** update docker.io/vladgh/gpg docker tag to v1.3.6 ([#1509](#1509)) ([e521e61](e521e61)) * **base-cluster/dependencies:** update external-dns docker tag to v8.8.4 ([#1510](#1510)) ([b8b3f80](b8b3f80)) * **base-cluster/dependencies:** update external-dns docker tag to v8.9.1 ([#1559](#1559)) ([f9c5642](f9c5642)) * **base-cluster/dependencies:** update external-dns docker tag to v8.9.2 ([#1575](#1575)) ([88b2630](88b2630)) * **base-cluster/dependencies:** update grafana-tempo docker tag to v4 ([#1570](#1570)) ([63c2593](63c2593)) * **base-cluster/dependencies:** update grafana-tempo docker tag to v4.0.13 ([#1580](#1580)) ([5b9df00](5b9df00)) * **base-cluster/dependencies:** update helm release alloy to v1 ([#1573](#1573)) ([013a670](013a670)) * **base-cluster/dependencies:** update helm release alloy to v1.2.0 ([#1596](#1596)) ([aec70ba](aec70ba)) * **base-cluster/dependencies:** update helm release descheduler to v0.33.0 ([#1525](#1525)) ([36d8eca](36d8eca)) * **base-cluster/dependencies:** update helm release ingress-nginx to v4.12.3 ([#1511](#1511)) ([3dd2aa7](3dd2aa7)) * **base-cluster/dependencies:** update helm release kube-prometheus-stack to v75.12.0 ([#1598](#1598)) ([eec214f](eec214f)) * **base-cluster/dependencies:** update helm release kyverno to v3.4.4 ([#1512](#1512)) ([bc20fb9](bc20fb9)) * **base-cluster/dependencies:** update helm release kyverno-policies to v3.4.4 ([#1513](#1513)) ([f79dce1](f79dce1)) * **base-cluster/dependencies:** update helm release loki to v6.30.1 ([#1526](#1526)) ([4bf6daa](4bf6daa)) * **base-cluster/dependencies:** update helm release loki to v6.31.0 ([#1566](#1566)) ([b188c09](b188c09)) * **base-cluster/dependencies:** update helm release loki to v6.32.0 ([#1586](#1586)) ([57c7d86](57c7d86)) * **base-cluster/dependencies:** update helm release reflector to v9 ([#1590](#1590)) ([e679195](e679195)) * **base-cluster/dependencies:** update helm release tetragon to v1.4.1 ([#1581](#1581)) ([ff4f27d](ff4f27d)) * **base-cluster/dependencies:** update helm release traefik to v35.4.0 ([#1527](#1527)) ([9d30e5e](9d30e5e)) * **base-cluster/dependencies:** update helm release traefik to v36 ([#1592](#1592)) ([8c9cafa](8c9cafa)) * **base-cluster/dependencies:** update helm release trivy-operator to v0.29.2 ([#1528](#1528)) ([a815190](a815190)) * **base-cluster/dependencies:** update helm release trivy-operator to v0.29.3 ([#1582](#1582)) ([b078902](b078902)) * **base-cluster/dependencies:** update helm release velero to v7.2.2 ([#1172](#1172)) ([a33e9cb](a33e9cb)) * **base-cluster/dependencies:** update metrics-server docker tag to v7.4.10 ([#1576](#1576)) ([c29757a](c29757a)) * **base-cluster/dependencies:** update metrics-server docker tag to v7.4.6 ([#1514](#1514)) ([c897fbd](c897fbd)) * **base-cluster/dependencies:** update metrics-server docker tag to v7.4.9 ([#1541](#1541)) ([134e3c1](134e3c1)) * **base-cluster/dependencies:** update oauth2-proxy docker tag to v6.2.13 ([#1515](#1515)) ([0c04e1c](0c04e1c)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **New Features** * Added an alert for suspended resources in the flux component. * Reduced CPU requests for Prometheus in the monitoring component. * Updated routing to exclude non-critical alerts from on-call notifications. * Improved RBAC configurations for better OIDC account support. * **Bug Fixes** * Clarified that there is no 9.0.0 release currently. * **Chores** * Updated multiple dependencies and docker image tags. * Bumped chart version to 8.2.0. <!-- end of auto-generated comment: release notes by coderabbit.ai --> --------- Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
Summary by CodeRabbit