-
Notifications
You must be signed in to change notification settings - Fork 16.8k
[stable/prometheus-operator] bump requirements #23347
Conversation
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: MrFreezeex The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Hi @MrFreezeex. Thanks for your PR. I'm waiting for a helm member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
c227906
to
f478139
Compare
The CI is green now :) (I was using helm3 instead of helm2 to update requirements files). /assign @bismarck |
/ok-to-test |
f478139
to
6fdd4b3
Compare
@MrFreezeex can you rebase? |
Signed-off-by: Arthur Outhenin-Chalandre <arthur@cri.epita.fr>
Signed-off-by: Arthur Outhenin-Chalandre <arthur@cri.epita.fr>
Signed-off-by: Arthur Outhenin-Chalandre <arthur@cri.epita.fr>
6fdd4b3
to
e48f90f
Compare
Done! I have also updated the grafana chart to 5.5.1. |
/hold Let’s hold PRs to stable/prometheus-operator chart for a short time until prometheus-community/community#28 (comment) is resolved. If you want to help with that effort, to test etc, that would be great! |
@MrFreezeex as a user and contributor, would you mind helping to review/test this PR? prometheus-community/helm-charts#1 No need to reply here. If you can, please just reply and help there. It would help this process move along faster. Thanks! |
🚧 This chart is now deprecated. Further development has moved to prometheus-community/helm-charts. The chart has been renamed kube-prometheus-stack to more clearly reflect that it installs the If applicable, please re-open this PR there. Thanks! |
Signed-off-by: Arthur Outhenin-Chalandre arthur@cri.epita.fr
What this PR does / why we need it:
This is a simple version bump on chart dependencies. I am interested in particular with the grafana chart 5.3.1 which fixes a bug for ldap configuration.
Special notes for your reviewer:
I can only bump the grafana version, if multiple bumps in one PR is too much.
Checklist
[stable/mychartname]
)