-
Notifications
You must be signed in to change notification settings - Fork 45
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
Bump prometheus-operator version to 8.13.0 #2557
Bump prometheus-operator version to 8.13.0 #2557
Conversation
Hello ebaneck,My role is to assist you with the merge of this Status report is not available. |
Integration data createdI have created the integration data for the additional destination branches.
The following branches will NOT be impacted:
You can set option
|
/approve |
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
Peer approvals must include at least 1 approval from the following list:
|
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
Peer approvals must include at least 1 approval from the following list:
The following options are set: approve |
/after_pull_request=2546 |
Waiting for other pull request(s)The current pull request is locked by the after_pull_request option. In order for me to merge this pull request, run the following actions first: ➡️ Merge the
Alternatively, delete all the after_pull_request comments from this pull request. The following options are set: approve, after_pull_request |
This chart is updated using: ``` $ rm -rf charts/prometheus-operator $ helm repo update $ helm fetch -d charts --untar stable/prometheus-operator $ ./charts/render.py prometheus-operator --namespace metalk8s-monitoring charts/prometheus-operator.yaml --service-config grafana metalk8s-grafana-config --service-config prometheus metalk8s-prometheus-config --service-config alertmanager metalk8s-alertmanager-config charts/prometheus-operator/ > salt/metalk8s/addons/prometheus-operator/deployed/chart.sls ```
Since we have updated the version of the prometheus-operator chart to version 8.13.0, we need to bump the versions of it's component in the buildchain ``` The following components now run with version numbers: Prometheus-operator => 0.38.1 Prometheus-config-reloader => 0.38.1 Prometheus => 2.16.0 kube-state-metrics => 1.9.5 Alertmanager => 0.20.0 Grafana => 6.7.1 ``` Closes: #2430
Grafana app labels have changed from `app=grafana` to `app.kubernetes.io/name=grafana` and so we need to make this changes.
Since we are updating the Prometheus-operator charts to `8.13.0`, we indeed check and document the new rules by running the rule_extractor python script and save the new changes introduced.
4b5a32c
to
f8ac04f
Compare
/reset |
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
Peer approvals must include at least 1 approval from the following list:
The following options are set: after_pull_request, approve |
/status |
StatusStatus report is not available. The following options are set: after_pull_request, approve |
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
Peer approvals must include at least 1 approval from the following list:
The following options are set: after_pull_request, approve |
In the queueThe changeset has received all authorizations and has been added to the The changeset will be merged in:
The following branches will NOT be impacted:
There is no action required on your side. You will be notified here once IMPORTANT Please do not attempt to modify this pull request.
If you need this pull request to be removed from the queue, please contact a The following options are set: after_pull_request, approve |
I have successfully merged the changeset of this pull request
The following branches have NOT changed:
Please check the status of the associated issue None. Goodbye ebaneck. |
Component:
'charts', 'prometheus-operator'
Context:
See #2430
Summary:
Now, that the python-kubernetes version has been updated, we can proceed
with the chart upgrade.
The following issue #2430 (comment) raised previously while upgrading the charts should have been fixed after the python-kubernetes version was bumped.
Acceptance criteria:
Green build
Closes: #2430