Skip to content

Logging custom facade - #2390

Merged
eyalkoren merged 15 commits into
elastic:masterfrom
eyalkoren:logging-custom-facade
Jan 18, 2022
Merged

Logging custom facade#2390
eyalkoren merged 15 commits into
elastic:masterfrom
eyalkoren:logging-custom-facade

Conversation

@eyalkoren

@eyalkoren eyalkoren commented Jan 11, 2022

Copy link
Copy Markdown
Contributor

What does this PR do?

Fixes #2371
Fixes #2350 (the LinkageError, not adding support for log correlation in all cases)

Checklist

@ghost

ghost commented Jan 11, 2022

Copy link
Copy Markdown

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2022-01-18T14:55:24.794+0000

  • Duration: 46 min 22 sec

  • Commit: 432d247

Test stats 🧪

Test Results
Failed 0
Passed 2443
Skipped 16
Total 2459

💚 Flaky test report

Tests succeeded.

🤖 GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

  • run benchmark tests : Run the benchmark test.

  • run compatibility tests : Run the JDK Compatibility test.

  • run integration tests : Run the APM-ITs.

  • run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)

@eyalkoren
eyalkoren marked this pull request as ready for review January 17, 2022 05:56
@eyalkoren
eyalkoren requested a review from felixbarny January 17, 2022 06:23
@AlexanderWert AlexanderWert added this to the 8.1 milestone Jan 17, 2022
felixbarny and others added 2 commits January 18, 2022 12:42
…t/java/co/elastic/apm/servlet/AbstractServletContainerIntegrationTest.java

Co-authored-by: Felix Barnsteiner <felixbarny@users.noreply.github.com>
@eyalkoren
eyalkoren enabled auto-merge (squash) January 18, 2022 14:57
@eyalkoren
eyalkoren merged commit 864e4e2 into elastic:master Jan 18, 2022
@eyalkoren
eyalkoren deleted the logging-custom-facade branch January 19, 2022 04:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Slf4j java.lang.LinkageError slf4j MDC with java.lang.LinkageError, within the transaction created by external plugin and log correlation enabled

4 participants