-
Notifications
You must be signed in to change notification settings - Fork 319
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
Disabling Log correlation when agent is inactive #787
Disabling Log correlation when agent is inactive #787
Conversation
eyalkoren
commented
Aug 8, 2019
•
edited
Loading
edited
- Implement code
- Add tests
- Update documentation
- Update CHANGELOG.md
What if active is set to false after the activation and before the deactivation? We should have a transaction-scoped flag (can we use the noop flag?) which is consistent for a whole transaction. If that is false, call no scope listeners. |
I only check in adding to MDC but always removing from MDC. |
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.
can you add a test where active is set to false in between an activation?
...pm-slf4j-plugin/src/test/java/co/elastic/apm/agent/slf4j/Slf4JMdcActivationListenerTest.java
Show resolved
Hide resolved
@@ -5,6 +5,7 @@ | |||
## Bug Fixes | |||
* A warning in logs saying APM server is not available when using 1.8 with APM server 6.x | |||
* `ApacheHttpAsyncClientInstrumentation` matching increases startup time considerably | |||
* Log correlation feature is active when `active==false` |
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.
* Log correlation feature is active when `active==false` | |
* Disable log correlation feature when `active=false` |
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.
The Bug Fixes
section usually describes the bugs and not the fixes