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
Merged

Fix handler use_ec2_instance_id options #44

merged 2 commits into from Feb 6, 2013

Conversation

cbuckley01
Copy link

Pass use_ec2_instance_id attributes to options for the handler and correct 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']
Copy link
Member

Choose a reason for hiding this comment

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

I'd say (use instead of user).

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

@cbuckley01
Copy link
Author

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
Fix handler use_ec2_instance_id options
@alq666 alq666 merged commit 47ba8e6 into DataDog:master Feb 6, 2013
@alq666
Copy link
Member

alq666 commented Feb 6, 2013

👍 @cbuckley01

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.

None yet

2 participants