You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@skywalking.apache.org by GitBox <gi...@apache.org> on 2020/07/18 16:22:00 UTC

[GitHub] [skywalking] dagmom opened a new issue #5124: Alarm support alert level

dagmom opened a new issue #5124:
URL: https://github.com/apache/skywalking/issues/5124


   Please answer these questions before submitting your issue.
   
   - Why do you submit this issue?
   - [x] Question or discussion
   - [ ] Bug
   - [ ] Requirement
   - [x] Feature or performance improvement
   
   ___
   ### Question
   - Alarm support level
   
   Many monitoring system alarms support levels, such as info, warn, critical.
   I want to enhance the alarm rules with alert level.
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [skywalking] dagmom commented on issue #5124: Alarm support alert level

Posted by GitBox <gi...@apache.org>.
dagmom commented on issue #5124:
URL: https://github.com/apache/skywalking/issues/5124#issuecomment-660575585


   @wu-sheng In large data centers, there can be many alerts.
   Some alarms must be dealt with immediately. And some may cause problems, we'll deal with that later.
   For example successful rate of endpoint lower than 75% is warning, and lower than 50% is critical.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [skywalking] wu-sheng commented on issue #5124: Alarm support alert level

Posted by GitBox <gi...@apache.org>.
wu-sheng commented on issue #5124:
URL: https://github.com/apache/skywalking/issues/5124#issuecomment-660572946


   What are the use cases of that? As the SkyWalking doesn't react from it, can't see the real value. 


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [skywalking] dagmom commented on issue #5124: Alarm support alert level

Posted by GitBox <gi...@apache.org>.
dagmom commented on issue #5124:
URL: https://github.com/apache/skywalking/issues/5124#issuecomment-660579401


   @wu-sheng 
   Like prometheus alertmanager?
   ```yml
   groups:
   - name: PromRules
     rules:
     - alert: Node-Load
       annotations: {description: 'descTest',
         summary: 'summaryTest'}
       expr: node_load1 > 0.01
       for: 30s
       labels: {resType: 'Node',severity: 'warning'}
     - alert: Node-Load
       annotations: {description: 'descTest',
         summary: 'summaryTest'}
       expr: node_load1 > 0.02
       for: 30s
       labels: {resType: 'Node',severity: 'critical'}
   ```
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [skywalking] wu-sheng closed issue #5124: Support alarm labels

Posted by GitBox <gi...@apache.org>.
wu-sheng closed issue #5124:
URL: https://github.com/apache/skywalking/issues/5124


   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [skywalking] dagmom edited a comment on issue #5124: Alarm support alert level

Posted by GitBox <gi...@apache.org>.
dagmom edited a comment on issue #5124:
URL: https://github.com/apache/skywalking/issues/5124#issuecomment-660579401


   @wu-sheng 
   Like prometheus alertmanager?
   ```yml
   groups:
   - name: PromRules
     rules:
     - alert: Node-Load
       annotations: {description: 'descTest',
         summary: 'summaryTest'}
       expr: node_load1 > 0.01
       for: 30s
       labels: {resType: 'Node',severity: 'warning'}
     - alert: Node-Load
       annotations: {description: 'descTest',
         summary: 'summaryTest'}
       expr: node_load1 > 0.02
       for: 30s
       labels: {resType: 'Node',severity: 'critical'}
   ```
   I agree that this is a good way.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [skywalking] wu-sheng commented on issue #5124: Support alarm labels

Posted by GitBox <gi...@apache.org>.
wu-sheng commented on issue #5124:
URL: https://github.com/apache/skywalking/issues/5124#issuecomment-817110742


   > @wu-sheng I think this feature is useful when we have a lot of alarm rules to manage. Now I distinguish by keywords in messge. if there are labels it will be more convenient. I will solve this issue.
   
   Like the above comments, yes, labels make sense. This feature just ends w/o more processing. You are involving in many issues already, we only respond to the developers contributing to the ecosystem, as too many people asking.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [skywalking] liqiangz commented on issue #5124: Support alarm labels

Posted by GitBox <gi...@apache.org>.
liqiangz commented on issue #5124:
URL: https://github.com/apache/skywalking/issues/5124#issuecomment-817084582


   @wu-sheng I think this feature is useful when we have a lot of alarm rules to manage. Now I distinguish by keywords in messge. if there are labels it will be more convenient. I will solve this issue.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [skywalking] wu-sheng commented on issue #5124: Alarm support alert level

Posted by GitBox <gi...@apache.org>.
wu-sheng commented on issue #5124:
URL: https://github.com/apache/skywalking/issues/5124#issuecomment-660576575


   OK, If you are talking about large scale env, I would recommend bringing the `tag` system into an alarm message. For alarm rule, you could set multiple tags(name: value), if the alarm happens, these tags would be sent with message too. What do you think?


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [skywalking] wu-sheng commented on issue #5124: Alarm support alert level

Posted by GitBox <gi...@apache.org>.
wu-sheng commented on issue #5124:
URL: https://github.com/apache/skywalking/issues/5124#issuecomment-660590925


   About the `labels` part, yes. 


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [skywalking] wu-sheng commented on issue #5124: Support alarm labels

Posted by GitBox <gi...@apache.org>.
wu-sheng commented on issue #5124:
URL: https://github.com/apache/skywalking/issues/5124#issuecomment-703231917


   No update.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org