Possibly redundant Sleep

[Kek5chen]

vicky/vicky-worker/src/main.rs

Line 163 in 8c5e046

tokio::time::sleep(std::time::Duration::from_secs(1)).await;

Why is there a sleep here? Can this be removed? The task is done at that point and logs should have sunk into the remote logger so finish can be called safely right away, or am I missing something?

[Kek5chen]

We're all a bit unsure but it seems to be some ancient logging thing from etcd that was used to protect against some desynchronized state.

Might remove and test.