Fix Helm chart deprecation notice in dashboard 1.84.0 blog#970
Fix Helm chart deprecation notice in dashboard 1.84.0 blog#970petersutter wants to merge 1 commit into
Conversation
Distinguish between the two deprecation cases: - gardener-dashboard chart: deprecated in favor of gardener-operator - identity chart: deprecated without replacement
✅ Deploy Preview for gardener-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (1)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including ⚙️ Run configurationConfiguration used: Path: .coderabbit.yml Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
|
/invite @grolu |
n-boshnakov
left a comment
There was a problem hiding this comment.
/lgtm
Waiting on @grolu's review, I will merge afterwards. Ping me if you want me to merge the PR right away.
|
LGTM label has been added. DetailsGit tree hash: 7093189810a7314c450e5c3fb33ee901eb5cbe06 |
/kind cleanup
What this PR does / why we need it:
Corrects the Helm Chart Deprecation Notice section to distinguish between the two deprecation cases:
gardener-dashboardchart: deprecated in favor ofgardener-operator(chore(charts): deprecate helm charts in favor of gardener-operator dashboard#2935)identitychart: deprecated without replacement (docs(charts): correct identity chart deprecation notice dashboard#2963)Which issue(s) this PR fixes:
Follow-up to #963
Special notes for your reviewer: