…ker should be used with to run condor_vault_storer

… introduce worker.Worker type

…ng func instances

…sForWorkers interface

… only accept InteractiveTokenStorer or NonInteractiveTokenStorer

…nboarding flag

… meant to be used externally

well as new StoreAndGetTokensInteractiveWorker

In discussion with stlammel about this code base, he brought up that it
wasn't easy to find where exactly these workers loop over the schedds to
store tokens.  To make that much clearer, I moved the loops out of
StoreAndGetTokenForSchedds, renamed that function to
StoreAndGetTokenForSchedd, and made it only handle the operations for a
single token store.  The loop then moved into the two worker funcs
mentioned above.  This required some rewriting of the error handling as
well, since in both cases, we need to check for and report the correct
errors across different operations.

…anonymous func for better structure

…rt fileCopier interface

… PingNodeStatus to worker package, as package ping never uses this type. Finally, changed Node.PingNode to Node.Ping

…utKey

…StoreAndGetTokenInteractiveWorkerType] map uninitialized

…ush package main

…hat back in and also added an info message for run-onboarding mode

…at means they are duplicated, but it removes a dependency

…scheddCache.go

…nless there were valid service configs

…rate collector goroutine, each config setup goroutine simply accesses the serviceConfigs and successfulServices maps directly. Access is mediated through mutexes for both

…tifications chan was getting blocked due to absence of listener. I introduced this bug in commit 861bc7a

…erNotificationsChans to a new func registerDummyServiceNotificationsChan that gets invoked in the service Config setup stage

… drain the notifications channels

…s --dont-notify

…essing - unused