Skip to content

Raise the priority for 'changed' event types so they're visible#156

Merged
talwai merged 1 commit intoDataDog:masterfrom
jenkins-infra:how-low-can-you-go
Jan 19, 2016
Merged

Raise the priority for 'changed' event types so they're visible#156
talwai merged 1 commit intoDataDog:masterfrom
jenkins-infra:how-low-can-you-go

Conversation

@rtyler
Copy link
Copy Markdown
Contributor

@rtyler rtyler commented Dec 22, 2015

If your Puppet infrastructure never fails, let's say because you have a good
testing pipeline, it might not be obvious to you that any Puppet events are
being reported because everything but failures is 'low' priority. Datadog's web
UI defaults to showing only things of 'normal' priority.

This changes the priority of a 'changed' resource to 'normal' since a change of
configuration is useful information to use when correlating data or showing
events in a dashboard.

…e default UI

If your Puppet infrastructure never fails, let's say because you have a good
testing pipeline, it might not be obvious to you that any Puppet events are
being reported because everything but failures is 'low' priority. Datadog's web
UI defaults to showing only things of 'normal' priority.

This changes the priority of a 'changed' resource to 'normal' since a change of
configuration is useful information to use when correlating data or showing
events in a dashboard.
@truthbk truthbk added this to the triage milestone Jan 8, 2016
@truthbk
Copy link
Copy Markdown
Member

truthbk commented Jan 8, 2016

@rtyler thank you very much for this! We will be reviewing this shortly.

@talwai talwai modified the milestones: 1.6.0, triage Jan 11, 2016
@talwai talwai self-assigned this Jan 11, 2016
talwai added a commit that referenced this pull request Jan 19, 2016
Raise the priority for 'changed' event types so they're visible
@talwai talwai merged commit 1f77799 into DataDog:master Jan 19, 2016
cegeka-jenkins pushed a commit to cegeka/puppet-datadog_agent that referenced this pull request Jan 31, 2018
Raise the priority for 'changed' event types so they're visible
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