3187: Text tweaks.
authorPeter Amstutz <peter.amstutz@curoverse.com>
Fri, 12 Sep 2014 12:52:03 +0000 (08:52 -0400)
committerPeter Amstutz <peter.amstutz@curoverse.com>
Fri, 12 Sep 2014 12:52:03 +0000 (08:52 -0400)
apps/workbench/app/views/pipeline_instances/_show_components.html.erb
apps/workbench/app/views/pipeline_instances/_show_components_running.html.erb

index b7a5711e7250c846f8e6f6541c46b1514d0d4820..94cbf984897c66753ff6728a7d2f2196d9f540dd 100644 (file)
@@ -1,6 +1,6 @@
 <% if !@object.state.in? ['New', 'Ready'] %>
 
-  <div class="pull-right" style="padding-left: 2em">
+  <div class="pull-right" style="padding-left: 1em">
     Current state: <span class="badge badge-info" data-pipeline-state="<%= @object.state %>">
       <% if @object.state == "RunningOnServer" %>
         Active
index f5eb17b63584acc244e69695e91bab7c9810c51c..0d27566e538d7d0c15cea447ed7901fee8e44c82 100644 (file)
@@ -8,19 +8,18 @@
 <% end %>
 
 <p>
-  This pipeline
   <% if @object.started_at %>
-    started at <span data-utc-date="<%= @object.started_at %>"><%= @object.started_at %></span> and
+    Started at <span data-utc-date="<%= @object.started_at %>"><%= @object.started_at %></span> and
   <% else %>
-    has not started.
+    Has not started.
   <% end %>
 
   <% if @object.state == 'Complete' %>
     completed in
   <% elsif @object.state == 'Failed' %>
     failed after
-  <% else %>
-    and has been active for
+  <% elsif @object.started_at %>
+    has been active for
   <% end %>
 
   <% walltime = if @object.started_at
       %>
 
   <%= runtime(walltime, true) %><% if @object.finished_at %> at <span data-utc-date="<%= @object.finished_at %>"><%= @object.finished_at %></span><% end %>.
+
+  <% if @object.state == 'Failed' %>
+    Check the Log tab for more detail about why this pipeline failed.
+  <% end %>
 </p>
 
 <p>
-  This pipeline
     <% tasks = JobTask.filter([['job_uuid', 'in', render_pipeline_jobs.map { |j| j[:job].andand[:uuid] }]]).results %>
     <% runningtime = determine_wallclock_runtime(render_pipeline_jobs.map {|j| j[:job]}) %>
 
-    <% if @object.state.start_with? == 'Running' %>
-      has run
+    <% if @object.state.start_with? 'Running' %>
+      Has run
       <% else %>
-      ran
+      Ran
     <% end %>
     for
-    <%= runtime(runningtime, true) %><% if tasks.size == 0 %>.<% else %> (<%= runtime(walltime - runningtime, true) %> queued),
+    <%= runtime(runningtime, true) %> (<%= runtime(walltime - runningtime, true) %> queued)<% if tasks.size == 0 %>.<% else %>
       and used
       <% cputime = tasks.map { |task|
            puts "started at #{task.started_at}"
         <div class="row">
           <div class="col-md-6">
             <table>
-              <% [:script, :repository, :supplied_script_version, :script_version, :nondeterministic].each do |k| %>
+              <% [:script, :repository, :script_version, :supplied_script_version, :nondeterministic].each do |k| %>
                 <tr>
                   <td style="padding-right: 1em">
                     <%= k.to_s %>: