Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Custom emitters #123

Merged
merged 2 commits into from
Dec 31, 2014
Merged

Custom emitters #123

merged 2 commits into from
Dec 31, 2014

Conversation

arthurnn
Copy link
Contributor

@arthurnn arthurnn commented May 5, 2014

Add custom_emitters configuration.
review @miketheman

cc @graemej

@graemej @arthurnn
add custom emitters
eac7c45 
Signed-off-by: Arthur Neves <arthurnn@gmail.com>
@miketheman
Copy link
Contributor

Thanks, will review soon.
Can also use other node attributes to do this for things we don't currently add:
https://github.com/DataDog/chef-datadog/blob/master/templates/default/datadog.conf.erb#L77-L92

@miketheman miketheman self-assigned this May 5, 2014
@miketheman miketheman added this to the Next minor milestone May 5, 2014
miketheman added a commit that referenced this pull request Dec 31, 2014
@miketheman
Merge pull request #123 from Shopify/custom_emitters
2fe916f 
Custom emitters
@miketheman miketheman merged commit 2fe916f into DataDog:master Dec 31, 2014
@miketheman
Copy link
Contributor

Thanks!

@arthurnn arthurnn deleted the custom_emitters branch December 31, 2014 18:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@miketheman miketheman

Labels
feature ready
Projects
None yet
Milestone
v2.1.0
Development

Successfully merging this pull request may close these issues.
3 participants
@arthurnn @miketheman @graemej