mesa_test 1.2.0 → 1.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 77716549c3af3f8960f9d630145619aed3fcc1a35b66f8303d0f5561fe70306a
-  data.tar.gz: 3a52ee9cbc1f853469979ffaff62ff2e72ef9422a9edc643a13f387747d414da
+  metadata.gz: 3c01327f74ca824acf8b7cb473561037281255de396cc623c6c636b9c21bb2ed
+  data.tar.gz: 52f1c9d09ece9a85bdfc3827249357ec1ada18da6b14d70793df9b082331bbe9
 SHA512:
-  metadata.gz: a1db1bf6457b33ea8e0ce55f4cd6c6fafe3ece040925176748bd49541cdb6ba0540c3c7a6354743685df210a82be0f9acbcd1a9ad692d0be88a012746971eb8c
-  data.tar.gz: 264f3d0dd767b4bc610cbbd2d736bd387840e675de87a52ef5748453a5c0ffef8a7f90f6a2f0f14019fdb974cee54e9c1ad4c86227c04de5d9e0553f29f5d2a2
+  metadata.gz: 91a42fce1cf44a643b465edf60351448ea863ebd8ac4e5f962733928b9c620c1dafb251c38cbc4bd9267309ee6298d96fb05a12b35847a974025aa8a183f9037
+  data.tar.gz: 19aca1c41bc5519d8082f7ee5e85398a3652d9b7455f21b4ecf2b2b767084e3abfe5fd52a2630f4a17a8248879e84375549b954453f3c2c79a1acdf6590cd6cf

data/lib/mesa_test.rb

@@ -330,7 +330,12 @@ e-mail and password will be stored in plain text.'
       password: password,
       computer_name: computer_name
     }.to_json
-    JSON.parse(https.request(request).body).to_hash
+    response = testhub_request(https, request)
+    # if the hub was unreachable, behave as an unverified computer; the
+    # network error has already been reported by testhub_request
+    return {} if response.nil?
+
+    JSON.parse(response.body).to_hash
   end
 
   # submit entire commit's worth of test cases, OR submit compilation status
@@ -366,11 +371,15 @@ e-mail and password will be stored in plain text.'
     request.body = request_data.to_json
 
     # actually do the submission
-    response = https.request request
+    response = testhub_request(https, request)
 
-    if !response.is_a? Net::HTTPCreated
+    if response.nil?
+      # network failure; testhub_request already explained why
+      false
+    elsif !response.is_a? Net::HTTPCreated
       shell.say "\nFailed to submit some or all test case instances and/or "\
-                'commit data.', :red
+                "commit data (server responded #{response.code} "\
+                "#{response.message}).", :red
       false
     else
       shell.say "\nSuccessfully submitted commit #{mesa.sha}.", :green
@@ -427,12 +436,16 @@ e-mail and password will be stored in plain text.'
     request.body = request_data.to_json
 
     # actually do the submission
-    response = https.request request
+    response = testhub_request(https, request)
 
-    if !response.is_a? Net::HTTPCreated
+    if response.nil?
+      # network failure; testhub_request already explained why
+      return false
+    elsif !response.is_a? Net::HTTPCreated
       shell.say "\nFailed to submit #{test_case.test_name} for commit "\
-                "#{mesa.sha}", :red
-      false
+                "#{mesa.sha} (server responded #{response.code} "\
+                "#{response.message}).", :red
+      return false
     else
       shell.say "\nSuccessfully submitted instance of #{test_case.test_name} "\
                 "for commit #{mesa.sha}.", :green
@@ -444,17 +457,50 @@ e-mail and password will be stored in plain text.'
     end
   end
 
+  # Perform an HTTP request against the test hub with bounded connect/read
+  # timeouts, so a slow or unreachable server (e.g. while it is under heavy
+  # load) fails fast with a clear message instead of hanging on the default
+  # 60-second connect timeout and then dumping a raw Ruby backtrace. Returns
+  # the Net::HTTPResponse, or +nil+ if the request could not be completed
+  # because of a network problem.
+  def testhub_request(https, request)
+    https.open_timeout = 10
+    https.read_timeout = 60
+    https.request(request)
+  rescue StandardError => e
+    shell.say "\nCould not reach the test hub at #{https.address} "\
+              "(#{e.class}: #{e.message}).", :red
+    nil
+  end
+
   # make generic request to LOGS server
   # +params+ is a hash of data to be encoded as JSON and sent off
+  #
+  # Returns the Net::HTTPResponse on success, or +nil+ if the request could
+  # not be completed because of a network problem (the LOGS server being
+  # unreachable, slow, or refusing connections). The LOGS server only receives
+  # diagnostic build/test output, so a failure here must never crash the run or
+  # fail a CI build whose actual test results already reached the test hub. We
+  # cap the connect/read time so we fail fast instead of hanging on the default
+  # 60-second open timeout for every test case.
   def submit_logs(params)
     #uri = URI('https://logs.mesastar.org/uploads')
     uri = URI('https://mesa-logs.flatironinstitute.org/uploads')
     https = Net::HTTP.new(uri.host, uri.port)
     https.use_ssl = true
+    https.open_timeout = 10
+    https.read_timeout = 30
     req = Net::HTTP::Post.new(uri.path, 'Content-Type' => 'application/json',
                               'X-Api-Key' => logs_token)
     req.body = params.to_json
-    https.request(req)
+    begin
+      https.request(req)
+    rescue StandardError => e
+      shell.say "\nCould not reach the LOGS server at #{uri.host} "\
+                "(#{e.class}: #{e.message}). Skipping log upload; this does "\
+                'not affect test results already sent to the test hub.', :yellow
+      nil
+    end
   end
 
   # send build log to the logs server
@@ -473,9 +519,13 @@ e-mail and password will be stored in plain text.'
     res = submit_logs(build_log_params(mesa))
 
     # report out results
-    if !res.is_a? Net::HTTPOK
+    if res.nil?
+      # network failure; submit_logs already explained why. Don't fail the run.
+      false
+    elsif !res.is_a? Net::HTTPOK
       shell.say "\nFailed to submit build.log to the LOGS server for commit "\
-                "#{mesa.sha}.", :red
+                "#{mesa.sha} (server responded #{res.code} #{res.message}).",
+                :red
       false
     else
       shell.say "\nSuccessfully submitted build.log to the LOGS server for "\
@@ -503,9 +553,13 @@ e-mail and password will be stored in plain text.'
     res = submit_logs(test_log_params(test_case))
 
     # report out results
-    if !res.is_a? Net::HTTPOK
+    if res.nil?
+      # network failure; submit_logs already explained why. Don't fail the run.
+      false
+    elsif !res.is_a? Net::HTTPOK
       shell.say "Failed to submit logs for test case #{test_case.test_name} "\
-                "in commit #{test_case.mesa.sha}.", :red
+                "in commit #{test_case.mesa.sha} (server responded "\
+                "#{res.code} #{res.message}).", :red
       false
     else
       shell.say "Successfully submitted logs for test case "\

metadata

@@ -1,14 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: mesa_test
 version: !ruby/object:Gem::Version
-  version: 1.2.0
+  version: 1.2.1
 platform: ruby
 authors:
 - William Wolf
-autorequire:
 bindir: bin
 cert_chain: []
-date: 2026-05-26 00:00:00.000000000 Z
+date: 2026-05-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: json
@@ -66,7 +65,6 @@ homepage: https://github.com/MESAHub/mesa_test
 licenses:
 - MIT
 metadata: {}
-post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -81,8 +79,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.4.17
-signing_key:
+rubygems_version: 3.7.1
 specification_version: 4
 summary: Command line tool for running and reporting the MESA test suites.
 test_files: []