roqua-support 0.3.0 → 0.3.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3b7684543b65ccc1553042df3e832dba5f40a8f42d0ff15fa31f42d865755b32
-  data.tar.gz: 3ba17388d2c2b947756ddb1ca3b4172a6bcbde85bef76b0c70a5eb9fe7fe16d5
+  metadata.gz: e5997425ea5b975ef1adf4b17e10d3d6c44bbdaa6507f696db1a5aad1ddf9513
+  data.tar.gz: cc565728bb3fc02e707b92d8ab507c996a793f9c6d5d16b0e6d8cfed1462ae70
 SHA512:
-  metadata.gz: 880ee19112ad1e4f596a451228b7b3b6f5b07ad381f0d1706ee00c42b070af2c8dd3048a26363183c09ba9ceb2f7e34c72b3289b11f8829ee0e02a613e173742
-  data.tar.gz: fa8da4fcb52839b129019a08f4ba852cc629af98944c9aef9add34c4ef2244ace706e59f53765b551c95da8a6231bed7a096afeb0e4dacfa7e7a1d02a5f14f17
+  metadata.gz: c9fc30367ece67f45800e8ab586f86f8aed50f8033be899265c5a03a87318ebdb960b87652c69dbb03da28c419242dffa1aa6977e82708b6e33a3b9ff445c352
+  data.tar.gz: 28c2f803f644b2244908cc52b5326074298e81b31c8e0f3f9cd6ac9a422dab00e51ac3d1e4383f2b09516fdc045bafe6b52cde2e3cf22353608d284ce9e046d9

data/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 0.3.1
+
+* Added Appsignal counters for probe calls and scheduling probe call tasks in particular.
+
 ## 0.3.0
 
 * Support for Ruby 2.5

data/Gemfile

@@ -17,4 +17,5 @@ group :test do
   gem 'responders'
   gem 'rspec-instrumentation-matcher'
   gem 'rspec-rails'
+  gem 'pry'
 end

data/Gemfile.lock

@@ -188,6 +188,7 @@ DEPENDENCIES
   delayed_job_active_record
   fakefs
   guard-rspec (~> 4.2.6)
+  pry
   rake
   responders
   roqua-support!
@@ -199,4 +200,4 @@ DEPENDENCIES
   timecop
 
 BUNDLED WITH
-   1.16.6
+   1.17.3

data/lib/roqua-support/version.rb

@@ -1,5 +1,5 @@
 module Roqua
   module Support
-    VERSION = '0.3.0'.freeze
+    VERSION = '0.3.1'.freeze
   end
 end

data/lib/roqua/probes/base_probe.rb

@@ -1,12 +1,24 @@
+require 'active_support/concern'
+
 module Roqua
   module Probes
     module BaseProbe
-      def enable
-        new.tap do |probe|
-          probe_sym = probe.class.to_s.to_sym
-          Appsignal::Minutely.probes.register(probe_sym, probe) unless Appsignal::Minutely.probes[probe_sym]
+      extend ActiveSupport::Concern
+
+      class_methods do
+        def enable
+          new.tap do |probe|
+            probe_sym = probe.class.to_s.to_sym
+            Appsignal::Minutely.probes.register(probe_sym, probe) unless Appsignal::Minutely.probes[probe_sym]
+          end
         end
       end
+
+      # do not override me, implement probes by implementing the #run method
+      def call
+        run
+        Appsignal.increment_counter("probe.call.completed.#{self.class.name.demodulize.underscore}", 1)
+      end
     end
   end
 end

data/lib/roqua/probes/delayed_job_probe.rb

@@ -3,13 +3,13 @@ require_relative 'base_probe'
 module Roqua
   module Probes
     class DelayedJobProbe
-      extend BaseProbe
+      include BaseProbe
 
       def backlog_count
         Delayed::Job.where(locked_at: nil).where('run_at < ?', Time.zone.now).count
       end
 
-      def call
+      def run
         Appsignal.set_gauge('delayed_job_backlog_count', backlog_count)
       end
     end

data/lib/roqua/probes/monitoring_probe.rb

@@ -3,7 +3,7 @@ require_relative 'base_probe'
 module Roqua
   module Probes
     class MonitoringProbe
-      extend BaseProbe
+      include BaseProbe
 
       def incomplete_jobs
         Roqua::Scheduling::CronJob.where('completed_at IS NULL OR completed_at < next_run_at')
@@ -19,7 +19,7 @@ module Roqua
         (longest_delay_in_seconds / 1.minute).to_i
       end
 
-      def call
+      def run
         Appsignal.set_gauge('scheduler_delay_in_minutes', longest_delay_in_minutes)
       end
     end

data/lib/roqua/probes/scheduling_probe.rb

@@ -3,9 +3,9 @@ require_relative 'base_probe'
 module Roqua
   module Probes
     class SchedulingProbe
-      extend BaseProbe
+      include BaseProbe
 
-      def call
+      def run
         ActiveRecord::Base.establish_connection
         require_relative Rails.root.join('config', 'schedule.rb')
         Roqua::Scheduling::Scheduler.new.ping

data/lib/roqua/scheduling/scheduler.rb

@@ -42,6 +42,7 @@ class Roqua::Scheduling::Scheduler
     task = schedule.tasks[cron_job.name]
     task.run
 
+    Appsignal.increment_counter("scheduler.run_task.completed.#{task.name}", 1)
     cron_job.update completed_at: Time.now, next_run_at: task.next_run_at
   end
 end

data/spec/roqua/probes/delayed_job_probe_spec.rb

@@ -7,7 +7,7 @@ require 'roqua/probes/delayed_job_probe'
 describe Roqua::Probes::DelayedJobProbe do
   before { Timecop.freeze }
   after { Timecop.return }
-  subject(:probe) { Roqua::Probes::DelayedJobProbe.new }
+  subject(:probe) { described_class.new }
 
   describe 'backlog_count' do
     context 'if a single job unlocked job exists that has a run_at in the past' do
@@ -24,12 +24,17 @@ describe Roqua::Probes::DelayedJobProbe do
     end
   end
 
-  describe 'call' do
+  describe '#call' do
     it 'sends the correct metric to Appsignal' do
       expect(probe).to receive(:backlog_count).and_return(12)
       expect(Appsignal).to receive(:set_gauge).with('delayed_job_backlog_count', 12)
       probe.call
     end
+
+    it 'increments the probe call counter' do
+      expect(Appsignal).to receive(:increment_counter).with('probe.call.completed.delayed_job_probe', 1)
+      probe.call
+    end
   end
 
   context '.enable' do

data/spec/roqua/probes/monitoring_probe_spec.rb

@@ -4,21 +4,26 @@ require 'timecop'
 
 require 'roqua/probes/delayed_job_probe'
 
-describe Roqua::Probes::DelayedJobProbe do
+describe Roqua::Probes::MonitoringProbe do
   before { Timecop.freeze }
   after { Timecop.return }
-  subject(:probe) { Roqua::Probes::MonitoringProbe.new }
+  subject(:probe) { described_class.new }
 
   before do
     Roqua::Scheduling::CronJob.delete_all
   end
 
-  describe 'call' do
+  describe '#call' do
     it 'sends data to AppSignal' do
       expect(Appsignal).to receive(:set_gauge).with('scheduler_delay_in_minutes', 10)
       expect(probe).to receive(:longest_delay_in_minutes).and_return(10)
       probe.call
     end
+
+    it 'increments the probe call counter' do
+      expect(Appsignal).to receive(:increment_counter).with('probe.call.completed.monitoring_probe', 1)
+      probe.call
+    end
   end
 
   describe 'longest_delay' do

data/spec/roqua/scheduling/scheduler_spec.rb

@@ -72,16 +72,23 @@ describe Roqua::Scheduling::Scheduler do
   describe 'callbacks' do
     let(:callback_spy) { spy(:callback) }
 
-    it 'get executed' do
+    before do
       Roqua::Scheduling::Schedule.setup do |cron|
         cron.add_task 'hourly', next_run_at: proc { 1.minute.ago } do
           callback_spy.execute
         end
       end
+    end
 
+    it 'get executed' do
       expect(callback_spy).to receive(:execute)
       subject.ping
     end
+
+    it 'counts task calls' do
+      expect(Appsignal).to receive(:increment_counter).with('scheduler.run_task.completed.hourly', 1)
+      subject.ping
+    end
   end
 
   describe 'when running multiple times' do

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: roqua-support
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.3.1
 platform: ruby
 authors:
 - Marten Veldthuis
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-05-06 00:00:00.000000000 Z
+date: 2019-06-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: active_interaction
@@ -281,8 +281,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.9
+rubygems_version: 3.0.3
 signing_key: 
 specification_version: 4
 summary: Helper objects and proxies used by a lot of RoQua applications