fix(monitor): detach alert (#20375)

pkg/monitor/models/monitor_resource.go

@@ -416,7 +416,20 @@ func (manager *SMonitorResourceManager) UpdateMonitorResourceAttachJoint(ctx con
 	}
 	deleteJointIds := make([]int64, 0)
 	for _, joint := range resourceAlerts {
-		if utils.IsInStringArray(joint.MonitorResourceId, matchResourceIds) {
+		evalData, err := joint.GetData()
+		if err != nil {
+			log.Warningf("get data of monitor_resource_alert %s: %s", jsonutils.Marshal(joint), err)
+			continue
+		}
+		metricName := evalData.Metric
+		isMetricFound := false
+		for _, match := range matches {
+			if match.Metric == metricName {
+				isMetricFound = true
+				break
+			}
+		}
+		if utils.IsInStringArray(joint.MonitorResourceId, matchResourceIds) && isMetricFound {
 			continue
 		}
 		deleteJointIds = append(deleteJointIds, joint.RowId)

pkg/monitor/models/monitor_resource_alert.go

@@ -152,6 +152,17 @@ func (obj *SMonitorResourceAlert) UpdateAlertRecordData(record *SAlertRecord, ma
 	return nil
 }
 
+func (obj *SMonitorResourceAlert) GetData() (*monitor.EvalMatch, error) {
+	if obj.Data == nil {
+		return nil, errors.Errorf("data is nil")
+	}
+	match := new(monitor.EvalMatch)
+	if err := obj.Data.Unmarshal(match); err != nil {
+		return nil, errors.Wrap(err, "unmarshal to monitor.EvalMatch")
+	}
+	return match, nil
+}
+
 func (m *SMonitorResourceAlertManager) ListItemFilter(ctx context.Context, q *sqlchemy.SQuery, userCred mcclient.TokenCredential, input *monitor.MonitorResourceJointListInput) (*sqlchemy.SQuery, error) {
 	var err error
 	q, err = m.SJointResourceBaseManager.ListItemFilter(ctx, q, userCred, input.JointResourceBaseListInput)