Cyrille Le Clerc

Status: Capturing Maven logs will be more complex than just replacing the `org.apache.maven.cli.event.ExecutionEventLogger` by an OtelLogsExecutionListener. We need to intercept "Logger" invocations, hopefully changing the implementation of the loggers to...

Update: we are at the moment polishing the integration of CI logs with OpenTelemetry on a CI server, Jenkins, before moving forward on the integration of Maven Logs with OpenTelemetry....

Clever idea, thanks @delanym

I tried to use `@Inject` rather than Plexus but I faced limitations with `@Inject`. I forgot what were these limitations, I have to look at the history of commits.

For the record, I reverted from JSR-330 / SISU to Plexus on commit https://github.com/elastic/opentelemetry-maven-extension/commit/39e2975f14d1f8f1e231d6699d8da314b00331a4 . FYI The Maven community is interested in understanding the limitations we faced: https://lists.apache.org/thread/bqm4zpjmvhmtqxjnz5qbp2yjxk9y9472

> Right, we think this is a great idea, but like #4736, it needs corresponding changes to our Pipeline jobs to invoke `./mvnw` rather than `mvn`. Thanks @basil , I'm...

@askoog sorry for the delay. Thanks for you submission. Unfortunately, the build fails on unit tests. Did you have a chance to look at this failure? https://travis-ci.org/jmxtrans/jmxtrans-agent/jobs/439558727

Hello @aakarshg , sorry for the delay. I am no longer much active on this project. Could you please propose a PR?

Hi @evmin, I trust you but I don't understand yet how it would work with indexed values... I would need to test Rather than `deleting` the existing `writeQueryResult(...)` method, I...

@evmin could you initiate the work on a PR? Maybe it will be the opportunity to cut a v2.0 of the plugin.