fluent-plugin-jfrog-metrics 0.2.11 → 0.2.12.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cb8dfdd51ee10875ad7d6ec7c539a92ebc703061e4bc87aa407229ebb1b97898
-  data.tar.gz: b5f7c2a0121c9100a579f6ce61d5845ae6058656f06031d06dbc4800a7b8bfe3
+  metadata.gz: e3874d94d40532a3bb834c0b5c7fa4e999a81d9ac9446fbfdf5787a43243b1b3
+  data.tar.gz: 56639f48a7989af20ad9444a90f036a17efbfd0d5dd4a053d9c2170506e2e705
 SHA512:
-  metadata.gz: a7fca5a309cb9c334116019c7914456870f31aa1ca8adcadb8803932327bea1a0d8ecfd6cd4f36346b6c3904dc83c795618877a2ceca8a78bcd4ae3a6d3ab667
-  data.tar.gz: 8ed3af580a9cea432a5a66983c5c87e2c28b46134f3772e4fdc9b712bdd55ca75ea4ca8247cd2bdbbf426a13e6982d4dee69151ab96b39d7f8f55dd040749c1e
+  metadata.gz: 167edf56bbab4ea71298eab1972b0dedfa437939dd617619f5b344f257dfe804b5a7634f35aa69fe27c0fa2a412af7b8e25a102e857b8a16febba9c346654e1e
+  data.tar.gz: 7485f266e6a5d9c987cc60f9326aa872e2f3ca6c0fc31968f29912f88c9f701990b83a4de08eaf633fc69279940b6d315d5efcb9b4533c0b0df87de960af4a7a

data/Gemfile

@@ -7,5 +7,5 @@ gemspec
 group :test do
   gem 'minitest'
   gem 'minitest-reporters', '>= 0.5.0'
-  gem 'rest-client', '>= 2.1.0'
+  gem 'rest-client', '~> 2.1.0'
 end

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    fluent-plugin-jfrog-metrics (0.2.11)
+    fluent-plugin-jfrog-metrics (0.2.12.1)
       fluentd (>= 0.14.10, < 2)
       rest-client (~> 2.1.0)
 

data/fluent-plugin-jfrog-metrics.gemspec

@@ -3,7 +3,7 @@ $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 
 Gem::Specification.new do |spec|
   spec.name    = 'fluent-plugin-jfrog-metrics'
-  spec.version = '0.2.11'
+  spec.version = '0.2.12.1'
   spec.authors = ['MahithaB, BenHarosh']
   spec.email   = ['cpe-support@jfrog.com']
 

data/lib/fluent/plugin/in_jfrog_metrics.rb

@@ -43,6 +43,8 @@ module Fluent
       config_param :common_jpd, :bool, default: false
       config_param :verify_ssl, :bool, default: true
 
+      $logger = nil
+
       # `configure` is called before `start`.
       # 'conf' is a `Hash` that includes the configuration parameters.
       # If the configuration is invalid, raise `Fluent::ConfigError`.
@@ -71,8 +73,9 @@ module Fluent
       def start
         super
         @running = true
+        $logger = log
         @thread = Thread.new do
-          run(log)
+          run
         end
       end
 
@@ -82,28 +85,34 @@ module Fluent
         super
       end
 
-      def run(logger)
-        logger.info("Preparing metrics collection, creating timer task")
+      def run
+        $logger.info("Preparing metrics collection, creating timer task")
         timer_task = Concurrent::TimerTask.new(execution_interval: @execution_interval, run_now: true) do
-          logger.info("Timer task execution started")
-          do_execute(logger)
-          logger.info("Timer task execution finished")
+          begin
+            $logger.info("Timer task execution started")
+            do_execute
+            next "Timer task execution finished successfully"
+          rescue => e
+            $logger.error("Error occurred when running Timer task: #{e.message}")
+            raise e
+          end
         end
+        # timer_task.add_observer(TaskObserver.new)
         timer_task.execute
         sleep 100
       end
 
-      def do_execute(logger)
+      def do_execute
         begin
-          logger.info("Metrics collection started")
-          metrics_helper = MetricsHelper.new(logger, @metric_prefix, @jpd_url, @username, @apikey, @token, @common_jpd, @verify_ssl, @request_timeout)
+          $logger.info("Metrics collection started")
+          metrics_helper = MetricsHelper.new($logger, @metric_prefix, @jpd_url, @username, @apikey, @token, @common_jpd, @verify_ssl, @request_timeout)
           platform_metrics = metrics_helper.get_metrics
 
           additional_metrics = metrics_helper.get_additional_metrics
           if !additional_metrics.nil? && additional_metrics != ''
             platform_metrics += additional_metrics.to_s
           end
-          logger.info("Metrics collection finished")
+          $logger.info("Metrics collection finished")
 
           if @target_platform == 'SPLUNK'
             parser = SplunkMetricsParser.new(@metric_prefix, router, @tag)
@@ -114,24 +123,34 @@ module Fluent
           else
             raise 'Parser Type is not valid. target_platform Should be SPLUNK or NEWRELIC or DATADOG'
           end
-          logger.debug("Emitting collected metrics started")
+          $logger.debug("Emitting collected metrics started")
           parser.emit_parsed_metrics(platform_metrics)
-          logger.debug("Emitting collected metrics finished")
+          $logger.debug("Emitting collected metrics finished")
 
         rescue RestClient::Exceptions::OpenTimeout
-          logger.info("The request timed out while trying to open a connection. The configured request timeout is: #{@request_timeout}")
+          $logger.error("The request timed out while trying to open a connection. The configured request timeout is: #{@request_timeout}")
         rescue RestClient::Exceptions::ReadTimeout
-          logger.info("The request timed out while waiting for a response. The configured request timeout is: #{@request_timeout}")
+          $logger.error("The request timed out while waiting for a response. The configured request timeout is: #{@request_timeout}")
         rescue  RestClient::Exceptions::RequestTimeout
-          logger.info("The request timed out. The configured request timeout is: #{@request_timeout}")
+          $logger.error("The request timed out. The configured request timeout is: #{@request_timeout}")
         rescue RestClient::ExceptionWithResponse => e
-          logger.info("HTTP request failed: #{e.response}")
+          $logger.error("HTTP request failed: #{e.response}")
         rescue StandardError => e
-          logger.info("An unexpected error occurred during metrics collection: #{e.message}")
+          $logger.error("An unexpected error occurred during metrics collection: #{e.message}")
         else
-          logger.debug("Metrics collection and emission do_execute finished with no errors")
+          $logger.debug("Metrics collection and emission do_execute finished with no errors")
         end
       end
+
+      # class TaskObserver
+      #   def update(time, result, e)
+      #     if result
+      #       $logger.info("Timer task Execution successfully returned: '#{result}' at: #{time}")
+      #     else
+      #       $logger.error("Timer task Execution failed with error: #{e} at: #{time}")
+      #     end
+      #   end
+      # end
     end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: fluent-plugin-jfrog-metrics
 version: !ruby/object:Gem::Version
-  version: 0.2.11
+  version: 0.2.12.1
 platform: ruby
 authors:
 - MahithaB, BenHarosh