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.

Sign up for GitHub

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

Jump to bottom

kube-prometheus-stack to 58.1.3 #120

Merged
merged 1 commit into from
Apr 22, 2024
Merged

kube-prometheus-stack to 58.1.3 #120

merged 1 commit into from
Apr 22, 2024

Conversation

Bdegraaf1234
Copy link
Collaborator

Description of the change

Update the kube-prometheus-stack chart from 47.1.3 to 58.1.3

Benefits

Add MSTeams support, use LTS versions of components

Possible drawbacks

Backwards compatibility?

Applicable issues

NA

Additional information

This code was largely modeled after f23277f

Checklist

  • Chart version bumped in Chart.yaml according to semver.
  • Variables are documented in the README.md
  • Title of the PR starts with chart name (e.g. [<name_of_the_chart>])

@Bdegraaf1234 Bdegraaf1234 self-assigned this Apr 22, 2024
@Bdegraaf1234 Bdegraaf1234 requested a review from keyvaann April 22, 2024 11:12
keyvaann
keyvaann approved these changes Apr 22, 2024
View reviewed changes
Copy link
Collaborator

@keyvaann keyvaann left a comment

Choose a reason for hiding this comment

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

LGTM

@Bdegraaf1234 Bdegraaf1234 merged commit 1edc6dd into main Apr 22, 2024
4 checks passed
@Bdegraaf1234 Bdegraaf1234 deleted the update-kube-promethues-stack branch April 22, 2024 11:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@keyvaann keyvaann keyvaann approved these changes

Assignees

@Bdegraaf1234 Bdegraaf1234

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Bdegraaf1234 @keyvaann