Fix #16 -n option: new docker version concatenating container names

sensu-plugins / sensu-plugins-docker

This plugin provides native Docker instrumentation for monitoring and metrics collection, including: container status, container number, and container metrics via `docker ps`.

http://sensu-plugins.io

MIT License

35 stars 57 forks source link

Fix #16 -n option: new docker version concatenating container names #75

Closed theyough closed 5 years ago

theyough commented 5 years ago

Pull Request Checklist

Fix #16

General

tested with Docker API version 1.39

majormoses commented 5 years ago

Thanks for your contribution to Sensu plugins! Without people like you submitting PRs we couldn't run the project. I will review it shortly.

majormoses commented 5 years ago

You can ignore that CI failure, I plan on dropping support for all EOL rubies after this PR which should fix it.

theyough commented 5 years ago

@majormoses, it should be ok. Thanks for the comment

majormoses commented 5 years ago

Hey could you please add a changelog entry? https://github.com/sensu-plugins/community/blob/master/HOW_WE_CHANGELOG.md#unreleased-change

majormoses commented 5 years ago

released: https://rubygems.org/gems/sensu-plugins-docker/versions/3.2.0