Introduce Tracer API for log capture. (#3515)

---------

Co-authored-by: Sylvain Juge <[email protected]>

Author: raphw

apm-agent-core/src/main/java/co/elastic/apm/agent/impl/ElasticApmTracer.java

@@ -26,6 +26,8 @@
 import co.elastic.apm.agent.configuration.CoreConfiguration;
 import co.elastic.apm.agent.configuration.MetricsConfiguration;
 import co.elastic.apm.agent.configuration.ServerlessConfiguration;
+import co.elastic.apm.agent.impl.metadata.ServiceFactory;
+import co.elastic.apm.agent.tracer.service.Service;
 import co.elastic.apm.agent.tracer.service.ServiceInfo;
 import co.elastic.apm.agent.configuration.SpanConfiguration;
 import co.elastic.apm.agent.context.ClosableLifecycleListenerAdapter;
@@ -973,4 +975,24 @@ public Set<String> getTraceHeaderNames() {
     public ServiceInfo autoDetectedServiceInfo() {
         return AutoDetectedServiceInfo.autoDetected();
     }
+
+    @Override
+    public void reportLog(String log) {
+        reporter.reportLog(log);
+    }
+
+    @Override
+    public void reportLog(byte[] log) {
+        reporter.reportLog(log);
+    }
+
+    @Nullable
+    @Override
+    public Service createService(String ephemeralId) {
+        return new ServiceFactory().createService(
+            coreConfiguration,
+            ephemeralId,
+            configurationRegistry.getConfig(ServerlessConfiguration.class).runsOnAwsLambda()
+        );
+    }
 }

apm-agent-core/src/main/java/co/elastic/apm/agent/impl/metadata/Node.java

@@ -23,7 +23,7 @@
 /**
  * A representation of a service node, ie JVM
  */
-public class Node {
+public class Node implements co.elastic.apm.agent.tracer.service.Node {
 
     /**
      * (Optional)
@@ -37,6 +37,7 @@ public Node(@Nullable String name) {
     }
 
     @Nullable
+    @Override
     public String getName() {
         return name;
     }

apm-agent-core/src/main/java/co/elastic/apm/agent/impl/metadata/Service.java

@@ -25,7 +25,7 @@
 /**
  * Information about the instrumented Service
  */
-public class Service {
+public class Service implements co.elastic.apm.agent.tracer.service.Service {
 
     /**
      * Name and version of the Elastic APM agent
@@ -127,10 +127,8 @@ public Service withLanguage(Language language) {
         return this;
     }
 
-    /**
-     * Representation of a service node
-     */
     @Nullable
+    @Override
     public Node getNode() {
         return node;
     }
@@ -143,11 +141,8 @@ public Service withNode(Node node) {
         return this;
     }
 
-    /**
-     * Immutable name of the service emitting this event
-     * (Required)
-     */
     @Nullable
+    @Override
     public String getName() {
         return name;
     }
@@ -161,10 +156,8 @@ public Service withName(String name) {
         return this;
     }
 
-    /**
-     * Environment name of the service, e.g. "production" or "staging"
-     */
     @Nullable
+    @Override
     public String getEnvironment() {
         return environment;
     }
@@ -193,10 +186,8 @@ public Service withRuntime(RuntimeInfo runtime) {
         return this;
     }
 
-    /**
-     * Version of the service emitting this event
-     */
     @Nullable
+    @Override
     public String getVersion() {
         return version;
     }

apm-agent-plugins/apm-ecs-logging-plugin/pom.xml

@@ -16,13 +16,6 @@
     </properties>
 
     <dependencies>
-        <!-- This plugin requires further refactoring to avoid agent dependencies. -->
-        <dependency>
-            <groupId>${project.groupId}</groupId>
-            <artifactId>apm-agent-core</artifactId>
-            <version>${project.version}</version>
-        </dependency>
-
         <!-- log4j 1.x -->
         <dependency>
             <groupId>co.elastic.logging</groupId>

apm-agent-plugins/apm-logging-plugin/apm-jul-plugin/src/main/java/co/elastic/apm/agent/jul/error/JulLoggerErrorCapturingInstrumentation.java

@@ -18,7 +18,6 @@
  */
 package co.elastic.apm.agent.jul.error;
 
-import co.elastic.apm.agent.impl.Tracer;
 import co.elastic.apm.agent.loginstr.error.AbstractLoggerErrorCapturingInstrumentation;
 import co.elastic.apm.agent.loginstr.error.LoggerErrorHelper;
 import net.bytebuddy.asm.Advice;

apm-agent-plugins/apm-logging-plugin/apm-jul-plugin/src/main/java/co/elastic/apm/agent/jul/reformatting/AbstractJulEcsReformattingHelper.java

@@ -21,10 +21,10 @@
 import co.elastic.apm.agent.jul.sending.JulLogSenderHandler;
 import co.elastic.apm.agent.loginstr.correlation.CorrelationIdMapAdapter;
 import co.elastic.apm.agent.loginstr.reformatting.AbstractEcsReformattingHelper;
-import co.elastic.apm.agent.report.Reporter;
 import co.elastic.apm.agent.sdk.logging.Logger;
 import co.elastic.apm.agent.sdk.logging.LoggerFactory;
 import co.elastic.apm.agent.sdk.internal.util.LoggerUtils;
+import co.elastic.apm.agent.tracer.Tracer;
 import co.elastic.logging.AdditionalField;
 import co.elastic.logging.jul.EcsFormatter;
 
@@ -151,8 +151,8 @@ private static FileHandler doCreateFileHandler(String pattern, int maxLogFileSiz
     protected abstract boolean isFileHandler(Handler originalHandler);
 
     @Override
-    protected T createAndStartLogSendingAppender(Reporter reporter, Formatter formatter) {
-        return (T) new JulLogSenderHandler(reporter, formatter);
+    protected T createAndStartLogSendingAppender(Tracer tracer, Formatter formatter) {
+        return (T) new JulLogSenderHandler(tracer, formatter);
     }
 
     @Override

apm-agent-plugins/apm-logging-plugin/apm-jul-plugin/src/main/java/co/elastic/apm/agent/jul/sending/JulLogSenderHandler.java

@@ -18,24 +18,24 @@
  */
 package co.elastic.apm.agent.jul.sending;
 
-import co.elastic.apm.agent.report.Reporter;
+import co.elastic.apm.agent.tracer.Tracer;
 
 import java.util.logging.Formatter;
 import java.util.logging.Handler;
 import java.util.logging.LogRecord;
 
 public class JulLogSenderHandler extends Handler {
-    private final Reporter reporter;
+    private final Tracer tracer;
     private final Formatter formatter;
 
-    public JulLogSenderHandler(Reporter reporter, Formatter formatter) {
-        this.reporter = reporter;
+    public JulLogSenderHandler(Tracer tracer, Formatter formatter) {
+        this.tracer = tracer;
         this.formatter = formatter;
     }
 
     @Override
     public void publish(LogRecord record) {
-        reporter.reportLog(formatter.format(record));
+        tracer.reportLog(formatter.format(record));
     }
 
     @Override

apm-agent-plugins/apm-logging-plugin/apm-log4j1-plugin/src/main/java/co/elastic/apm/agent/log4j1/reformatting/Log4J1EcsReformattingHelper.java

@@ -22,9 +22,9 @@
 import co.elastic.apm.agent.loginstr.reformatting.AbstractEcsReformattingHelper;
 import co.elastic.apm.agent.loginstr.reformatting.Utils;
 
-import co.elastic.apm.agent.report.Reporter;
 import co.elastic.apm.agent.sdk.logging.Logger;
 import co.elastic.apm.agent.sdk.logging.LoggerFactory;
+import co.elastic.apm.agent.tracer.Tracer;
 import co.elastic.logging.log4j.EcsLayout;
 import org.apache.log4j.Appender;
 import org.apache.log4j.FileAppender;
@@ -112,8 +112,8 @@ protected void closeShadeAppender(WriterAppender shadeAppender) {
     }
 
     @Override
-    protected Appender createAndStartLogSendingAppender(Reporter reporter, Layout formatter) {
-        return new LogSenderAppender(reporter, formatter);
+    protected Appender createAndStartLogSendingAppender(Tracer tracer, Layout formatter) {
+        return new LogSenderAppender(tracer, formatter);
     }
 
     @Override

apm-agent-plugins/apm-logging-plugin/apm-log4j1-plugin/src/main/java/co/elastic/apm/agent/log4j1/sending/LogSenderAppender.java

@@ -18,17 +18,17 @@
  */
 package co.elastic.apm.agent.log4j1.sending;
 
-import co.elastic.apm.agent.report.Reporter;
+import co.elastic.apm.agent.tracer.Tracer;
 import org.apache.log4j.AppenderSkeleton;
 import org.apache.log4j.Layout;
 import org.apache.log4j.spi.LoggingEvent;
 
 public class LogSenderAppender extends AppenderSkeleton {
-    private final Reporter reporter;
+    private final Tracer tracer;
     private final Layout formatter;
 
-    public LogSenderAppender(Reporter reporter, Layout formatter) {
-        this.reporter = reporter;
+    public LogSenderAppender(Tracer tracer, Layout formatter) {
+        this.tracer = tracer;
         this.formatter = formatter;
     }
 
@@ -39,7 +39,7 @@ public synchronized void doAppend(LoggingEvent event) {
 
     @Override
     protected void append(LoggingEvent event) {
-        reporter.reportLog(formatter.format(event));
+        tracer.reportLog(formatter.format(event));
     }
 
     @Override

apm-agent-plugins/apm-logging-plugin/apm-log4j2-plugin/src/main/java/co/elastic/apm/agent/log4j2/correlation/Log4j2_7PlusLogCorrelationHelper.java

@@ -19,7 +19,6 @@
 package co.elastic.apm.agent.log4j2.correlation;
 
 import co.elastic.apm.agent.tracer.GlobalTracer;
-import co.elastic.apm.agent.impl.error.ErrorCapture;
 import co.elastic.apm.agent.loginstr.correlation.AbstractLogCorrelationHelper;
 import co.elastic.apm.agent.loginstr.correlation.CorrelationIdMapAdapter;
 import co.elastic.apm.agent.tracer.Tracer;