Skip to content

[docker] set exec events' priority to low#2960

Merged
remh merged 1 commit intomasterfrom
haissam/docker-healthcheck-event-prio
Oct 27, 2016
Merged

[docker] set exec events' priority to low#2960
remh merged 1 commit intomasterfrom
haissam/docker-healthcheck-event-prio

Conversation

@hkaj
Copy link
Member

@hkaj hkaj commented Oct 26, 2016

Note: Please remember to review the Datadog Contribution Guidelines
if you have not yet done so.

What does this PR do?

Set exec events priority to low.

Motivation

Docker health checks create tons of these, let's make them discreet

Additional Notes

Hopefully nobody is relying on their priority for important stuff.
Failing health checks will be caught by service checks by #2859 anyway

@hkaj hkaj force-pushed the haissam/docker-healthcheck-event-prio branch from 15192d1 to 0cf2f30 Compare October 26, 2016 14:03
@hkaj hkaj force-pushed the haissam/docker-healthcheck-event-prio branch from 0cf2f30 to 52e31ab Compare October 26, 2016 14:10
@hkaj hkaj added this to the 5.10.0 milestone Oct 26, 2016
Copy link

@remh remh left a comment

Choose a reason for hiding this comment

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

Looks good to me

@remh remh merged commit cf6b834 into master Oct 27, 2016
@olivielpeau olivielpeau deleted the haissam/docker-healthcheck-event-prio branch October 29, 2016 00:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants