chore(main): [bot] release base-cluster:8.2.0#1530
Conversation
|
Warning Rate limit exceeded@teutonet-bot has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 8 minutes and 30 seconds before requesting another review. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📒 Files selected for processing (2)
WalkthroughThe changes update the base-cluster Helm chart from version 8.1.0 to 8.2.0. This includes updating version references in the manifest and chart metadata, adding a new changelog entry for 8.2.0 with features, bug fixes, and dependency updates, but no source URL changes. Changes
Assessment against linked issues
Possibly related PRs
Suggested reviewers
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 (
|
fb98612 to
1d6f8a5
Compare
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
charts/base-cluster/README.md (1)
249-250: Normalize list item style to use dashes.markdownlint flags unordered lists using
*. Switch to-for consistency with linting rules.- * https://github.com/teutonet/teutonet-helm-charts/tree/base-cluster-v8.1.1/charts/base-cluster - * https://github.com/teutonet/teutonet-helm-charts/tree/main/charts/base-cluster + - https://github.com/teutonet/teutonet-helm-charts/tree/base-cluster-v8.1.1/charts/base-cluster + - https://github.com/teutonet/teutonet-helm-charts/tree/main/charts/base-cluster🧰 Tools
🪛 markdownlint-cli2 (0.17.2)
249-249: Unordered list style
Expected: dash; Actual: asterisk(MD004, ul-style)
250-250: Unordered list style
Expected: dash; Actual: asterisk(MD004, ul-style)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
charts/base-cluster/Chart.yaml(3 hunks)charts/base-cluster/README.md(3 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
- charts/base-cluster/Chart.yaml
🧰 Additional context used
🪛 markdownlint-cli2 (0.17.2)
charts/base-cluster/README.md
249-249: Unordered list style
Expected: dash; Actual: asterisk
(MD004, ul-style)
⏰ Context from checks skipped due to timeout of 90000ms (3)
- GitHub Check: update metadata file for release
- GitHub Check: lint helm chart (base-cluster)
- GitHub Check: wait-for-checks
🔇 Additional comments (2)
charts/base-cluster/README.md (2)
3-3: Approve version badge update.The version badge has been correctly updated from 8.1.0 to 8.1.1 to match the new release.
258-258: Approve dependency table version bump.The
commonchart version has been correctly incremented to 1.5.0 in the dependencies table.
🤖 I have diffed this beep boop"/$namespace/$kind/$name.yaml" for normal resources
|
f7eea01 to
ef9c34f
Compare
20dd29c to
e50194b
Compare
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
charts/base-cluster/README.md (1)
249-250: Standardize list style per markdownlint.
markdownlintrecommends using dashes (-) instead of asterisks (*) for unordered lists.
Apply this diff for consistency:- * https://github.com/teutonet/teutonet-helm-charts/tree/base-cluster-v8.2.0/charts/base-cluster - * https://github.com/teutonet/teutonet-helm-charts/tree/main/charts/base-cluster + - https://github.com/teutonet/teutonet-helm-charts/tree/base-cluster-v8.2.0/charts/base-cluster + - https://github.com/teutonet/teutonet-helm-charts/tree/main/charts/base-cluster
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (4)
.github/release-please/manifest.json(1 hunks)charts/base-cluster/CHANGELOG.md(1 hunks)charts/base-cluster/Chart.yaml(1 hunks)charts/base-cluster/README.md(3 hunks)
✅ Files skipped from review due to trivial changes (1)
- .github/release-please/manifest.json
🚧 Files skipped from review as they are similar to previous changes (2)
- charts/base-cluster/Chart.yaml
- charts/base-cluster/CHANGELOG.md
🧰 Additional context used
🪛 markdownlint-cli2 (0.17.2)
charts/base-cluster/README.md
249-249: Unordered list style
Expected: dash; Actual: asterisk
(MD004, ul-style)
⏰ Context from checks skipped due to timeout of 90000ms (2)
- GitHub Check: update metadata file for release
- GitHub Check: lint helm chart (base-cluster)
🔇 Additional comments (2)
charts/base-cluster/README.md (2)
3-3: Updated version badge correctly.The chart version badge now reflects
8.2.0, matching the release.
258-258: Verified dependency version bump.The
commonchart dependency version was updated to1.5.0as expected for this release.
2812dff to
c6bdf3e
Compare
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
charts/base-cluster/README.md (1)
249-250: Consistent unordered list style.
The list under “Source Code” uses*markers, but the project follows dash-style bullets.Apply this diff:
- * https://github.com/teutonet/teutonet-helm-charts/tree/base-cluster-v8.2.0/charts/base-cluster - * https://github.com/teutonet/teutonet-helm-charts/tree/main/charts/base-cluster + - https://github.com/teutonet/teutonet-helm-charts/tree/base-cluster-v8.2.0/charts/base-cluster + - https://github.com/teutonet/teutonet-helm-charts/tree/main/charts/base-cluster
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
charts/base-cluster/Chart.yaml(1 hunks)charts/base-cluster/README.md(3 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
- charts/base-cluster/Chart.yaml
🧰 Additional context used
🪛 markdownlint-cli2 (0.17.2)
charts/base-cluster/README.md
249-249: Unordered list style
Expected: dash; Actual: asterisk
(MD004, ul-style)
🔇 Additional comments (2)
charts/base-cluster/README.md (2)
3-3: Approve version badge update.
The chart version badge has been correctly updated to 8.2.0, matching the new release metadata.
258-258: Approve dependency version bump.
Thecommonchart requirement has been correctly updated to 1.5.0.
2386365 to
7dbb77b
Compare
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
charts/base-cluster/README.md (1)
249-251: Conform to markdownlint-cli MD004 (list marker style).The linter expects dashes (
-) for unordered lists, but an asterisk (*) is used here. Aligning with the existing lint configuration avoids CI noise.-* <https://github.com/teutonet/teutonet-helm-charts/tree/base-cluster-v8.2.0/charts/base-cluster> -* <https://github.com/teutonet/teutonet-helm-charts/tree/main/charts/base-cluster> +- <https://github.com/teutonet/teutonet-helm-charts/tree/base-cluster-v8.2.0/charts/base-cluster> +- <https://github.com/teutonet/teutonet-helm-charts/tree/main/charts/base-cluster>
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
charts/base-cluster/Chart.yaml(1 hunks)charts/base-cluster/README.md(3 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
- charts/base-cluster/Chart.yaml
🧰 Additional context used
🪛 markdownlint-cli2 (0.17.2)
charts/base-cluster/README.md
249-249: Unordered list style
Expected: dash; Actual: asterisk
(MD004, ul-style)
⏰ Context from checks skipped due to timeout of 90000ms (2)
- GitHub Check: update metadata file for release
- GitHub Check: lint helm chart (base-cluster)
🔇 Additional comments (2)
charts/base-cluster/README.md (2)
258-258: Dependency version bump acknowledged.
commonsub-chart reference updated to1.5.0; change is consistent with the release notes.
3-3: VerifyChart.yamlversion matches the badge.The README shows
8.2.0; double-check thatcharts/base-cluster/Chart.yamland anyappVersionfields were bumped in lock-step to avoid version skew.#!/bin/bash # Fail if the Chart.yaml version differs from the README badge. badge_version="8.2.0" yaml_version=$(grep -E '^version:' charts/base-cluster/Chart.yaml | awk '{print $2}') test "$badge_version" = "$yaml_version" || { echo "Chart.yaml version ($yaml_version) differs from README badge ($badge_version)"; exit 1; }
0da706b to
ea84b3d
Compare
09354bf to
1c6e29c
Compare
e42e60e to
111141d
Compare
3fdc47b to
9288e54
Compare
8610536 to
48d8af1
Compare
|
🤖 Created releases: 🌻 |
🤖 I have created a release beep boop
8.2.0 (2025-07-21)
Features
Bug Fixes
Miscellaneous Chores
This PR was generated with Release Please. See documentation.
Summary by CodeRabbit
New Features
Bug Fixes
Chores