Skip to content

Add kubernetes autodiscover docs#6065

Merged
exekias merged 1 commit intoelastic:masterfrom
vjsamuel:kube_autodocs
Jan 15, 2018
Merged

Add kubernetes autodiscover docs#6065
exekias merged 1 commit intoelastic:masterfrom
vjsamuel:kube_autodocs

Conversation

@vjsamuel
Copy link
Copy Markdown
Contributor

No description provided.

@elasticmachine
Copy link
Copy Markdown
Contributor

Can one of the admins verify this patch?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

there is no need for this line this time, as Add autodiscover for kubernetes is already there :)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This is a very good example for the configuration! Perhaps we should use the prometheus.io.scrape: "true" annotation?

@exekias
Copy link
Copy Markdown
Contributor

exekias commented Jan 15, 2018

Thank you for taking this @vjsamuel, I left some comments

@vjsamuel
Copy link
Copy Markdown
Contributor Author

@exekias done

Copy link
Copy Markdown
Contributor

@exekias exekias left a comment

Choose a reason for hiding this comment

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

LGTM

@exekias
Copy link
Copy Markdown
Contributor

exekias commented Jan 15, 2018

ok to test

@exekias exekias merged commit 30e8ccb into elastic:master Jan 15, 2018
@vjsamuel vjsamuel deleted the kube_autodocs branch January 15, 2018 20:47
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.

3 participants