-
Notifications
You must be signed in to change notification settings - Fork 14
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
Remote connection to target without JUL daemon does not handle error message gracefully (lttng 2.13) #192
Comments
For reference, the previous version of our target produced this output from the same command:
I assume it's a difference in how we built, configured, or installed lttng on the target. (we have no java runtime on the target, so no need to log JUL events) |
I think it fails because the command exits with a non zero value. Could you please confirm by printing the exit code (e.g. |
Good point, it returns exit code 1 with / without the --quiet flag
|
So the question remains: is it possible to configure TraceCompass to not query JUL sessions, or is this an implicit requirement of the design? |
Thanks for confirming. No, it's not a implicit requirement. I think it's just a matter of ignoring any errors when checking for such UST provider (e.g. JUL, python etc). If it is not supported it should not prevent using other logger providers: [1] is the relevant code line. Instead of throwing an exception it could just return. Alternatively, we could ignore some of the errors like done for the default UST provider [2]. [1] Line 1144 in 1801704
[2] Line 626 in 1801704
I'll try to submit a PR for this. |
Excellent, thanks! |
Don't fail if logger agents are not available (e.g. JUL). [Fixed] listing UST domains if there are no logger agents Fixes eclipse-tracecompass#192 Signed-off-by: Bernd Hufmann <[email protected]>
Don't fail if logger agents are not available (e.g. JUL). [Fixed] listing UST domains if there are no logger agents Fixes eclipse-tracecompass#192 Signed-off-by: Bernd Hufmann <[email protected]>
Don't fail if logger agents are not available (e.g. JUL). [Fixed] listing UST domains if there are no logger agents Fixes eclipse-tracecompass#192 Signed-off-by: Bernd Hufmann <[email protected]>
Some additional information: We rolled back to previous versions of lttng and re-built the system. No other changes were made. It seems that this is a general incompatibility with lttng v2.13 specifically - it's now returning an error whereas previously in v2.12 it did not. Turns out our system has never been built with --enable-java-agent-jul nor has it ever had any java runtime installed. The behavior of lttng in this situation is simply different between 2.12 and 2.13. System built with lttng version we were using previously (no error)
System built with lttng 2.12 (latest version) (also no error)
I updated the title of the issue accordingly. |
Don't fail if logger agents are not available (e.g. JUL). [Fixed] listing UST domains if there are no logger agents Fixes #192 Signed-off-by: Bernd Hufmann <[email protected]>
@RyanFindley The fix is integrated in the nightly build. Please let me know if it works in your environment. Thanks. |
@bhufmann Confirmed! Thanks for the quick turnaround. :) |
Recently, we updated our embedded target and apparently liblttng-ust is no longer being built with --enable-java-agent-jul.
When connecting to this target over ssh from TraceCompass, the following error occurs:
Actual output of this command when run on the target:
I suspect that the parser is choking on the bare error message emitted before the XML response content.
This can probably be solved by adding the --quiet arg to the request, to suppress the bare error output if the parser is not prepared to handle it:
or by redirecting stderr to /dev/null:
To workaround in the short term: Is it possible to disable TraceCompass from querying for JUL domain session list? (via a properties or prefs file, command line arg, etc) Search didn't turn up any clues...
TraceCompass version: 10.2.0
lttng version on target:
The text was updated successfully, but these errors were encountered: