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

fix alert label error #2521

Merged
merged 1 commit into from
Apr 9, 2019
Merged

fix alert label error #2521

merged 1 commit into from
Apr 9, 2019

Conversation

xudifsd
Copy link
Member

@xudifsd xudifsd commented Apr 9, 2019

fix alert label error in pai_node_count alert.

This metric is scrapped by watchdog, so the instance label will be where watchdog was deployed. We can not use this label, there are name label in watchdog to represent the node's IP, so we should use name here instead of instance.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.01%) to 52.943% when pulling e462bc3 on dixu/fix-alert-label-error into 5b3f044 on master.

3 similar comments
@coveralls
Copy link

Coverage Status

Coverage increased (+0.01%) to 52.943% when pulling e462bc3 on dixu/fix-alert-label-error into 5b3f044 on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.01%) to 52.943% when pulling e462bc3 on dixu/fix-alert-label-error into 5b3f044 on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.01%) to 52.943% when pulling e462bc3 on dixu/fix-alert-label-error into 5b3f044 on master.

@xudifsd xudifsd merged commit ebb1273 into master Apr 9, 2019
@xudifsd xudifsd deleted the dixu/fix-alert-label-error branch April 9, 2019 10:06
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.

None yet

3 participants