Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[release-v2.7] Remove logging and monitoring charts #614

Merged

Conversation

mallardduck
Copy link
Member

@mallardduck mallardduck commented Sep 30, 2024

A release- targeting version of: #613

RE:

Per recent slack chats, these charts are not necessary for use anymore as all logging and monitoring charts should come from rancher/charts repo for 2.7+; doing a manual verification I cannot find references to indicate that these should be kept around.

If anyone happens to incorrectly still be using these versions then they likely have larger issues than removing this will cause and should opt to update to a version coming from rancher/charts sources. We can likely also backport this to 2.7 as well - though that is less urgent/necessary as that version will go EOL sooner than later.

Copy link
Contributor

@nicholasSUSE nicholasSUSE left a comment

Choose a reason for hiding this comment

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

Assuming these charts are not being used by anyone, as per the PR owner.
LGTM.

@nicholasSUSE nicholasSUSE merged commit a0fc936 into rancher:release-v2.7 Oct 1, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants