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

Ability to override default log file directory #273

Closed

Conversation

Projects
None yet
2 participants
@SupermanScott
Copy link
Contributor

commented Feb 4, 2016

Summary:
This adds a node attribute that sets up the directory for the collector,
forwarder, dogstatsd and pup log files. Doing this allows consumers of
the recipe to change the directory from node attributes.

Scott Reynolds
Ability to override default log file directory
Summary:
This adds a node attribute that sets up the directory for the collector,
forwarder, dogstatsd and pup log files. Doing this allows consumers of
the recipe to change the directory from node attributes.

@miketheman miketheman added the feature label Mar 12, 2016

@miketheman miketheman added this to the Next minor milestone Mar 12, 2016

@miketheman

This comment has been minimized.

Copy link
Collaborator

commented Mar 12, 2016

Thanks! We'll merge in the next minor release cycle.

collector_log_file: <%= node['datadog']['log_file_directory'] %>/collector.log
forwarder_log_file: <%= node['datadog']['log_file_directory'] %>/forwarder.log
dogstatsd_log_file: <%= node['datadog']['log_file_directory'] %>/dogstatsd.log
pup_log_file: <%= node['datadog']['log_file_directory'] %>/pup.log

This comment has been minimized.

Copy link
@miketheman

@miketheman miketheman self-assigned this Apr 21, 2016

miketheman added a commit that referenced this pull request Apr 21, 2016

Ability to override default log file directory (#273)
Summary:
This adds a node attribute that sets up the directory for the collector,
forwarder, dogstatsd and pup log files. Doing this allows consumers of
the recipe to change the directory from node attributes.
@miketheman

This comment has been minimized.

Copy link
Collaborator

commented Apr 21, 2016

Merged in ba34f7e
Thanks!

@miketheman miketheman closed this Apr 21, 2016

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.