Merge pull request #61183 from ingvagabund/node-status-be-more-verbose

Automatic merge from submit-queue. 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>. Node status be more verbose **What this PR does / why we need it**: Improve logging ability of node status so it is easier to debug update of a node status ```release-note NONE ```

Merge pull request #61183 from ingvagabund/node-status-be-more-verbose
1d030799 · Kubernetes Submit Queue · GitHub · 9939f709 · 6d820d5a · 1d030799
Unverified Commit 1d030799 authored Apr 06, 2018 by Kubernetes Submit Queue Committed by GitHub Apr 06, 2018
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 1 deletion

kubelet_node_status.go pkg/kubelet/kubelet_node_status.go +3 -1

No files found.
--- a/pkg/kubelet/kubelet_node_status.go
+++ b/pkg/kubelet/kubelet_node_status.go
@@ -369,6 +369,7 @@ func (kl *Kubelet) syncNodeStatus() {

 // updateNodeStatus updates node status to master with retries.
 func (kl *Kubelet) updateNodeStatus() error {
+	glog.V(5).Infof("Updating node status")
 	for i := 0; i < nodeStatusUpdateRetry; i++ {
 		if err := kl.tryUpdateNodeStatus(i); err != nil {
 			glog.Errorf("Error updating node status, will retry: %v", err)
@@ -1041,7 +1042,8 @@ func (kl *Kubelet) setNodeVolumesInUseStatus(node *v1.Node) {
 // TODO(madhusudancs): Simplify the logic for setting node conditions and
 // refactor the node status condition code out to a different file.
 func (kl *Kubelet) setNodeStatus(node *v1.Node) {
-	for _, f := range kl.setNodeStatusFuncs {
+	for i, f := range kl.setNodeStatusFuncs {
+		glog.V(5).Infof("Setting node status at position %v", i)
 		if err := f(node); err != nil {
 			glog.Warningf("Failed to set some node status fields: %s", err)
 		}