reconciler: support disconnected clients#12058

Merged

DerekStrickland merged 25 commits intof-disconnected-client-allocation-handlingfrom f-reconciler-disconnected-clients

Feb 16, 2022

+943-56

Commits on Feb 14, 2022

Add merge helper for string maps
DerekStrickland
committed
structs: add statuses, MaxClientDisconnect, and helper funcs
DerekStrickland
committed
taintedNodes: Include disconnected nodes
DerekStrickland
committed
upsertAllocsImpl: don't use existing ClientStatus when upserting unknown
DerekStrickland
committed
allocSet: update filterByTainted and add delayByMaxClientDisconnect
DerekStrickland
committed
allocReconciler: support disconnecting and reconnecting allocs
DerekStrickland
committed
GenericScheduler: upsert unknown and queue reconnecting
DerekStrickland
committed

Commits on Feb 15, 2022

Apply suggestions from code review

DerekStrickland
and
tgross
authored
AllocMetric.MaxNormScore: use first score from pre-sorted heap
DerekStrickland
committed
scheduler tests: remove table tests and commented code
DerekStrickland
committed
reconciler tests: remove TODO comments
DerekStrickland
committed
reconcile_util: fix stale comment
DerekStrickland
committed
filterByTainted: add negative tests to ensure terminal allocs don't come back
DerekStrickland
committed
filterAndStopAll: add docstring
DerekStrickland
committed
computeReconnecting: remove log entries, unneeded conditionals, and queue language from comments
DerekStrickland
committed
computeStopByReconnecting: improve docstring
DerekStrickland
committed
computeStopByReconnecting: add guard, combine conditionals
DerekStrickland
committed
createTimeoutLaterEvals: update docstring to describe returned value.
DerekStrickland
committed
createTimeoutLaterEvals: Fix if/else structural error. Change log level to Error.
DerekStrickland
committed
AppendUnknownAlloc: remove comments after code review
DerekStrickland
committed

Commits on Feb 16, 2022

Apply suggestions from code review

DerekStrickland
and
tgross
authored
remove queue language from comment
DerekStrickland
committed
DisconnectTimeout: add guard clause for nil receiver or job
DerekStrickland
committed
filterByTainted: add pending allocs on disconnected nodes to lost
DerekStrickland
committed
scheduler tests: remove unneccessary WaitForResult and WatchSet
DerekStrickland
committed