Skip to content

Commit c59203e

Browse files
authored
Merge pull request kubernetes#121967 from torredil/update-logging
Update log verbosity for node health and taint checks
2 parents 71e7f9e + 1354bb9 commit c59203e

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

pkg/controller/volume/attachdetach/reconciler/reconciler.go

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -215,7 +215,9 @@ func (rc *reconciler) reconcile(ctx context.Context) {
215215

216216
isHealthy, err := rc.nodeIsHealthy(attachedVolume.NodeName)
217217
if err != nil {
218-
logger.Error(err, "Failed to get health of node", "node", klog.KRef("", string(attachedVolume.NodeName)))
218+
logger.V(5).Info("Failed to get health of node",
219+
"node", klog.KRef("", string(attachedVolume.NodeName)),
220+
"err", err)
219221
}
220222

221223
// Force detach volumes from unhealthy nodes after maxWaitForUnmountDuration if force detach is enabled
@@ -228,7 +230,9 @@ func (rc *reconciler) reconcile(ctx context.Context) {
228230

229231
hasOutOfServiceTaint, err := rc.hasOutOfServiceTaint(attachedVolume.NodeName)
230232
if err != nil {
231-
logger.Error(err, "Failed to get taint specs for node", "node", klog.KRef("", string(attachedVolume.NodeName)))
233+
logger.V(5).Info("Failed to get taint specs for node",
234+
"node", klog.KRef("", string(attachedVolume.NodeName)),
235+
"err", err)
232236
}
233237

234238
// Check whether volume is still mounted. Skip detach if it is still mounted unless we have

0 commit comments

Comments
 (0)