[maven-extension] Propagate OTel context to plugin mojos #786
Conversation
…s and span attributes
| | ||
| private static final Logger logger = LoggerFactory.getLogger(OtelExecutionListener.class); | ||
| | ||
| private static final ThreadLocal<Scope> MOJO_EXECUTION_SCOPE = new ThreadLocal<>(); |
There was a problem hiding this comment.
Does this cause any issue with parallel Maven builds with -T?
There was a problem hiding this comment.
Yes, using a threadlocal from ExecutionListener#mojoStarted() to ExecutionListener#mojoSucceeded() / ExecutionListener#mojoFailed() works even when parallelising the Maven builds with -T.
The code contains state checks to ensure this assumption continues to be valid with newer versions of Maven parallelization.
I tested with 8 cores and -T 1C on a Macbok Air m2 2022 building https://github.com/apache/maven. I could confirm looking at the debug logs that the build was massively parallelised and that the threadlocal worked as desired.
There was a problem hiding this comment.
I just added a comment in the code to explain the rationale.
| Works for me. Thanks a lot @cyrille-leclerc ! |
maven-extension/src/main/java/io/opentelemetry/maven/OtelExecutionListener.java Show resolved Hide resolved
maven-extension/src/main/java/io/opentelemetry/maven/OtelExecutionListener.java Show resolved Hide resolved
4 participants
Description:
Propagate OTel context to plugin mojos so they can add their own attributes and spans to traces.
Existing Issue(s):
None
Testing:
No unit test framework compatible with JUnit 5 for Maven builds yet.
Documentation:
See updated README.md
Outstanding items:
None