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

add option in datadog_agent to extract hostnames #173

Closed
wants to merge 2 commits into from

Conversation

LeoCavaille
Copy link
Member

Fixes #160.

This option is a string regex which can be enabled and used to extract
the hostname from puppet hostname strings with a capture group.
It is useful for people that have specific puppet hostnames and that
would like to sanitize them before reporting those in Datadog.

cc @olivielpeau @diranged

There are unrelated rake lint and rake spec errors that I can fix as part of another PR.

@irabinovitch
Copy link

CC @truthbk

@truthbk
Copy link
Member

truthbk commented Apr 7, 2016

@LeoCavaille will merge this in for the next release I'm working on. I don't see any spec or lint issues with master... could you please elaborate a little more? Thanks!

@truthbk truthbk modified the milestones: triage, 1.7.0 Apr 7, 2016
@LeoCavaille
Copy link
Member Author

@truthbk boo this one is on me fixing!

Fixes DataDog#160.

This option is a string regex which can be enabled and used to extract
the hostname from puppet hostname strings with a capture group.
It is useful for people that have specific puppet hostnames and that
would like to sanitize them before reporting those in Datadog.
@LeoCavaille LeoCavaille force-pushed the leo/hostnameextract branch from 15f8ef9 to 830ffe8 Compare April 11, 2016 08:07
@LeoCavaille
Copy link
Member Author

@truthbk should be fixed! I added one more commit on top that is unrelated to my PR but makes the lint command works.

@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
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow modifications to reported hostnames
3 participants