fix(shutdown): implement 5 seconds shutdown period

will already help prevent false positives from other kubenurse pods when
trying to reach me_ingress through the ingress controller during
teardown. without this 5sec wait, in-flight requests from e.g. the
ingress controller will reach a pod that is already terminated.
Might not be sufficient for similar for "path" errors, as there is no
filter for terminating pods.

Signed-off-by: Clément Nussbaumer <[email protected]>

internal/kubenurse/server.go

@@ -198,6 +198,12 @@ func (s *Server) Shutdown(ctx context.Context) error {
 	s.ready = false
 	s.mu.Unlock()
 
+	// wait 5 second before actually shutting down the http/s server, as the updated
+	// endpoints for the kubenurse service might not have propagated everywhere
+	// (other kubenurse/ingress controller) yet, which will lead to
+	// me_ingress or path errors in other pods
+	time.Sleep(5 * time.Second)
+
 	// stop the scheduled checker
 	s.checker.StopScheduled()