web47core 3.2.3.49 → 3.2.3.50

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 70781cb08eb5be20273e40a90835d91799d40c28c266631d5f905fcc8443047f
-  data.tar.gz: e876bd9972ca5ea75eb074578b98c1ba435e7100bd4c946b4ca9fa283fbe03cd
+  metadata.gz: 6919700b0f69654fb86470e9ff727198e1e5e8da157240eb58f869597c95b497
+  data.tar.gz: eaa6281463cd954391791d53eeaee3b21875764bed20f554886de54d3209d7cd
 SHA512:
-  metadata.gz: 1e9c9401038a170e3acf2e6677aaa7f840742771036acf8d1f50c0234299892e150d05d0395fad30169359e6c700c9fe62fa2b8bf55cdf2ba9b6e16dbf1df3ae
-  data.tar.gz: e998d7542d10b5c21088697a883eb39ac8137da958b6e1e2d9712629465a5802ce11ec7b78831750bee238976e56c429827cbfd7828bb47a62d8674575a8c926
+  metadata.gz: 7657edbc6f8674bee74604a85aa58bc1e61e5562052ad096b20beaeff701345820e801f6af00f589a9b9b6dddd669e6257767e1fca7d05b9b4cd8bbdf161cda1
+  data.tar.gz: c85f3e3a93c533fa1f5863cf48204dc933e6a81ebf1c7a907d4a60a9a4ccea73f3562b27cf7eb9fcdade153abffa3c34f0cb61df84f7a7bbb7b2492a73f49479

data/lib/app/models/command_job.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require 'open3'
+
 #
 # Base class for all jobs that will be run on builds
 #
@@ -329,11 +331,9 @@ class CommandJob
   #
   def run_command(command, dir = '/tmp', options = {})
     command = command.join(' ') if command.is_a?(Array)
-    output = Tempfile.open('run-command-', '/tmp') do |f|
-      Dir.chdir(dir) { `#{command} > #{f.path} 2>&1` }
-      mask_keywords(f.open.read, options[:mask_texts])
-    end
-    output = 'Success' if output.blank?
+    output, status = Open3.capture2e(command, chdir: dir)
+    output = mask_keywords(output, options[:mask_texts])
+    output = status.success? ? 'Success' : 'No output' if output.blank?
     command = mask_keywords(command, options[:mask_texts])
     if block_given?
       yield output
@@ -341,6 +341,7 @@ class CommandJob
       logs.create!(dir: dir, command: command, message: output)
     end
     options[:output_limit] ||= -1
+    check_for_success(command, status, output, output_limit: options[:output_limit])
     check_for_text(output, options[:error_texts], output_limit: options[:output_limit])
     check_for_text(output, options[:required_texts], inclusive_check: false, output_limit: options[:output_limit])
     output
@@ -359,10 +360,22 @@ class CommandJob
     output
   end
 
-  #
+  # @abstract Check if we were success
+  # @param command  [String] The command that was run
+  # @param status  [Process::Status] Status returned from the command
+  # @param output  [String] The output returned from the command
+  # @param output_limit [Integer] Limit of output to show in error message
+  # @return [Boolean] true if we were success
+  def check_for_success(command, status, output, output_limit: -1)
+    return if status.success?
+
+    reason = status.signaled? ? "signal #{status.termsig}" : "exit status #{status.exitstatus}"
+    error_output = output.slice(0, output_limit) || output
+    raise "Command failed with #{reason}: #{command} - #{error_output}"
+  end
+
   # Check if any occurrences were found (or not found)
   # For most command jobs, we want to see the full output. -1 accomplishes this
-  #
   def check_for_text(output, texts = [], inclusive_check: true, output_limit: -1)
     return if texts.blank?
 

data/lib/web47core/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Web47core
-  VERSION = '3.2.3.49'
+  VERSION = '3.2.3.50'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: web47core
 version: !ruby/object:Gem::Version
-  version: 3.2.3.49
+  version: 3.2.3.50
 platform: ruby
 authors:
 - Chris Schroeder