Skip to content

Commit f90c0e0

Browse files
committed
change to Infof s
1 parent 2159a5f commit f90c0e0

File tree

1 file changed

+12
-6
lines changed

1 file changed

+12
-6
lines changed

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

+12-6
Original file line numberDiff line numberDiff line change
@@ -576,7 +576,8 @@ func (cloud *CloudProvider) DeleteDisk(ctx context.Context, project string, volK
576576

577577
func (cloud *CloudProvider) deleteZonalDisk(ctx context.Context, project, zone, name string) error {
578578
op, err := cloud.service.Disks.Delete(project, zone, name).Context(ctx).Do()
579-
// klog.V(5).Infof("DeleteDisk operation %s", op.Name)
579+
opName := op.Name
580+
klog.V(5).Infof("DeleteDisk operation %s", opName)
580581
if err != nil {
581582
if IsGCEError(err, "notFound") {
582583
// Already deleted
@@ -593,7 +594,8 @@ func (cloud *CloudProvider) deleteZonalDisk(ctx context.Context, project, zone,
593594

594595
func (cloud *CloudProvider) deleteRegionalDisk(ctx context.Context, project, region, name string) error {
595596
op, err := cloud.service.RegionDisks.Delete(project, region, name).Context(ctx).Do()
596-
// klog.V(5).Infof("DeleteDisk operation %s", op.Name)
597+
opName := op.Name
598+
klog.V(5).Infof("DeleteDisk operation %s", opName)
597599
if err != nil {
598600
if IsGCEError(err, "notFound") {
599601
// Already deleted
@@ -625,7 +627,8 @@ func (cloud *CloudProvider) AttachDisk(ctx context.Context, project string, volK
625627
}
626628

627629
op, err := cloud.service.Instances.AttachDisk(project, instanceZone, instanceName, attachedDiskV1).Context(ctx).Do()
628-
// klog.V(5).Infof("AttachDisk operation %s", op.Name)
630+
opName := op.Name
631+
klog.V(5).Infof("AttachDisk operation %s", opName)
629632
if err != nil {
630633
return fmt.Errorf("failed cloud service attach disk call: %v", err)
631634
}
@@ -639,7 +642,8 @@ func (cloud *CloudProvider) AttachDisk(ctx context.Context, project string, volK
639642
func (cloud *CloudProvider) DetachDisk(ctx context.Context, project, deviceName, instanceZone, instanceName string) error {
640643
klog.V(5).Infof("Detaching disk %v from %v", deviceName, instanceName)
641644
op, err := cloud.service.Instances.DetachDisk(project, instanceZone, instanceName, deviceName).Context(ctx).Do()
642-
// klog.V(5).Infof("DetachDisk operation %s", op.Name)
645+
opName := op.Name
646+
klog.V(5).Infof("DetachDisk operation %s", opName)
643647
if err != nil {
644648
return err
645649
}
@@ -852,7 +856,8 @@ func (cloud *CloudProvider) resizeZonalDisk(ctx context.Context, project string,
852856
SizeGb: requestGb,
853857
}
854858
op, err := cloud.service.Disks.Resize(project, volKey.Zone, volKey.Name, resizeReq).Context(ctx).Do()
855-
// klog.V(5).Infof("ResizeDisk operation %s", op.Name)
859+
opName := op.Name
860+
klog.V(5).Infof("ResizeDisk operation %s", opName)
856861
if err != nil {
857862
return -1, fmt.Errorf("failed to resize zonal volume %v: %v", volKey.String(), err)
858863
}
@@ -870,7 +875,8 @@ func (cloud *CloudProvider) resizeRegionalDisk(ctx context.Context, project stri
870875
SizeGb: requestGb,
871876
}
872877
op, err := cloud.service.RegionDisks.Resize(project, volKey.Region, volKey.Name, resizeReq).Context(ctx).Do()
873-
// klog.V(5).Infof("ResizeDisk operation %s", op.Name)
878+
opName := op.Name
879+
klog.V(5).Infof("ResizeDisk operation %s", opName)
874880
if err != nil {
875881
return -1, fmt.Errorf("failed to resize regional volume %v: %v", volKey.String(), err)
876882
}

0 commit comments

Comments
 (0)