Merge pull request #44487 from resouer/fix-ss-error

Automatic merge from submit-queue (batch tested with PRs 41498, 44487) Use len of pods in stateful set error **What this PR does / why we need it**: Sync stateful set reports wrong error, we need to fix it. **Release note**: ```release-note `NONE` ```

Merge pull request #44487 from resouer/fix-ss-error
cd37aaaf · Kubernetes Submit Queue · GitHub · cdc0cbda · cef629c6 · cd37aaaf
Commit cd37aaaf authored Apr 21, 2017 by Kubernetes Submit Queue Committed by GitHub Apr 21, 2017
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 3 deletions

stateful_set.go pkg/controller/statefulset/stateful_set.go +2 -3

No files found.
--- a/pkg/controller/statefulset/stateful_set.go
+++ b/pkg/controller/statefulset/stateful_set.go
@@ -416,11 +416,10 @@ func (ssc *StatefulSetController) sync(key string) error {

 // syncStatefulSet syncs a tuple of (statefulset, []*v1.Pod).
 func (ssc *StatefulSetController) syncStatefulSet(set *apps.StatefulSet, pods []*v1.Pod) error {
-	glog.V(2).Infof("Syncing StatefulSet %v/%v with %d pods", set.Namespace, set.Name, len(pods))
+	glog.V(4).Infof("Syncing StatefulSet %v/%v with %d pods", set.Namespace, set.Name, len(pods))
 	if err := ssc.control.UpdateStatefulSet(set, pods); err != nil {
-		glog.V(2).Infof("Error syncing StatefulSet %s/%s with %d pods : %s", set.Namespace, set.Name, err)
 		return err
 	}
-	glog.V(2).Infof("Successfully synced StatefulSet %s/%s successful", set.Namespace, set.Name)
+	glog.V(4).Infof("Successfully synced StatefulSet %s/%s successful", set.Namespace, set.Name)
 	return nil
 }