-
Notifications
You must be signed in to change notification settings - Fork 245
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
feat: add prefix monitor #3621
feat: add prefix monitor #3621
Conversation
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.
We are always happy to welcome new contributors ❤️ To make things easier for everyone, please make sure to follow our contribution guidelines, check if you have already signed the ECA, and relate this pull request to an existing issue or discussion.
Codecov ReportAttention:
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #3621 +/- ##
===========================================
- Coverage 71.77% 17.27% -54.51%
===========================================
Files 909 910 +1
Lines 18104 18126 +22
Branches 1027 1027
===========================================
- Hits 12994 3131 -9863
- Misses 4666 14908 +10242
+ Partials 444 87 -357 ☔ View full report in Codecov by Sentry. |
spi/common/core-spi/src/main/java/org/eclipse/edc/spi/monitor/PrefixMonitor.java
Outdated
Show resolved
Hide resolved
...nitor/monitor-jdk-logger/src/test/java/org/eclipse/edc/monitor/logger/PrefixMonitorTest.java
Outdated
Show resolved
Hide resolved
spi/common/core-spi/src/main/java/org/eclipse/edc/spi/monitor/PrefixMonitor.java
Outdated
Show resolved
Hide resolved
spi/common/core-spi/src/main/java/org/eclipse/edc/spi/monitor/Monitor.java
Outdated
Show resolved
Hide resolved
spi/common/core-spi/src/main/java/org/eclipse/edc/spi/monitor/PrefixMonitor.java
Outdated
Show resolved
Hide resolved
spi/common/core-spi/src/main/java/org/eclipse/edc/spi/monitor/PrefixMonitor.java
Outdated
Show resolved
Hide resolved
spi/common/core-spi/src/main/java/org/eclipse/edc/spi/monitor/PrefixMonitor.java
Outdated
Show resolved
Hide resolved
...nitor/monitor-jdk-logger/src/test/java/org/eclipse/edc/monitor/logger/PrefixMonitorTest.java
Outdated
Show resolved
Hide resolved
...nitor/monitor-jdk-logger/src/test/java/org/eclipse/edc/monitor/logger/PrefixMonitorTest.java
Outdated
Show resolved
Hide resolved
...nitor/monitor-jdk-logger/src/test/java/org/eclipse/edc/monitor/logger/PrefixMonitorTest.java
Outdated
Show resolved
Hide resolved
spi/common/core-spi/src/main/java/org/eclipse/edc/spi/monitor/Monitor.java
Show resolved
Hide resolved
Please resolve the outstanding comments and dependency review. Thx. |
What this PR changes/adds
This will add a new monitor implementation so that logs can be prefixed with a custom string.
Why it does that
Give logs more meaning by providing a context.
Further notes
Linked Issue(s)
Closes #3510