foreman_remote_execution 0.0.7 → 0.0.8

checksums.yaml

@@ -1,7 +1,15 @@
 ---
-SHA1:
-  metadata.gz: f7b17c729ed77837816fc91f385aec89be98be24
-  data.tar.gz: 32d6a582df1ffb8e3d20163481176e651070127e
+!binary "U0hBMQ==":
+  metadata.gz: !binary |-
+    NWE1NDJlYTY5N2UyNDEyYmVmNzBlMzRhYzAxOGYwMTM1ZWU0NjIyZg==
+  data.tar.gz: !binary |-
+    ZDNiN2JhOGU3NTJiMTJjNGRmZGU5Mjc3OTM0YmM0Yzk5YWMyYzJkYQ==
 SHA512:
-  metadata.gz: 733f0235a691db6d71bc9ad72bcf1d3c45db6c95530ed9a57d39203d59401b92858ffb9954c2dccba622ae5a3819147fae35415a1073e1cb920fb138ef89ef46
-  data.tar.gz: 35cfdf9e050b3ebe132c955c0f21c41fbd97e33a3dee871f52e556b446a6d64625b66b0c558bd5d01a23465695b0a1cb233d2e39d5ac3f43c48e5aa6793e3d62
+  metadata.gz: !binary |-
+    ZWY2MmU2NjU2MjQxYjZmYjM1NGJjNDIzODE2NzlkMjlhYjVmM2QyYmY1Yzk1
+    MWJmYWVmNzQ0OTIwOTIxYmEyYjE4NTliNTAzMTk4NTA1NGMwMzc0N2M4OTMx
+    MzZiODQ1ZmYwMDk3NzE4ZGU3MmExYjE4NmYyNmY4MTAwMThiYmM=
+  data.tar.gz: !binary |-
+    Mjc1NDY5OWVhNjEwNTVhNTQ3YTNlNWM1NTQ2NWMzZGQzZWIwZWRiMzdkMWJh
+    MmUxY2I0MjlkYWUxNzBhNjQ4MDNmZjU0MjkzMjhjNGExMjE5NmNmNzJhOTc4
+    ZjhjYzY3NjYxMDU1N2UzMGRhZWEyZDExMjg0NTBmNmQ0NTg3ZjI=

data/app/assets/javascripts/template_invocation.js

@@ -57,3 +57,16 @@ function delayed_refresh(url, data){
     )
   }, 1000);
 }
+
+function job_invocation_refresh_data(){
+  return {
+    hosts_needs_refresh: $("div#hosts").data('refresh_required'),
+    host_ids_needing_status_update: fetch_ids_of_hosts('status'),
+    host_ids_needing_provider_update: fetch_ids_of_hosts('provider'),
+    host_ids_needing_actions_update: fetch_ids_of_hosts('actions')
+  }
+}
+
+function fetch_ids_of_hosts(attribute){
+  return _.map($('div#hosts td.host_' + attribute + '[data-refresh_required="true"]'), function(elem) { return $(elem).data('id') });
+}

data/app/controllers/job_invocations_controller.rb

@@ -43,6 +43,8 @@ class JobInvocationsController < ApplicationController
   def show
     @job_invocation = resource_base.find(params[:id])
     @auto_refresh = @job_invocation.last_task.try(:pending?)
+    hosts_base = @job_invocation.targeting.hosts.authorized(:view_hosts, Host)
+    @hosts = hosts_base.search_for(params[:search], :order => params[:order] || 'name ASC').paginate(:page => params[:page])
   end
 
   def index

data/app/helpers/remote_execution_helper.rb

@@ -158,4 +158,15 @@ module RemoteExecutionHelper
             :text => renderer.error_message.html_safe
     end
   end
+
+  def job_invocation_active_tab(tab, params)
+    active = 'active'
+    inactive = ''
+    hosts_tab_active = params[:page].present? || params[:search].present? || params[:order].present?
+    if hosts_tab_active
+      tab == :hosts ? active : inactive
+    else
+      tab == :overview ? active : inactive
+    end
+  end
 end

data/app/models/concerns/foreman_remote_execution/errors_flattener.rb

@@ -0,0 +1,62 @@
+module ForemanRemoteExecution
+  module ErrorsFlattener
+    def flattened_validation_exception
+      ActiveRecord::RecordNotSaved.new(I18n.t("activerecord.errors.messages.record_invalid", :errors => flattened_errors.join(', ')))
+    end
+
+    def flattened_errors
+      errors = Hash.new { |h, k| h[k] = [] }
+      self.errors.keys.each do |key|
+        messages = self.errors[key]
+        invalid_objects = invalid_objects_for_attribute(key)
+        if invalid_objects.blank?
+          errors[key] = messages
+        else
+          invalid_objects.each do |invalid_object|
+            errors.merge!(sub_object_errors(key, invalid_object, messages))
+          end
+        end
+      end
+      errors.map { |key, messages| self.errors.full_message(key, messages.join(', ')) }
+    end
+
+    private
+
+    def invalid_objects_for_attribute(attribute)
+      if self.respond_to?(attribute)
+        invalid_object = self.public_send(attribute)
+        if invalid_object.is_a? Enumerable
+          invalid_object.select { |o| o.respond_to?(:invalid?) && o.invalid? }
+        else
+          [invalid_object]
+        end
+      end
+    end
+
+    def flattened_error_key(key, object)
+      mapping = if defined? self.class::FLATTENED_ERRORS_MAPPING
+                  self.class::FLATTENED_ERRORS_MAPPING
+                else
+                  {}
+                end
+      mapped_key = mapping.fetch(key, key)
+      if mapped_key.is_a? Proc
+        mapped_key.call(object)
+      else
+        mapped_key
+      end
+    end
+
+    def sub_object_errors(key, object, original_message)
+      key = flattened_error_key(key, object)
+      errors = if object.respond_to? :flattened_errors
+                 object.flattened_errors
+               elsif object.respond_to? :errors
+                 object.errors.full_messages
+               else
+                 original_message
+               end
+      return { "#{key}:" => errors }
+    end
+  end
+end

data/app/models/job_invocation.rb

@@ -1,6 +1,11 @@
 class JobInvocation < ActiveRecord::Base
   include Authorizable
 
+  include ForemanRemoteExecution::ErrorsFlattener
+  FLATTENED_ERRORS_MAPPING = { :template_invocations => lambda do |template_invocation|
+                                                          _("template") + " #{template_invocation.template.name}"
+                                                        end }
+
   belongs_to :targeting, :dependent => :destroy
   has_many :template_invocations, :inverse_of => :job_invocation, :dependent => :destroy
 
@@ -10,7 +15,7 @@ class JobInvocation < ActiveRecord::Base
 
   scoped_search :on => :job_name
 
-  delegate :bookmark, :to => :targeting, :allow_nil => true
+  delegate :bookmark, :resolved?, :to => :targeting, :allow_nil => true
 
   include ForemanTasks::Concerns::ActionSubject
 
@@ -51,10 +56,10 @@ class JobInvocation < ActiveRecord::Base
   end
 
   def total_hosts_count
-    if targeting.resolved_at.nil?
-      _('N/A')
-    else
+    if targeting.resolved?
       targeting.hosts.count
+    else
+      _('N/A')
     end
   end
 
@@ -109,4 +114,8 @@ class JobInvocation < ActiveRecord::Base
   def available_providers(host)
     return RemoteExecutionProvider.provider_names
   end
+
+  def sub_task_for_host(host)
+    sub_tasks.joins(:locks).where("#{ForemanTasks::Lock.table_name}.resource_type" => 'Host::Managed', "#{ForemanTasks::Lock.table_name}.resource_id" => host.id).first
+  end
 end

data/app/models/job_invocation_api_composer.rb

@@ -4,7 +4,6 @@ class JobInvocationApiComposer
   delegate :job_name, :to => :job_invocation
   delegate :targeting, :to => :job_invocation
   delegate :template_invocations, :to => :job_invocation
-  delegate :save!, :to => :job_invocation
 
   def initialize(job_invocation, user, params)
     @job_invocation = job_invocation
@@ -56,6 +55,14 @@ class JobInvocationApiComposer
     targeting.valid? & job_invocation.valid? & template_invocations.all?(&:valid?)
   end
 
+  def save!
+    if valid?
+      job_invocation.save!
+    else
+      raise job_invocation.flattened_validation_exception
+    end
+  end
+
   def available_bookmarks
     Bookmark.authorized(:view_bookmarks).my_bookmarks
   end

data/app/models/targeting.rb

@@ -42,6 +42,10 @@ class Targeting < ActiveRecord::Base
     ids.map { |id| "name = #{hosts[id].first.name}" }.join(' or ')
   end
 
+  def resolved?
+    self.resolved_at.present?
+  end
+
   private
 
   def bookmark_or_query_is_present

data/app/models/template_invocation.rb

@@ -1,6 +1,12 @@
 class TemplateInvocation < ActiveRecord::Base
   include ForemanTasks::Concerns::ActionSubject
 
+  include ForemanRemoteExecution::ErrorsFlattener
+  FLATTENED_ERRORS_MAPPING = { :input_values => lambda do |input_value|
+                                                 _("Input") + " #{input_value.template_input.name}"
+                                                end }
+
+
   belongs_to :template, :class_name => 'JobTemplate', :foreign_key => 'template_id'
   belongs_to :job_invocation, :inverse_of => :template_invocations
   has_many :input_values, :class_name => 'TemplateInvocationInputValue', :dependent => :destroy
@@ -16,9 +22,10 @@ class TemplateInvocation < ActiveRecord::Base
   private
 
   def provides_required_input_values
-    required_input_ids = self.template.template_inputs.where(:required => true).pluck(:id)
-    unless (required_input_ids - self.input_values.map(&:template_input_id)).empty?
-      errors.add(:base, _("Not all required inputs have values."))
+    required_inputs = self.template.template_inputs.where(:required => true)
+    missing_inputs = required_inputs.reject { |input| self.input_values.map(&:template_input_id).include?(input.id) }
+    unless missing_inputs.empty?
+      errors.add(:base, _("Not all required inputs have values. Missing inputs: %s") % missing_inputs.map(&:name).join(', '))
     end
   end
 end

data/app/models/template_invocation_input_value.rb

@@ -3,7 +3,7 @@ class TemplateInvocationInputValue < ActiveRecord::Base
   belongs_to :template_invocation
   belongs_to :template_input
 
-  validates :value, :presence => true
+  validates :value, :presence => true, :if => proc { |v| v.template_input.required? || v.value.nil? }
 
   validates :value, :inclusion => { :in => proc { |v| v.template_input.options_array } },
                     :if => proc { |v| v.template_input.input_type == 'user' && v.template_input.options_array.present? }

data/app/views/job_invocations/_host_actions_td.html.erb

@@ -0,0 +1,3 @@
+<td class="host_actions" id="<%= dom_id(host) %>-actions" data-refresh_required="<%= task.nil? ? 'true' : '' %>" data-id="<%= host.id %>">
+  <%= action_buttons(template_invocation_actions(task, host)) %>
+</td>

data/app/views/job_invocations/_host_provider_td.html.erb

@@ -0,0 +1,3 @@
+<td class="host_provider" id="<%= dom_id(host) %>-provider" data-refresh_required="<%= task.nil? ? 'true' : '' %>" data-id="<%= host.id %>">
+  <%= remote_execution_provider_for(task) %>
+</td>

data/app/views/job_invocations/_host_status_td.html.erb

@@ -0,0 +1 @@
+<td class="host_status" id="<%= dom_id(host) %>-status" data-refresh_required="<%= task.nil? || task.pending? ? 'true' : '' %>" data-id="<%= host.id %>"><%= template_invocation_status(task) %></td>

data/app/views/job_invocations/_tab_hosts.html.erb

@@ -1,8 +1,24 @@
-<% if job_invocation.targeting.resolved_at %>
+<% if job_invocation.resolved? %>
+  <%= form_tag job_invocation_path(job_invocation), :method => "get",:id =>"search-form" do %>
+    <div class="row">
+      <div class="title_filter col-md-6">
+        <div class="input-group">
+            <%= auto_complete_search(:search, params[:search].try(:squeeze, " "), :placeholder => _("Filter") + ' ...', :path => hosts_path) %>
+            <span class="input-group-btn">
+              <button class="btn btn-default" type="submit">
+                <%= icon_text("search", content_tag(:span, _("Search"), :class => "hidden-xs")) %>
+              </button>
+            </span>
+        </div>
+      </div>
+    </div>
+  <% end %>
+  <br>
+
   <table class="table table-bordered table-striped table-condensed">
     <thead>
     <tr>
-      <th><%= _('Host') %></th>
+      <th><%= sort :name, :as => _('Host') %></th>
       <th><%= _('Provider') %></th>
       <th><%= _('Status') %></th>
       <th><%= _('Actions') %></th>
@@ -10,17 +26,19 @@
     </thead>
 
     <tbody>
-      <% job_invocation.targeting.hosts.each do |host| %>
-        <tr>
-          <% task = job_invocation.sub_tasks.joins(:locks).where("#{ForemanTasks::Lock.table_name}.resource_type" => 'Host::Managed', "#{ForemanTasks::Lock.table_name}.resource_id" => host.id).first %>
-          <td><%= link_to_if_authorized host.name, hash_for_host_path(host).merge(:auth_object => host, :permission => :view_hosts) %></td>
-          <td><%= remote_execution_provider_for(task) %></td>
-          <td><%= template_invocation_status(task) %></td>
-          <td><%= action_buttons(template_invocation_actions(task, host)) %></td>
-        </tr>
-      <% end %>
+    <% hosts.each do |host| %>
+      <% task = job_invocation.sub_task_for_host(host) %>
+      <tr>
+        <td><%= link_to_if_authorized host.name, hash_for_host_path(host).merge(:auth_object => host, :permission => :view_hosts) %></td>
+        <%= render('host_provider_td', :host => host, :task => task) %>
+        <%= render('host_status_td', :host => host, :task => task) %>
+        <%= render('host_actions_td', :host => host, :task => task) %>
+      </tr>
+    <% end %>
     </tbody>
   </table>
+
+  <%= will_paginate_with_info @hosts %>
 <% else %>
   <div class="alert alert-warning">
     <%= _("The dynamic query '#{job_invocation.targeting.search_query}' was not resolved yet. The list of hosts to which it would resolve now can be seen") %>

data/app/views/job_invocations/show.html.erb

@@ -7,22 +7,22 @@
 <% end %>
 
 <ul class="nav nav-tabs" data-tabs="tabs">
-  <li class="active"><a href="#primary" data-toggle="tab"><%= _('Overview') %></a></li>
-  <li><a href="#hosts" data-toggle="tab"><%= _('Hosts') %></a></li>
+  <li class="<%= job_invocation_active_tab(:overview, params) %>"><a href="#primary" data-toggle="tab"><%= _('Overview') %></a></li>
+  <li class="<%= job_invocation_active_tab(:hosts, params) %>"><a href="#hosts" data-toggle="tab"><%= _('Hosts') %></a></li>
 </ul>
 
 <div class="tab-content">
-  <div class="tab-pane active" id="primary">
+  <div class="tab-pane <%= job_invocation_active_tab(:overview, params) %>" id="primary">
     <%= render 'tab_overview', :job_invocation => @job_invocation %>
   </div>
 
-  <div class="tab-pane" id="hosts">
-    <%= render 'tab_hosts', :job_invocation => @job_invocation %>
+  <div class="tab-pane <%= job_invocation_active_tab(:hosts, params) %>" id="hosts" data-refresh_required="<%= @job_invocation.resolved? ? '' : 'true' %>">
+    <%= render 'tab_hosts', :job_invocation => @job_invocation, :hosts => @hosts %>
   </div>
 </div>
 
 <script id="job_invocation_refresh" data-refresh-url="<%= job_invocation_path(@job_invocation) %>">
 <% if @auto_refresh %>
-  delayed_refresh($('script#job_invocation_refresh').data('refresh-url'));
+  delayed_refresh($('script#job_invocation_refresh').data('refresh-url'), job_invocation_refresh_data());
 <% end %>
 </script>

data/app/views/job_invocations/show.js.erb

@@ -1,6 +1,22 @@
 $('div#status_chart').html('<%=j job_invocation_chart(@job_invocation) %>');
 $('div#status').flot_pie();
 
+<% if params[:hosts_needs_refresh] == 'true' && @job_invocation.resolved? %>
+  var hosts_tab = $('div#hosts');
+  hosts_tab.html('<%=j render('tab_hosts', :job_invocation => @job_invocation, :hosts => @hosts) %>');
+  hosts_tab.data('refresh_required', '');
+<% end %>
+
+<% ['status', 'actions', 'provider'].each do |attribute| %>
+  <% if params["host_ids_needing_#{attribute}_update"].present? %>
+    var td_element;
+    <% Host.authorized(:view_hosts, Host).where(:id => params["host_ids_needing_#{attribute}_update"]).each do |host| %>
+      td_element= $('#<%= dom_id(host) %>-<%= attribute %>');
+      td_element.replaceWith('<%=j render("host_#{attribute}_td", :host => host, :task => @job_invocation.sub_task_for_host(host)) %>');
+    <% end %>
+  <% end %>
+<% end %>
+
 <% if @auto_refresh %>
-  delayed_refresh($('script#job_invocation_refresh').data('refresh-url'));
+  delayed_refresh($('script#job_invocation_refresh').data('refresh-url'), job_invocation_refresh_data());
 <% end %>

data/doc/public/404.html

@@ -0,0 +1,101 @@
+<!doctype html>
+<html itemscope itemtype="http://schema.org/Organization" lang="en">
+  <head>
+    <base href="/foreman_remote_execution/">
+    
+    
+    
+
+    
+    
+    
+  <title> :: Foreman Remote Execution</title>
+
+  <script type="text/javascript" src="/foreman_remote_execution/static/js/jquery.js"></script>
+  <script type="text/javascript" src="/foreman_remote_execution/static/js/bootstrap.min.js"></script>
+  <script type="text/javascript" src="/foreman_remote_execution/static/js/jquery-ui-1.9.2.custom.min.js"></script>
+  <script type="text/javascript" src="/foreman_remote_execution/static/js/jquery.tocify.min.js"></script>
+  <script type="text/javascript" src="/foreman_remote_execution/static/js/scroll.js"></script>
+
+  <link rel="stylesheet" media="all" href="/foreman_remote_execution/static/css/bootstrap.min.css"/>
+  <link rel="stylesheet" media="all" href="/foreman_remote_execution/static/css/bootstrap-responsive.min.css"/>
+  <link rel="stylesheet" media="all" href="/foreman_remote_execution/static/css/jquery.tocify.css"/>
+  <link rel="stylesheet" media="all" href="/foreman_remote_execution/static/css/syntax.css" />
+  <link rel="stylesheet" media="all" href="/foreman_remote_execution/static/css/style.css"/>
+  <meta name="apple-mobile-web-app-capable" content="yes">
+  <meta name="viewport" content="user-scalable=no, width=device-width, initial-scale=1.0, maximum-scale=1.0;" />
+  <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
+</head>
+
+<body>
+  <div class="navbar navbar-fixed-top">
+      <div class="navbar-inner">
+          <div class="container">
+              <button type="button" class="btn btn-navbar" data-toggle="collapse" data-target=".nav-collapse">
+                  <span class="icon-bar"></span>
+                  <span class="icon-bar"></span>
+                  <span class="icon-bar"></span>
+              </button>
+              <img alt="Foreman" class="logo" src="/foreman_remote_execution/static/images/foreman.png" />
+              <div class="brand">
+                <a  href="/foreman_remote_execution/">Foreman</a>
+                <a class="subtitle" href="/foreman_remote_execution/">Remote Execution</a>
+              </div>
+              <div class="nav-collapse collapse">
+                <ul class="nav">
+                  <li><a href="/foreman_remote_execution/">About</a></li>
+                  <li><a href="/foreman_remote_execution/design/">Design</a></li>
+                </ul>
+              </div>
+          </div>
+      </div>
+  </div>
+<script>
+$(function(){
+  $('.nav > li a[href="'+window.location.pathname+'"]').parents('.nav > li').addClass('active')
+})
+</script>
+
+
+  <div id="main">
+    <div id="content" class="container">
+      <div class="row">
+        <div id="toc" class="span3">
+        </div>
+        <div id="doc" class="offset3 span9">
+          <p>We are sorry, page You are looking for does not exist :&#39;(</p>
+
+        </div>
+      </div>
+    </div>
+  </div>
+
+<script type="text/javascript">
+  $(function() {
+    //Calls the tocify method on your HTML div.
+    $("#toc").tocify({context: '#doc', scrollTo: '60', selectors: "h1,h2,h3"});
+  });
+</script>
+
+<div class="footer">
+  <div class="container">
+    <p>This web site is licensed under a <a rel="license" href="http://creativecommons.org/licenses/by-sa/3.0/deed.en_GB">Creative Commons Attribution-ShareAlike 3.0 Unported License</a>.  Source available: <a xmlns:dct="http://purl.org/dc/terms/" href="https://github.com/theforeman/theforeman.org" rel="dct:source">github/theforeman/theforeman.org</a>.</p>
+		<a href="https://plus.google.com/102496134326414788199" rel="publisher">Google+ community</a>
+  </div>
+</div>
+
+<script type="text/javascript">
+  var _gaq = _gaq || [];
+  _gaq.push(['_setAccount', 'UA-2134730-5']);
+  _gaq.push(['_trackPageview']);
+
+  (function() {
+      var ga = document.createElement('script'); ga.type = 'text/javascript'; ga.async = true;
+      ga.src = ('https:' == document.location.protocol ? 'https://ssl' : 'http://www') + '.google-analytics.com/ga.js';
+      var s = document.getElementsByTagName('script')[0]; s.parentNode.insertBefore(ga, s);
+  })();
+</script>
+
+</body>
+</html>
+