Skip to content

Commit

Permalink
add more logging
Browse files Browse the repository at this point in the history
  • Loading branch information
stephsachrajda committed Oct 18, 2023
1 parent c7ece4b commit 774c878
Show file tree
Hide file tree
Showing 4 changed files with 10 additions and 3 deletions.
1 change: 1 addition & 0 deletions ruby/lib/ci/queue/redis/base.rb
Original file line number Diff line number Diff line change
Expand Up @@ -80,6 +80,7 @@ def queue_initializing?
end

def increment_test_failed
puts "---- incrementing cache test failed"
redis.incr(key('test_failed_count'))
end

Expand Down
4 changes: 4 additions & 0 deletions ruby/lib/minitest/queue.rb
Original file line number Diff line number Diff line change
Expand Up @@ -239,13 +239,16 @@ def run_from_queue(reporter, *)
result.mark_as_flaked!
failed = false
elsif failed
puts "---- reporting as failure in poll"
queue.report_failure!
else
queue.report_success!
end

requeued = false

if failed && CI::Queue.requeueable?(result) && queue.requeue(example)
puts "---- requeable in poll"
requeued = true
result.requeue!
reporter.record(result)
Expand All @@ -256,6 +259,7 @@ def run_from_queue(reporter, *)
end

if !requeued && failed
puts "---- incrementing test failed in poll"
queue.increment_test_failed
end
end
Expand Down
3 changes: 3 additions & 0 deletions ruby/lib/minitest/queue/build_status_recorder.rb
Original file line number Diff line number Diff line change
Expand Up @@ -43,13 +43,16 @@ def record(test)
elsif test.skipped?
self.skips += 1
elsif test.error?
puts "--- increment error in record"
self.errors += 1
elsif test.failure
puts "--- increment failure in record"
self.failures += 1
end

stats = COUNTERS.zip(COUNTERS.map { |c| send(c) }).to_h
if (test.failure || test.error?) && !test.skipped?
puts "--- recording error in record"
build.record_error("#{test.klass}##{test.name}", dump(test), stats: stats)
else
build.record_success("#{test.klass}##{test.name}", stats: stats, skip_flaky_record: test.skipped?)
Expand Down
5 changes: 2 additions & 3 deletions ruby/lib/minitest/queue/runner.rb
Original file line number Diff line number Diff line change
Expand Up @@ -234,10 +234,9 @@ def report_command
if queue_config.failure_file
reporter = BuildStatusReporter.new(build: supervisor.build)
failures = reporter.error_reports.map(&:to_h).to_json
puts "----"
puts failures
puts "----"
File.write(queue_config.failure_file, failures)

reporter.report
end

puts('Encountered too many failed tests. Test run was ended early.')
Expand Down

0 comments on commit 774c878

Please sign in to comment.