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

   … timedout_scripts field
   
   Three bugs in app-redis_sentinel.yml:
   
   1. helpLink pointed to influxdb_promql docs instead of redis_sentinel docs 
(copy-paste error).
   
   2. All five sentinel section fields (sentinel_masters, sentinel_tilt, 
sentinel_running_scripts, sentinel_scripts_queue_length, 
sentinel_simulate_failure_flags) were declared as type: 1 (string) but Redis 
INFO sentinel returns integers for all of them. Corrected to type: 0 (number) 
so dashboards and alerting thresholds work correctly.
   
   3. Added missing sentinel_timedout_scripts field (present in Redis INFO 
sentinel output, tracks scripts that exceeded the timeout deadline — useful for 
detecting runaway Lua scripts in HA pipelines).
   
   ## What's changed?
   
   <!-- Describe Your PR Here -->
   
   
   ## Checklist
   
   - [ ]  I have read the [Contributing 
Guide](https://hertzbeat.apache.org/docs/community/code_style_and_quality_guide)
   - [ ]  I have written the necessary doc or comment.
   - [ ]  I have added the necessary unit tests and all cases have passed.
   
   ## Add or update API
   
   - [ ] I have added the necessary [e2e 
tests](https://github.com/apache/hertzbeat/tree/master/e2e) and all cases have 
passed.
   


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