From 367a23e4d9a3dae30c65abf381b55c0c82b78c89 Mon Sep 17 00:00:00 2001 From: Patrick Ohly Date: Thu, 2 Apr 2020 15:04:00 +0200 Subject: [PATCH] mock tests: remove redundant retrieval of log output The code became obsolete with the introduction of parseMockLogs because that will retrieve the log itself. For debugging of a running test the normal pod output logging is sufficient. --- test/e2e/storage/csi_mock_volume.go | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/test/e2e/storage/csi_mock_volume.go b/test/e2e/storage/csi_mock_volume.go index f9e9498cd5d..346438790d2 100644 --- a/test/e2e/storage/csi_mock_volume.go +++ b/test/e2e/storage/csi_mock_volume.go @@ -885,13 +885,7 @@ func checkPodLogs(cs clientset.Interface, namespace, driverPodName, driverContai expectedAttributes["csi.storage.k8s.io/ephemeral"] = strconv.FormatBool(ephemeralVolume) } - // Load logs of driver pod - log, err := e2epod.GetPodLogs(cs, namespace, driverPodName, driverContainerName) - if err != nil { - return fmt.Errorf("could not load CSI driver logs: %s", err) - } - framework.Logf("CSI driver logs:\n%s", log) - // Find NodePublish in the logs + // Find NodePublish in the GRPC calls. foundAttributes := sets.NewString() numNodePublishVolume := 0 numNodeUnpublishVolume := 0