chihsuan commented on PR #10857: URL: https://github.com/apache/ozone/pull/10857#issuecomment-5130969704
> Would it be possible to surface the actual failure detail here — e.g. the assertion message like `expected: <1> but was: <2>` or the exception reason? > > The goal is to let devs see _where_ and _what_ failed directly in the summary, without opening the full CI log — that log is slow to load and tends to stall the browser. Thanks for the review and context! @peterxcli Kafka's `junit.py` does not truncate at all. It keeps the full message and just converts newlines to `<br>`. But that works because Gradle message attribute is usually short. In our case, like `TestLeaseRecovery` flooding the assertion message, no limit could blow past the 1 MiB step summary cap. I'll update to keep only the first non-empty line of the message, so the useful part stays visible. Will verify how it looks and let you know. -- 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]
