log operation id for attach and detach

Author: amacaskill

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

@@ -415,12 +415,14 @@ func (cloud *CloudProvider) insertRegionalDisk(
 		insertOp, err = cloud.betaService.RegionDisks.Insert(project, volKey.Region, betaDiskToCreate).Context(ctx).Do()
 		if insertOp != nil {
 			opName = insertOp.Name
+			klog.V(5).Infof("InsertDisk operation %s for disk %s", opName, betaDiskToCreate.Name)
 		}
 	} else {
 		var insertOp *computev1.Operation
 		insertOp, err = cloud.service.RegionDisks.Insert(project, volKey.Region, diskToCreate).Context(ctx).Do()
 		if insertOp != nil {
 			opName = insertOp.Name
+			klog.V(5).Infof("InsertDisk operation %s for disk %s", opName, diskToCreate.Name)
 		}
 	}
 	if err != nil {
@@ -509,12 +511,14 @@ func (cloud *CloudProvider) insertZonalDisk(
 		insertOp, err = cloud.betaService.Disks.Insert(project, volKey.Zone, betaDiskToCreate).Context(ctx).Do()
 		if insertOp != nil {
 			opName = insertOp.Name
+			klog.V(5).Infof("InsertDisk operation %s for disk %s", opName, betaDiskToCreate.Name)
 		}
 	} else {
 		var insertOp *computev1.Operation
 		insertOp, err = cloud.service.Disks.Insert(project, volKey.Zone, diskToCreate).Context(ctx).Do()
 		if insertOp != nil {
 			opName = insertOp.Name
+			klog.V(5).Infof("InsertDisk operation %s for disk %s", opName, diskToCreate.Name)
 		}
 	}
 
@@ -574,6 +578,9 @@ 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()
+	if op != nil {
+		klog.V(5).Infof("DeleteDisk operation %s for disk %s", op.Name, name)
+	}
 	if err != nil {
 		if IsGCEError(err, "notFound") {
 			// Already deleted
@@ -590,6 +597,9 @@ 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()
+	if op != nil {
+		klog.V(5).Infof("DeleteDisk operation %s for disk %s", op.Name, name)
+	}
 	if err != nil {
 		if IsGCEError(err, "notFound") {
 			// Already deleted
@@ -621,6 +631,9 @@ func (cloud *CloudProvider) AttachDisk(ctx context.Context, project string, volK
 	}
 
 	op, err := cloud.service.Instances.AttachDisk(project, instanceZone, instanceName, attachedDiskV1).Context(ctx).Do()
+	if op != nil {
+		klog.V(5).Infof("AttachDisk operation %s for disk %s", op.Name, attachedDiskV1.DeviceName)
+	}
 	if err != nil {
 		return fmt.Errorf("failed cloud service attach disk call: %v", err)
 	}
@@ -634,6 +647,9 @@ 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()
+	if op != nil {
+		klog.V(5).Infof("DetachDisk operation %s for disk %s", op.Name, deviceName)
+	}
 	if err != nil {
 		return err
 	}
@@ -846,6 +862,9 @@ 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()
+	if op != nil {
+		klog.V(5).Infof("ResizeDisk operation %s for disk %s", op.Name, volKey.Name)
+	}
 	if err != nil {
 		return -1, fmt.Errorf("failed to resize zonal volume %v: %v", volKey.String(), err)
 	}
@@ -864,6 +883,9 @@ func (cloud *CloudProvider) resizeRegionalDisk(ctx context.Context, project stri
 	}
 
 	op, err := cloud.service.RegionDisks.Resize(project, volKey.Region, volKey.Name, resizeReq).Context(ctx).Do()
+	if op != nil {
+		klog.V(5).Infof("ResizeDisk operation %s for disk %s", op.Name, volKey.Name)
+	}
 	if err != nil {
 		return -1, fmt.Errorf("failed to resize regional volume %v: %v", volKey.String(), err)
 	}