Merge pull request #54244 from gnufied/fix-log-spam-from-node-status

Automatic merge from submit-queue (batch tested with PRs 53809, 54244, 54142). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. Fixes spam from node status updates Some of us are looking at reducing unncessary logging spam which happens in k8s. This message is anyway logged by caller of the function and hence we should not log it again. cc @kubernetes/sig-storage-pr-reviews @eparis

Merge pull request #54244 from gnufied/fix-log-spam-from-node-status
44f89806 · Kubernetes Submit Queue · GitHub · c7a1a061 · e3f0c8bb · 44f89806
Commit 44f89806 authored Oct 19, 2017 by Kubernetes Submit Queue Committed by GitHub Oct 19, 2017
Hide whitespace changes
Inline Side-by-side

Showing with 0 additions and 1 deletion

actual_state_of_world.go ...roller/volume/attachdetach/cache/actual_state_of_world.go +0 -1

No files found.
--- a/pkg/controller/volume/attachdetach/cache/actual_state_of_world.go
+++ b/pkg/controller/volume/attachdetach/cache/actual_state_of_world.go
@@ -475,7 +475,6 @@ func (asw *actualStateOfWorld) updateNodeStatusUpdateNeeded(nodeName types.NodeN
 		// should not happen
 		errMsg := fmt.Sprintf("Failed to set statusUpdateNeeded to needed %t because nodeName=%q  does not exist",
 			needed, nodeName)
-		glog.Errorf(errMsg)
 		return fmt.Errorf(errMsg)
 	}