web47command 0.0.4 → 0.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f7a5d228fd6711bc83e6321edb8488455218fd1c88fd0f2002b28b56806515a5
-  data.tar.gz: 32044f2e427b2d62e50f35801291ca26a32bf875cf066267f7592602a4059758
+  metadata.gz: 507c57273933c17f93c493bc950163b4434ffc5ac3a0396ee83aabc3c60d9355
+  data.tar.gz: 2036dbb4fcb481b60c377c91dd2eaec8b9fbf61628526a063358b1b7bab58b95
 SHA512:
-  metadata.gz: d642520ec8e7aa6dab40703a8228cd05917f87a4a6ad2ba606fb8434dbd59497f96b8d30682e13f26a5356c5c5401635ff925b706b9061954fe078baeb40f6fe
-  data.tar.gz: aadce6c8f98a4d7ef683d4c0591be8371bfeae8fe5236a0c366a8b5a6238a4ca6c642d2275cca8bc87d5904b497dd06cb583f912588d608311bcd889d9f060d7
+  metadata.gz: 391a11146443d5312c30d595a2e494339f1461854117512a9ebd1cb13b24f32c25f91fff710290265e94b774ecc27c4b31437e5cc3e48953110ca20fee965991
+  data.tar.gz: 16b2084ce7f199e310462056e386d69fb559ef227b20e48b1f4b92e169826a9543c24e13e61fba045b30935ddb7febd5f8bc1b1283bc84fa6fc5e37ec49d731a

data/README.md

@@ -73,3 +73,5 @@ _Please do not ship to production code using the git repo, as the production ser
 2.  `JobLog` 
 3.  `CommandLogLog` 
 4.  `TrimJobs` 
+### Remove the following files
+1. Change `status` on a job to `current_status`

data/lib/app/models/command_job.rb

@@ -41,7 +41,7 @@ class CommandJob
   # Who started this job
   #
   def display_started_by
-    started_by.present? ? started_by.display_name : 'System'
+    started_by.present? ? started_by.name : 'System'
   end
 
   #
@@ -62,113 +62,164 @@ class CommandJob
   # True if in new status
   #
   def new_job?
-    STATE_NEW.eql?(state)
+    job_state?(STATE_NEW)
   end
 
   #
   # True if in WIP status
   #
   def work_in_progress?
-    [STATE_RETRYING, STATE_WIP].include?(state)
+    job_state?([STATE_WIP, STATE_RETRYING])
   end
 
   #
   # True if in success status
   #
   def succeeded?
-    STATE_SUCCESS.eql?(state)
+    job_state?(STATE_SUCCESS)
   end
 
   #
   # True if in fail status
   #
-  def failed?
-    STATE_FAIL.eql?(state)
+  def failure?
+    job_state?(STATE_FAIL)
   end
 
   #
   # If we is finished, failed or success
   #
   def completed?
-    [STATE_CANCELLED, STATE_FAIL, STATE_SUCCESS].include?(state)
+    job_state?([STATE_CANCELLED, STATE_FAIL, STATE_SUCCESS])
   end
 
   #
   # Job has not finished, failure or success
   #
-  def incomplete?
+  def running?
     !completed?
   end
 
-  alias running? incomplete?
+  alias incomplete? running?
 
   #
   # If we are cancelled
   #
   def cancelled?
-    STATE_CANCELLED.eql?(state)
+    job_state?(STATE_CANCELLED)
   end
 
-  def job_cancelled?(job = self)
-    job.reload
-    job.cancelled?
-  rescue StandardError => error
-    App47Logger.log_warn "Unable to check job cancelled #{job}", error
-    true
+  def failure_or_cancelled?
+    job_state?([STATE_FAIL, STATE_CANCELLED], true)
   end
 
-  def failure_or_cancelled?(job = self)
-    job.reload
-    job.failure? || job.cancelled?
+  #
+  # Fetch the latest version of this instance from the database and check the state against the required
+  # state. If there is a match, then return true, otherwise return false.
+  # If there is an error, return the default.
+  #
+  def job_state?(states, default = false)
+    states.is_a?(Array) ? states.include?(state) : states.eql?(state)
   rescue StandardError => error
-    App47Logger.log_warn "Unable to check job cancelled #{job}", error
-    true
+    App47Logger.log_warn "Unable to check job failed or cancelled #{self.inspect}", error
+    default
   end
 
   #
   # Return the job's status and information in a hash that could be used to return to a calling
   # api
   #
-  def status
+  def current_status
     status = { state: state }
     status[:message] = error_message if error_message.present?
     status
   end
 
-  def perform
-    if fail?
-      self.retries = 0
-      self.error_message = 'Retrying'
+  #
+  # Perform this job in the background
+  #
+  def perform_later
+    perform
+  end
+
+  handle_asynchronously :perform_later
+
+  #
+  # Steps to execute before a run
+  #
+  def before_run
+    case state
+    when STATE_NEW
+      set retries: 0,
+          started_at: Time.now.utc,
+          finished_at: nil,
+          error_message: nil,
+          result: nil,
+          state: STATE_WIP
+    when STATE_RETRYING
+      set retries: 0,
+          started_at: Time.now.utc,
+          finished_at: nil,
+          error_message: nil,
+          result: nil
+    when STATE_FAIL
+      set retries: 0,
+          started_at: Time.now.utc,
+          finished_at: nil,
+          error_message: nil,
+          state: STATE_RETRYING,
+          result: nil
+    else
+      set retries: 0, started_at: Time.now.utc, finished_at: nil, result: nil
     end
+  end
 
-    begin
-      save!
-      run
-    rescue StandardError => error
-      log_error 'Unable to start job', error
-      set state: STATE_FAIL, error_message: error.message
+  #
+  # Steps to execute after a run
+  #
+  def after_run
+    case state
+    when STATE_RETRYING, STATE_WIP
+      set finished_at: Time.now.utc, error_message: nil, state: STATE_SUCCESS
+    when STATE_SUCCESS
+      set finished_at: Time.now.utc, error_message: nil
+    else
+      set finished_at: Time.now.utc
     end
   end
 
+  #
+  #
+  # Perform the command job
+  #
+  def perform
+    before_run
+    run
+    after_run
+  rescue StandardError => error
+    log_error 'Unable to start job', error
+    set state: STATE_FAIL, error_message: error.message
+  end
+
+  alias perform_now perform
+
+  #
+  # Run the job, handling any failures that might happen
+  #
   def run
-    set state: STATE_WIP
-    # Run the job
-    run!
-    set error_message: '', state: STATE_SUCCESS
+    run! unless cancelled?
   rescue StandardError => error
-    if retries >= max_retries
+    if (retries + 1) >= max_retries
       log_error "Unable to run job id: #{id}, done retrying", error
       set state: STATE_FAIL, error_message: "Failed final attempt: #{error.message}"
     else
       log_error "Unable to run job id: #{id}, retrying!!", error
       add_log "Unable to run job: #{error.message}, retrying!!"
-      set error_message: "Failed attempt # #{retries}: #{error.message}", retries: retries + 1
-      delay(run_at: 10.seconds.from_now).run
+      set error_message: "Failed attempt # #{retries}: #{error.message}", retries: retries + 1, state: STATE_RETRYING
+      run
     end
   end
 
-  handle_asynchronously :run if Delayed::Worker.delay_jobs
-
   #
   # Determine the correct action to take and get it started
   #
@@ -191,6 +242,8 @@ class CommandJob
     download = URI.parse(file_url).open
     IO.copy_stream(download, file_path)
     add_log "Downloaded file: #{file_url} to #{file_path}"
+  rescue StandardError => error
+    raise "Unable to download file from #{file_url} to #{file_path}, error: ##{error.message}"
   end
 
   #
@@ -203,6 +256,8 @@ class CommandJob
     else
       add_log "File not found: #{from_path}, copy not performed"
     end
+  rescue StandardError => error
+    raise "Unable to copy file from #{from_path} to #{to_path}, error: ##{error.message}"
   end
 
   #
@@ -263,7 +318,11 @@ class CommandJob
     end
     output = 'Success' if output.blank?
     command = mask_keywords(command, options[:mask_texts])
-    logs.create!(dir: dir, command: command, message: output)
+    if block_given?
+      yield output
+    else
+      logs.create!(dir: dir, command: command, message: output)
+    end
     check_for_text(output, options[:error_texts], true)
     check_for_text(output, options[:required_texts], false)
     output
@@ -311,4 +370,5 @@ class CommandJob
   def sort_fields
     %i[created_at]
   end
+
 end

data/lib/web47command/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Web47command
-  VERSION = '0.0.4'
+  VERSION = '0.1.0'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: web47command
 version: !ruby/object:Gem::Version
-  version: 0.0.4
+  version: 0.1.0
 platform: ruby
 authors:
 - Chris Schroeder
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-04-11 00:00:00.000000000 Z
+date: 2021-01-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport