Skip to content

Fix Helm chart deprecation notice in dashboard 1.84.0 blog#970

Open
petersutter wants to merge 1 commit into
masterfrom
fix/helm-chart-deprecation-notice
Open

Fix Helm chart deprecation notice in dashboard 1.84.0 blog#970
petersutter wants to merge 1 commit into
masterfrom
fix/helm-chart-deprecation-notice

Conversation

@petersutter
Copy link
Copy Markdown
Member

/kind cleanup

What this PR does / why we need it:

Corrects the Helm Chart Deprecation Notice section to distinguish between the two deprecation cases:

Which issue(s) this PR fixes:

Follow-up to #963

Special notes for your reviewer:

Distinguish between the two deprecation cases:
- gardener-dashboard chart: deprecated in favor of gardener-operator
- identity chart: deprecated without replacement
@petersutter petersutter requested a review from a team as a code owner May 12, 2026 21:26
@gardener-prow gardener-prow Bot added the kind/cleanup Something that is not needed anymore and can be cleaned up label May 12, 2026
@netlify
Copy link
Copy Markdown

netlify Bot commented May 12, 2026

Deploy Preview for gardener-docs ready!

Name Link
🔨 Latest commit 3782a44
🔍 Latest deploy log https://app.netlify.com/projects/gardener-docs/deploys/6a039b22b3f25900092ea6c2
😎 Deploy Preview https://deploy-preview-970--gardener-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@gardener-prow
Copy link
Copy Markdown

gardener-prow Bot commented May 12, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign kostov6 for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 12, 2026

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (1)
  • website/blog/2026/05/05-11-gardener-dashboard-1840.md is excluded by !website/**

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

Run ID: e431f295-470f-4f9f-89b7-67a514435d8a

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/helm-chart-deprecation-notice

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@gardener-prow gardener-prow Bot added size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. cla: yes Indicates the PR's author has signed the cla-assistant.io CLA. labels May 12, 2026
@petersutter
Copy link
Copy Markdown
Member Author

/invite @grolu

Copy link
Copy Markdown
Contributor

@n-boshnakov n-boshnakov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

Waiting on @grolu's review, I will merge afterwards. Ping me if you want me to merge the PR right away.

@gardener-prow gardener-prow Bot added the lgtm Indicates that a PR is ready to be merged. label May 13, 2026
@gardener-prow
Copy link
Copy Markdown

gardener-prow Bot commented May 13, 2026

LGTM label has been added.

DetailsGit tree hash: 7093189810a7314c450e5c3fb33ee901eb5cbe06

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes Indicates the PR's author has signed the cla-assistant.io CLA. kind/cleanup Something that is not needed anymore and can be cleaned up lgtm Indicates that a PR is ready to be merged. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants