-
Notifications
You must be signed in to change notification settings - Fork 183
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
chore(kube-prometheus-stack): update kube-prometheus-stack chart to 39.11.0 #2446
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This pull request contains invalid labels. Please remove all of the following labels: ['do-not-merge/hold']
You should also mark this as a breaking change. |
Migration script: SumoLogic/sumologic-kubernetes-tools#275 |
3d695d1
to
41398f3
Compare
ToDo: To add CRD migrations to migration doc |
e951d29
to
980ec6b
Compare
Signed-off-by: Dominik Rosiek <drosiek@sumologic.com>
Signed-off-by: Dominik Rosiek <drosiek@sumologic.com>
Signed-off-by: Dominik Rosiek <drosiek@sumologic.com>
Signed-off-by: Dominik Rosiek <drosiek@sumologic.com>
Signed-off-by: Dominik Rosiek <drosiek@sumologic.com>
Signed-off-by: Dominik Rosiek <drosiek@sumologic.com>
Signed-off-by: Dominik Rosiek <drosiek@sumologic.com>
Signed-off-by: Dominik Rosiek <drosiek@sumologic.com>
Signed-off-by: Dominik Rosiek <drosiek@sumologic.com>
f964d34
to
0afc851
Compare
Signed-off-by: Dominik Rosiek <drosiek@sumologic.com>
Probably need to add this version to the table at https://github.com/SumoLogic/sumologic-kubernetes-collection/tree/main/deploy#support-matrix |
Signed-off-by: Dominik Rosiek <drosiek@sumologic.com>
Signed-off-by: Dominik Rosiek <drosiek@sumologic.com>
Signed-off-by: Dominik Rosiek <drosiek@sumologic.com>
Signed-off-by: Dominik Rosiek <drosiek@sumologic.com>
Signed-off-by: Dominik Rosiek <drosiek@sumologic.com>
…ated_reason metric Signed-off-by: Dominik Rosiek <drosiek@sumologic.com>
deploy/helm/sumologic/values.yaml
Outdated
repository: k8s.gcr.io/kube-state-metrics/kube-state-metrics | ||
tag: v1.9.8 | ||
# Do not collect metrics for CertificateSigningRequest or Ingress resources. | ||
# This is because the v1 version of kube-state-metrics uses API versions | ||
# that are removed in Kubernetes v1.22. | ||
collectors: | ||
certificatesigningrequests: false | ||
ingresses: false | ||
tag: v2.3.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need to specify the repository
ang tag
? Maybe we could just use the defaults now?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also I believe we could remove the kube-prometheus-stack.prometheus-node-exporter.image.tag
?
Signed-off-by: Dominik Rosiek <drosiek@sumologic.com>
Description
Update kube-state-metrics to v2.3.0. I need to revisit this PR in order to upgrade to the newest possible version and also in order to add this change to migration script
Checklist
Testing performed