[remoteaccess] Track unwakable machines, ignore restricted locations

- Locations that are set to "never" or "only outside business hours"
  will not be considered during WOL, and not delivered to the guacamole
  proxy.
- If we cannot wake as much machines per remote group as configured,
  we'll display a little number next to the WOL count setting.

1 files changed, 41 insertions, 20 deletions

diff --git a/modules-available/remoteaccess/inc/remoteaccess.inc.php b/modules-available/remoteaccess/inc/remoteaccess.inc.php
index 1910c595..ce6bf0db 100644
--- a/modules-available/remoteaccess/inc/remoteaccess.inc.php
+++ b/modules-available/remoteaccess/inc/remoteaccess.inc.php
@@ -11,14 +11,20 @@ class RemoteAccess
 
 	const PROP_PLUGIN_VERSION = 'remoteaccess.plugin-version';
 
-	public static function getEnabledLocations($group = 0)
+	public static function getEnabledLocations(int $group = 0)
 	{
 		if ($group === 0) {
-			return Database::queryColumnArray("SELECT DISTINCT rxl.locationid FROM remoteaccess_x_location rxl
+			$list = Database::queryColumnArray("SELECT DISTINCT rxl.locationid FROM remoteaccess_x_location rxl
 				INNER JOIN remoteaccess_group g ON (g.groupid = rxl.groupid AND g.active = 1)");
-		}
-		return Database::queryColumnArray("SELECT DISTINCT locationid FROM remoteaccess_x_location
+		} else {
+			$list = Database::queryColumnArray("SELECT DISTINCT locationid FROM remoteaccess_x_location
 				WHERE groupid = :gid", ['gid' => $group]);
+		}
+		return array_filter($list, function (int $lid) {
+			$mode = Scheduler::getLocationOptions($lid)['ra-mode'];
+			return ($mode !== Scheduler::RA_NEVER
+				&& ($mode !== Scheduler::RA_SELECTIVE || !OpeningTimes::isRoomOpen($lid, 5, 5)));
+		});
 	}
 
 	public static function ensureMachinesRunning()
@@ -28,7 +34,9 @@ class RemoteAccess
 			return;
 		}
 
-		$res = Database::simpleQuery("SELECT rg.groupid, rg.groupname, rg.wolcount, GROUP_CONCAT(rxl.locationid) AS locs FROM remoteaccess_group rg
+		$res = Database::simpleQuery("SELECT rg.groupid, rg.groupname, rg.wolcount,
+			GROUP_CONCAT(rxl.locationid) AS locs
+			FROM remoteaccess_group rg
 			INNER JOIN remoteaccess_x_location rxl USING (groupid)
 			WHERE rg.active = 1
 			GROUP BY groupid");
@@ -36,25 +44,40 @@ class RemoteAccess
 		// Consider machines we tried to wake in the past 90 seconds as online
 		$wolDeadline = time() - 90;
 		foreach ($res as $row) {
-			if ($row['wolcount'] <= 0)
-				continue;
+			$wantNum = $row['wolcount'];
 			// This can't really be anything but a CSV list, but better be safe
 			$locs = preg_replace('/[^0-9,]/', '', $row['locs']);
-			if (empty($locs))
-				continue;
-			$active = Database::queryFirst("SELECT Count(*) AS cnt FROM machine m
+			if (!empty($locs)) {
+				// Filter out locations for which remote-access is disabled
+				$locArray = explode(',', $locs);
+				$locArray = array_filter($locArray, function (int $lid) {
+					$mode = Scheduler::getLocationOptions($lid)['ra-mode'];
+					return ($mode !== Scheduler::RA_NEVER
+						&& ($mode !== Scheduler::RA_SELECTIVE || !OpeningTimes::isRoomOpen($lid, 5, 5)));
+				});
+				$locs = implode(',', $locArray);
+			}
+			if ($wantNum > 0 && !empty($locs)) {
+				$active = Database::queryFirst("SELECT Count(*) AS cnt FROM machine m
 					INNER JOIN remoteaccess_machine rm USING (machineuuid)
 					WHERE m.locationid IN ($locs) AND (m.state = 'IDLE' OR rm.woltime > $wolDeadline)");
-			$active = (isset($active['cnt']) ? $active['cnt'] : 0);
-			$wantNum = $row['wolcount'] - $active;
-			if ($wantNum <= 0)
-				continue;
-			self::tryWakeMachines($locs, $wantNum);
+				$active = ($active['cnt'] ?? 0);
+				$wantNum -= $active;
+			}
+			if ($wantNum > 0) {
+				$numFailed = self::tryWakeMachines($locs, $wantNum);
+			} else {
+				$numFailed = 0;
+			}
+			Database::exec("UPDATE remoteaccess_group SET unwoken = :num WHERE groupid = :groupid",
+				['num' => $numFailed, 'groupid' => $row['groupid']]);
 		}
 	}
 
-	private static function tryWakeMachines($locs, $num)
+	private static function tryWakeMachines(string $locs, int $num): int
 	{
+		if (empty($locs))
+			return $num;
 		$res = Database::simpleQuery("SELECT m.machineuuid, m.macaddr, m.clientip, m.locationid FROM machine m
 				LEFT JOIN remoteaccess_machine rm USING (machineuuid)
 				WHERE m.locationid IN ($locs) AND m.state IN ('OFFLINE', 'STANDBY')
@@ -72,7 +95,7 @@ class RemoteAccess
 			if (empty($list))
 				break; // No more clients in this location
 			RebootControl::wakeMachines($list, $fails);
-			$num -= count($list) - count($fails);
+			$num -= (count($list) - count($fails));
 			if (!empty($fails)) {
 				$failIds = ArrayUtil::flattenByKey($fails, 'machineuuid');
 				// Reduce time so they won't be marked as wol_in_progress
@@ -80,9 +103,7 @@ class RemoteAccess
 					['faketime' => $NOW - 95, 'fails' => $failIds]);
 			}
 		}
-		if ($num > 0) {
-			error_log("Could not wake $num clients in ($locs)...");
-		}
+		return $num;
 	}
 
 }