rails-autoscale-core 1.5.2 → 1.5.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 03aadf04a101c5dfdaf9a715a25f36d3304943833c4feeba551c11645590418c
-  data.tar.gz: 8e46c89f87301ff458ae77640ebf1dad10abf53ffde7110448d4c6c6ffafd5c0
+  metadata.gz: ecb527ab4d6f7a97247bd1aa47424414ea57ef62eaea688e24674820d9c20e8b
+  data.tar.gz: d1fa82cb48feb2fe4dd8136647b51d83a439632a19cc14f2cc6114d2e3677a54
 SHA512:
-  metadata.gz: 1a754daf2618a1e9d7a7dadd773b798493ecdbbfc27882231df528170ffa44e159941bd7f52ecf1878df53e50291abd9b00aafbe8938ae3f263aa1accb4fa700
-  data.tar.gz: f3b11817285f7afbc3c656419d06f797d75f829ec329cd6a526ec51ea2322fb4a060e3d9fe4c827eb89e03b99fc58999d01f794308d0aead18013b81cbded9db
+  metadata.gz: 4f4403334de767802741291af1799bfa1f0f911e002d89a6f8d9e82465da3e1bef724aa75196fd09e25342fa43e66e516852463e8c8e4063eab526d597605f61
+  data.tar.gz: d3f82c8c7387ff7627c30b70571c72bca3f4c8d000933e5917a595b6cb6a7a20827f52c1fe4ad9b55d343907085710e59ac21426bfea9a241ecd63b2899e5df1

data/lib/judoscale/config.rb

@@ -65,7 +65,7 @@ module Judoscale
     end
 
     attr_accessor :api_base_url, :report_interval_seconds,
-      :max_request_size_bytes, :logger, :log_tag, :current_runtime_container, :allow_rake_tasks
+      :max_request_size_bytes, :logger, :log_tag, :current_runtime_container
     attr_reader :log_level
 
     def initialize
@@ -77,7 +77,6 @@ module Judoscale
       @log_tag = "Judoscale"
       @max_request_size_bytes = 100_000 # ignore request payloads over 100k since they skew the queue times
       @report_interval_seconds = 10
-      @allow_rake_tasks = []
 
       self.log_level = ENV["JUDOSCALE_LOG_LEVEL"] || ENV["RAILS_AUTOSCALE_LOG_LEVEL"]
       @logger = ::Logger.new($stdout)

data/lib/judoscale/logger.rb

@@ -6,7 +6,11 @@ require "logger"
 module Judoscale
   module Logger
     def logger
-      @logger ||= LoggerProxy.new(Config.instance.logger, Config.instance.log_level)
+      if @logger && @logger.log_level == Config.instance.log_level
+        @logger
+      else
+        @logger = LoggerProxy.new(Config.instance.logger, Config.instance.log_level)
+      end
     end
   end
 

data/lib/judoscale/metric.rb

@@ -5,7 +5,7 @@ module Judoscale
     # No queue_name is assumed to be a web request metric
     # Metrics: qt = queue time (default), qd = queue depth, busy
     def initialize(identifier, value, time, queue_name = nil)
-      super identifier, value.to_i, time.utc, queue_name
+      super(identifier, value.to_i, time.utc, queue_name)
     end
   end
 end

data/lib/judoscale/metrics_collector.rb

@@ -3,23 +3,11 @@
 module Judoscale
   class MetricsCollector
     def self.collect?(config)
-      in_rake_task = defined?(::Rake) && Rake.respond_to?(:application) && Rake.application.top_level_tasks.any?
-      in_generator = defined?(::Rails::Command::GenerateCommand)
-
-      !in_generator && (!in_rake_task || in_whitelisted_rake_tasks?(config.allow_rake_tasks))
+      true
     end
 
     def collect
       []
     end
-
-    def self.in_whitelisted_rake_tasks?(allowed_rake_tasks)
-      # Get the tasks that were invoked from the command line.
-      tasks = Rake.application.top_level_tasks
-
-      allowed_rake_tasks.any? do |task_regex|
-        tasks.any? { |task| task =~ task_regex }
-      end
-    end
   end
 end

data/lib/judoscale/reporter.rb

@@ -19,15 +19,13 @@ module Judoscale
     def start!(config, adapters)
       @pid = Process.pid
 
-      if !config.api_base_url
+      if config.api_base_url.nil? || config.api_base_url.strip.empty?
         logger.debug "Set api_base_url to enable metrics reporting"
         return
       end
 
       enabled_adapters, skipped_adapters = adapters.partition { |adapter|
-        if adapter.metrics_collector&.collect?(config)
-          adapter.enabled = true
-        end
+        adapter.metrics_collector&.collect?(config)
       }
       metrics_collectors_classes = enabled_adapters.map(&:metrics_collector)
       adapters_msg = enabled_adapters.map(&:identifier).concat(
@@ -74,17 +72,26 @@ module Judoscale
       @_thread&.terminate
       @_thread = nil
       @pid = nil
+      @reported = false
     end
 
     private
 
     def report(config, metrics)
+      # Make sure we report at least once, even if there are no metrics,
+      # so Judoscale knows the adapter is installed and running.
+      if @reported && metrics.empty?
+        logger.debug "No metrics to report - skipping"
+        return
+      end
+
       report = Report.new(Judoscale.adapters, config, metrics)
       logger.info "Reporting #{report.metrics.size} metrics"
       result = AdapterApi.new(config).report_metrics(report.as_json)
 
       case result
       when AdapterApi::SuccessResponse
+        @reported = true
         logger.debug "Reported successfully"
       when AdapterApi::FailureResponse
         logger.error "Reporter failed: #{result.failure_message}"

data/lib/judoscale/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Judoscale
-  VERSION = "1.5.2"
+  VERSION = "1.5.3"
 end

data/lib/judoscale-ruby.rb

@@ -21,17 +21,8 @@ module Judoscale
   end
 
   class Adapter < Struct.new(:identifier, :adapter_info, :metrics_collector)
-    attr_accessor :enabled
-
-    def initialize(identifier, adapter_info, metrics_collector)
-      super
-      self.enabled = false
-    end
-
     def as_json
-      {
-        identifier => adapter_info.merge(enabled: enabled)
-      }
+      {identifier => adapter_info}
     end
   end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rails-autoscale-core
 version: !ruby/object:Gem::Version
-  version: 1.5.2
+  version: 1.5.3
 platform: ruby
 authors:
 - Adam McCrea
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-08-10 00:00:00.000000000 Z
+date: 2024-01-25 00:00:00.000000000 Z
 dependencies: []
 description: 
 email:
@@ -20,7 +20,6 @@ extensions: []
 extra_rdoc_files: []
 files:
 - Gemfile
-- Gemfile.lock
 - Rakefile
 - judoscale-ruby.gemspec
 - lib/judoscale-ruby.rb

data/Gemfile.lock

@@ -1,48 +0,0 @@
-PATH
-  remote: .
-  specs:
-    judoscale-ruby (1.5.2)
-
-GEM
-  remote: https://rubygems.org/
-  specs:
-    addressable (2.8.1)
-      public_suffix (>= 2.0.2, < 6.0)
-    crack (0.4.5)
-      rexml
-    debug (1.7.1)
-      irb (>= 1.5.0)
-      reline (>= 0.3.1)
-    hashdiff (1.0.1)
-    io-console (0.6.0)
-    irb (1.6.2)
-      reline (>= 0.3.0)
-    minitest (5.17.0)
-    minitest-stub-const (0.6)
-    public_suffix (5.0.1)
-    rake (13.0.6)
-    reline (0.3.2)
-      io-console (~> 0.5)
-    rexml (3.2.5)
-    webmock (3.18.1)
-      addressable (>= 2.8.0)
-      crack (>= 0.3.2)
-      hashdiff (>= 0.4.0, < 2.0.0)
-
-PLATFORMS
-  arm64-darwin-20
-  arm64-darwin-21
-  arm64-darwin-22
-  x86_64-darwin-21
-  x86_64-linux
-
-DEPENDENCIES
-  debug
-  judoscale-ruby!
-  minitest
-  minitest-stub-const
-  rake (>= 12.3.3)
-  webmock
-
-BUNDLED WITH
-   2.3.9