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.

Sign up for GitHub

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

Jump to bottom

[jmx] fix jmx commands false warning 🐛 #1612

Merged
merged 1 commit into from
May 12, 2015
Merged

[jmx] fix jmx commands false warning 🐛 #1612

merged 1 commit into from
May 12, 2015

Conversation

yannmh
Copy link
Member

@yannmh yannmh commented May 11, 2015

Bugfix 🐛
jmx commands print a false warning, when it just ran fine.
Command outputs were ending with

Couldn't find any valid JMX configuration in your conf.d directory: /etc/dd-agent/conf.d
Have you enabled any JMX check ?
If you think it's not normal please get in touch with Datadog Support

@yannmh yannmh added the bugfix label May 11, 2015
@yannmh yannmh self-assigned this May 11, 2015
@yannmh yannmh added this to the 5.4.0 milestone May 11, 2015
@remh
Copy link

remh commented May 12, 2015

looks good to me

yannmh added a commit that referenced this pull request May 12, 2015
@yannmh
Merge pull request #1612 from DataDog/yann/jmx-check-cmd
de7e6f3 
[jmx] fix `jmx` commands false warning 🐛
@yannmh yannmh merged commit de7e6f3 into master May 12, 2015
@yannmh yannmh deleted the yann/jmx-check-cmd branch May 13, 2015 18:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@yannmh yannmh

Labels
bugfix
Projects
None yet
Milestone
5.4.0
Development

Successfully merging this pull request may close these issues.
2 participants
@yannmh @remh