fluent-plugin-zabbix 0.2.3 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 1e840411e29db42fc60e77d0b56235e8012bb9e6
-  data.tar.gz: 1ed7a4df13454605e0c1a9dd00003353db7d5aac
+  metadata.gz: 35423c2cf732ed7048b36bb6c4c5882fb2526110
+  data.tar.gz: 46dbb7031bfb41f3ea4d7a9acd87dcef14714f3b
 SHA512:
-  metadata.gz: 27c58fc74df1d1d0ba806ec89586588eb63d236bf4c35edf638986f4e06aa8d305005e5845800206a090b9a53dc8a6d246f0b770b85fd92543931b3130f20bd8
-  data.tar.gz: 1a18cfb1ba839c90ec7908fe820f564ddfb75ec20fcfa5c517e16c855417a5b1e93e5b479e008bf8b3649ac079779521ea49556d61390ebd61870c6943e62777
+  metadata.gz: 33cfd15134ba66af5163ec247b4696708d89960b86fb3a5a67f9ec9dffb33c7db8eabba8b8c8f55542f22e0601452af684fd27e6d134ad640949f955dddaf29e
+  data.tar.gz: bab731439abad3a3a7242ff35840d29a305ab9d529aa83f7fe75ba4ee9b5ee5fc811368bebf0b5117478b81987f5d552b2c2c9bd6fe022588ec63588b040318f

data/.travis.yml

@@ -3,6 +3,7 @@ rvm:
   - 2.1
   - 2.2
   - 2.3
+  - 2.4.0
 gemfile:
   - Gemfile
 before_install:

data/Rakefile

@@ -9,4 +9,4 @@ Rake::TestTask.new(:test) do |test|
   test.verbose = true
 end
 
-task :default => :test
+task default: :test

data/fluent-plugin-zabbix.gemspec

@@ -14,9 +14,9 @@ Gem::Specification.new do |gem|
   gem.test_files    = gem.files.grep(%r{^(test|spec|features)/})
   gem.name          = "fluent-plugin-zabbix"
   gem.require_paths = ["lib"]
-  gem.version       = "0.2.3"
+  gem.version       = "0.3.0"
 
-  gem.add_runtime_dependency "fluentd", [">= 0.10", "< 2"]
+  gem.add_runtime_dependency "fluentd", [">= 0.14.8", "< 2"]
   gem.add_runtime_dependency "yajl-ruby", "~> 1.0"
   gem.add_runtime_dependency "fluent-mixin-config-placeholders", "~> 0.3"
   gem.add_development_dependency "rake", ">= 0.9.2"

data/lib/fluent/plugin/out_zabbix.rb

@@ -1,40 +1,28 @@
+require 'socket'
+require 'yajl'
+require 'fluent/plugin/output'
 require 'fluent/mixin/config_placeholders'
 
-class Fluent::ZabbixOutput < Fluent::Output
+class Fluent::Plugin::ZabbixOutput < Fluent::Plugin::Output
   Fluent::Plugin.register_output('zabbix', self)
 
   ZBXD = "ZBXD\x01"
   ZBX_PROTO_VALUE_SENDER_DATA = 'sender data'
 
-  def initialize
-    super
-    require 'socket'
-    require 'yajl'
-  end
-
   config_param :zabbix_server, :string
-  config_param :port, :integer,            :default => 10051
-  config_param :host, :string,             :default => Socket.gethostname
-  config_param :host_key, :string,         :default => nil
-  config_param :name_keys, :string,        :default => nil
-  config_param :name_key_pattern, :string, :default => nil
-  config_param :add_key_prefix, :string,   :default => nil
-  config_param :prefix_key, :string,       :default => nil
+  config_param :port, :integer,            default: 10051
+  config_param :host, :string,             default: Socket.gethostname
+  config_param :host_key, :string,         default: nil
+  config_param :name_keys, :string,        default: nil
+  config_param :name_key_pattern, :string, default: nil
+  config_param :add_key_prefix, :string,   default: nil
+  config_param :prefix_key, :string,       default: nil
 
   include Fluent::Mixin::ConfigPlaceholders
 
-  # Define `log` method for v0.10.42 or earlier
-  unless method_defined?(:log)
-    define_method("log") { $log }
-  end
-
   def configure(conf)
     super
 
-    if @zabbix_server.nil?
-      raise Fluent::ConfigError, "missing zabbix_server"
-    end
-
     if @name_keys.nil? and @name_key_pattern.nil?
       raise Fluent::ConfigError, "missing both of name_keys and name_key_pattern"
     end
@@ -49,12 +37,8 @@ class Fluent::ZabbixOutput < Fluent::Output
     end
   end
 
-  def start
-    super
-  end
-
-  def shutdown
-    super
+  def multi_workers_ready?
+    true
   end
 
   def bulk_send(time, bulk)
@@ -76,17 +60,17 @@ class Fluent::ZabbixOutput < Fluent::Output
     end
   end
 
-  def emit(tag, es, chain)
+  def process(tag, es)
     if @name_keys
       es.each {|time,record|
         host = gen_host(record)
         bulk = []
         @name_keys.each {|key|
           if record[key]
-            bulk.push({ :key => format_key(tag, key, record),
-                        :value => format_value(record[key]),
-                        :host => host,
-                        :time => time.to_i,
+            bulk.push({ key: format_key(tag, key, record),
+                        value: format_value(record[key]),
+                        host: host,
+                        time: time.to_i,
                       })
           end
         }
@@ -98,17 +82,16 @@ class Fluent::ZabbixOutput < Fluent::Output
         bulk = []
         record.keys.each {|key|
           if @name_key_pattern.match(key) && record[key]
-            bulk.push({ :key => format_key(tag, key, record),
-                        :value => format_value(record[key]),
-                        :host => host,
-                        :time => time.to_i,
+            bulk.push({ key: format_key(tag, key, record),
+                        value: format_value(record[key]),
+                        host: host,
+                        time: time.to_i,
                       })
           end
         }
         bulk_send(time, bulk) if bulk.size > 0
       }
     end
-    chain.next
   end
 
   def gen_host(record)
@@ -155,9 +138,9 @@ class Fluent::ZabbixOutput < Fluent::Output
 
   def send_to_zabbix(sock, time, bulk)
     req = Yajl::Encoder.encode({
-      :request => ZBX_PROTO_VALUE_SENDER_DATA,
-      :clock => time.to_i,
-      :data => bulk,
+      request: ZBX_PROTO_VALUE_SENDER_DATA,
+      clock: time.to_i,
+      data: bulk,
     })
     sock.write(ZBXD + [ req.size ].pack('q') + req)
     sock.flush

data/test/plugin/test_out_zabbix.rb

@@ -1,4 +1,5 @@
 require 'helper'
+require 'fluent/test/driver/output'
 
 if ENV['LIVE_TEST']
   require "glint"
@@ -11,7 +12,7 @@ class ZabbixOutputTest < Test::Unit::TestCase
     Fluent::Test.setup
     if ENV['LIVE_TEST']
       $dir = Dir.mktmpdir
-      $server = Glint::Server.new(10051, { :timeout => 3 }) do |port|
+      $server = Glint::Server.new(10051, { timeout: 3 }) do |port|
         exec "./mockserver", $dir.to_s + "/trapper.log"
       end
       $server.start
@@ -25,21 +26,22 @@ class ZabbixOutputTest < Test::Unit::TestCase
     name_keys      foo, bar, baz, f1, f2
   ]
 
-  def create_driver(conf = CONFIG, tag='test')
-    Fluent::Test::OutputTestDriver.new(Fluent::ZabbixOutput, tag).configure(conf)
+  def create_driver(conf = CONFIG)
+    Fluent::Test::Driver::Output.new(Fluent::Plugin::ZabbixOutput).configure(conf)
   end
 
   def test_write
     d = create_driver
     if ENV['LIVE_TEST']
-      d.emit({"foo" => "test value of foo"})
-      d.emit({"bar" => "test value of bar"})
-      d.emit({"baz" => 123.4567 })
-      d.emit({"foo" => "yyy", "zabbix_host" => "alternative-hostname"})
-      d.emit({"f1" => 0.000001})
-      d.emit({"f2" => 0.01})
-      d.run
-      sleep 1
+      d.run(default_tag: 'test') do
+        d.feed({"foo" => "test value of foo"})
+        d.feed({"bar" => "test value of bar"})
+        d.feed({"baz" => 123.4567 })
+        d.feed({"foo" => "yyy", "zabbix_host" => "alternative-hostname"})
+        d.feed({"f1" => 0.000001})
+        d.feed({"f2" => 0.01})
+        sleep 1
+      end
       $server.stop
       assert_equal open($dir + "/trapper.log").read, <<END
 host:test_host	key:test.foo	value:test value of foo
@@ -60,8 +62,8 @@ END
     name_keys      foo, bar, baz
   ]
 
-  def create_driver_host_key(conf = CONFIG_HOST_KEY, tag='test')
-    Fluent::Test::OutputTestDriver.new(Fluent::ZabbixOutput, tag).configure(conf)
+  def create_driver_host_key(conf = CONFIG_HOST_KEY)
+    Fluent::Test::Driver::Output.new(Fluent::Plugin::ZabbixOutput).configure(conf)
   end
 
   CONFIG_PREFIX_KEY = %[
@@ -71,17 +73,18 @@ END
     name_keys      foo, bar, baz
   ]
 
-  def create_driver_prefix_key(conf = CONFIG_PREFIX_KEY, tag='test')
-    Fluent::Test::OutputTestDriver.new(Fluent::ZabbixOutput, tag).configure(conf)
+  def create_driver_prefix_key(conf = CONFIG_PREFIX_KEY)
+    Fluent::Test::Driver::Output.new(Fluent::Plugin::ZabbixOutput).configure(conf)
   end
 
   def test_write_host_key
     d = create_driver_host_key
     if ENV['LIVE_TEST']
-      d.emit({"foo" => "AAA" })
-      d.emit({"foo" => "BBB", "host" => "alternative-hostname"})
-      d.run
-      sleep 1
+      d.run(default_tag: 'test') do
+        d.feed({"foo" => "AAA" })
+        d.feed({"foo" => "BBB", "host" => "alternative-hostname"})
+        sleep 1
+      end
       $server.stop
       assert_equal open($dir + "/trapper.log").read, <<END
 host:test_host	key:test.foo	value:AAA
@@ -93,10 +96,11 @@ END
   def test_write_prefix_key
     d = create_driver_prefix_key
     if ENV['LIVE_TEST']
-      d.emit({"foo" => "AAA"})
-      d.emit({"foo" => "BBB", "prefix" => "p"})
-      d.run
-      sleep 1
+      d.run(default_tag: 'test') do
+        d.feed({"foo" => "AAA"})
+        d.feed({"foo" => "BBB", "prefix" => "p"})
+        sleep 1
+      end
       $server.stop
       assert_equal open($dir + "/trapper.log").read, <<END
 host:test_host	key:foo	value:AAA

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: fluent-plugin-zabbix
 version: !ruby/object:Gem::Version
-  version: 0.2.3
+  version: 0.3.0
 platform: ruby
 authors:
 - FUJIWARA Shunichiro
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-08-30 00:00:00.000000000 Z
+date: 2018-02-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: fluentd
@@ -16,7 +16,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0.10'
+        version: 0.14.8
     - - "<"
       - !ruby/object:Gem::Version
         version: '2'
@@ -26,7 +26,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0.10'
+        version: 0.14.8
     - - "<"
       - !ruby/object:Gem::Version
         version: '2'
@@ -110,7 +110,6 @@ files:
 - ".gitignore"
 - ".travis.yml"
 - Gemfile
-- Gemfile.fluentd.lt.0.10.43
 - LICENSE
 - README.md
 - Rakefile

data/Gemfile.fluentd.lt.0.10.43

@@ -1,5 +0,0 @@
-source 'https://rubygems.org'
-
-# Specify your gem's dependencies in fluent-plugin-zabbix.gemspec
-gem 'fluentd', '= 0.10.42'
-gemspec