Skip to content

fix hostname in output - #130

Open
bmx0r wants to merge 9 commits into
sensu-plugins:masterfrom
bmx0r:master
Open

bmx0r wants to merge 9 commits into
sensu-plugins:masterfrom
bmx0r:master

Conversation

@bmx0r

@bmx0r bmx0r commented May 27, 2020

Copy link
Copy Markdown

Pull Request Checklist

Is this in reference to an existing issue?
no

General

  • Update Changelog following the conventions laid out here

  • Update README with any necessary configuration snippets

  • Binstubs are created if needed

  • RuboCop passes

  • Existing tests pass

New Plugins

  • Tests

  • Add the plugin to the README

  • Does it have a complete header as outlined here

Purpose

  • When using the script with -h 127.0.0.1 we end up with 127 as hostname, I change to have a hostname more graphite compliant
  • include MR Collect more innodb variables #111: Collect more innodb variables Collect more innodb variables #111
  • include metrics gathering from https://github.com/sensu-plugins/sensu-plugins-percona in metrics-mysql-graphite.rb (there is no reason to run the gathering twice and no reasons to keep two code base)

Known Compatibility Issues

Comment thread bin/metrics-mysql-graphite.rb Outdated
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.

4 participants