fix(monitor): set monitors to nil after closing them

Set monitors to nil after closing them, so the controller can recreate them when the node is back. Longhorn 10035 Signed-off-by: Derek Su <[email protected]> (cherry picked from commit a65a6ce)
longhorn · Dec 23, 2024 · 790b6dc · 790b6dc
1 parent a5dad03
commit 790b6dc
Show file tree

Hide file tree

Showing 5 changed files with 58 additions and 572 deletions.
diff --git a/controller/monitor/disk_monitor.go b/controller/monitor/disk_monitor.go
@@ -133,7 +133,6 @@ func (m *NodeMonitor) GetCollectedData() (interface{}, error) {
 func (m *NodeMonitor) run(value interface{}) error {
 	node, err := m.ds.GetNode(m.nodeName)
 	if err != nil {
-		logrus.WithError(err).Errorf("Failed to get longhorn node %v", m.nodeName)
 		return errors.Wrapf(err, "failed to get longhorn node %v", m.nodeName)
 	}