Skip to content
This repository has been archived by the owner on Jun 6, 2024. It is now read-only.

[Alert manager] Add firing status to the cert expiration checker alert #5440

Merged
merged 2 commits into from
Apr 23, 2021

Conversation

yiyione
Copy link
Contributor

@yiyione yiyione commented Apr 23, 2021

Add "status": "firing" to the alert.
So the alert will be shown on the webportal home page.
image

@yiyione yiyione requested a review from Binyang2014 April 23, 2021 08:35
@coveralls
Copy link

coveralls commented Apr 23, 2021

Coverage Status

Coverage remained the same at 33.818% when pulling e9a26e6 on yiyi/fix_alert_from_cert_expiration_checker into ce9b957 on master.

@yiyione yiyione merged commit f9d98be into master Apr 23, 2021
@yiyione yiyione deleted the yiyi/fix_alert_from_cert_expiration_checker branch April 23, 2021 09:01
yiyione added a commit that referenced this pull request Apr 28, 2021
* Add docs for renew k8s API server cert

* fix kubernetes typo

* skip the drain step in master node

* change !prodheads000001 to !master-node

* delete token step should be run from master node

* fix chown commands run by other user

* [Alert manager] Add firing status to the cert expiration checker alert (#5440)

* Add firing status to the cert expiration checker alert

* update docs

* update docs
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants