chore(main): [bot] release base-cluster:10.0.0#1733
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 24 minutes and 16 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 (1)
WalkthroughBumps the charts/base-cluster Helm chart to version 10.0.0 by updating the release manifest and Chart.yaml, and adds a new 10.0.0 changelog entry listing breaking changes, features, bug fixes, and chores. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Poem
Pre-merge checks and finishing touches✅ Passed checks (5 passed)
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. Comment |
aa6f397 to
1126769
Compare
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
charts/base-cluster/README.md (1)
3516-3598: Clarify what valuesdns.zonesexpects.
external-dnstypically wants provider-specific zone identifiers (e.g. Cloudflare zone IDs), which differ from the domain names that stay indns.domains. Without spelling that out here, readers can easily feed the wrong values and zone filtering will fail silently. Please explain the expected format and add a short example for Cloudflare or whichever provider you’ve validated.-| - [zones](#dns_zones ) | No | array of string | No | - | - | +| - [zones](#dns_zones ) | No | array of string | No | - | List of provider-specific zone identifiers (e.g. Cloudflare zone IDs); falls back to domains if omitted |And extend the section below with a brief example showing the difference between
domains(FQDNs) andzones(IDs) for at least one provider.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
charts/base-cluster/Chart.yaml(2 hunks)charts/base-cluster/README.md(6 hunks)
🧰 Additional context used
🧠 Learnings (1)
📚 Learning: 2025-07-24T09:55:53.655Z
Learnt from: cwrau
PR: teutonet/teutonet-helm-charts#1601
File: charts/base-cluster/templates/dns/external-dns.yaml:30-32
Timestamp: 2025-07-24T09:55:53.655Z
Learning: In charts/base-cluster/templates/dns/external-dns.yaml, the dns.provider field in values.yaml has always been expected to be a map format (e.g., `{ cloudflare: {} }`), never a string format. The template correctly uses `{{ .Values.dns.provider | keys | first }}` to extract the provider name from the map keys.
Applied to files:
charts/base-cluster/README.md
⏰ 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). (2)
- GitHub Check: update metadata file for release
- GitHub Check: lint helm chart (base-cluster)
1b3a1e4 to
159189a
Compare
There was a problem hiding this comment.
Actionable comments posted: 1
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (3)
.github/release-please/manifest.json(1 hunks)charts/base-cluster/CHANGELOG.md(1 hunks)charts/base-cluster/Chart.yaml(1 hunks)
🚧 Files skipped from review as they are similar to previous changes (2)
- charts/base-cluster/Chart.yaml
- .github/release-please/manifest.json
⏰ 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). (2)
- GitHub Check: Update release-please config file for a possibly new chart
- GitHub Check: update metadata file for release
532e7b1 to
67a4faf
Compare
6c27264 to
577d1fc
Compare
2406e12 to
42dd719
Compare
|
🤖 Created releases: 🌻 |
🤖 I have created a release beep boop
10.0.0 (2025-10-23)
⚠ BREAKING CHANGES
Features
Bug Fixes
true, not just truthy (#1738) (7f46f4e)Miscellaneous Chores
This PR was generated with Release Please. See documentation.
Summary by CodeRabbit
Release Notes - Version 10.0.0
Breaking Changes
New Features
Bug Fixes
Chores