foreman-tasks 7.0.0 → 7.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1c6773ac47a0c5b8e3e28da1461a348510e8595da0bd2d08feafd78092394fb2
-  data.tar.gz: 40d4c506cd3d31b72355e8c2d4255c854c703be6be49644b8d30a55032bd9b2b
+  metadata.gz: 7b653c4a46abfb110f3a5c1ff0f130814110475989e364d39aa2756bc76c4ae2
+  data.tar.gz: b7ad044c96a9cd2f7f7dbf10416ce42b88e5f2ddaf4fe5cbed696cfde39dee32
 SHA512:
-  metadata.gz: 1d05cdb24aea11a647028a4b1c314cf5f0db3d95459bbcfa0c52aa8f53263b6deb3d27ee23bbb7ec2a227e1ebbead0a432c0ecf6701131b2f7921b39145085ab
-  data.tar.gz: 777127d69a84b7c94780f78ad5a8aa5754ad246e6019a9f35c518e0b246d73b8f6f31acc84029b1b0648b8ff3ba8f167efc4fad7f4116cb4eb2f3fad7b850591
+  metadata.gz: b0525a6e659bf05476e86b69c6147600463f16274e71fc15b1ccee628011a785dc53fc6b8cad0ef5f3439cac8c6581a73cea7a1175c4489778f5204fe3914c76
+  data.tar.gz: feab08efc57055c27fab6010dd1f5dac438f74091dee9c239da5722a7236eacf467433f17838066e3ef707f2268e39e2f3130615a25a9094286a9c65d66b2485

data/.github/workflows/ruby_tests.yml

@@ -29,7 +29,7 @@ jobs:
       fail-fast: false
       matrix:
         foreman-core-branch: [develop]
-        ruby-version: [2.5, 2.6, 2.7]
+        ruby-version: [2.7]
         node-version: [12]
     steps:
       - run: sudo apt-get update

data/app/lib/actions/helpers/with_delegated_action.rb

@@ -3,7 +3,7 @@ module Actions
     module WithDelegatedAction
       include ::Actions::Helpers::WithContinuousOutput
 
-      def plan_delegated_action(proxy, klass, options)
+      def plan_delegated_action(proxy, klass, options, proxy_action_class: ::Actions::ProxyAction)
         case proxy
         when :not_defined
           if klass.is_a?(String)
@@ -14,7 +14,7 @@ module Actions
         when :not_available
           raise Foreman::Exception, _('All proxies with the required feature are unavailable at the moment')
         when ::SmartProxy
-          delegated_action = plan_action(::Actions::ProxyAction, proxy, klass, options)
+          delegated_action = plan_action(proxy_action_class, proxy, klass, options)
         end
 
         input[:delegated_action_id] = delegated_action.id

data/app/lib/actions/proxy_action.rb

@@ -22,7 +22,15 @@ module Actions
       end
     end
 
-    class ProxyActionStopped; end
+    class ProxyActionStopped < RuntimeError
+      def backtrace
+        []
+      end
+    end
+
+    ProxyActionStoppedEvent = ::Algebrick.type do
+      fields! exception: type { variants NilClass, Exception }
+    end
 
     def plan(proxy, klass, options)
       options[:connection_options] ||= {}
@@ -52,8 +60,8 @@ module Actions
           on_data(event.data, event.meta)
         when ProxyActionMissing
           on_proxy_action_missing
-        when ProxyActionStopped
-          on_proxy_action_stopped
+        when ProxyActionStoppedEvent
+          on_proxy_action_stopped(event)
         else
           raise "Unexpected event #{event.inspect}"
         end
@@ -94,6 +102,8 @@ module Actions
       else
         suspend
       end
+    rescue RestClient::NotFound
+      on_proxy_action_missing
     end
 
     def cancel_proxy_task
@@ -133,8 +143,12 @@ module Actions
       error! ProxyActionMissing.new(_('Proxy task gone missing from the smart proxy'))
     end
 
-    def on_proxy_action_stopped
-      check_task_status
+    def on_proxy_action_stopped(event)
+      if event.exception
+        error! ProxyActionStopped.new(_('Failed to trigger task on the smart proxy: ') + event.exception.message)
+      else
+        check_task_status
+      end
     end
 
     # @override String name of an action to be triggered on server

data/app/lib/actions/trigger_proxy_batch.rb

@@ -42,7 +42,10 @@ module Actions
     rescue => e
       action_logger.warn "Could not trigger task on the smart proxy"
       action_logger.warn e
-      batch.each { |remote_task| remote_task.update_from_batch_trigger({}) }
+      # The response contains non-serializable objects
+      # TypeError: no _dump_data is defined for class Monitor
+      e.response = nil
+      batch.each { |remote_task| remote_task.update_from_batch_trigger({ 'exception' => e }) }
       output[:failed_count] += batch.size
     end
 

data/app/models/foreman_tasks/remote_task.rb

@@ -7,7 +7,7 @@ module ForemanTasks
                       :foreign_key => :execution_plan_id,
                       :inverse_of  => :remote_tasks
 
-    scope :triggered, -> { where(:state => 'triggered') }
+    scope :triggered, -> { where(:state => ['triggered', 'parent-triggered']) }
     scope :pending,   -> { where(:state => 'new') }
     scope :external,  -> { where(:state => 'external') }
 
@@ -49,10 +49,11 @@ module ForemanTasks
         self.parent_task_id = parent['task_id']
         self.state = 'parent-triggered'
       else
+        exception = data['exception']
         # Tell the action the task on the smart proxy stopped
         ForemanTasks.dynflow.world.event execution_plan_id,
                                          step_id,
-                                         ::Actions::ProxyAction::ProxyActionStopped.new,
+                                         ::Actions::ProxyAction::ProxyActionStoppedEvent[exception],
                                          optional: true
       end
       save!

data/app/views/foreman_tasks/recurring_logics/index.html.erb

@@ -38,16 +38,16 @@
 <table class="<%= table_css_classes('table-condensed table-fixed') %>">
   <thead>
     <th class="col-md-1"><%= N_("ID") %></th>
-    <th><%= N_("Cron line") %></th>
-    <th><%= N_("Task count") %></th>
-    <th><%= N_("Action") %></th>
-    <th><%= N_("Last occurrence") %></th>
-    <th><%= N_("Next occurrence") %></th>
-    <th><%= N_("Current iteration") %></th>
-    <th><%= N_("Iteration limit") %></th>
-    <th><%= N_("Repeat until") %></th>
-    <th><%= N_("State") %></th>
-    <th><%= N_("Purpose") %></th>
+    <th><%= _("Cron line") %></th>
+    <th><%= _("Task count") %></th>
+    <th><%= _("Action") %></th>
+    <th><%= _("Last occurrence") %></th>
+    <th><%= _("Next occurrence") %></th>
+    <th><%= _("Current iteration") %></th>
+    <th><%= _("Iteration limit") %></th>
+    <th><%= _("Repeat until") %></th>
+    <th><%= _("State") %></th>
+    <th><%= _("Purpose") %></th>
     <th/>
   </thead>
   <% @recurring_logics.each do |recurring_logic| %>

data/app/views/foreman_tasks/task_groups/_common.html.erb

@@ -1,11 +1,11 @@
 <div>
   <table class='<%= table_css_classes('table-condensed') %>'>
     <tr>
-      <th><%= N_('ID') %></th>
+      <th><%= _('ID') %></th>
       <td><%= link_to(task_group.id, foreman_tasks_task_group_url(task_group)) %></td>
     </tr>
     <tr>
-      <th><%= N_('Task count') %></th>
+      <th><%= _('Task count') %></th>
       <td><%= link_to(task_group.tasks.count, foreman_tasks_tasks_url(:search => "task_group.id = #{task_group.id}")) %></td>
     </tr>
   </table>

data/app/views/foreman_tasks/task_groups/recurring_logic_task_groups/_recurring_logic_task_group.html.erb

@@ -1,47 +1,47 @@
 <% recurring_logic = task_group.recurring_logic -%>
 <table class='<%= table_css_classes('table-condensed') %>'>
   <tr>
-    <th>ID</th>
+    <th><%= _('ID') %></th>
     <td><%= link_to(recurring_logic.id, recurring_logic) %></td>
   </tr>
   <tr>
-    <th><%= N_("Cron line") %></th>
+    <th><%= _("Cron line") %></th>
     <td><%= recurring_logic.cron_line %></td>
   </tr>
   <tr>
-    <th><%= N_("Action") %></th>
+    <th><%= _("Action") %></th>
     <td><%= format_task_input(recurring_logic.tasks.last) %></td>
   </tr>
   <tr>
-    <th><%= N_("Last occurrence") %></th>
+    <th><%= _("Last occurrence") %></th>
     <td><%= recurring_logic.tasks.order(:started_at).where('started_at IS NOT NULL').last.try(:started_at) || '-' %></td>
   </tr>
   <tr>
-    <th><%= N_("Next occurrence") %></th>
+    <th><%= _("Next occurrence") %></th>
     <td><%= recurring_logic_next_occurrence recurring_logic %></td>
   </tr>
   <tr>
-    <th><%= N_("Current iteration") %></th>
+    <th><%= _("Current iteration") %></th>
     <td><%= recurring_logic.iteration %></td>
   </tr>
   <tr>
-    <th><%= N_("Iteration limit") %></th>
+    <th><%= _("Iteration limit") %></th>
     <td><%= format_recurring_logic_limit recurring_logic.max_iteration %></td>
   </tr>
   <tr>
-    <th><%= N_("Repeat until") %></th>
+    <th><%= _("Repeat until") %></th>
     <td><%= format_recurring_logic_limit recurring_logic.end_time.try(:in_time_zone) %></td>
   </tr>
   <tr>
-    <th><%= N_("State") %></th>
+    <th><%= _("State") %></th>
     <td><%= recurring_logic_state(recurring_logic) %></td>
   </tr>
   <tr>
-    <th><%= N_("Purpose") %></th>
+    <th><%= _("Purpose") %></th>
     <td><%= recurring_logic.purpose %></td>
   </tr>
   <tr>
-    <th><%= N_("Task count") %></th>
+    <th><%= _("Task count") %></th>
     <td><%= link_to(task_group.tasks.count, foreman_tasks_tasks_url(:search => "task_group.id = #{task_group.id}")) %></td>
   </tr>
 </table>

data/lib/foreman_tasks/version.rb

@@ -1,3 +1,3 @@
 module ForemanTasks
-  VERSION = '7.0.0'.freeze
+  VERSION = '7.1.0'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: foreman-tasks
 version: !ruby/object:Gem::Version
-  version: 7.0.0
+  version: 7.1.0
 platform: ruby
 authors:
 - Ivan Nečas
 autorequire:
 bindir: bin
 cert_chain: []
-date: 1980-01-01 00:00:00.000000000 Z
+date: 2022-11-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: dynflow
@@ -608,7 +608,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.26
+rubygems_version: 3.3.20
 signing_key:
 specification_version: 4
 summary: Foreman plugin for showing tasks information for resources and users