Skip to content

[service_discovery] adding jmx check for service discovery#296

Merged
truthbk merged 1 commit intoDataDog:masterfrom
alvin-huang:add_jmx_check_sd
Mar 10, 2017
Merged

[service_discovery] adding jmx check for service discovery#296
truthbk merged 1 commit intoDataDog:masterfrom
alvin-huang:add_jmx_check_sd

Conversation

@alvin-huang
Copy link
Copy Markdown
Contributor

Closes #287

@truthbk
Copy link
Copy Markdown
Member

truthbk commented Feb 8, 2017

Thanks @alvin-huang 👍 This was surely due, and makes plenty of sense given the new feature. We're going to have to pin nokogiri to a lower version, I think. Mainly because we have to maintain compatibility with older puppets.

Thanks!

@truthbk truthbk added this to the 1.10.0 milestone Mar 3, 2017
@truthbk
Copy link
Copy Markdown
Member

truthbk commented Mar 3, 2017

@alvin-huang could you be so kind as to rebase this to the latest master? That should fix up the CI issues we encountered in the past with the unwanted nokogiri upgrade (it's not pinned).

@alvin-huang
Copy link
Copy Markdown
Contributor Author

@truthbk not a problem-- rebased and ready to go!

@truthbk
Copy link
Copy Markdown
Member

truthbk commented Mar 10, 2017

Thanks for the rebase! Merging in this one!

@truthbk truthbk merged commit 83835cb into DataDog:master Mar 10, 2017
cegeka-jenkins pushed a commit to cegeka/puppet-datadog_agent that referenced this pull request Jan 31, 2018
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