change to Infof s

Author: amacaskill

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

@@ -576,8 +576,10 @@ func (cloud *CloudProvider) DeleteDisk(ctx context.Context, project string, volK
 
 func (cloud *CloudProvider) deleteZonalDisk(ctx context.Context, project, zone, name string) error {
 	op, err := cloud.service.Disks.Delete(project, zone, name).Context(ctx).Do()
-	opName := op.Name
-	klog.V(5).Infof("DeleteDisk operation %s", opName)
+	if op != nil {
+		opName := op.Name
+		klog.V(5).Infof("DeleteDisk operation %s", opName)
+	}
 	if err != nil {
 		if IsGCEError(err, "notFound") {
 			// Already deleted
@@ -594,8 +596,10 @@ func (cloud *CloudProvider) deleteZonalDisk(ctx context.Context, project, zone,
 
 func (cloud *CloudProvider) deleteRegionalDisk(ctx context.Context, project, region, name string) error {
 	op, err := cloud.service.RegionDisks.Delete(project, region, name).Context(ctx).Do()
-	opName := op.Name
-	klog.V(5).Infof("DeleteDisk operation %s", opName)
+	if op != nil {
+		opName := op.Name
+		klog.V(5).Infof("DeleteDisk operation %s", opName)
+	}
 	if err != nil {
 		if IsGCEError(err, "notFound") {
 			// Already deleted
@@ -627,8 +631,10 @@ func (cloud *CloudProvider) AttachDisk(ctx context.Context, project string, volK
 	}
 
 	op, err := cloud.service.Instances.AttachDisk(project, instanceZone, instanceName, attachedDiskV1).Context(ctx).Do()
-	opName := op.Name
-	klog.V(5).Infof("AttachDisk operation %s", opName)
+	if op != nil {
+		opName := op.Name
+		klog.V(5).Infof("AttachDisk operation %s", opName)
+	}
 	if err != nil {
 		return fmt.Errorf("failed cloud service attach disk call: %v", err)
 	}
@@ -642,8 +648,10 @@ func (cloud *CloudProvider) AttachDisk(ctx context.Context, project string, volK
 func (cloud *CloudProvider) DetachDisk(ctx context.Context, project, deviceName, instanceZone, instanceName string) error {
 	klog.V(5).Infof("Detaching disk %v from %v", deviceName, instanceName)
 	op, err := cloud.service.Instances.DetachDisk(project, instanceZone, instanceName, deviceName).Context(ctx).Do()
-	opName := op.Name
-	klog.V(5).Infof("DetachDisk operation %s", opName)
+	if op != nil {
+		opName := op.Name
+		klog.V(5).Infof("DetachDisk operation %s", opName)
+	}
 	if err != nil {
 		return err
 	}
@@ -856,8 +864,10 @@ func (cloud *CloudProvider) resizeZonalDisk(ctx context.Context, project string,
 		SizeGb: requestGb,
 	}
 	op, err := cloud.service.Disks.Resize(project, volKey.Zone, volKey.Name, resizeReq).Context(ctx).Do()
-	opName := op.Name
-	klog.V(5).Infof("ResizeDisk operation %s", opName)
+	if op != nil {
+		opName := op.Name
+		klog.V(5).Infof("ResizeDisk operation %s", opName)
+	}
 	if err != nil {
 		return -1, fmt.Errorf("failed to resize zonal volume %v: %v", volKey.String(), err)
 	}
@@ -875,8 +885,10 @@ func (cloud *CloudProvider) resizeRegionalDisk(ctx context.Context, project stri
 		SizeGb: requestGb,
 	}
 	op, err := cloud.service.RegionDisks.Resize(project, volKey.Region, volKey.Name, resizeReq).Context(ctx).Do()
-	opName := op.Name
-	klog.V(5).Infof("ResizeDisk operation %s", opName)
+	if op != nil {
+		opName := op.Name
+		klog.V(5).Infof("ResizeDisk operation %s", opName)
+	}
 	if err != nil {
 		return -1, fmt.Errorf("failed to resize regional volume %v: %v", volKey.String(), err)
 	}