PR review changes

component/otelcol/extension/jaeger_remote_sampling/internal/jaegerremotesampling/extension.go

@@ -73,7 +73,7 @@ func (jrse *jrsExtension) Start(ctx context.Context, host component.Host) error
 	}
 
 	if jrse.cfg.Source.Remote != nil {
-		conn, err := jrse.cfg.Source.Remote.ToClientConn(context.Background(), host, jrse.telemetry)
+		conn, err := jrse.cfg.Source.Remote.ToClientConn(ctx, host, jrse.telemetry)
 		if err != nil {
 			return fmt.Errorf("error while connecting to the remote sampling source: %w", err)
 		}

component/otelcol/receiver/loki/loki.go

@@ -124,6 +124,8 @@ func convertLokiEntryToPlog(lokiEntry loki.Entry) plog.Logs {
 		// transformation happens.
 		lr.Attributes().PutStr("log.file.path", filenameStr)
 		lr.Attributes().PutStr("log.file.name", path.Base(filenameStr))
+		// TODO(@tpaschalis) Remove the addition of "log.file.path" and "log.file.name",
+		// because the Collector doesn't do it and we would be more in line with it.
 	}
 
 	var lbls []string

pkg/traces/instance.go

@@ -167,7 +167,7 @@ func (i *Instance) buildAndStartPipeline(ctx context.Context, cfg InstanceConfig
 		Extensions:               extension.NewBuilder(otelConfig.Extensions, factories.Extensions),
 		OtelMetricViews:          servicegraphprocessor.OtelMetricViews(),
 		OtelMetricReader:         *promExporter,
-		UseExternalMetricsServer: false,
+		UseExternalMetricsServer: true,
 		TracerProvider:           trace.NewNoopTracerProvider(),
 		//TODO: Plug in an AsyncErrorChannel to shut down the Agent in case of a fatal event
 		LoggingOptions: []zap.Option{