ci-queue 0.17.0 → 0.17.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 85d4db3b22fa289a9fed0d6456071f79a5e4ced83c6fc4fdee7cc4d8725df647
-  data.tar.gz: e4635ddf545a73f4680eac9660bfd82e07962e52cc23281a7328e8c7beac6b7d
+  metadata.gz: 27cf10eb673c561ed2dac70e1c0d91ac026aafd331902f5e71fb278dcac953d6
+  data.tar.gz: 75df09bcd1018694fd49c05432a08d2904032baedce5bd24a9a07b6d46be6a28
 SHA512:
-  metadata.gz: 79b7d97bfb440d829695c56aaf6634805098f95967afd07a52a7140c7d8b610eca8ae07029cf6e81846a51890960be4eba41e2b430f27b10a43dd426e9c3a038
-  data.tar.gz: 199500511e7864d46d4de0a4cb8f01dab40149005137b3d8cb8ef17d7050c4a5d5c7bc2d4c088ff325a996929207a837badd5af82d5ee42136ecb342f93fa796
+  metadata.gz: 7bd15e57e3c5298ecfce7dc8fa881b98e2c7406ba7f43832f6746035e9fe850ff015ee5494bef13515195a4fc778ef7181fd670c0bd1ff037b6b755d26e5ac43
+  data.tar.gz: 642c70bc89150e8f48826b3167598f5da0cfc3eff18124331a6abbaaea9b0a6fe6295d5d131ce31afb06fbca8812a5f5fd40f406f01b61d87991a9bf862f1369

data/lib/ci/queue/configuration.rb

@@ -4,7 +4,7 @@ module CI
     class Configuration
       attr_accessor :timeout, :build_id, :worker_id, :max_requeues, :grind_count, :failure_file
       attr_accessor :requeue_tolerance, :namespace, :seed, :failing_test, :statsd_endpoint
-      attr_accessor :max_test_duration, :max_test_duration_percentile
+      attr_accessor :max_test_duration, :max_test_duration_percentile, :track_test_duration
       attr_reader :circuit_breakers
 
       class << self
@@ -30,7 +30,7 @@ module CI
         timeout: 30, build_id: nil, worker_id: nil, max_requeues: 0, requeue_tolerance: 0,
         namespace: nil, seed: nil, flaky_tests: [], statsd_endpoint: nil, max_consecutive_failures: nil,
         grind_count: nil, max_duration: nil, failure_file: nil, max_test_duration: nil,
-        max_test_duration_percentile: 0.5
+        max_test_duration_percentile: 0.5, track_test_duration: false
       )
         @circuit_breakers = [CircuitBreaker::Disabled]
         @build_id = build_id
@@ -46,6 +46,7 @@ module CI
         @worker_id = worker_id
         @max_test_duration = max_test_duration
         @max_test_duration_percentile = max_test_duration_percentile
+        @track_test_duration = track_test_duration
         self.max_duration = max_duration
         self.max_consecutive_failures = max_consecutive_failures
       end

data/lib/ci/queue/version.rb

@@ -2,7 +2,7 @@
 
 module CI
   module Queue
-    VERSION = '0.17.0'
+    VERSION = '0.17.1'
     DEV_SCRIPTS_ROOT = ::File.expand_path('../../../../../redis', __FILE__)
     RELEASE_SCRIPTS_ROOT = ::File.expand_path('../redis', __FILE__)
   end

data/lib/minitest/queue/runner.rb

@@ -91,14 +91,18 @@ module Minitest
         queue_config.grind_count = grind_count
 
         reporter_queue = CI::Queue::Redis::Grind.new(queue_url, queue_config)
-        test_time_record = CI::Queue::Redis::TestTimeRecord.new(queue_url, queue_config)
 
         Minitest.queue = queue
         reporters = [
           GrindRecorder.new(build: reporter_queue.build),
           TestDataReporter.new(namespace: queue_config&.namespace),
-          TestTimeRecorder.new(build: test_time_record)
         ]
+
+        if queue_config.track_test_duration
+          test_time_record = CI::Queue::Redis::TestTimeRecord.new(queue_url, queue_config)
+          reporters << TestTimeRecorder.new(build: test_time_record)
+        end
+
         if queue_config.statsd_endpoint
           reporters << Minitest::Reporters::StatsdReporter.new(statsd_endpoint: queue_config.statsd_endpoint)
         end
@@ -205,15 +209,21 @@ module Minitest
         grind_reporter = GrindReporter.new(build: supervisor.build)
         grind_reporter.report
 
-        test_time_record = CI::Queue::Redis::TestTimeRecord.new(queue_url, queue_config)
-        test_time_reporter = Minitest::Queue::TestTimeReporter.new(
-          build: test_time_record,
-          limit: queue_config.max_test_duration,
-          percentile: queue_config.max_test_duration_percentile,
-        )
-        test_time_reporter.report
+        test_time_reporter_success = if queue_config.track_test_duration
+          test_time_record = CI::Queue::Redis::TestTimeRecord.new(queue_url, queue_config)
+          test_time_reporter = Minitest::Queue::TestTimeReporter.new(
+            build: test_time_record,
+            limit: queue_config.max_test_duration,
+            percentile: queue_config.max_test_duration_percentile,
+          )
+          test_time_reporter.report
+
+          test_time_reporter.success?
+        else
+          true
+        end
 
-        exit! grind_reporter.success? && test_time_reporter.success? ? 0 : 1
+        exit! grind_reporter.success? && test_time_reporter_success ? 0 : 1
       end
 
       private
@@ -416,6 +426,13 @@ module Minitest
             queue_config.max_consecutive_failures = max
           end
 
+          help = <<~EOS
+            Must set this option in report and report_grind command if you set --max-test-duration in the report_grind
+          EOS
+          opts.on('--track-test-duration', help) do
+            queue_config.track_test_duration = true
+          end
+
           help = <<~EOS
             Set the time limit of the execution time from grinds on a given test.
             For example, when max-test-duration is set to 10 and

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ci-queue
 version: !ruby/object:Gem::Version
-  version: 0.17.0
+  version: 0.17.1
 platform: ruby
 authors:
 - Jean Boussier
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2020-01-06 00:00:00.000000000 Z
+date: 2020-02-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ansi