From ed1f30e799d1341b807394e9b255ce16b2467ccd Mon Sep 17 00:00:00 2001 From: Preetha Appan <460133+preetapan@users.noreply.github.com> Date: Thu, 12 Dec 2019 12:05:29 -0600 Subject: [PATCH] Use debug logging for scheduler internals We currently log an error if preemption is unable to find a suitable set of allocations to preempt. This commit changes that to debug level since not finding preemptable allocations is not an error condition. --- scheduler/rank.go | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/scheduler/rank.go b/scheduler/rank.go index d90d74bd4..4c7cb132e 100644 --- a/scheduler/rank.go +++ b/scheduler/rank.go @@ -243,7 +243,7 @@ OUTER: netPreemptions := preemptor.PreemptForNetwork(ask, netIdx) if netPreemptions == nil { - iter.ctx.Logger().Named("binpack").Error("preemption not possible ", "network_resource", ask) + iter.ctx.Logger().Named("binpack").Debug("preemption not possible ", "network_resource", ask) netIdx.Release() continue OUTER } @@ -307,7 +307,7 @@ OUTER: netPreemptions := preemptor.PreemptForNetwork(ask, netIdx) if netPreemptions == nil { - iter.ctx.Logger().Named("binpack").Error("preemption not possible ", "network_resource", ask) + iter.ctx.Logger().Named("binpack").Debug("preemption not possible ", "network_resource", ask) netIdx.Release() continue OUTER } @@ -352,7 +352,7 @@ OUTER: devicePreemptions := preemptor.PreemptForDevice(req, devAllocator) if devicePreemptions == nil { - iter.ctx.Logger().Named("binpack").Error("preemption not possible", "requested_device", req) + iter.ctx.Logger().Named("binpack").Debug("preemption not possible", "requested_device", req) netIdx.Release() continue OUTER }