cluster: check add CPU frequency check output (#1663)

pingcap · Dec 13, 2021 · 3f0c568 · 3f0c568
1 parent b01e507
commit 3f0c568
Showing 1 changed file with 15 additions and 2 deletions.
diff --git a/pkg/cluster/operation/check.go b/pkg/cluster/operation/check.go
@@ -224,10 +224,23 @@ func checkCPU(opt *CheckOptions, cpuInfo *sysinfo.CPU) []*CheckResult {
 	}
 
 	// check for CPU frequency governor
-	if cpuInfo.Governor != "" && cpuInfo.Governor != "performance" {
+	if cpuInfo.Governor != "" {
+		if cpuInfo.Governor != "performance" {
+			results = append(results, &CheckResult{
+				Name: CheckNameCPUGovernor,
+				Err:  fmt.Errorf("CPU frequency governor is %s, should use performance", cpuInfo.Governor),
+			})
+		} else {
+			results = append(results, &CheckResult{
+				Name: CheckNameCPUGovernor,
+				Msg:  fmt.Sprintf("CPU frequency governor is %s", cpuInfo.Governor),
+			})
+		}
+	} else {
 		results = append(results, &CheckResult{
 			Name: CheckNameCPUGovernor,
-			Err:  fmt.Errorf("CPU frequency governor is %s, should use performance", cpuInfo.Governor),
+			Err:  fmt.Errorf("Unable to determine current CPU frequency governor policy"),
+			Warn: true,
 		})
 	}