Skip to content

Commit 3c074d8

Browse files
committed
change to Infof s
1 parent 644f5ad commit 3c074d8

File tree

1 file changed

+7
-12
lines changed

1 file changed

+7
-12
lines changed

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

+7-12
Original file line numberDiff line numberDiff line change
@@ -577,8 +577,7 @@ func (cloud *CloudProvider) DeleteDisk(ctx context.Context, project string, volK
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()
579579
if op != nil {
580-
opName := op.Name
581-
klog.V(5).Infof("DeleteDisk operation %s", opName)
580+
klog.V(5).Infof("DeleteDisk operation %s", op.Name)
582581
}
583582
if err != nil {
584583
if IsGCEError(err, "notFound") {
@@ -597,8 +596,7 @@ func (cloud *CloudProvider) deleteZonalDisk(ctx context.Context, project, zone,
597596
func (cloud *CloudProvider) deleteRegionalDisk(ctx context.Context, project, region, name string) error {
598597
op, err := cloud.service.RegionDisks.Delete(project, region, name).Context(ctx).Do()
599598
if op != nil {
600-
opName := op.Name
601-
klog.V(5).Infof("DeleteDisk operation %s", opName)
599+
klog.V(5).Infof("DeleteDisk operation %s", op.Name)
602600
}
603601
if err != nil {
604602
if IsGCEError(err, "notFound") {
@@ -632,8 +630,7 @@ func (cloud *CloudProvider) AttachDisk(ctx context.Context, project string, volK
632630

633631
op, err := cloud.service.Instances.AttachDisk(project, instanceZone, instanceName, attachedDiskV1).Context(ctx).Do()
634632
if op != nil {
635-
opName := op.Name
636-
klog.V(5).Infof("AttachDisk operation %s", opName)
633+
klog.V(5).Infof("AttachDisk operation %s", op.Name)
637634
}
638635
if err != nil {
639636
return fmt.Errorf("failed cloud service attach disk call: %v", err)
@@ -649,8 +646,7 @@ func (cloud *CloudProvider) DetachDisk(ctx context.Context, project, deviceName,
649646
klog.V(5).Infof("Detaching disk %v from %v", deviceName, instanceName)
650647
op, err := cloud.service.Instances.DetachDisk(project, instanceZone, instanceName, deviceName).Context(ctx).Do()
651648
if op != nil {
652-
opName := op.Name
653-
klog.V(5).Infof("DetachDisk operation %s", opName)
649+
klog.V(5).Infof("DetachDisk operation %s", op.Name)
654650
}
655651
if err != nil {
656652
return err
@@ -865,8 +861,7 @@ func (cloud *CloudProvider) resizeZonalDisk(ctx context.Context, project string,
865861
}
866862
op, err := cloud.service.Disks.Resize(project, volKey.Zone, volKey.Name, resizeReq).Context(ctx).Do()
867863
if op != nil {
868-
opName := op.Name
869-
klog.V(5).Infof("ResizeDisk operation %s", opName)
864+
klog.V(5).Infof("ResizeDisk operation %s", op.Name)
870865
}
871866
if err != nil {
872867
return -1, fmt.Errorf("failed to resize zonal volume %v: %v", volKey.String(), err)
@@ -884,10 +879,10 @@ func (cloud *CloudProvider) resizeRegionalDisk(ctx context.Context, project stri
884879
resizeReq := &computev1.RegionDisksResizeRequest{
885880
SizeGb: requestGb,
886881
}
882+
887883
op, err := cloud.service.RegionDisks.Resize(project, volKey.Region, volKey.Name, resizeReq).Context(ctx).Do()
888884
if op != nil {
889-
opName := op.Name
890-
klog.V(5).Infof("ResizeDisk operation %s", opName)
885+
klog.V(5).Infof("ResizeDisk operation %s", op.Name)
891886
}
892887
if err != nil {
893888
return -1, fmt.Errorf("failed to resize regional volume %v: %v", volKey.String(), err)

0 commit comments

Comments
 (0)