-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Enable some form of Slack notifications parsing #586
Comments
I think that makes sense. I only wonder whether it should be configurable (or always on), but probably configurable. |
is this gonna be merged any time soon ? |
Following up now, sorry for letting this slip |
We have same issue, can you fix it? |
* Resolves prometheus/node_exporter#585 * - removed 'docker rm' as it is not allowed on CircleCI See discussion: https://discuss.circleci.com/t/docker-error-removing-intermediate-container/70
#912 added this. |
Looks like nowadays |
Hi,
right now mentioning slack users or teams is pretty tedious as it has to be done with slack IDs. Think
<!subteam^ID|team>
or<!@U12345|nickname>
instead of normal@team
or@nickname
.This is because slack does not parse messages received from webhooks (default
parse=none
).I think
parse=full
is unnecessary, but addinglink_names=1
herealertmanager/notify/impl.go
Line 465 in 2433eeb
What do you think? Are there any reasons why this is not there or will you accept a simple PR? :)
The text was updated successfully, but these errors were encountered: