[GitHub] [skywalking] llinder commented on pull request #6888: Include event(s) to alarms.

[GitBox <gi...@apache.org>]

llinder commented on pull request #6888:
URL: https://github.com/apache/skywalking/pull/6888#issuecomment-841846547


   > > @kezhenxu94 Do you think we should include these events in the notification messages? If so, should we include these in all channels? I think it isn't easy to format them in slack, wechat, etc.
   > 
   > I feel like it’s unnecessary to include events in notification channels, users tend to open the web ui for more details after receiving an alert message. I don’t think one can deal with the alert only in the chat channel.
   
   Makes sense to me.  Quickly opening an alert from a message to get more context is probably better UX than adding too much info in Slack.  At the same time, being able to correlate which events might be related to an alert seems really powerful for things like web hooks.  If the events aren't delivered with the web hook that might be OK but I would imagine it would be something that an automated process monitoring alerts might be interested in.  What comes to mind are things like, notify me if a recent change of my service is correlated to another teams alert etc.  Having that context in Slack or WeChat probably isn't necessary but access from custom web hooks or other integrations might make sense.


-- 
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