Skip to content

Commit 5ef59ae

Browse files
authored
Merge pull request kubernetes#78250 from qingsenLi/k8s-190523
klog format error
2 parents 2daad0b + 5b737a0 commit 5ef59ae

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

pkg/cloudprovider/providers/openstack/openstack_routes.go

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -111,12 +111,12 @@ func updateRoutes(network *gophercloud.ServiceClient, router *routers.Router, ne
111111
}
112112

113113
unwinder := func() {
114-
klog.V(4).Info("Reverting routes change to router ", router.ID)
114+
klog.V(4).Infof("Reverting routes change to router %v", router.ID)
115115
_, err := routers.Update(network, router.ID, routers.UpdateOpts{
116116
Routes: origRoutes,
117117
}).Extract()
118118
if err != nil {
119-
klog.Warning("Unable to reset routes during error unwind: ", err)
119+
klog.Warningf("Unable to reset routes during error unwind: %v", err)
120120
}
121121
}
122122

@@ -134,12 +134,12 @@ func updateAllowedAddressPairs(network *gophercloud.ServiceClient, port *neutron
134134
}
135135

136136
unwinder := func() {
137-
klog.V(4).Info("Reverting allowed-address-pairs change to port ", port.ID)
137+
klog.V(4).Infof("Reverting allowed-address-pairs change to port %v", port.ID)
138138
_, err := neutronports.Update(network, port.ID, neutronports.UpdateOpts{
139139
AllowedAddressPairs: &origPairs,
140140
}).Extract()
141141
if err != nil {
142-
klog.Warning("Unable to reset allowed-address-pairs during error unwind: ", err)
142+
klog.Warningf("Unable to reset allowed-address-pairs during error unwind: %v", err)
143143
}
144144
}
145145

@@ -200,7 +200,7 @@ func (r *Routes) CreateRoute(ctx context.Context, clusterName string, nameHint s
200200
found := false
201201
for _, item := range port.AllowedAddressPairs {
202202
if item.IPAddress == route.DestinationCIDR {
203-
klog.V(4).Info("Found existing allowed-address-pair: ", item)
203+
klog.V(4).Infof("Found existing allowed-address-pair: %v", item)
204204
found = true
205205
break
206206
}

0 commit comments

Comments
 (0)