fixup! fix(preflight): improved error reporting for storage container checks

Remove unnecessary type conversions

Author: dlipovetsky

pkg/webhook/preflight/nutanix/storagecontainer.go

@@ -154,44 +154,39 @@ func getStorageContainer(
 		return nil, fmt.Errorf("failed to list storage containers: %w", err)
 	}
 
-	switch resp.GetData().(type) {
+	switch data := resp.GetData().(type) {
 	case nil:
 		return nil, fmt.Errorf("failed to find a matching storage container")
 
 	case clustermgmtv4errors.ErrorResponse:
-		errResp, ok := resp.GetData().(clustermgmtv4errors.ErrorResponse)
-		if !ok {
-			return nil, fmt.Errorf("failed to parse error response from %v", resp.GetData())
-		}
-
-		return nil, fmt.Errorf("failed to list storage containers: %v", errResp.GetError())
+		return nil, fmt.Errorf("failed to list storage containers: %v", data.GetError())
 
 	case []clustermgmtv4.StorageContainer:
-		containers, ok := resp.GetData().([]clustermgmtv4.StorageContainer)
-		if !ok {
-			return nil, fmt.Errorf("failed to parse storage containers from %v", resp.GetData())
-		}
-
-		if len(containers) == 0 {
+		if len(data) == 0 {
 			return nil, fmt.Errorf(
 				"no storage container named %q found on cluster named %q",
 				storageContainerName,
 				*cluster.Name,
 			)
 		}
 
-		if len(containers) > 1 {
+		if len(data) > 1 {
 			return nil, fmt.Errorf(
 				"multiple storage containers found with name %q on cluster %q",
 				storageContainerName,
 				*cluster.Name,
 			)
 		}
 
-		return ptr.To(containers[0]), nil
+		return ptr.To(data[0]), nil
+	default:
+		return nil,
+			fmt.Errorf(
+				"unexpected response type from ListStorageContainers(filter=%q): %T",
+				fltr,
+				resp.GetData(),
+			)
 	}
-
-	return nil, fmt.Errorf("unexpected response type from ListStorageContainers(filter=%q): %T", fltr, resp.GetData())
 }
 
 func getCluster(