Clear static clients on scheduler reset (#332)

* scheduler: clear clients upon scheduler reset

* scheduler: also clear-up this host used slots

* dist-tests: actually use separate planner

* planner: move logging to trace

src/planner/Planner.cpp

@@ -151,7 +151,7 @@ bool Planner::registerHost(const Host& hostIn, bool overwrite)
         it->second->set_slots(hostIn.slots());
         it->second->set_usedslots(hostIn.usedslots());
     } else if (it != state.hostMap.end()) {
-        SPDLOG_DEBUG("NOT overwritting host {} with {} slots (used {})",
+        SPDLOG_TRACE("NOT overwritting host {} with {} slots (used {})",
                      hostIn.ip(),
                      hostIn.slots(),
                      hostIn.usedslots());

src/scheduler/Scheduler.cpp

@@ -193,6 +193,11 @@ void Scheduler::reset()
     // Clear the point to point broker
     broker.clear();
 
+    // Clear the clients
+    functionCallClients.clear();
+    snapshotClients.clear();
+    plannerClient.clear();
+
     faabric::util::FullLock lock(mx);
 
     // Ensure host is set correctly
@@ -202,6 +207,7 @@ void Scheduler::reset()
     thisHostResources = faabric::HostResources();
     thisHostResources.set_slots(faabric::util::getUsableCores());
     thisHostResources.set_usedslots(0);
+    thisHostUsedSlots.store(0, std::memory_order_release);
 
     // Reset scheduler state
     availableHostsCache.clear();

tests/utils/fixtures.h

@@ -114,7 +114,7 @@ class PlannerClientServerFixture
 {
   public:
     PlannerClientServerFixture()
-      : plannerCli(LOCALHOST)
+      : plannerCli(faabric::util::getSystemConfig().plannerHost)
     {
         plannerServer.start();
         plannerCli.ping();