sensu-plugins-varnish 1.1.0 → 1.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: e5e0f886fc302904f4fbec469d3d29bb3ede8093
-  data.tar.gz: 2a9a72e264fb432cc0ff878257a784ca2ab015f4
+  metadata.gz: 4bb123e60a211227bf52003d63bd5c35722e1438
+  data.tar.gz: 210c42bf554975d06d9d3b67b7715e905d0dd33e
 SHA512:
-  metadata.gz: 33b7405d02ff2acd1065f87e449d6bda8a85e3794914b65604e8fc4224cd24230ebef4592dd99c905ff615f01594b5b266183dd633382ee17b3687375682186b
-  data.tar.gz: 62fc22513eb76c53caeff5b61d5e88f1066411ad71ff8c6b34f65ae088f2ef3226244403584661b8379635703f53e6c99a5f5ab0be61af1ecdc3b47cafd7efe0
+  metadata.gz: cb7369976fff7287ceef9924f0f551eec9f4ac398388a55138d9772e30fa9bf1254b0b21e186faab671388bf5f51ae966cbe145fda5763d6ce7736fd382ad16f
+  data.tar.gz: eee1e5fe3632d8d4f8e2897060931ff58c29e18e0a544ebc3cb6110ed6cee542f7a6f5fe169b9db0e6509e9bec37c5a94dd9f66c1303c3d9e78554a6fbf31ec7

data/CHANGELOG.md

@@ -5,6 +5,13 @@ This CHANGELOG follows the format listed at [Keep A Changelog](http://keepachang
 
 ## [Unreleased]
 
+## [1.1.1] - 2017-08-18
+### Fixed
+ - `which` returns a newline terminated string in all cases - strip it before trying to run the command otherwise the generated command will be split over two lines and fail (@warmfusion)
+
+### Changed
+- switched from `strip` to `chomp` as it will be more performant for the use case (@majormoses)
+
 ## [1.1.0] - 2017-08-01
 ### Changed
 - Use full path for varnish scripts to allow sudoers files with explicit binaries to work (@warmfusion)
@@ -49,6 +56,7 @@ This CHANGELOG follows the format listed at [Keep A Changelog](http://keepachang
 - initial release
 
 [Unreleased]: https://github.com/sensu-plugins/sensu-plugins-varnish/compare/1.1.0...HEAD
+[1.1.1]: https://github.com/sensu-plugins/sensu-plugins-varnish/compare/1.1.0...1.1.1
 [1.1.0]: https://github.com/sensu-plugins/sensu-plugins-varnish/compare/1.0.0...1.1.0
 [1.0.0]: https://github.com/sensu-plugins/sensu-plugins-varnish/compare/0.1.0...1.0.0
 [0.1.0]: https://github.com/sensu-plugins/sensu-plugins-varnish/compare/0.0.5...0.1.0

data/README.md

@@ -20,7 +20,7 @@ manage your sudo lists;
 
 ```
 # Assuming that your varnish scripts are located in /usr/bin;
-sensu ALL=(ALL) NOPASSWD: /usr/bin/varnishadm /usr/bin/varnishstat
+sensu ALL=(ALL) NOPASSWD: /usr/bin/varnishadm, /usr/bin/varnishstat
 ```
 
 

data/bin/check-varnish-status.rb

@@ -77,7 +77,7 @@ class CheckVarnishStatus < Sensu::Plugin::Check::CLI
   # Main Function
   def run
     # Keep a full reference for the varnish binary so sudo uses a full path
-    varnishadm = `which varnishadm 2>/dev/null`.to_s
+    varnishadm = `which varnishadm 2>/dev/null`.chomp
     unknown 'varnishadm is not installed!' unless varnishadm.include? 'varnish'
     command = `sudo #{varnishadm} -T #{config[:host]}:#{config[:port]} -S #{config[:secret]} -t #{config[:timeout]} #{config[:command]}`
     if config[:command] == 'status'

data/bin/metrics-varnish.rb

@@ -62,7 +62,7 @@ class VarnishMetrics < Sensu::Plugin::Metric::CLI::Graphite
 
   def run
     # Keep a full reference for the varnish binary so sudo uses a full path
-    varnishstat_command = `which varnishstat 2>/dev/null`.to_s
+    varnishstat_command = `which varnishstat 2>/dev/null`.chomp
     unknown 'varnishstat is not installed!' unless varnishstat_command.include? 'varnish'
 
     begin

data/lib/sensu-plugins-varnish/version.rb

@@ -2,7 +2,7 @@ module SensuPluginsVarnish
   module Version
     MAJOR = 1
     MINOR = 1
-    PATCH = 0
+    PATCH = 1
 
     VER_STRING = [MAJOR, MINOR, PATCH].compact.join('.')
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: sensu-plugins-varnish
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.1.1
 platform: ruby
 authors:
 - Sensu Plugins and contributors
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-08-01 00:00:00.000000000 Z
+date: 2017-08-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: crack