perf: Only call getVolumeWWN if we didn't get it in response of Creat…

…eVolume call
Seagate · Oct 11, 2023 · bb2a524 · bb2a524
1 parent 5e00c39
commit bb2a524
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/pkg/controller/provisioner.go b/pkg/controller/provisioner.go
@@ -118,14 +118,18 @@ func (controller *Controller) CreateVolume(ctx context.Context, req *csi.CreateV
 			}
 
 		} else {
-			_, _, err2 := controller.client.CreateVolume(volumeName, sizeStr, parameters[common.PoolConfigKey], poolType)
+			volume, _, err2 := controller.client.CreateVolume(volumeName, sizeStr, parameters[common.PoolConfigKey], poolType)
 			if err2 != nil {
 				return nil, err
 			}
+			if volume != nil {
+				wwn = volume.Wwn
+			}
 		}
 	}
-
-	wwn, err = controller.client.GetVolumeWwn(volumeName)
+	if wwn == "" {
+		wwn, err = controller.client.GetVolumeWwn(volumeName)
+	}
 	if err != nil {
 		klog.ErrorS(err, "Error retrieving WWN of new volume", "volumeName", volumeName)
 		return nil, err