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

Parameter expansion #79

Closed
wants to merge 8 commits into from
Closed

Conversation

obowersa
Copy link

Pull request to expand the current testing spec to cover paramaters in datadog.conf.erb in addition to expanding the parameters to cover (almost) all config options.

Not necessarily expecting this to get pulled but it's probably worth figuring out of this is a good approach to take.

@LeoCavaille
Copy link
Member

Nice work! We've queued some work on the puppet module, and try to figure out a solution to the problem you raised. We'll review that by then.
Thanks again for contributing @obowersa

@LeoCavaille LeoCavaille added this to the 2.0.0 milestone Apr 28, 2015
@ryandoherty
Copy link

Any info on when this will be merged? I've run in to a problem where I can't change the listen address, which is a PITA.

@truthbk
Copy link
Member

truthbk commented Apr 8, 2016

@ryandoherty I'm going to try to squeeze this into 1.7.0, I'm running some tests to make sure everything looks good. It's been a long time coming, our apologies for that.

@truthbk truthbk modified the milestones: 1.7.0, 2.0.0 Apr 8, 2016
@truthbk
Copy link
Member

truthbk commented Apr 12, 2016

Included in #176, thanks!

@truthbk truthbk closed this Apr 12, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants