fix(obcluster): list obcluster parameters group by name

internal/dashboard/business/oceanbase/obcluster.go

@@ -934,7 +934,7 @@ func ListOBClusterParameters(ctx context.Context, nn *param.K8sObjectIdentity) (
 		logger.Info("Failed to get OceanBase database connection")
 		return nil, errors.Wrap(err, "Get OceanBase database connection")
 	}
-	parameters, err := conn.ListParametersWithTenantID(ctx, 1)
+	parameters, err := conn.ListClusterParameters(ctx)
 	if err != nil {
 		logger.WithError(err).Error("Failed to query parameters")
 		return nil, errors.Wrap(err, "Query parameters")

internal/dashboard/business/oceanbase/obtenant.go

@@ -211,6 +211,7 @@ func buildOverviewFromApiType(t *v1alpha1.OBTenant) *response.OBTenantOverview {
 	rt.Charset = t.Spec.Charset
 	rt.Locality = t.Status.TenantRecordInfo.Locality
 	rt.PrimaryZone = t.Status.TenantRecordInfo.PrimaryZone
+	rt.DeletionProtection = t.Annotations[oceanbaseconst.AnnotationsIgnoreDeletion] == "true"
 
 	for i := range t.Status.Pools {
 		pool := t.Status.Pools[i]

internal/dashboard/handler/obtenant_handler.go

@@ -243,9 +243,6 @@ func PatchTenant(c *gin.Context) (*response.OBTenantDetail, error) {
 	if err != nil {
 		return nil, httpErr.NewBadRequest(err.Error())
 	}
-	if patch.UnitNumber == nil && patch.UnitConfig == nil {
-		return nil, httpErr.NewBadRequest("unitNumber or unitConfig is required")
-	}
 	logger.Infof("Patch obtenant with param: %+v", patch)
 	tenant, err := oceanbase.PatchTenant(c, types.NamespacedName{
 		Namespace: nn.Namespace,

internal/dashboard/model/response/obtenant.go

@@ -27,6 +27,8 @@ type OBTenantOverview struct {
 	Locality    string            `json:"locality" binding:"required"`            // Locality of the tenant units
 	Charset     string            `json:"charset" binding:"required"`             // Charset of the tenant
 	PrimaryZone string            `json:"primaryZone" binding:"required"`         // Primary zone of the tenant
+
+	DeletionProtection bool `json:"deletionProtection" binding:"required"` // Whether the tenant is protected from deletion
 }
 
 type OBTenantDetail struct {

pkg/oceanbase-sdk/const/sql/parameter.go

@@ -22,4 +22,5 @@ const (
 const (
 	ListParametersWithTenantID = "select name, value, data_type, info, section, default_value, isdefault, edit_level, scope from GV$OB_PARAMETERS where tenant_id = ?"
 	SelectCompatibleOfTenants  = "select name, value, tenant_id from GV$OB_PARAMETERS where name = 'compatible'"
+	ListParametersGroupByName  = "select name, value, data_type, info, section, default_value, isdefault, edit_level, scope, svr_ip, zone, svr_port, IFNULL(tenant_id, 0) as tenant_id from GV$OB_PARAMETERS group by name"
 )

pkg/oceanbase-sdk/operation/cluster.go

@@ -60,3 +60,12 @@ func (m *OceanbaseOperationManager) GetVersion(ctx context.Context) (*model.OBVe
 	}
 	return version, nil
 }
+
+func (m *OceanbaseOperationManager) ListClusterParameters(ctx context.Context) ([]*model.Parameter, error) {
+	params := make([]*model.Parameter, 0)
+	err := m.QueryList(ctx, &params, sql.ListParametersGroupByName)
+	if err != nil {
+		return nil, errors.Wrap(err, "List cluster parameters")
+	}
+	return params, nil
+}