FIX: native ipmi client not close

Signed-off-by: dingxin05 <dingxin05@baidu.com>

Author: dingxin05

collector_bmc_native.go

@@ -52,6 +52,11 @@ func (c BMCNativeCollector) Collect(_ freeipmi.Result, ch chan<- prometheus.Metr
 	if err != nil {
 		return 0, err
 	}
+	defer func() {
+		if closeErr := client.Close(context.TODO()); closeErr != nil {
+			logger.Debug("Failed to close IPMI client", "target", targetName(target.host), "error", closeErr)
+		}
+	}()
 	res, err := client.GetDeviceID(context.TODO())
 	if err != nil {
 		return 0, err

collector_bmc_watchdog_native.go

@@ -99,6 +99,11 @@ func (c BMCWatchdogNativeCollector) Collect(_ freeipmi.Result, ch chan<- prometh
 	if err != nil {
 		return 0, err
 	}
+	defer func() {
+		if closeErr := client.Close(context.TODO()); closeErr != nil {
+			logger.Debug("Failed to close IPMI client", "target", targetName(target.host), "error", closeErr)
+		}
+	}()
 	res, err := client.GetWatchdogTimer(context.TODO())
 	if err != nil {
 		return 0, err

collector_chassis_native.go

@@ -62,6 +62,11 @@ func (c ChassisNativeCollector) Collect(_ freeipmi.Result, ch chan<- prometheus.
 	if err != nil {
 		return 0, err
 	}
+	defer func() {
+		if closeErr := client.Close(context.TODO()); closeErr != nil {
+			logger.Debug("Failed to close IPMI client", "target", targetName(target.host), "error", closeErr)
+		}
+	}()
 	res, err := client.GetChassisStatus(context.TODO())
 	if err != nil {
 		return 0, err

collector_dcmi_native.go

@@ -50,6 +50,11 @@ func (c DCMINativeCollector) Collect(_ freeipmi.Result, ch chan<- prometheus.Met
 	if err != nil {
 		return 0, err
 	}
+	defer func() {
+		if closeErr := client.Close(context.TODO()); closeErr != nil {
+			logger.Debug("Failed to close IPMI client", "target", targetName(target.host), "error", closeErr)
+		}
+	}()
 	res, err := client.GetDCMIPowerReading(context.TODO())
 	if err != nil {
 		logger.Error("Failed to collect DCMI data", "target", targetName(target.host), "error", err)

collector_ipmi_native.go

@@ -150,6 +150,11 @@ func (c IPMINativeCollector) Collect(_ freeipmi.Result, ch chan<- prometheus.Met
 	if err != nil {
 		return 0, err
 	}
+	defer func() {
+		if closeErr := client.Close(context.TODO()); closeErr != nil {
+			logger.Debug("Failed to close IPMI client", "target", targetName(target.host), "error", closeErr)
+		}
+	}()
 	res, err := client.GetSensors(context.TODO(), filter)
 	if err != nil {
 		return 0, err

collector_sel_events_native.go

@@ -64,6 +64,11 @@ func (c SELEventsNativeCollector) Collect(_ freeipmi.Result, ch chan<- prometheu
 	if err != nil {
 		return 0, err
 	}
+	defer func() {
+		if closeErr := client.Close(context.TODO()); closeErr != nil {
+			logger.Debug("Failed to close IPMI client", "target", targetName(target.host), "error", closeErr)
+		}
+	}()
 	res, err := client.GetSELEntries(context.TODO(), 0)
 	if err != nil {
 		return 0, err

collector_sel_native.go

@@ -57,6 +57,11 @@ func (c SELNativeCollector) Collect(_ freeipmi.Result, ch chan<- prometheus.Metr
 	if err != nil {
 		return 0, err
 	}
+	defer func() {
+		if closeErr := client.Close(context.TODO()); closeErr != nil {
+			logger.Debug("Failed to close IPMI client", "target", targetName(target.host), "error", closeErr)
+		}
+	}()
 	res, err := client.GetSELInfo(context.TODO())
 	if err != nil {
 		return 0, err

collector_sm_lan_mode_native.go

@@ -52,6 +52,11 @@ func (c SMLANModeNativeCollector) Collect(_ freeipmi.Result, ch chan<- prometheu
 	if err != nil {
 		return 0, err
 	}
+	defer func() {
+		if closeErr := client.Close(context.TODO()); closeErr != nil {
+			logger.Debug("Failed to close IPMI client", "target", targetName(target.host), "error", closeErr)
+		}
+	}()
 	if _, err := client.SetSessionPrivilegeLevel(context.TODO(), ipmi.PrivilegeLevelAdministrator); err != nil {
 		logger.Error("Failed to set privilege level to admin", "target", targetName(target.host))
 		return 0, fmt.Errorf("failed to set privilege level to admin")