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

[kubernetes] add kubernetes container name #3368

Merged
merged 1 commit into from Jul 10, 2017
Merged

Conversation

hkaj
Copy link
Member

@hkaj hkaj commented Jun 2, 2017

Note: Please remember to review the Datadog Contribution Guidelines
if you have not yet done so.

What does this PR do?

Add the kubernetes container name to auto discovery's additional tags.

Motivation

Inspired by #3365

Testing Guidelines

An overview on testing
is available in our contribution guidelines.

Additional Notes

Anything else we should know when reviewing?

@hkaj hkaj requested a review from xvello June 2, 2017 09:25
@hkaj hkaj added this to the 5.15 milestone Jun 2, 2017
Copy link
Contributor

@xvello xvello left a comment

Choose a reason for hiding this comment

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

LGTM, let's merge once 5.14.x is branched-off

@hkaj hkaj merged commit e924aff into master Jul 10, 2017
@hkaj hkaj deleted the haissam/ad-k8s-cname branch July 10, 2017 09:11
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

2 participants