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

remove cgo dependeny with forking sensors command #1414

Closed
wants to merge 1 commit into from
Closed

remove cgo dependeny with forking sensors command #1414

wants to merge 1 commit into from

Conversation

zbindenren
Copy link
Contributor

@zbindenren zbindenren commented Jun 24, 2016

Plugin rewritten, fork sensor exec instead of cgo dependeny.
This pull request closes #649

Required for all PRs:

  • CHANGELOG.md updated
  • Sign CLA (if not already signed)
  • README.md updated (if adding a new plugin)

## Remove numbers from field names.
## If true, a field name like 'temp1_input' will be changed to 'temp_input'.
# remove_numbers = true
#
Copy link
Contributor

Choose a reason for hiding this comment

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

please remove the trailing #

@sparrc
Copy link
Contributor

sparrc commented Jul 19, 2016

looks good, just the trailing # and then update the changelog and I'll merge

@sparrc sparrc closed this in 1989a58 Aug 9, 2016
aurrelhebert pushed a commit to aurrelhebert/telegraf that referenced this pull request Aug 9, 2016
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.

'sensors' input unavailable for telegraf-0.10.2-1.x86_64 RPM
2 participants