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

[consul] Support multiple instances #2948

Merged
merged 1 commit into from
Oct 21, 2016

Conversation

olivielpeau
Copy link
Member

What does this PR do?

Support multiple instances of the consul check

Motivation

Customer request: the Agent needs to support multiple instances of the check when an Agent monitors multiple containers running consul servers.

Testing Guidelines

Updated the mock tests.

Additional Notes

Also adds support of yaml-configured tags on each instance to
distinguish data from different instances running on the same Agent

Also adds support of yaml-configured tags on each instance to
distinguish data from different instances running on the same Agent
@olivielpeau olivielpeau added this to the 5.10.0 milestone Oct 21, 2016
@yannmh yannmh self-assigned this Oct 21, 2016
@yannmh
Copy link
Member

yannmh commented Oct 21, 2016

Looks good to me !

@olivielpeau olivielpeau merged commit 18262e2 into master Oct 21, 2016
@olivielpeau olivielpeau deleted the olivielpeau/multiple-consul-instances branch October 21, 2016 20:20
jcejohnson pushed a commit to EFXCIA/dd-agent that referenced this pull request Oct 24, 2016
Also adds support of yaml-configured tags on each instance to
distinguish data from different instances running on the same Agent
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.

2 participants