Merge pull request #50846 from FengyunPan/change-log-level

Automatic merge from submit-queue (batch tested with PRs 51480, 49616, 50123, 50846, 50404) Use 'Infof' instead of 'Errorf' for a debug log Outputing error log for a debug is confused. **Release note**: ```release-note NONE ```

Merge pull request #50846 from FengyunPan/change-log-level
b5fad6eb · Kubernetes Submit Queue · GitHub · 00846fc7 · 4738a88d · b5fad6eb
Commit b5fad6eb authored Aug 31, 2017 by Kubernetes Submit Queue Committed by GitHub Aug 31, 2017
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 1 deletion

range_allocator.go pkg/controller/node/ipam/range_allocator.go +3 -1

No files found.
--- a/pkg/controller/node/ipam/range_allocator.go
+++ b/pkg/controller/node/ipam/range_allocator.go
@@ -233,8 +233,10 @@ func (r *rangeAllocator) updateCIDRAllocation(data nodeAndCIDR) error {
 			continue
 		}
 		if node.Spec.PodCIDR != "" {
-			glog.Errorf("Node %v already has allocated CIDR %v. Releasing assigned one if different.", node.Name, node.Spec.PodCIDR)
+			glog.V(4).Infof("Node %v already has allocated CIDR %v. Releasing assigned one if different.", node.Name, node.Spec.PodCIDR)
 			if node.Spec.PodCIDR != data.cidr.String() {
+				glog.Errorf("Node %q PodCIDR seems to have changed (original=%v, current=%v), releasing original and occupying new CIDR",
+					node.Name, node.Spec.PodCIDR, data.cidr.String())
 				if err := r.cidrs.Release(data.cidr); err != nil {
 					glog.Errorf("Error when releasing CIDR %v", data.cidr.String())
 				}