zmuxuny commented on PR #4517: URL: https://github.com/apache/rocketmq-dashboard/pull/4517#issuecomment-5718801481
Audit follow-up: I found one scope-preservation edge while re-reviewing the open PR set. Per-group UNAVAILABLE Apache lag samples were dropping the group's `clusterId`, so a native alert rule with `clusterName` could fail to match the unavailable sample during reconciliation. The branch now preserves `group.clusterId` on per-group/group-topic UNAVAILABLE samples while keeping whole-instance failure markers unscoped. The existing unknown-queue regression now asserts the cluster id too. Re-verified on Java 21: `ApacheRocketMqBusinessMetricsCollectorTest,RocketMQMetadataProviderTest,NativeAlertProcessorTest` **71/71 passed**, Checkstyle 0; `mvn -B -ntp -DskipTests package` BUILD SUCCESS; `git diff --check` clean. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
