-
-
Notifications
You must be signed in to change notification settings - Fork 67
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add stack trace print for jlm test connect exceptions #403
Add stack trace print for jlm test connect exceptions #403
Conversation
Signed-off-by: Simon Rushton <srushton@redhat.com>
Fixes #402 |
Test run here: #402 The test did hit an occurrence of the exception which was being reported simply as The output from this PR now gives full information about the exception:
|
This should help provide information to resolve eclipse-openj9/openj9#7397, eclipse-openj9/openj9#9046 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Thanks for improving the error reporting in these tests, @lumpfish . The PR looks good, I've approved it. |
Signed-off-by: Simon Rushton srushton@redhat.com