Merge 7df3a8b94c into 53186b30e3

collector/bonding_linux.go

@@ -21,14 +21,20 @@ import (
 	"log/slog"
 	"os"
 	"path/filepath"
+	"strconv"
 	"strings"
 
 	"github.com/prometheus/client_golang/prometheus"
 )
 
 type bondingCollector struct {
-	slaves, active typedDesc
-	logger         *slog.Logger
+	slaves, active, miimon typedDesc
+	logger                 *slog.Logger
+}
+
+type bondingStats struct {
+	name                   string
+	slaves, active, miimon int
 }
 
 func init() {
@@ -49,6 +55,11 @@ func NewBondingCollector(logger *slog.Logger) (Collector, error) {
 			"Number of active slaves per bonding interface.",
 			[]string{"master"}, nil,
 		), prometheus.GaugeValue},
+		miimon: typedDesc{prometheus.NewDesc(
+			prometheus.BuildFQName(namespace, "bonding", "miimon"),
+			"MII link monitoring frequency in milliseconds.",
+			[]string{"master"}, nil,
+		), prometheus.GaugeValue},
 		logger: logger,
 	}, nil
 }
@@ -64,15 +75,16 @@ func (c *bondingCollector) Update(ch chan<- prometheus.Metric) error {
 		}
 		return err
 	}
-	for master, status := range bondingStats {
-		ch <- c.slaves.mustNewConstMetric(float64(status[0]), master)
-		ch <- c.active.mustNewConstMetric(float64(status[1]), master)
+	for _, bond := range bondingStats {
+		ch <- c.slaves.mustNewConstMetric(float64(bond.slaves), bond.name)
+		ch <- c.active.mustNewConstMetric(float64(bond.active), bond.name)
+		ch <- c.miimon.mustNewConstMetric(float64(bond.miimon), bond.name)
 	}
 	return nil
 }
 
-func readBondingStats(root string) (status map[string][2]int, err error) {
-	status = map[string][2]int{}
+func readBondingStats(root string) (status []bondingStats, err error) {
+	status = []bondingStats{}
 	masters, err := os.ReadFile(filepath.Join(root, "bonding_masters"))
 	if err != nil {
 		return nil, err
@@ -97,7 +109,22 @@ func readBondingStats(root string) (status map[string][2]int, err error) {
 				sstat[1]++
 			}
 		}
-		status[master] = sstat
+
+		miimon, err := os.ReadFile(filepath.Join(root, master, "bonding", "miimon"))
+		if err != nil {
+			return nil, err
+		}
+		intMiimon, err := strconv.Atoi(strings.TrimSpace(string(miimon)))
+		if err != nil {
+			return nil, err
+		}
+
+		status = append(status, bondingStats{
+			name:   master,
+			slaves: sstat[0],
+			active: sstat[1],
+			miimon: intMiimon,
+		})
 	}
 	return status, err
 }

collector/bonding_linux_test.go

@@ -24,15 +24,15 @@ func TestBonding(t *testing.T) {
 	if err != nil {
 		t.Fatal(err)
 	}
-	if bondingStats["bond0"][0] != 0 || bondingStats["bond0"][1] != 0 {
+	if bondingStats[0].name != "bond0" || bondingStats[0].slaves != 0 || bondingStats[0].active != 0 || bondingStats[0].miimon != 100 {
 		t.Fatal("bond0 in unexpected state")
 	}
 
-	if bondingStats["int"][0] != 2 || bondingStats["int"][1] != 1 {
-		t.Fatal("int in unexpected state")
+	if bondingStats[1].name != "dmz" || bondingStats[1].slaves != 2 || bondingStats[1].active != 2 || bondingStats[1].miimon != 0 {
+		t.Fatal("dmz in unexpected state")
 	}
 
-	if bondingStats["dmz"][0] != 2 || bondingStats["dmz"][1] != 2 {
-		t.Fatal("dmz in unexpected state")
+	if bondingStats[2].name != "int" || bondingStats[2].slaves != 2 || bondingStats[2].active != 1 || bondingStats[2].miimon != 200 {
+		t.Fatal("int in unexpected state")
 	}
 }

collector/fixtures/sys.ttar

@@ -1741,6 +1741,11 @@ Path: sys/class/net/bond0/bonding/slaves
 Lines: 0
 Mode: 644
 # ttar - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
+Path: sys/class/net/bond0/bonding/miimon
+Lines: 1
+100
+Mode: 644
+# ttar - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
 Path: sys/class/net/bond0/broadcast
 Lines: 1
 ff:ff:ff:ff:ff:ff
@@ -1883,6 +1888,11 @@ Lines: 1
 eth0 eth4
 Mode: 644
 # ttar - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
+Path: sys/class/net/dmz/bonding/miimon
+Lines: 1
+0
+Mode: 644
+# ttar - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
 Path: sys/class/net/dmz/broadcast
 Lines: 1
 ff:ff:ff:ff:ff:ff
@@ -2055,6 +2065,11 @@ Lines: 1
 eth5 eth1
 Mode: 644
 # ttar - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
+Path: sys/class/net/int/bonding/miimon
+Lines: 1
+200
+Mode: 644
+# ttar - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
 Path: sys/class/net/int/broadcast
 Lines: 1
 ff:ff:ff:ff:ff:ff