From 5582856e23b956a5d3f54d28ffeaa80eb6a89f71 Mon Sep 17 00:00:00 2001 From: Wojciech Tyczynski Date: Tue, 6 Sep 2016 08:39:06 +0200 Subject: [PATCH] Fix debugging log in RC --- pkg/controller/replication/replication_controller.go | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/pkg/controller/replication/replication_controller.go b/pkg/controller/replication/replication_controller.go index 77cbde0761..679ca827ed 100644 --- a/pkg/controller/replication/replication_controller.go +++ b/pkg/controller/replication/replication_controller.go @@ -320,7 +320,7 @@ func (rm *ReplicationManager) updateRC(old, cur interface{}) { rm.lookupCache.InvalidateAll() } // TODO: Remove when #31981 is resolved! - glog.Infof("Observed updated replication controller %v. Pod count change: %d->%d", curRC.Name, oldRC.Status.Replicas, curRC.Status.Replicas) + glog.Infof("Observed updated replication controller %v. Desired pod count change: %d->%d", curRC.Name, oldRC.Spec.Replicas, curRC.Spec.Replicas) // You might imagine that we only really need to enqueue the // controller when Spec changes, but it is safer to sync any @@ -335,6 +335,7 @@ func (rm *ReplicationManager) updateRC(old, cur interface{}) { // that bad as rcs that haven't met expectations yet won't // sync, and all the listing is done using local stores. if oldRC.Status.Replicas != curRC.Status.Replicas { + // TODO: Should we log status or spec? glog.V(4).Infof("Observed updated replica count for rc: %v, %d->%d", curRC.Name, oldRC.Status.Replicas, curRC.Status.Replicas) } rm.enqueueController(cur)