Respect running_job_log_records_to_fetch in JobsController#logs.

[arvados.git] / apps / workbench / app / controllers / application_controller.rb

diff --git a/apps/workbench/app/controllers/application_controller.rb b/apps/workbench/app/controllers/application_controller.rb
index 7ab2403f3cbb38c82a9d6bbc8fa76b7279abad5b..648cae85a67006dc46e2410f3b7e3afc99637f8e 100644 (file)
--- a/apps/workbench/app/controllers/application_controller.rb
+++ b/apps/workbench/app/controllers/application_controller.rb
@@ -115,7 +115,7 @@ class ApplicationController < ActionController::Base
   # Column names should always be qualified by a table name and a direction is optional, defaulting to asc
   # (e.g. "collections.name" or "collections.name desc").
   # If a column name is specified, that table will be sorted by that column.
-  # If there are objects from different models that will be shown (such as in Jobs and Pipelines tab),
+  # If there are objects from different models that will be shown (such as in Pipelines and processes tab),
   # then a sort column name can optionally be specified for each model, passed as an comma-separated list (e.g. "jobs.script, pipeline_instances.name")
   # Currently only one sort column name and direction can be specified for each model.
   def load_filters_and_paging_params
@@ -828,57 +828,20 @@ class ApplicationController < ActionController::Base
     pi
   end
 
-  helper_method :running_processes
-  def running_processes lim
-    lim = 8 if lim.nil?
+  helper_method :recent_processes
+  def recent_processes lim
+    lim = 12 if lim.nil?
 
-    pipelines = PipelineInstance.limit(lim).order(["started_at desc", "created_at desc"]).filter([["state", "in", ["RunningOnServer", "RunningOnClient"]]])
-
-    crs = ContainerRequest.order(["modified_at desc"]).filter([["requesting_container_uuid", "=", nil], ["state", "=", "Committed"]])
-    cr_uuids = crs.results.collect { |c| c.container_uuid }
-    containers = Container.limit(lim).order(["started_at desc", "created_at desc"]).filter([["uuid", "in", cr_uuids], ["state", "=", "Running"]]).results if cr_uuids.any?
-
-    procs = {}
-    pipelines.results.each { |pi| procs[pi] = (pi.started_at || pi.created_at)}
-    containers.each { |c| procs[c] = c.created_at } if !containers.nil?
-
-    Hash[procs.sort_by {|key, value| value}].keys.reverse.first(lim)
-  end
-
-  helper_method :finished_pipelines
-  def finished_pipelines lim
-    PipelineInstance.limit(lim).order(["finished_at desc"]).filter([["state", "in", ["Complete", "Failed", "Paused"]], ["finished_at", "!=", nil]])
-  end
-
-  helper_method :finished_processes
-  def finished_processes lim
-    lim = 8 if lim.nil?
-
-    pipelines = PipelineInstance.limit(lim).order(["finished_at desc"]).filter([["state", "in", ["Complete", "Failed", "Paused"]], ["finished_at", "!=", nil]])
-
-    crs = ContainerRequest.order(["modified_at desc"]).filter([["requesting_container_uuid", "=", nil], ["state", "=", "Final"]])
-    cr_uuids = crs.results.collect { |c| c.container_uuid }
-    containers = Container.limit(lim).order(["finished_at desc"]).filter([["uuid", "in", cr_uuids], ["state", "in", ["Complete", "Canceled"]], ["finished_at", "!=", nil]]).results if cr_uuids.any?
+    pipelines = PipelineInstance.limit(lim).order(["created_at desc"])
 
+    crs = ContainerRequest.limit(lim).order(["created_at desc"]).filter([["requesting_container_uuid", "=", nil]])
     procs = {}
-    pipelines.results.each { |pi| procs[pi] = pi.finished_at }
-    containers.each { |pi| procs[pi] = pi.finished_at } if !containers.nil?
+    pipelines.results.each { |pi| procs[pi] = pi.created_at }
+    crs.results.each { |c| procs[c] = c.created_at }
 
     Hash[procs.sort_by {|key, value| value}].keys.reverse.first(lim)
   end
 
-  helper_method :queued_processes
-  def queued_processes
-    procs = {}
-    queued_jobs = Job.queue
-    queued_jobs.each { |j| procs[j] = j.priority }
-
-    queued_containers = Container.order(["priority desc", "created_at desc"]).filter([["state", "in", ["Queued", "Locked"]]])
-    queued_containers.results.each { |c| procs[c] = c.priority }
-
-    Hash[procs.sort_by {|key, value| value}].keys.reverse
-  end
-
   helper_method :recent_collections
   def recent_collections lim
     c = Collection.limit(lim).order(["modified_at desc"]).filter([["owner_uuid", "is_a", "arvados#group"]])