[INFRAUTILS-48] diagstatus JSON response should include a systemReadyStateErrorCause Created: 22/Aug/18  Updated: 04/Sep/18  Resolved: 04/Sep/18

Status: Resolved
Project: infrautils
Component/s: diagstatus
Affects Version/s: None
Fix Version/s: Oxygen-SR4, Fluorine-SR1, Neon

Type: Improvement Priority: Medium
Reporter: Michael Vorburger Assignee: Michael Vorburger
Resolution: Done Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified


 Description   

Looking over trozet's INFRAUTILS-47, I'm finding it would be very useful to have details in a GET /diagstatus/ in case of a "systemReadyState": "FAILURE" ... somewhat similar to jluhrsen's work in INFRAUTILS-31 where we added an errorCause for each service (if available), it would really make sense to have an additional field named e.g. systemReadyStateErrorCause (OK for everyone?) which tells us WHY we are in a failed system ready state - without having to dig through a karaf.log.

This should be relatively easy to implement - SystemReadyMonitorMXBean should have not only an SystemState getSystemState() but also a Throwable getFailureCause() method, which SystemReadyImpl could provide, and DiagStatusServiceImpl could stick into the JSON (just like in INFRAUTILS-31).

k.faseela you certainly would have no objections if I implemented this?



 Comments   
Comment by Faseela K [ 23/Aug/18 ]

Please go ahead vorburger! And sorry that I still did not find time to push patch for INFRAUTILS-45

Generated at Wed Feb 07 20:02:07 UTC 2024 using Jira 8.20.10#820010-sha1:ace47f9899e9ee25d7157d59aa17ab06aee30d3d.