From 62805dd73c032dd9cc5276abbed72742d699acbb Mon Sep 17 00:00:00 2001
From: Eelco Dolstra <eelco.dolstra@logicblox.com>
Date: Thu, 26 Feb 2015 13:14:10 +0100
Subject: [PATCH] Queue: Remove the scheduling priority

Scheduling is mostly based on jobset shares these days. So showing and
sorting by priority just wastes space and gives the incorrect
impression that Hydra executes builds in the order shown on the queue
page.
---
 src/lib/Hydra/Controller/Root.pm | 2 +-
 src/root/common.tt               | 6 ------
 2 files changed, 1 insertion(+), 7 deletions(-)

diff --git a/src/lib/Hydra/Controller/Root.pm b/src/lib/Hydra/Controller/Root.pm
index 696d8e25..c4765065 100644
--- a/src/lib/Hydra/Controller/Root.pm
+++ b/src/lib/Hydra/Controller/Root.pm
@@ -88,7 +88,7 @@ sub queue_GET {
     $c->stash->{flashMsg} //= $c->flash->{buildMsg};
     $self->status_ok(
         $c,
-        entity => [$c->model('DB::Builds')->search({finished => 0}, { order_by => ["priority DESC", "id"]})]
+        entity => [$c->model('DB::Builds')->search({finished => 0}, { order_by => ["id"]})]
     );
 }
 
diff --git a/src/root/common.tt b/src/root/common.tt
index 30e7a075..b27ec992 100644
--- a/src/root/common.tt
+++ b/src/root/common.tt
@@ -78,9 +78,6 @@ BLOCK renderBuildListHeader %]
           <th></th>
         [% END %]
         <th>#</th>
-        [% IF showSchedulingInfo %]
-          <th>P</th>
-        [% END %]
         [% IF !hideJobName %]
         <th>Job</th>
         [% END %]
@@ -108,9 +105,6 @@ BLOCK renderBuildListBody;
         <td>[% IF build.busy %]<span class="label label-success">Started</span>[% ELSE %]<span class="label">Queued</span>[% END %]</td>
       [% END %]
       <td><a class="row-link" href="[% c.uri_for('/build' build.id) %]">[% build.id %]</a></td>
-      [% IF showSchedulingInfo %]
-        <td>[% build.priority %]</td>
-      [% END %]
       [% IF !hideJobName %]
         <td>[% INCLUDE renderFullJobNameOfBuild %]</td>
       [% END %]