Merge pull request #122644 from gyuho/logs-removing-taints

chores(controller/nodelifecycle): make node taint removal logs more a…
This commit is contained in:
Kubernetes Prow Robot
2024-10-23 01:17:15 +01:00
committed by GitHub

View File

@@ -809,7 +809,7 @@ func (nc *Controller) processTaintBaseEviction(ctx context.Context, node *v1.Nod
logger.Error(nil, "Failed to remove taints from node. Will retry in next iteration", "node", klog.KObj(node)) logger.Error(nil, "Failed to remove taints from node. Will retry in next iteration", "node", klog.KObj(node))
} }
if removed { if removed {
logger.V(2).Info("Node is healthy again, removing all taints", "node", klog.KObj(node)) logger.V(2).Info("Node is healthy again, removed all taints", "node", klog.KObj(node))
} }
} }
} }
@@ -1258,12 +1258,12 @@ func (nc *Controller) markNodeAsReachable(ctx context.Context, node *v1.Node) (b
err := controller.RemoveTaintOffNode(ctx, nc.kubeClient, node.Name, node, UnreachableTaintTemplate) err := controller.RemoveTaintOffNode(ctx, nc.kubeClient, node.Name, node, UnreachableTaintTemplate)
logger := klog.FromContext(ctx) logger := klog.FromContext(ctx)
if err != nil { if err != nil {
logger.Error(err, "Failed to remove taint from node", "node", klog.KObj(node)) logger.Error(err, "Failed to remove unreachable taint from node", "node", klog.KObj(node))
return false, err return false, err
} }
err = controller.RemoveTaintOffNode(ctx, nc.kubeClient, node.Name, node, NotReadyTaintTemplate) err = controller.RemoveTaintOffNode(ctx, nc.kubeClient, node.Name, node, NotReadyTaintTemplate)
if err != nil { if err != nil {
logger.Error(err, "Failed to remove taint from node", "node", klog.KObj(node)) logger.Error(err, "Failed to remove not-ready taint from node", "node", klog.KObj(node))
return false, err return false, err
} }
nc.evictorLock.Lock() nc.evictorLock.Lock()