From 7c2fb6187a080e4c91564a288a0e341e9852ee29 Mon Sep 17 00:00:00 2001 From: Madhu Rajanna Date: Tue, 6 Aug 2019 12:16:56 +0530 Subject: [PATCH] remove post validation of rbd device once we map the rbd image on a node we will get the device name its mapped in the map output itself,no need to check the devicepath post rbd mapping Signed-off-by: Madhu Rajanna --- pkg/rbd/rbd_attach.go | 12 +----------- 1 file changed, 1 insertion(+), 11 deletions(-) diff --git a/pkg/rbd/rbd_attach.go b/pkg/rbd/rbd_attach.go index 4be1dcd1a..16beb27be 100644 --- a/pkg/rbd/rbd_attach.go +++ b/pkg/rbd/rbd_attach.go @@ -269,10 +269,8 @@ func createPath(volOpt *rbdVolume, cr *util.Credentials) (string, error) { klog.V(5).Infof("rbd: map mon %s", volOpt.Monitors) - useNBD := false cmdName := rbd if volOpt.Mounter == rbdTonbd && hasNBD { - useNBD = true cmdName = rbdTonbd } @@ -282,15 +280,7 @@ func createPath(volOpt *rbdVolume, cr *util.Credentials) (string, error) { klog.Warningf("rbd: map error %v, rbd output: %s", err, string(output)) return "", fmt.Errorf("rbd: map failed %v, rbd output: %s", err, string(output)) } - devicePath, found := waitForPath(volOpt.Pool, image, 10, useNBD) - if !found { - output, err := execCommand(cmdName, []string{ - "unmap", imagePath, "--id", cr.ID, "-m", volOpt.Monitors, "--keyfile=" + cr.KeyFile}) - if err != nil { - klog.Warningf("rbd: unmap error %v, rbd output: %s", err, string(output)) - } - return "", fmt.Errorf("could not map image %s, Timeout after 10s", imagePath) - } + devicePath := strings.TrimSuffix(string(output), "\n") return devicePath, nil }