Change up log msg

Nick Sardo · Nick Sardo · commit a869ac852ca6 · 2017-08-30T10:13:48.000-07:00
diff --git a/pkg/cloudprovider/providers/gce/gce_address_manager.go b/pkg/cloudprovider/providers/gce/gce_address_manager.go
@@ -61,7 +61,7 @@ func (am *addressManager) HoldAddress() (string, error) {
 	// could be reserving another address; therefore, it would need to be deleted. In the normal
 	// case of using a controller address, retrieving the address by name results in the fewest API
 	// calls since it indicates whether a Delete is necessary before Reserve.
-	glog.V(2).Infof("%v: attempting hold of IP %q Type %q", am.logPrefix, am.targetIP, am.addressType)
+	glog.V(4).Infof("%v: attempting hold of IP %q Type %q", am.logPrefix, am.targetIP, am.addressType)
 	// Get the address in case it was orphaned earlier
 	addr, err := am.svc.GetBetaRegionAddress(am.name, am.region)
 	if err != nil && !isNotFound(err) {
diff --git a/pkg/cloudprovider/providers/gce/gce_loadbalancer_internal.go b/pkg/cloudprovider/providers/gce/gce_loadbalancer_internal.go
@@ -82,12 +82,12 @@ func (gce *GCECloud) ensureInternalLoadBalancer(clusterName, clusterID string, s
 	// Determine IP which will be used for this LB. If no forwarding rule has been established
 	// or specified in the Service spec, then requestedIP = "".
 	requestedIP := determineRequestedIP(svc, existingFwdRule)
-	glog.V(2).Infof("ensureInternalLoadBalancer(%v): Existing forwarding rule or service spec says to use IP: %q", loadBalancerName, requestedIP)
 	addrMgr := newAddressManager(gce, nm.String(), gce.Region(), gce.SubnetworkURL(), loadBalancerName, requestedIP, schemeInternal)
 	ipToUse, err := addrMgr.HoldAddress()
 	if err != nil {
 		return nil, err
 	}
+	glog.V(2).Infof("ensureInternalLoadBalancer(%v): reserved IP %q for the forwarding rule", loadBalancerName, ipToUse)
 
 	// Ensure firewall rules if necessary
 	if gce.OnXPN() {
@@ -136,6 +136,7 @@ func (gce *GCECloud) ensureInternalLoadBalancer(clusterName, clusterID string, s
 		if err = gce.CreateRegionForwardingRule(expectedFwdRule, gce.region); err != nil {
 			return nil, err
 		}
+		glog.V(2).Infof("ensureInternalLoadBalancer(%v): created forwarding rule", loadBalancerName)
 	}
 
 	// Delete the previous internal load balancer resources if necessary

Original file line number	Diff line number	Diff line change
`@@ -82,12 +82,12 @@ func (gce *GCECloud) ensureInternalLoadBalancer(clusterName, clusterID string, s`
`82`	`82`	`// Determine IP which will be used for this LB. If no forwarding rule has been established`
`83`	`83`	`// or specified in the Service spec, then requestedIP = "".`
`84`	`84`	`requestedIP := determineRequestedIP(svc, existingFwdRule)`
`85`		`- glog.V(2).Infof("ensureInternalLoadBalancer(%v): Existing forwarding rule or service spec says to use IP: %q", loadBalancerName, requestedIP)`
`86`	`85`	`addrMgr := newAddressManager(gce, nm.String(), gce.Region(), gce.SubnetworkURL(), loadBalancerName, requestedIP, schemeInternal)`
`87`	`86`	`ipToUse, err := addrMgr.HoldAddress()`
`88`	`87`	`if err != nil {`
`89`	`88`	`return nil, err`
`90`	`89`	`}`
	`90`	`+ glog.V(2).Infof("ensureInternalLoadBalancer(%v): reserved IP %q for the forwarding rule", loadBalancerName, ipToUse)`
`91`	`91`
`92`	`92`	`// Ensure firewall rules if necessary`
`93`	`93`	`if gce.OnXPN() {`
`@@ -136,6 +136,7 @@ func (gce *GCECloud) ensureInternalLoadBalancer(clusterName, clusterID string, s`
`136`	`136`	`if err = gce.CreateRegionForwardingRule(expectedFwdRule, gce.region); err != nil {`
`137`	`137`	`return nil, err`
`138`	`138`	`}`
	`139`	`+ glog.V(2).Infof("ensureInternalLoadBalancer(%v): created forwarding rule", loadBalancerName)`
`139`	`140`	`}`
`140`	`141`
`141`	`142`	`// Delete the previous internal load balancer resources if necessary`