Merge pull request kubernetes#90338 from gaurav1086/azure_remove_sprintf

k8s-ci-robot · web-flow · commit 8de8d25f6e04 · 2020-05-16T07:36:17.000-07:00
provider/azure: Remove unnecessary Sprintf in azure_routes.go
diff --git a/staging/src/k8s.io/legacy-cloud-providers/azure/azure_routes.go b/staging/src/k8s.io/legacy-cloud-providers/azure/azure_routes.go
@@ -405,7 +405,7 @@ func mapNodeNameToRouteName(ipv6DualStackEnabled bool, nodeName types.NodeName,
 // Used with mapNodeNameToRouteName. See comment on mapNodeNameToRouteName.
 func mapRouteNameToNodeName(ipv6DualStackEnabled bool, routeName string) types.NodeName {
 	if !ipv6DualStackEnabled {
-		return types.NodeName(fmt.Sprintf("%s", routeName))
+		return types.NodeName(routeName)
 	}
 	parts := strings.Split(routeName, routeNameSeparator)
 	nodeName := parts[0]

Original file line number	Diff line number	Diff line change
`@@ -405,7 +405,7 @@ func mapNodeNameToRouteName(ipv6DualStackEnabled bool, nodeName types.NodeName,`
`405`	`405`	`// Used with mapNodeNameToRouteName. See comment on mapNodeNameToRouteName.`
`406`	`406`	`func mapRouteNameToNodeName(ipv6DualStackEnabled bool, routeName string) types.NodeName {`
`407`	`407`	`if !ipv6DualStackEnabled {`
`408`		`- return types.NodeName(fmt.Sprintf("%s", routeName))`
	`408`	`+ return types.NodeName(routeName)`
`409`	`409`	`}`
`410`	`410`	`parts := strings.Split(routeName, routeNameSeparator)`
`411`	`411`	`nodeName := parts[0]`