Include duration in logs when evicting pods

[mikkeloscar]

Would be nice if the CLM logged for how long it had tried to evict each pod, to make it more clear which pods are not evictable for a long time.

[AlexanderYastrebov]

I think it eventually logs how long it took to complete

cluster-lifecycle-manager/pkg/updatestrategy/drain.go

Line 387 in dc1763e

localLogger.Infof("Pod evicted. (Observed termination period: %s)", time.Now().UTC().Sub(start))

which takes up to

cluster-lifecycle-manager/pkg/updatestrategy/drain.go

Line 420 in dc1763e

gracePeriod := time.Duration(*pod.Spec.TerminationGracePeriodSeconds)*time.Second + podEvictionHeadroom

I am not sure it makes sense to log the retry progress as that may unnecessary pollute logs.

[aermakov-zalando]

This is about force eviction (kubernetes delete -> grace period -> pod deletion), I think Mikkel meant the total time.