knapsack_pro 1.21.0 → 1.22.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 59df1c0c620e9c92fb18e3963029fd02df6f04754ffe1b5c590cd7ca45159f55
-  data.tar.gz: 5cf6b36b8f908d8dbb9788918f7586c3b0334ec91a3a79afd768fbde3a8088b2
+  metadata.gz: 6c1620234d6399fb4f66840cc101d290f70e9b8c6bbb6c7f2df7b2f2fa16062a
+  data.tar.gz: f0d3b9c93244b37baa1817bb2b910d1e4e1f3a84e13482f90755a528a04c8a5a
 SHA512:
-  metadata.gz: ae228667bbd897ad31512140d6c6382aefe2fce977748f909b3da7cca7a39495731cde4e079cd8a0247bd2d332fe8809aecf69fe773c34f864e74febc0a4dbbc
-  data.tar.gz: fc8255bc114f010a6a85a14aa98dab171767769e8476c3c422fd92e17198588e767f08bdb1072ca36c5038aef23af4396a8b44fa271b720845f26bd16fbae32b
+  metadata.gz: 5bfe8f48e23514d4dc3c9dd957fbaa5d14d39fae4b9c6eb5c9426eeaae9a57007fb379d6a00860ea30b4cb9766444678e55bf306bba742cf80d470387e22c9a7
+  data.tar.gz: 171de7e1e9c7dbf18b291ffd0f183459e0b9a41b0c60ff2c15de4867db6c07caefcae8e3d57718a001e04c49320878cc022656cfc82709b948e48770dafb2b5c

data/CHANGELOG.md

@@ -1,5 +1,16 @@
 # Change Log
 
+### 1.22.0
+
+* Increase request retry timebox from 4s to 8s to not flood Knapsack Pro API with too many requests in a short period of time and to give time for API server to autoscale and add additional machines to serve traffic
+* When Fallback Mode is disabled with env `KNAPSACK_PRO_FALLBACK_MODE_ENABLED=false` then retry the request to Knapsack Pro API for 6 times instead of only 3 times.
+
+  Here is related [info why some users want to disable Fallback Mode](https://github.com/KnapsackPro/knapsack_pro-ruby#required-ci-configuration-if-you-use-retry-single-failed-ci-node-feature-on-your-ci-server-when-knapsack_pro_fixed_queue_splittrue-in-queue-mode-or-knapsack_pro_fixed_test_suite_splittrue-in-regular-mode).
+
+    https://github.com/KnapsackPro/knapsack_pro-ruby/pull/112
+
+https://github.com/KnapsackPro/knapsack_pro-ruby/compare/v1.21.0...v1.22.0
+
 ### 1.21.0
 
 * Automatically detect slow test files for RSpec and split them by test examples when `KNAPSACK_PRO_RSPEC_SPLIT_BY_TEST_EXAMPLES=true`

data/lib/knapsack_pro/client/connection.rb

@@ -4,8 +4,8 @@ module KnapsackPro
       class ServerError < StandardError; end
 
       TIMEOUT = 15
-      MAX_RETRY = 3
-      REQUEST_RETRY_TIMEBOX = 4
+      MAX_RETRY = -> { KnapsackPro::Config::Env.fallback_mode_enabled? ? 3 : 6 }
+      REQUEST_RETRY_TIMEBOX = 8
 
       def initialize(action)
         @action = action
@@ -118,7 +118,7 @@ module KnapsackPro
       rescue ServerError, Errno::ECONNREFUSED, Errno::ETIMEDOUT, Errno::EPIPE, EOFError, SocketError, Net::OpenTimeout, Net::ReadTimeout, OpenSSL::SSL::SSLError => e
         logger.warn(e.inspect)
         retries += 1
-        if retries < MAX_RETRY
+        if retries < MAX_RETRY.call
           wait = retries * REQUEST_RETRY_TIMEBOX
           logger.warn("Wait #{wait}s and retry request to Knapsack Pro API.")
           Kernel.sleep(wait)

data/lib/knapsack_pro/version.rb

@@ -1,3 +1,3 @@
 module KnapsackPro
-  VERSION = '1.21.0'
+  VERSION = '1.22.0'
 end

data/spec/knapsack_pro/client/connection_spec.rb

@@ -72,12 +72,13 @@ shared_examples 'when retry request' do
 
     before do
       expect(KnapsackPro).to receive(:logger).at_least(1).and_return(logger)
+    end
+
+    it do
       expect(logger).to receive(:debug).exactly(3).with("#{expected_http_method} http://api.knapsackpro.test:3000/v1/fake_endpoint")
       expect(logger).to receive(:debug).exactly(3).with('API request UUID: fake-uuid')
       expect(logger).to receive(:debug).exactly(3).with('API response:')
-    end
 
-    it do
       parsed_response = { 'error' => 'Internal Server Error' }
 
       expect(logger).to receive(:error).exactly(3).with(parsed_response)
@@ -85,16 +86,51 @@ shared_examples 'when retry request' do
       server_error = described_class::ServerError.new(parsed_response)
       expect(logger).to receive(:warn).exactly(3).with(server_error.inspect)
 
-      expect(logger).to receive(:warn).with("Wait 4s and retry request to Knapsack Pro API.")
       expect(logger).to receive(:warn).with("Wait 8s and retry request to Knapsack Pro API.")
-      expect(Kernel).to receive(:sleep).with(4)
+      expect(logger).to receive(:warn).with("Wait 16s and retry request to Knapsack Pro API.")
       expect(Kernel).to receive(:sleep).with(8)
+      expect(Kernel).to receive(:sleep).with(16)
 
       expect(subject).to eq(parsed_response)
 
       expect(connection.success?).to be false
       expect(connection.errors?).to be true
     end
+
+    context 'when Fallback Mode is disabled' do
+      before do
+        expect(KnapsackPro::Config::Env).to receive(:fallback_mode_enabled?).at_least(1).and_return(false)
+      end
+
+      it do
+        expect(logger).to receive(:debug).exactly(6).with("#{expected_http_method} http://api.knapsackpro.test:3000/v1/fake_endpoint")
+        expect(logger).to receive(:debug).exactly(6).with('API request UUID: fake-uuid')
+        expect(logger).to receive(:debug).exactly(6).with('API response:')
+
+        parsed_response = { 'error' => 'Internal Server Error' }
+
+        expect(logger).to receive(:error).exactly(6).with(parsed_response)
+
+        server_error = described_class::ServerError.new(parsed_response)
+        expect(logger).to receive(:warn).exactly(6).with(server_error.inspect)
+
+        expect(logger).to receive(:warn).with("Wait 8s and retry request to Knapsack Pro API.")
+        expect(logger).to receive(:warn).with("Wait 16s and retry request to Knapsack Pro API.")
+        expect(logger).to receive(:warn).with("Wait 24s and retry request to Knapsack Pro API.")
+        expect(logger).to receive(:warn).with("Wait 32s and retry request to Knapsack Pro API.")
+        expect(logger).to receive(:warn).with("Wait 40s and retry request to Knapsack Pro API.")
+        expect(Kernel).to receive(:sleep).with(8)
+        expect(Kernel).to receive(:sleep).with(16)
+        expect(Kernel).to receive(:sleep).with(24)
+        expect(Kernel).to receive(:sleep).with(32)
+        expect(Kernel).to receive(:sleep).with(40)
+
+        expect(subject).to eq(parsed_response)
+
+        expect(connection.success?).to be false
+        expect(connection.errors?).to be true
+      end
+    end
   end
 end
 
@@ -186,7 +222,7 @@ describe KnapsackPro::Client::Connection do
       let(:http_method) { :post }
 
       before do
-        expect(http).to receive(:post).exactly(3).with(
+        expect(http).to receive(:post).at_least(3).with(
           endpoint_path,
           request_hash.to_json,
           {
@@ -210,7 +246,7 @@ describe KnapsackPro::Client::Connection do
       before do
         uri = URI.parse("http://api.knapsackpro.test:3000#{endpoint_path}")
         uri.query = URI.encode_www_form(request_hash)
-        expect(http).to receive(:get).exactly(3).with(
+        expect(http).to receive(:get).at_least(3).with(
           uri,
           {
             'Content-Type' => 'application/json',

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: knapsack_pro
 version: !ruby/object:Gem::Version
-  version: 1.21.0
+  version: 1.22.0
 platform: ruby
 authors:
 - ArturT
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-05-06 00:00:00.000000000 Z
+date: 2020-05-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake