Skip to content

Conversation

erplsf
Copy link
Contributor

@erplsf erplsf commented Jan 29, 2021

We noticed a use case that was not covered - updates from alertmanager updated the summary but not the description.
That's a big piece of missing functionality - we would like the description to also be updated on new alerts, not only summary.

We also think it would be useful for other users, so that's why this pull-request is here.

Signed-off-by: Andriy Mykhaylyk <[email protected]>
Copy link
Member

@bwplotka bwplotka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. this feels clean and nice to have.

Thank you!

Copy link
Member

@erdii erdii left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm! thank you very much - this is a pain point, that I have experienced as well!

@erdii erdii merged commit a288def into prometheus-community:master Mar 5, 2021
erdii added a commit to erdii/jiralert that referenced this pull request Mar 9, 2021
erdii added a commit to erdii/jiralert that referenced this pull request Mar 10, 2021
erdii added a commit that referenced this pull request Mar 10, 2021
* hash group labels to get the issue search label

Signed-off-by: Josh Gwosdz <[email protected]>

* make group label hashing easier to read

and disable errcheck linting for hash.Write call

Signed-off-by: Josh Gwosdz <[email protected]>

* run go mod tidy

Signed-off-by: Josh Gwosdz <[email protected]>

* hide label hashing behavior behind flag

Signed-off-by: Josh Gwosdz <[email protected]>

* ignore never occuring error the go way 💪

Signed-off-by: Josh Gwosdz <[email protected]>

* use JIRALERT{hash...} issue label to make the origin clear

Signed-off-by: Josh Gwosdz <[email protected]>

* document -hash-jira-label

Signed-off-by: Josh Gwosdz <[email protected]>

* document -hash-jira-label changes

Signed-off-by: Josh Gwosdz <[email protected]>

* adapt tests of #75

Signed-off-by: Josh Gwosdz <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants