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

Add MQTT output. #241

Closed
wants to merge 6 commits into from
Closed

Add MQTT output. #241

wants to merge 6 commits into from

Conversation

shirou
Copy link
Contributor

@shirou shirou commented Oct 4, 2015

Add output which sends metrics to MQTT server.

The topic format is

<topic_prefix>/host/<hostname>/<pluginname>/<metric_name>
ex: prefix/host/web1.example.com/mem/available

hostname in a topic is specified in the outputs.mqtt config section, not agent Because I could not found how to inherit hostname value from agant.

@sparrc
Copy link
Contributor

sparrc commented Oct 5, 2015

@shirou you should not have to manually update Godeps.json, try doing this:

$ go get github.com/tools/godep
$ godep save ./...

@sparrc
Copy link
Contributor

sparrc commented Oct 5, 2015

@shirou thank you for the contribution! 👍 You shouldn't need to inherit the hostname tag, the agent should add that tag automatically in addition to the ones you add within this plugin

@shirou
Copy link
Contributor Author

shirou commented Oct 5, 2015

@sparrc Thank you for the comments. I type

godep restore
godep save ./...

Then, the other packages are also updated. If I made a mistake, I will revert it.

And, Thank you for the suggestion about hostname. I removed the settings in the MQTT section.

@sparrc sparrc closed this in 63e9a4a Oct 5, 2015
@sparrc
Copy link
Contributor

sparrc commented Oct 5, 2015

Thanks @shirou! There was one problem with the Godep.json file. It was trying to manage the telegraf repo itself as a dependency. I'm not sure how that happened but I rebased your changes and removed that. Thanks again!

@sparrc
Copy link
Contributor

sparrc commented Oct 5, 2015

@shirou don't forget to sign up for Hacktoberfest! https://hacktoberfest.digitalocean.com/, between this and gopsutil you should be able to get a free t-shirt ;-D

@sparrc sparrc mentioned this pull request Oct 13, 2015
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants