RockteMQ-AI commented on issue #1298:
URL: 
https://github.com/apache/rocketmq-dashboard/issues/1298#issuecomment-5231675946

   ## Issue Evaluation
   
   **Category:** `type/bug` | **Status:** Confirmed
   
   The issue is well-described and the fix scope is clear. 
`AlertService.exportPrometheusRulesYaml()` should filter persisted rules by 
`enabled` before conversion, and fall back to bundled default templates when no 
enabled rules remain.
   
   **Assessment:**
   - **Root Cause:** Missing `enabled` filter in the export path
   - **Impact:** Disabled rules remain active in Prometheus after YAML import — 
operational risk
   - **Severity:** Medium — silent misconfiguration, no data loss
   - **Scope:** Narrow — single service method + regression tests
   
   This is a straightforward fix. A PR is welcome.
   
   ---
   *Automated evaluation by RockteMQ-AI*


-- 
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]

Reply via email to