move check op is error check

Author: amacaskill

pkg/gce-cloud-provider/compute/gce-compute.go

@@ -587,9 +587,7 @@ func (cloud *CloudProvider) deleteZonalDisk(ctx context.Context, project, zone,
 	if err != nil {
 		return err
 	}
-	// if op != nil {
 	klog.V(5).Infof("DeleteDisk operation %s for disk %s", op.Name, name)
-	// }
 	return nil
 }
 
@@ -606,9 +604,7 @@ func (cloud *CloudProvider) deleteRegionalDisk(ctx context.Context, project, reg
 	if err != nil {
 		return err
 	}
-	// if op != nil {
 	klog.V(5).Infof("DeleteDisk operation %s for disk %s", op.Name, name)
-	// }
 	return nil
 }
 
@@ -636,9 +632,7 @@ func (cloud *CloudProvider) AttachDisk(ctx context.Context, project string, volK
 	if err != nil {
 		return fmt.Errorf("failed when waiting for zonal op: %v", err)
 	}
-	// if op != nil {
 	klog.V(5).Infof("AttachDisk operation %s for disk %s", op.Name, attachedDiskV1.DeviceName)
-	// }
 	return nil
 }
 
@@ -652,9 +646,7 @@ func (cloud *CloudProvider) DetachDisk(ctx context.Context, project, deviceName,
 	if err != nil {
 		return err
 	}
-	if op != nil {
-		klog.V(5).Infof("DetachDisk operation %s for disk %s", op.Name, deviceName)
-	}
+	klog.V(5).Infof("DetachDisk operation %s for disk %s", op.Name, deviceName)
 	return nil
 }
 
@@ -868,9 +860,8 @@ func (cloud *CloudProvider) resizeZonalDisk(ctx context.Context, project string,
 	if err != nil {
 		return -1, fmt.Errorf("failed waiting for op for zonal resize for %s: %v", volKey.String(), err)
 	}
-	// if op != nil {
+
 	klog.V(5).Infof("ResizeDisk operation %s for disk %s", op.Name, volKey.Name)
-	// }
 	return requestGb, nil
 }
 
@@ -888,9 +879,8 @@ func (cloud *CloudProvider) resizeRegionalDisk(ctx context.Context, project stri
 	if err != nil {
 		return -1, fmt.Errorf("failed waiting for op for regional resize for %s: %v", volKey.String(), err)
 	}
-	// if op != nil {
+
 	klog.V(5).Infof("ResizeDisk operation %s for disk %s", op.Name, volKey.Name)
-	// }
 	return requestGb, nil
 }