From fcab3a442dae8bcd860196ce93c9bada7e200431 Mon Sep 17 00:00:00 2001 From: Clayton Coleman Date: Mon, 13 Mar 2017 00:01:45 -0400 Subject: [PATCH] Log how much time it takes e2e tests to clean up the namespace Will get a better handle on deletion test wasted time --- test/e2e/framework/util.go | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/test/e2e/framework/util.go b/test/e2e/framework/util.go index e4d373e405f..3d37583c5c3 100644 --- a/test/e2e/framework/util.go +++ b/test/e2e/framework/util.go @@ -954,12 +954,13 @@ func CheckTestingNSDeletedExcept(c clientset.Interface, skip string) error { // deleteNS deletes the provided namespace, waits for it to be completely deleted, and then checks // whether there are any pods remaining in a non-terminating state. func deleteNS(c clientset.Interface, clientPool dynamic.ClientPool, namespace string, timeout time.Duration) error { + startTime := time.Now() if err := c.Core().Namespaces().Delete(namespace, nil); err != nil { return err } // wait for namespace to delete or timeout. - err := wait.PollImmediate(5*time.Second, timeout, func() (bool, error) { + err := wait.PollImmediate(2*time.Second, timeout, func() (bool, error) { if _, err := c.Core().Namespaces().Get(namespace, metav1.GetOptions{}); err != nil { if apierrs.IsNotFound(err) { return true, nil @@ -1007,6 +1008,7 @@ func deleteNS(c clientset.Interface, clientPool dynamic.ClientPool, namespace st // no remaining content, but namespace was not deleted (namespace controller is probably wedged) return fmt.Errorf("namespace %v was not deleted with limit: %v, namespace is empty but is not yet removed", namespace, err) } + Logf("namespace %v deletion completed in %s", namespace, time.Now().Sub(startTime)) return nil }