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

   **Issue Evaluation**
   
   Category: `bug` | Status: **Confirmed**
   
   The bug is valid. `AlertService.alertName()` strips non-alphanumeric 
characters, which can produce duplicate alert names when two rules normalize to 
the same string (e.g., `"Consumer Lag!"` → `"ConsumerLag"` and `"Consumer 
Lag?"` → `"ConsumerLag"`). Prometheus rejects rule files with duplicate alert 
names.
   
   **Root Cause:** No deduplication in `exportPrometheusRulesYaml()` after name 
sanitization.
   **Impact:** Exported Prometheus rules files fail to load entirely when name 
collisions occur.
   **Severity:** Medium — affects monitoring reliability for users who export 
alert rules.
   
   The proposed fix (tracking emitted names with a `Set<String>` and appending 
`_2`, `_3` suffixes on collision) is sound and minimal.
   
   ---
   *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