zqr10159 opened a new pull request, #4284:
URL: https://github.com/apache/hertzbeat/pull/4284

   ## Summary
   
   - normalize Zabbix external alerts before passing them to the reducer
   - let the persistence layer assign the database identity, matching the 
default external alert handler
   - add a captured-argument contract for the normalized alert
   
   ## Validation
   
   - `./mvnw -pl hertzbeat-alerter -am -Dtest=ZabbixExternAlertServiceImplTest 
test -DskipITs -Dsurefire.failIfNoSpecifiedTests=false -DfailIfNoTests=false`
   - `git diff --cached --check`
   
   AI assistance: used for draft implementation and test iteration.
   Human validation: the focused Maven reactor completed successfully; the 
Zabbix external alert test passed and Checkstyle reported no violations.
   Risk notes: external Zabbix alert ids are no longer retained; alert 
fingerprinting and database identity are assigned by the existing reducer and 
persistence flow.
   


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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to