riemann-tools 1.8.0 → 1.8.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e19bf35e435918b5107c15cbcaabbe6da5c707e771dead6f971f38babe6967c7
-  data.tar.gz: fab496cdc89a6fdfa45b5740189b09d23dc3014d47a2e4fc889e19464eb365f8
+  metadata.gz: 4332ce7f616ae97148796f5563e9d1920e104da9f22ce5fade96204a46e83538
+  data.tar.gz: 4a84364a07bba6d111d124491ac65bf310075cf439fac30717c9aa5f95f0b68c
 SHA512:
-  metadata.gz: d0f40cd275ad0e71191bd021d16bc197be89b3ef869bd2ac3e0734b5444e89abb1e4db1976dffcb9911d567f24be17d2fd4103e85098866152fa5a4cd426f2cf
-  data.tar.gz: e141ca404e46fd6e4a95030dade5f0062ed670a191ed03022d1cfcab607ab46579c32dc75e80ea8ddaed3799477869dceaab44d5685dfbe91b17d906cdd1dc63
+  metadata.gz: 9791c57e10683c4509f08a75ef454545bf90fc3ab33a852c2797b302937a6d17aeb2ce19de6e4f1ee91d99291f442c5ac06e019cf8d893718487c70645a431e7
+  data.tar.gz: 7ab5ebb5ede098d648ff78033bac1b0743f99a98fb8a68a9898e70a2e7f29d60901b089f905c4b7afa2e32c50bdab025e68df7d2b6bc0c9c3806733ebcd8693c

data/.github/dependabot.yml

@@ -5,7 +5,14 @@
 
 version: 2
 updates:
+  # Open PR for gem updates
   - package-ecosystem: "bundler" # See documentation for possible values
     directory: "/" # Location of package manifests
     schedule:
       interval: "daily"
+
+  # Open PR for GitHub Actions updates
+  - package-ecosystem: "github-actions"
+    directory: "/"
+    schedule:
+      interval: "daily"

data/CHANGELOG.md

@@ -1,6 +1,23 @@
 # Changelog
 
-## [v1.8.0](https://github.com/riemann/riemann-tools/tree/v1.8.0) (2023-02-01)
+## [v1.8.2](https://github.com/riemann/riemann-tools/tree/v1.8.2) (2023-05-22)
+
+[Full Changelog](https://github.com/riemann/riemann-tools/compare/v1.8.1...v1.8.2)
+
+**Fixed bugs:**
+
+- Gracefully handle all communication errors [\#268](https://github.com/riemann/riemann-tools/pull/268) ([smortex](https://github.com/smortex))
+
+## [v1.8.1](https://github.com/riemann/riemann-tools/tree/v1.8.1) (2023-02-28)
+
+[Full Changelog](https://github.com/riemann/riemann-tools/compare/v1.8.0...v1.8.1)
+
+**Fixed bugs:**
+
+- Improve event sending thread lifecycle management [\#265](https://github.com/riemann/riemann-tools/pull/265) ([smortex](https://github.com/smortex))
+- Make sure all events are send before terminating [\#264](https://github.com/riemann/riemann-tools/pull/264) ([smortex](https://github.com/smortex))
+
+## [v1.8.0](https://github.com/riemann/riemann-tools/tree/v1.8.0) (2023-02-02)
 
 [Full Changelog](https://github.com/riemann/riemann-tools/compare/v1.7.1...v1.8.0)
 

data/lib/riemann/tools/riemann_client_wrapper.rb

@@ -7,52 +7,63 @@ require 'riemann/client'
 module Riemann
   module Tools
     class RiemannClientWrapper
-      include Singleton
+      attr_reader :options
+
+      def initialize(options)
+        @options = options
 
-      def initialize
-        @client = nil
         @queue = Queue.new
         @max_bulk_size = 1000
-      end
-
-      def configure(options)
-        return self unless @client.nil?
-
-        r = Riemann::Client.new(
-          host: options[:host],
-          port: options[:port],
-          timeout: options[:timeout],
-          ssl: options[:tls],
-          key_file: options[:tls_key],
-          cert_file: options[:tls_cert],
-          ca_file: options[:tls_ca_cert],
-          ssl_verify: options[:tls_verify],
-        )
-
-        @client = if options[:tcp] || options[:tls]
-                    r.tcp
-                  else
-                    r
-                  end
+        @draining = false
 
         @worker = Thread.new do
+          Thread.current.abort_on_exception = true
           loop do
             events = []
 
             events << @queue.pop
             events << @queue.pop while !@queue.empty? && events.size < @max_bulk_size
 
-            @client.bulk_send(events)
+            client.bulk_send(events)
+          rescue StandardError => e
+            warn "Dropping #{events.size} event#{'s' if events.size > 1} due to #{e}"
           end
         end
-        @worker.abort_on_exception = true
 
-        self
+        at_exit { drain }
+      end
+
+      def client
+        @client ||= begin
+          r = Riemann::Client.new(
+            host: options[:host],
+            port: options[:port],
+            timeout: options[:timeout],
+            ssl: options[:tls],
+            key_file: options[:tls_key],
+            cert_file: options[:tls_cert],
+            ca_file: options[:tls_ca_cert],
+            ssl_verify: options[:tls_verify],
+          )
+
+          if options[:tcp] || options[:tls]
+            r.tcp
+          else
+            r
+          end
+        end
       end
 
       def <<(event)
+        raise('Cannot queue events when draining') if @draining
+
         @queue << event
       end
+
+      def drain
+        @draining = true
+        sleep(1) until @queue.empty? || @worker.stop?
+      end
     end
   end
 end

data/lib/riemann/tools/version.rb

@@ -2,6 +2,6 @@
 
 module Riemann
   module Tools # :nodoc:
-    VERSION = '1.8.0'
+    VERSION = '1.8.2'
   end
 end

data/lib/riemann/tools.rb

@@ -73,7 +73,7 @@ module Riemann
     end
 
     def riemann
-      @riemann ||= RiemannClientWrapper.instance.configure(options)
+      @riemann ||= RiemannClientWrapper.new(options)
     end
     alias r riemann
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: riemann-tools
 version: !ruby/object:Gem::Version
-  version: 1.8.0
+  version: 1.8.2
 platform: ruby
 authors:
 - Kyle Kingsbury
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-02-02 00:00:00.000000000 Z
+date: 2023-05-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: json
@@ -367,7 +367,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.4.2
+rubygems_version: 3.4.12
 signing_key:
 specification_version: 4
 summary: Utilities which submit events to Riemann.