chore: support kbcli cluster update (#7015)

apecloud · Apr 23, 2024 · 912b51c · 912b51c
1 parent 6aa6a2d
commit 912b51c
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/pkg/cmd/cluster/update.go b/pkg/cmd/cluster/update.go
@@ -283,8 +283,8 @@ func (o *UpdateOptions) buildPatch() error {
 		"tolerations": {field: "tolerations", obj: spec, fn: buildTolObj},
 
 		// monitor and logs
-		"monitoring-interval": {field: "monitor", obj: nil, fn: buildComps},
-		"enable-all-logs":     {field: "enable-all-logs", obj: nil, fn: buildComps},
+		"monitor-enabled": {field: "monitor", obj: nil, fn: buildComps},
+		"enable-all-logs": {field: "enable-all-logs", obj: nil, fn: buildComps},
 
 		// backup config
 		"backup-enabled":                   {field: "enabled", obj: nil, fn: buildBackup},
@@ -603,13 +603,16 @@ func buildLogsReconfiguringOps(clusterName, namespace, compName, configName, key
 }
 
 func (o *UpdateOptions) updateMonitor(val string) error {
-	intVal, err := strconv.ParseInt(val, 10, 32)
+	enabled, err := strconv.ParseBool(val)
 	if err != nil {
 		return err
 	}
 
 	for i := range o.cluster.Spec.ComponentSpecs {
-		o.cluster.Spec.ComponentSpecs[i].Monitor = intVal != 0
+		// TODO: update kb version depend
+		// 1. add metrics sidecar to the component
+		// 2. enable monitor
+		o.cluster.Spec.ComponentSpecs[i].Monitor = enabled
 	}
 	return nil
 }