Skip to content
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

Option to send Equinox Framework trace to the OSGi LogService #766

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tjwatson
Copy link
Contributor

The trace messages in the framework are currently sent to the System.out. This commit adds an option to send the Framework trace messages to the OSGi LogService using the LogLevel of Trace. The Logger name used is the trace option key.

To set Equinox trace options with LoggerAdmin the logger name EQUINOX.TRACE is set to LogLevel TRACE. When EQUINOX.TRACE logger is set to TRACE then all other logger names in the LoggerContext configuration set to TRACE are also assumed to be trace names to be enabled.

Copy link

github-actions bot commented Jan 24, 2025

Test Results

  669 files  +3    669 suites  +3   1h 13m 13s ⏱️ - 1m 19s
2 222 tests +3  2 175 ✅ +3   47 💤 ±0  0 ❌ ±0 
6 810 runs  +9  6 667 ✅ +9  143 💤 ±0  0 ❌ ±0 

Results for commit 8f4fe61. ± Comparison against base commit 7461945.

♻️ This comment has been updated with latest results.

The trace messages in the framework are currently sent to the
System.out.  This commit adds an option to send the Framework
trace messages to the OSGi LogService using the LogLevel
of Trace.  The Logger name used is the trace option key.

To set Equinox trace options with LoggerAdmin the logger
name EQUINOX.TRACE is set to LogLevel TRACE. When EQUINOX.TRACE
logger is set to TRACE then all other logger names in the
LoggerContext configuration set to TRACE are also assumed to
be trace names to be enabled.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant