feat(sysadvisor): emit bound type code for knob status

kubewharf · Dec 1, 2023 · dcaf01f · dcaf01f
1 parent 1d2526f
commit dcaf01f
Show file tree

Hide file tree

Showing 3 changed files with 21 additions and 9 deletions.
diff --git a/pkg/agent/sysadvisor/plugin/metric-emitter/syncer/node/advisor.go b/pkg/agent/sysadvisor/plugin/metric-emitter/syncer/node/advisor.go
@@ -54,10 +54,6 @@ func (n *MetricSyncerNode) advisorMetric(ctx context.Context) {
 					Key: fmt.Sprintf("%s%s", data.CustomMetricLabelSelectorPrefixKey, "region"),
 					Val: fmt.Sprintf("%v", regionName),
 				},
-				{
-					Key: fmt.Sprintf("%s%s", data.CustomMetricLabelSelectorPrefixKey, "bound"),
-					Val: fmt.Sprintf("%v", regionInfo.RegionStatus.BoundType),
-				},
 			}...)
 		for indicator, overshot := range regionInfo.RegionStatus.OvershootStatus {
 			regionTag = append(regionTag, metrics.MetricTag{
@@ -66,7 +62,12 @@ func (n *MetricSyncerNode) advisorMetric(ctx context.Context) {
 			})
 		}
 
-		_ = n.dataEmitter.StoreFloat64(apimetricnode.CustomMetricNodeAdvisorKnobStatus, 1, metrics.MetricTypeNameRaw, regionTag...)
+		if code, ok := types.BoundTypeCodeMap[regionInfo.RegionStatus.BoundType]; ok {
+			_ = n.dataEmitter.StoreFloat64(apimetricnode.CustomMetricNodeAdvisorKnobStatus, float64(code), metrics.MetricTypeNameRaw, regionTag...)
+		} else {
+			general.Errorf("invalid bound type %v for ")
+		}
+
 		return true
 	})
 

diff --git a/pkg/agent/sysadvisor/types/consts.go b/pkg/agent/sysadvisor/types/consts.go
@@ -18,6 +18,13 @@ package types
 
 import "time"
 
+var BoundTypeCodeMap = map[BoundType]int{
+	BoundUpper:   BoundUpperCode,
+	BoundLower:   BoundLowerCode,
+	BoundNone:    BoundNoneCode,
+	BoundUnknown: BoundUnknownCode,
+}
+
 const (
 	RegionNameSeparator = "-"
 )

diff --git a/pkg/agent/sysadvisor/types/cpu.go b/pkg/agent/sysadvisor/types/cpu.go
@@ -137,16 +137,20 @@ type BoundType string
 
 const (
 	// BoundUpper indicates reaching resource upper bound, with highest priority
-	BoundUpper BoundType = "upper"
+	BoundUpper     BoundType = "upper"
+	BoundUpperCode int       = 0
 
 	// BoundLower indicates reaching resource lower bound
-	BoundLower BoundType = "lower"
+	BoundLower     BoundType = "lower"
+	BoundLowerCode int       = 1
 
 	// BoundNone indicates between resource upper and lower bound
-	BoundNone BoundType = "none"
+	BoundNone     BoundType = "none"
+	BoundNoneCode int       = 2
 
 	// BoundUnknown indicates unknown bound status
-	BoundUnknown BoundType = "unknown"
+	BoundUnknown     BoundType = "unknown"
+	BoundUnknownCode int       = 3
 )
 
 // RegionStatus holds stability accounting info of region