From eb2029a30bd25e2fece7147f8419259125a92207 Mon Sep 17 00:00:00 2001 From: Mike Rogers Date: Wed, 1 Jul 2020 11:50:08 +0100 Subject: [PATCH 1/5] Fix: Redis#exists(key) Warning from redis 4.2.0 --- lib/sidekiq/cron/job.rb | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/sidekiq/cron/job.rb b/lib/sidekiq/cron/job.rb index 313f8db5..5e855cb0 100644 --- a/lib/sidekiq/cron/job.rb +++ b/lib/sidekiq/cron/job.rb @@ -461,7 +461,7 @@ def save #add information about last time! - don't enque right after scheduler poller starts! time = Time.now.utc - conn.zadd(job_enqueued_key, time.to_f.to_s, formated_last_time(time).to_s) unless conn.exists(job_enqueued_key) + conn.zadd(job_enqueued_key, time.to_f.to_s, formated_last_time(time).to_s) unless conn.exists?(job_enqueued_key) end logger.info { "Cron Jobs - add job with name: #{@name}" } end @@ -540,7 +540,7 @@ def formated_last_time now = Time.now.utc def self.exists? name out = false Sidekiq.redis do |conn| - out = conn.exists redis_key name + out = conn.exists? redis_key name end out end From 48f08625ce6ea4354cca3d00dc5afe0cda3ccc7f Mon Sep 17 00:00:00 2001 From: Mike Rogers Date: Sun, 8 Nov 2020 15:49:06 +0000 Subject: [PATCH 2/5] Increasing redis-namespace requirement --- sidekiq-cron.gemspec | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/sidekiq-cron.gemspec b/sidekiq-cron.gemspec index 5962d75d..b0cb6733 100644 --- a/sidekiq-cron.gemspec +++ b/sidekiq-cron.gemspec @@ -68,7 +68,7 @@ Gem::Specification.new do |s| s.add_runtime_dependency(%q.freeze, ["~> 1.1"]) s.add_development_dependency(%q.freeze, [">= 0"]) s.add_development_dependency(%q.freeze, [">= 0"]) - s.add_development_dependency(%q.freeze, [">= 1.5.2"]) + s.add_development_dependency(%q.freeze, [">= 1.8.0"]) s.add_development_dependency(%q.freeze, [">= 0"]) s.add_development_dependency(%q.freeze, [">= 0"]) s.add_development_dependency(%q.freeze, [">= 0"]) @@ -88,7 +88,7 @@ Gem::Specification.new do |s| s.add_dependency(%q.freeze, ["~> 1.1"]) s.add_dependency(%q.freeze, [">= 0"]) s.add_dependency(%q.freeze, [">= 0"]) - s.add_dependency(%q.freeze, [">= 1.5.2"]) + s.add_dependency(%q.freeze, [">= 1.8.0"]) s.add_dependency(%q.freeze, [">= 0"]) s.add_dependency(%q.freeze, [">= 0"]) s.add_dependency(%q.freeze, [">= 0"]) @@ -109,7 +109,7 @@ Gem::Specification.new do |s| s.add_dependency(%q.freeze, ["~> 1.1"]) s.add_dependency(%q.freeze, [">= 0"]) s.add_dependency(%q.freeze, [">= 0"]) - s.add_dependency(%q.freeze, [">= 1.5.2"]) + s.add_dependency(%q.freeze, [">= 1.8.0"]) s.add_dependency(%q.freeze, [">= 0"]) s.add_dependency(%q.freeze, [">= 0"]) s.add_dependency(%q.freeze, [">= 0"]) From 6d24cb0e98116009cb81b855b68176fe3cd20fb9 Mon Sep 17 00:00:00 2001 From: Mike Rogers Date: Wed, 11 Nov 2020 14:55:03 +0000 Subject: [PATCH 3/5] Adding redis dependency requirement --- sidekiq-cron.gemspec | 3 +++ 1 file changed, 3 insertions(+) diff --git a/sidekiq-cron.gemspec b/sidekiq-cron.gemspec index b0cb6733..4907d93f 100644 --- a/sidekiq-cron.gemspec +++ b/sidekiq-cron.gemspec @@ -66,6 +66,7 @@ Gem::Specification.new do |s| if Gem::Version.new(Gem::VERSION) >= Gem::Version.new('1.2.0') then s.add_runtime_dependency(%q.freeze, [">= 4.2.1"]) s.add_runtime_dependency(%q.freeze, ["~> 1.1"]) + s.add_runtime_dependency(%q.freeze, [">= 4.2.0"]) s.add_development_dependency(%q.freeze, [">= 0"]) s.add_development_dependency(%q.freeze, [">= 0"]) s.add_development_dependency(%q.freeze, [">= 1.8.0"]) @@ -92,6 +93,7 @@ Gem::Specification.new do |s| s.add_dependency(%q.freeze, [">= 0"]) s.add_dependency(%q.freeze, [">= 0"]) s.add_dependency(%q.freeze, [">= 0"]) + s.add_dependency(%q.freeze, [">= 4.2.0"]) s.add_dependency(%q.freeze, [">= 0"]) s.add_dependency(%q.freeze, [">= 0"]) s.add_dependency(%q.freeze, [">= 0"]) @@ -113,6 +115,7 @@ Gem::Specification.new do |s| s.add_dependency(%q.freeze, [">= 0"]) s.add_dependency(%q.freeze, [">= 0"]) s.add_dependency(%q.freeze, [">= 0"]) + s.add_dependency(%q.freeze, [">= 4.2.0"]) s.add_dependency(%q.freeze, [">= 0"]) s.add_dependency(%q.freeze, [">= 0"]) s.add_dependency(%q.freeze, [">= 0"]) From fa0b577022185eee8fca3a3d4979775ee0592978 Mon Sep 17 00:00:00 2001 From: Mike Rogers Date: Thu, 10 Dec 2020 20:32:24 +0000 Subject: [PATCH 4/5] Adjusting requirements - it makes my local tests pass --- docker-compose.yml | 2 ++ sidekiq-cron.gemspec | 1 + 2 files changed, 3 insertions(+) diff --git a/docker-compose.yml b/docker-compose.yml index a21b9d52..54d06d36 100644 --- a/docker-compose.yml +++ b/docker-compose.yml @@ -11,6 +11,8 @@ services: - 8.8.4.4 redis: image: redis + ports: + - "6379:6379" tests: image: sidekiq-cron-test environment: *environment diff --git a/sidekiq-cron.gemspec b/sidekiq-cron.gemspec index 4907d93f..30e99cbc 100644 --- a/sidekiq-cron.gemspec +++ b/sidekiq-cron.gemspec @@ -89,6 +89,7 @@ Gem::Specification.new do |s| s.add_dependency(%q.freeze, ["~> 1.1"]) s.add_dependency(%q.freeze, [">= 0"]) s.add_dependency(%q.freeze, [">= 0"]) + s.add_dependency(%q.freeze, [">= 4.2.0"]) s.add_dependency(%q.freeze, [">= 1.8.0"]) s.add_dependency(%q.freeze, [">= 0"]) s.add_dependency(%q.freeze, [">= 0"]) From a8af9750327248ab4e3a77a36bd9b93c98b97ab0 Mon Sep 17 00:00:00 2001 From: Mike Rogers Date: Sun, 3 Jan 2021 22:02:59 +0000 Subject: [PATCH 5/5] Update sidekiq-cron.gemspec --- sidekiq-cron.gemspec | 2 -- 1 file changed, 2 deletions(-) diff --git a/sidekiq-cron.gemspec b/sidekiq-cron.gemspec index 30e99cbc..e1575eb3 100644 --- a/sidekiq-cron.gemspec +++ b/sidekiq-cron.gemspec @@ -94,7 +94,6 @@ Gem::Specification.new do |s| s.add_dependency(%q.freeze, [">= 0"]) s.add_dependency(%q.freeze, [">= 0"]) s.add_dependency(%q.freeze, [">= 0"]) - s.add_dependency(%q.freeze, [">= 4.2.0"]) s.add_dependency(%q.freeze, [">= 0"]) s.add_dependency(%q.freeze, [">= 0"]) s.add_dependency(%q.freeze, [">= 0"]) @@ -130,4 +129,3 @@ Gem::Specification.new do |s| s.add_dependency(%q.freeze, [">= 0"]) end end -