synchronised_migration 2.1.1 → 2.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cc7b4ada6d890ac183de30b375e6ead7de8dde32827aa870808552420637a006
-  data.tar.gz: 1826cd77772547266f1dcf309a52574960aa0fdd6efb96a148264ad995979b08
+  metadata.gz: 9881480f34d86d8a177756df3a12e3ec54d1f6b9cfa80d2847084101749bca5a
+  data.tar.gz: e79ad9a4ceddcd69a1936789d485c50daa3bbdc95ef74f2052eac8aef08d65b2
 SHA512:
-  metadata.gz: 5748c0d00c031d6bea8400d598c39f97a6bde99eebfe9273e018fd346ca79186a809ef6f16d9ce6a4213a583e88b7f1fce3e713fd2a9dc40a7adb70340016a0c
-  data.tar.gz: 18442e4f7ddcbdb497af3a05852b59677840871c24844945ddf4da3fca9a5edf2857f6090c5c01067d69c88ee7098a7e5e0ca5f8632e273399c614b537c36de2
+  metadata.gz: b921a8a53758ec6a7654c3decd2b86b2e60baf58336ce31a1eaded4cf7a266ad35e1603c2bc81547b94fc4617f7e5175e5bae7e052013c68004c410d6f410263
+  data.tar.gz: 2f934aebd9d2c5978fe368be30e606622861480ed2fd61d1a422c70d33ad8a391b7d19802743c29c3e16cff9e61972bc80051a3a460928d0f195169ce0c8d0de

data/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Synchronised Migration
 
+## 2.1.2
+
+* [TT-7511] Use the correct `ex` option for key value expiry times instead of `ttl`
+
 ## 2.1.1
 
 * [TT-5896] Early exit from migration inside lock if already completed

data/lib/synchronised_migration/main.rb

@@ -50,7 +50,7 @@ class SynchronisedMigration::Main
 
   def mark_successful
     if success_key
-      redis.set success_key, timestamp, ttl: 3600*24*30
+      redis.set success_key, timestamp, ex: 3600*24*30
     end
   end
 
@@ -60,7 +60,7 @@ class SynchronisedMigration::Main
   end
 
   def mark_failed
-    redis.set fail_key, timestamp, ttl: 3600
+    redis.set fail_key, timestamp, ex: 3600
   end
 
   def remove_fail_marker

data/lib/synchronised_migration/version.rb

@@ -1,3 +1,3 @@
 module SynchronisedMigration
-  VERSION = '2.1.1'
+  VERSION = '2.1.2'
 end

data/spec/synchronised_migration/main_spec.rb

@@ -65,8 +65,8 @@ describe SynchronisedMigration::Main do
         expect(result).to be_success
         expect(redlock).to have_received(:lock!)
         expect(redis).to have_received(:get).with('migration-failed-bork')
-        expect(redis).to have_received(:set).with('migration-failed-bork', 123456789, ttl: 3600)
-        expect(redis).to have_received(:set).with('migration-success-bork', 123456789, ttl: 3600*24*30)
+        expect(redis).to have_received(:set).with('migration-failed-bork', 123456789, ex: 3600)
+        expect(redis).to have_received(:set).with('migration-success-bork', 123456789, ex: 3600*24*30)
         expect(Kernel).to have_received(:system)
         expect(Bundler).not_to have_received(:with_original_env)
         expect(redis).to have_received(:del).with('migration-failed-bork')
@@ -128,7 +128,7 @@ describe SynchronisedMigration::Main do
 
       it 'marks the failure in Redis' do
         expect(result).not_to be_success
-        expect(redis).to have_received(:set).with('migration-failed-bork', 123456789, ttl: 3600)
+        expect(redis).to have_received(:set).with('migration-failed-bork', 123456789, ex: 3600)
         expect(redis).not_to have_received(:del)
       end
     end
@@ -141,7 +141,7 @@ describe SynchronisedMigration::Main do
           expect(result).to be_success
           expect(redlock).to have_received(:lock!)
           expect(redis).to have_received(:get).with('migration-failed')
-          expect(redis).to have_received(:set).with('migration-failed', 123456789, ttl: 3600)
+          expect(redis).to have_received(:set).with('migration-failed', 123456789, ex: 3600)
           expect(Kernel).to have_received(:system)
           expect(Bundler).not_to have_received(:with_original_env)
           expect(redis).to have_received(:del).with('migration-failed')
@@ -155,7 +155,7 @@ describe SynchronisedMigration::Main do
 
         it 'marks the failure in Redis' do
           expect(result).not_to be_success
-          expect(redis).to have_received(:set).with('migration-failed', 123456789, ttl: 3600)
+          expect(redis).to have_received(:set).with('migration-failed', 123456789, ex: 3600)
           expect(redis).not_to have_received(:del)
         end
       end

data/synchronised_migration.gemspec

@@ -17,6 +17,7 @@ Gem::Specification.new do |spec|
   spec.require_paths = ['lib']
 
   spec.add_dependency 'redlock', '>= 0.2'
+  spec.add_dependency 'redis'
   spec.add_development_dependency 'rake'
   spec.add_development_dependency 'simplecov-rcov', '>= 0.2'
   spec.add_development_dependency 'rspec', '>= 3.6'

metadata

@@ -1,15 +1,15 @@
 --- !ruby/object:Gem::Specification
 name: synchronised_migration
 version: !ruby/object:Gem::Version
-  version: 2.1.1
+  version: 2.1.2
 platform: ruby
 authors:
 - Alvin Yim
 - Stefan Cooper
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2019-08-22 00:00:00.000000000 Z
+date: 2020-06-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: redlock
@@ -25,6 +25,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0.2'
+- !ruby/object:Gem::Dependency
+  name: redis
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -107,7 +121,7 @@ files:
 homepage: https://github.com/sealink/synchronised-migration-rb
 licenses: []
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -123,7 +137,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubygems_version: 3.0.3
-signing_key: 
+signing_key:
 specification_version: 4
 summary: For deploying to multiple instances simultaneously
 test_files: