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.

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

Fix handler use_ec2_instance_id options #44

Merged
merged 2 commits into from Feb 6, 2013

Conversation

Projects
None yet
2 participants
@cbuckley01
Copy link
Contributor

cbuckley01 commented Feb 6, 2013

Pass use_ec2_instance_id attributes to options for the handler and correct comments for the default attribute

cbuckley01
Pass use_ec2_instance_id attributes to options for the handler and co…
…rrect comments for the default attribute
@@ -40,7 +40,8 @@
source "chef/handler/datadog"
arguments [
:api_key => node['datadog']['api_key'],
:application_key => node['datadog']['application_key']
:application_key => node['datadog']['application_key'],
:use_ec2_instance_id => node['datadog']['user_ec2_instance_id']

This comment has been minimized.

Copy link
@alq666

alq666 Feb 6, 2013

Contributor

I'd say (use instead of user).

:use_ec2_instance_id => node['datadog']['use_ec2_instance_id']
cbuckley01
@cbuckley01

This comment has been minimized.

Copy link
Contributor Author

cbuckley01 commented Feb 6, 2013

sorry bout that, made that same typo twice, but fixed it in one spot...

alq666 added a commit that referenced this pull request Feb 6, 2013

Merge pull request #44 from novu/master
Fix handler use_ec2_instance_id options

@alq666 alq666 merged commit 47ba8e6 into DataDog:master Feb 6, 2013

1 check failed

default The Travis build could not complete due to an error
Details
@alq666

This comment has been minimized.

Copy link
Contributor

alq666 commented Feb 6, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.