[ARVADOS] updated: 92211c8bed5aaf3abd83d02c4adbaf2329123bac

[git at public.curoverse.com]

Summary of changes:
 apps/workbench/app/helpers/pipeline_instances_helper.rb             | 2 +-
 apps/workbench/app/views/application/_job_status_label.html.erb     | 2 +-
 .../app/views/pipeline_instances/_running_component.html.erb        | 2 +-
 .../app/views/pipeline_instances/_show_components_running.html.erb  | 6 +++---
 apps/workbench/app/views/projects/_show_dashboard.html.erb          | 2 +-
 5 files changed, 7 insertions(+), 7 deletions(-)

       via  92211c8bed5aaf3abd83d02c4adbaf2329123bac (commit)
      from  1d5d09e8d1568fdae039571306bd5e56a8273ac7 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 92211c8bed5aaf3abd83d02c4adbaf2329123bac
Author: Peter Amstutz <peter.amstutz at curoverse.com>
Date:   Tue Sep 23 14:48:17 2014 -0400

    Bugfix time display, also bugfix for job state "Completed" -> "Complete".  refs #3187

diff --git a/apps/workbench/app/helpers/pipeline_instances_helper.rb b/apps/workbench/app/helpers/pipeline_instances_helper.rb
index 599da42..7e5324b 100644
--- a/apps/workbench/app/helpers/pipeline_instances_helper.rb
+++ b/apps/workbench/app/helpers/pipeline_instances_helper.rb
@@ -256,7 +256,7 @@ module PipelineInstancesHelper
       if minutes > 0 then
         s << "#{minutes} minute#{'s' if minutes != 1}"
       end
-      if seconds > 0 and not round_to_min 
+      if not round_to_min or s.size == 0
         s << "#{seconds} second#{'s' if seconds != 1}"
       end
       s = s * " "
diff --git a/apps/workbench/app/views/application/_job_status_label.html.erb b/apps/workbench/app/views/application/_job_status_label.html.erb
index 86c3415..ece8167 100644
--- a/apps/workbench/app/views/application/_job_status_label.html.erb
+++ b/apps/workbench/app/views/application/_job_status_label.html.erb
@@ -1,7 +1,7 @@
 <% status = Job.state j %>
 <% to_label = {
      "Cancelled" => "danger",
-     "Completed" => "success",
+     "Complete" => "success",
      "Running" => "info",
      "Failed" => "danger",
      "Queued" => "default",
diff --git a/apps/workbench/app/views/pipeline_instances/_running_component.html.erb b/apps/workbench/app/views/pipeline_instances/_running_component.html.erb
index 8f89916..1d52e28 100644
--- a/apps/workbench/app/views/pipeline_instances/_running_component.html.erb
+++ b/apps/workbench/app/views/pipeline_instances/_running_component.html.erb
@@ -31,7 +31,7 @@
               <% end %>
             </div>
 
-            <% if Job::state(current_job).in? ["Completed", "Failed", "Canceled"] %>
+            <% if Job::state(current_job).in? ["Complete", "Failed", "Canceled"] %>
               <div class="col-md-5 text-overflow-ellipsis">
                 <% if pj[:output_uuid] %>
                   <%= link_to_if_arvados_object pj[:output_uuid], friendly_name: true %>
diff --git a/apps/workbench/app/views/pipeline_instances/_show_components_running.html.erb b/apps/workbench/app/views/pipeline_instances/_show_components_running.html.erb
index 8af8c88..acaf9d3 100644
--- a/apps/workbench/app/views/pipeline_instances/_show_components_running.html.erb
+++ b/apps/workbench/app/views/pipeline_instances/_show_components_running.html.erb
@@ -29,9 +29,9 @@
                   end %>
     
     <%= if walltime > runningtime 
-          render_runtime(walltime, true) 
+          render_runtime(walltime, true, false) 
         else 
-          render_runtime(runningtime, true) 
+          render_runtime(runningtime, true, false) 
         end %><% if @object.finished_at %> at <%= render_localized_date(@object.finished_at) %><% end %>.
     <% else %>
       This pipeline is <%= if @object.state.start_with? 'Running' then 'active' else @object.state.downcase end %>.
@@ -61,7 +61,7 @@
     <%= render_runtime(runningtime, true, false) %><% if (walltime - runningtime) > 0 %>
       (<%= render_runtime(walltime - runningtime, true, false) %> queued)<% end %><% if cputime == 0 %>.<% else %>
       and used
-    <%= render_runtime(cputime, true) %>
+    <%= render_runtime(cputime, true, false) %>
     of CPU time (<%= (cputime/runningtime).round(1) %>⨯ scaling).
     <% end %>
 </p>
diff --git a/apps/workbench/app/views/projects/_show_dashboard.html.erb b/apps/workbench/app/views/projects/_show_dashboard.html.erb
index 4731594..2841637 100644
--- a/apps/workbench/app/views/projects/_show_dashboard.html.erb
+++ b/apps/workbench/app/views/projects/_show_dashboard.html.erb
@@ -38,7 +38,7 @@
                           <% running << k %>
                         <% elsif Job::state(v[:job]) == "Failed" or Job::state(v[:job]) == "Cancelled" %>
                           <% failed << k %>
-                        <% elsif Job::state(v[:job]) == "Completed" %>
+                        <% elsif Job::state(v[:job]) == "Complete" %>
                           <% completed << k %>
                         <% elsif Job::state(v[:job]) == "Queued" %>
                           <% queued << k %>

-----------------------------------------------------------------------


hooks/post-receive
--