From 170c7c98d081d4e22949ef980e40ccec4696734b Mon Sep 17 00:00:00 2001
From: Shea Levy <shea@shealevy.com>
Date: Mon, 4 Mar 2013 16:03:04 -0500
Subject: [PATCH] machines: Read /etc/nix.machines instead of using the
 BuildMachines table

---
 src/lib/Hydra/Controller/Admin.pm |  6 +-----
 src/root/machines.tt              | 25 ++++++++-----------------
 2 files changed, 9 insertions(+), 22 deletions(-)

diff --git a/src/lib/Hydra/Controller/Admin.pm b/src/lib/Hydra/Controller/Admin.pm
index 61835b93..3b935956 100644
--- a/src/lib/Hydra/Controller/Admin.pm
+++ b/src/lib/Hydra/Controller/Admin.pm
@@ -56,11 +56,7 @@ sub users : Chained('admin') PathPart('users') Args(0) {
 
 sub machines : Chained('admin') PathPart('machines') Args(0) {
     my ($self, $c) = @_;
-    $c->stash->{machines} = [$c->model('DB::BuildMachines')->search({}, {order_by => "hostname"})];
-    $c->stash->{systems} = [$c->model('DB::SystemTypes')->search({}, {select => ["system"], order_by => "system" })];
-    $c->stash->{nixMachines} = nixMachines($c);
-    $c->stash->{nixMachinesWritable} = (-e "/etc/nix.machines" && -w "/etc/nix.machines");
-
+    $c->stash->{machines} = getMachines;
     $c->stash->{template} = 'machines.tt';
 }
 
diff --git a/src/root/machines.tt b/src/root/machines.tt
index d498676d..eea0efe8 100644
--- a/src/root/machines.tt
+++ b/src/root/machines.tt
@@ -8,32 +8,23 @@
             <th>Machine</th>
             <th>Max concurrent</th>
             <th>Speed factor</th>
-            [% FOREACH s IN systems %]<th>[% s.system %]</th>[% END %]
+            <th>Systems</th>
         </tr>
     </thead>
     <tbody>
         [% FOREACH m IN machines %]
         <tr>
-            <td><input type="checkbox" name="enabled" [% IF m.enabled == 1  %]CHECKED[% END %] onclick="window.location='[% IF m.enabled == 1 %][%c.uri_for('/admin/machine' m.hostname 'disable' )%][% ELSE %][%c.uri_for('/admin/machine' m.hostname 'enable' )%][% END %]'"/></td>
-            <td><a href="[% c.uri_for(c.controller('Admin').action_for('machine_edit'), [m.hostname]) %]">[% m.hostname %]</a></td>
-            <td>[% m.maxconcurrent %]</td>
-            <td>[% m.speedfactor %]</td>
-            [% FOREACH s IN systems %]
-                <td>
-                [% checked = false %]
-                [% FOREACH ms IN m.buildmachinesystemtypes %]
-                    [% checked = ms.system == s.system  %]
-                    [% BREAK IF checked %]
-                [% END %]
-                <input type="checkbox" name="system" DISABLED [% IF checked %]CHECKED[% END %]/>
-                </td>
-            [% END %]
+            <td><input type="checkbox" name="enabled" [% IF m.value.maxJobs > 0  %]CHECKED[% END %] disabled="true" /></td>
+            <td>[% m.key %]</a></td>
+            <td>[% m.value.maxJobs %]</td>
+            <td>[% m.value.speedFactor %]</td>
+            <td>
+                [% comma=0; FOREACH system IN m.value.systemTypes %][% IF comma; %], [% ELSE; comma = 1; END; system; END %]
+            </td>
         </tr>
         [% END %]
     </tbody>
 
 </table>
 
-<p><a class="btn" href="[% c.uri_for(c.controller('Admin').action_for('create_machine')) %]"><i class="icon-plus"></i> Add a new machine</a></p>
-
 [% END %]