diff --git a/examples/autoload_sdk.php b/examples/autoload_sdk.php index 9043bca29..a114b70ec 100644 --- a/examples/autoload_sdk.php +++ b/examples/autoload_sdk.php @@ -4,12 +4,12 @@ namespace OpenTelemetry\Example; +use OpenTelemetry\API\Logs\LogRecord; + putenv('OTEL_PHP_AUTOLOAD_ENABLED=true'); -putenv('OTEL_TRACES_EXPORTER=otlp'); -putenv('OTEL_METRICS_EXPORTER=otlp'); -putenv('OTEL_LOGS_EXPORTER=otlp'); -putenv('OTEL_EXPORTER_OTLP_PROTOCOL=grpc'); -putenv('OTEL_EXPORTER_OTLP_ENDPOINT=http://collector:4317'); +putenv('OTEL_TRACES_EXPORTER=console'); +putenv('OTEL_METRICS_EXPORTER=console'); +putenv('OTEL_LOGS_EXPORTER=console'); putenv('OTEL_PROPAGATORS=b3,baggage,tracecontext'); echo 'autoloading SDK example starting...' . PHP_EOL; @@ -21,6 +21,6 @@ $instrumentation->tracer()->spanBuilder('root')->startSpan()->end(); $instrumentation->meter()->createCounter('cnt')->add(1); -$instrumentation->eventLogger()->emit('foo', 'hello, otel'); +$instrumentation->logger()->emit((new LogRecord('hello, otel'))->setEventName('foo')); echo 'Finished!' . PHP_EOL; diff --git a/examples/load_config.php b/examples/load_config.php index 874159bc3..c087e3853 100644 --- a/examples/load_config.php +++ b/examples/load_config.php @@ -2,6 +2,7 @@ declare(strict_types=1); +use OpenTelemetry\API\Logs\LogRecord; use OpenTelemetry\Config\SDK\Configuration; require __DIR__ . '/../vendor/autoload.php'; @@ -16,13 +17,13 @@ $tracer = $sdk->getTracerProvider()->getTracer('demo'); $meter = $sdk->getMeterProvider()->getMeter('demo'); -$eventLogger = $sdk->getEventLoggerProvider()->getEventLogger('demo'); +$logger = $sdk->getLoggerProvider()->getLogger('demo'); $span = $tracer->spanBuilder('root')->startSpan(); $scope = $span->activate(); $meter->createCounter('cnt')->add(1); -$eventLogger->emit('foo', 'hello, otel'); +$logger->emit((new LogRecord('hello, otel'))->setEventName('foo')); $scope->detach(); $span->end(); diff --git a/examples/load_config_env.php b/examples/load_config_env.php index 1c90d2683..cdbe85836 100644 --- a/examples/load_config_env.php +++ b/examples/load_config_env.php @@ -2,6 +2,7 @@ declare(strict_types=1); +use OpenTelemetry\API\Logs\LogRecord; use OpenTelemetry\Config\SDK\Configuration; require __DIR__ . '/../vendor/autoload.php'; @@ -21,10 +22,10 @@ $tracer = $sdk->getTracerProvider()->getTracer('demo'); $meter = $sdk->getMeterProvider()->getMeter('demo'); -$eventLogger = $sdk->getEventLoggerProvider()->getEventLogger('demo'); +$logger = $sdk->getLoggerProvider()->getLogger('demo'); $tracer->spanBuilder('root')->startSpan()->end(); $meter->createCounter('cnt')->add(1); -$eventLogger->emit('foo', 'hello, otel'); +$logger->emit((new LogRecord('hello, otel'))->setEventName('foo')); echo 'Finished!' . PHP_EOL; diff --git a/examples/logs/exporters/otlp_grpc.php b/examples/logs/exporters/otlp_grpc.php index 8fa7dd7b4..3baf6db5d 100644 --- a/examples/logs/exporters/otlp_grpc.php +++ b/examples/logs/exporters/otlp_grpc.php @@ -5,13 +5,13 @@ namespace OpenTelemetry\Example; use OpenTelemetry\API\Common\Time\Clock; +use OpenTelemetry\API\Logs\LogRecord; use OpenTelemetry\API\Logs\Severity; use OpenTelemetry\API\Signals; use OpenTelemetry\Contrib\Grpc\GrpcTransportFactory; use OpenTelemetry\Contrib\Otlp\LogsExporter; use OpenTelemetry\Contrib\Otlp\OtlpUtil; use OpenTelemetry\SDK\Common\Instrumentation\InstrumentationScopeFactory; -use OpenTelemetry\SDK\Logs\EventLoggerProvider; use OpenTelemetry\SDK\Logs\LoggerProvider; use OpenTelemetry\SDK\Logs\LogRecordLimitsBuilder; use OpenTelemetry\SDK\Logs\Processor\BatchLogRecordProcessor; @@ -29,18 +29,9 @@ (new LogRecordLimitsBuilder())->build()->getAttributeFactory() ) ); -$eventLoggerProvider = new EventLoggerProvider($loggerProvider); -$eventLogger = $eventLoggerProvider->getEventLogger('demo', '1.0', 'http://schema.url', ['foo' => 'bar']); +$logger = $loggerProvider->getLogger('demo', '1.0', 'http://schema.url', ['foo' => 'bar']); -$eventLogger->emit( - name: 'foo', - body: ['foo' => 'bar', 'baz' => 'bat', 'msg' => 'hello world'], - severityNumber: Severity::INFO -); - -$eventLogger->emit( - 'bar', - 'otel is great' -); +$logger->emit((new LogRecord(['foo' => 'bar', 'baz' => 'bat', 'msg' => 'hello world']))->setEventName('foo')->setSeverityNumber(Severity::INFO)); +$logger->emit((new LogRecord('otel is great'))->setEventName('bar')); $loggerProvider->shutdown(); diff --git a/examples/logs/exporters/otlp_http.php b/examples/logs/exporters/otlp_http.php index 870b967d5..57933cf0c 100644 --- a/examples/logs/exporters/otlp_http.php +++ b/examples/logs/exporters/otlp_http.php @@ -9,7 +9,6 @@ use OpenTelemetry\Contrib\Otlp\LogsExporter; use OpenTelemetry\SDK\Common\Export\Http\PsrTransportFactory; use OpenTelemetry\SDK\Common\Instrumentation\InstrumentationScopeFactory; -use OpenTelemetry\SDK\Logs\EventLoggerProvider; use OpenTelemetry\SDK\Logs\LoggerProvider; use OpenTelemetry\SDK\Logs\LogRecordLimitsBuilder; use OpenTelemetry\SDK\Logs\Processor\SimpleLogRecordProcessor; @@ -27,12 +26,10 @@ (new LogRecordLimitsBuilder())->build()->getAttributeFactory() ) ); -$eventLoggerProvider = new EventLoggerProvider($loggerProvider); -$eventLogger = $eventLoggerProvider->getEventLogger('demo', '1.0', 'https://opentelemetry.io/schemas/1.7.1', ['foo' => 'bar']); - -$eventLogger->emit( - name: 'foo', - body: ['foo' => 'bar', 'baz' => 'bat', 'msg' => 'hello world'], - timestamp: (new \DateTime())->getTimestamp() * LogRecord::NANOS_PER_SECOND, - severityNumber: Severity::INFO, +$logger = $loggerProvider->getLogger('demo', '1.0', 'https://opentelemetry.io/schemas/1.7.1', ['foo' => 'bar']); +$logger->emit( + (new LogRecord(['foo' => 'bar', 'baz' => 'bat', 'msg' => 'hello world'])) + ->setEventName('foo') + ->setSeverityNumber(Severity::INFO) + ->setTimestamp((new \DateTime())->getTimestamp() * LogRecord::NANOS_PER_SECOND) ); diff --git a/examples/logs/features/batch_exporting.php b/examples/logs/features/batch_exporting.php index 732eb22e3..c52f4cbb6 100644 --- a/examples/logs/features/batch_exporting.php +++ b/examples/logs/features/batch_exporting.php @@ -5,11 +5,10 @@ namespace OpenTelemetry\Example; use OpenTelemetry\API\Common\Time\Clock; +use OpenTelemetry\API\Logs\LogRecord; use OpenTelemetry\API\Logs\Severity; use OpenTelemetry\SDK\Common\Attribute\Attributes; use OpenTelemetry\SDK\Common\Instrumentation\InstrumentationScopeFactory; -use OpenTelemetry\SDK\Logs\EventLogger; -use OpenTelemetry\SDK\Logs\EventLoggerProvider; use OpenTelemetry\SDK\Logs\Exporter\ConsoleExporterFactory; use OpenTelemetry\SDK\Logs\LoggerProvider; use OpenTelemetry\SDK\Logs\Processor\BatchLogRecordProcessor; @@ -23,16 +22,15 @@ ), new InstrumentationScopeFactory(Attributes::factory()) ); -$eventLoggerProvider = new EventLoggerProvider($loggerProvider); -//get a logger, and emit a log record from an EventLogger. -$eventLoggerOne = $eventLoggerProvider->getEventLogger('demo', '1.0'); -$eventLoggerTwo = $eventLoggerProvider->getEventLogger('demo', '2.0'); +//get a logger, and emit an event. +$loggerOne = $loggerProvider->getLogger('demo', '1.0'); +$loggerTwo = $loggerProvider->getLogger('demo', '2.0'); $payload = ['foo' => 'bar', 'baz' => 'bat', 'msg' => 'hello world']; -$eventLoggerOne->emit(name: 'foo', body: $payload, severityNumber: Severity::INFO); -$eventLoggerOne->emit('bar', 'hello world'); -$eventLoggerTwo->emit(name: 'foo', body: $payload, severityNumber: Severity::INFO); +$loggerOne->emit((new LogRecord($payload))->setEventName('foo')->setSeverityNumber(Severity::INFO)); +$loggerOne->emit((new LogRecord('hello world'))->setEventName('bar')); +$loggerTwo->emit((new LogRecord($payload))->setEventName('foo')->setSeverityNumber(Severity::INFO)); //shut down logger provider $loggerProvider->shutdown(); diff --git a/examples/logs/getting_started.php b/examples/logs/getting_started.php index af8725411..3eebe8044 100644 --- a/examples/logs/getting_started.php +++ b/examples/logs/getting_started.php @@ -2,10 +2,10 @@ declare(strict_types=1); +use OpenTelemetry\API\Logs\LogRecord; use OpenTelemetry\API\Logs\Severity; use OpenTelemetry\SDK\Common\Attribute\Attributes; use OpenTelemetry\SDK\Common\Instrumentation\InstrumentationScopeFactory; -use OpenTelemetry\SDK\Logs\EventLoggerProvider; use OpenTelemetry\SDK\Logs\Exporter\ConsoleExporterFactory; use OpenTelemetry\SDK\Logs\LoggerProvider; use OpenTelemetry\SDK\Logs\Processor\SimpleLogRecordProcessor; @@ -26,7 +26,6 @@ ), new InstrumentationScopeFactory(Attributes::factory()) ); -$eventLoggerProvider = new EventLoggerProvider($loggerProvider); $tracerProvider = new TracerProvider(); $tracer = $tracerProvider->getTracer('demo-tracer'); @@ -36,12 +35,12 @@ echo 'Trace id: ' . $span->getContext()->getTraceId() . PHP_EOL; echo 'Span id: ' . $span->getContext()->getSpanId() . PHP_EOL; -//get an event logger, and emit an event. The active context (trace id + span id) will be +//get a logger, and emit an event. The active context (trace id + span id) will be //attached to the log record -$eventLogger = $eventLoggerProvider->getEventLogger('demo', '1.0', 'http://schema.url', ['foo' => 'bar']); +$logger = $loggerProvider->getLogger('demo', '1.0', 'http://schema.url', ['foo' => 'bar']); $payload = ['foo' => 'bar', 'baz' => 'bat', 'msg' => 'hello world']; -$eventLogger->emit(name: 'foo', body: $payload, severityNumber: Severity::INFO); +$logger->emit((new LogRecord($payload))->setEventName('foo')->setSeverityNumber(Severity::INFO)); //end span $span->end(); diff --git a/examples/src/_register.php b/examples/src/_register.php index 41a0c3c39..4df640cd9 100644 --- a/examples/src/_register.php +++ b/examples/src/_register.php @@ -3,8 +3,8 @@ declare(strict_types=1); use Nevay\SPI\ServiceLoader; -use OpenTelemetry\API\Instrumentation\AutoInstrumentation\Instrumentation; use OpenTelemetry\API\Configuration\Config\ComponentProvider; +use OpenTelemetry\API\Instrumentation\AutoInstrumentation\Instrumentation; use OpenTelemetry\Example\ExampleConfigProvider; use OpenTelemetry\Example\ExampleInstrumentation; use OpenTelemetry\Example\TestResourceDetectorFactory; diff --git a/examples/traces/exporters/otlp_file_autoload.php b/examples/traces/exporters/otlp_file_autoload.php index 5c1342541..b26305cb8 100644 --- a/examples/traces/exporters/otlp_file_autoload.php +++ b/examples/traces/exporters/otlp_file_autoload.php @@ -5,6 +5,7 @@ namespace OpenTelemetry\Example; use OpenTelemetry\API\Instrumentation\CachedInstrumentation; +use OpenTelemetry\API\Logs\LogRecord; putenv('OTEL_PHP_AUTOLOAD_ENABLED=true'); putenv('OTEL_TRACES_EXPORTER=otlp/stdout'); @@ -17,7 +18,7 @@ $instrumentation->tracer()->spanBuilder('root')->startSpan()->end(); $instrumentation->meter()->createCounter('cnt')->add(1); -$instrumentation->eventLogger()->emit('foo', 'hello, otel'); +$instrumentation->logger()->emit(new LogRecord('foo')); echo PHP_EOL . 'OTLP/stdout autoload example complete!'; echo PHP_EOL; diff --git a/phpstan.neon.dist b/phpstan.neon.dist index bb7cc4feb..64b8d19b7 100644 --- a/phpstan.neon.dist +++ b/phpstan.neon.dist @@ -51,10 +51,6 @@ parameters: message: "#.*return with type T is not subtype.*#" paths: - src/SDK/Common/InstrumentationScope - - - message: "#.*assertCount\\(\\) expects Countable\\|iterable.*#" - paths: - - tests/Unit/Contrib/Otlp/ - message: "#.*expects Google\\\\Protobuf\\\\RepeatedField.*#" paths: diff --git a/src/API/Globals.php b/src/API/Globals.php index f7f6e0e81..5d8d7a0ef 100644 --- a/src/API/Globals.php +++ b/src/API/Globals.php @@ -9,7 +9,6 @@ use OpenTelemetry\API\Behavior\LogsMessagesTrait; use OpenTelemetry\API\Instrumentation\Configurator; use OpenTelemetry\API\Instrumentation\ContextKeys; -use OpenTelemetry\API\Logs\EventLoggerProviderInterface; use OpenTelemetry\API\Logs\LoggerProviderInterface; use OpenTelemetry\API\Metrics\MeterProviderInterface; use OpenTelemetry\API\Trace\TracerProviderInterface; @@ -33,7 +32,6 @@ public function __construct( private readonly TracerProviderInterface $tracerProvider, private readonly MeterProviderInterface $meterProvider, private readonly LoggerProviderInterface $loggerProvider, - private readonly EventLoggerProviderInterface $eventLoggerProvider, private readonly TextMapPropagatorInterface $propagator, ) { } @@ -58,15 +56,6 @@ public static function loggerProvider(): LoggerProviderInterface return Context::getCurrent()->get(ContextKeys::loggerProvider()) ?? self::globals()->loggerProvider; } - /** - * @deprecated - * @phan-suppress PhanDeprecatedFunction - */ - public static function eventLoggerProvider(): EventLoggerProviderInterface - { - return Context::getCurrent()->get(ContextKeys::eventLoggerProvider()) ?? self::globals()->eventLoggerProvider; - } - /** * @param Closure(Configurator): Configurator $initializer * @@ -108,11 +97,10 @@ private static function globals(): self $meterProvider = $context->get(ContextKeys::meterProvider()); $propagator = $context->get(ContextKeys::propagator()); $loggerProvider = $context->get(ContextKeys::loggerProvider()); - $eventLoggerProvider = $context->get(ContextKeys::eventLoggerProvider()); - assert(isset($tracerProvider, $meterProvider, $loggerProvider, $eventLoggerProvider, $propagator)); + assert(isset($tracerProvider, $meterProvider, $loggerProvider, $propagator)); - return self::$globals = new self($tracerProvider, $meterProvider, $loggerProvider, $eventLoggerProvider, $propagator); + return self::$globals = new self($tracerProvider, $meterProvider, $loggerProvider, $propagator); } /** diff --git a/src/API/Instrumentation/CachedInstrumentation.php b/src/API/Instrumentation/CachedInstrumentation.php index 27b755b2f..deb0f58f2 100644 --- a/src/API/Instrumentation/CachedInstrumentation.php +++ b/src/API/Instrumentation/CachedInstrumentation.php @@ -5,8 +5,6 @@ namespace OpenTelemetry\API\Instrumentation; use OpenTelemetry\API\Globals; -use OpenTelemetry\API\Logs\EventLoggerInterface; -use OpenTelemetry\API\Logs\EventLoggerProviderInterface; use OpenTelemetry\API\Logs\LoggerInterface; use OpenTelemetry\API\Logs\LoggerProviderInterface; use OpenTelemetry\API\Metrics\MeterInterface; @@ -31,8 +29,6 @@ final class CachedInstrumentation private WeakMap $meters; /** @var WeakMap */ private WeakMap $loggers; - /** @var WeakMap */ - private WeakMap $eventLoggers; /** * @psalm-suppress PropertyTypeCoercion @@ -46,7 +42,6 @@ public function __construct( $this->tracers = new \WeakMap(); $this->meters = new \WeakMap(); $this->loggers = new \WeakMap(); - $this->eventLoggers = new \WeakMap(); } public function tracer(): TracerInterface @@ -68,15 +63,4 @@ public function logger(): LoggerInterface return $this->loggers[$loggerProvider] ??= $loggerProvider->getLogger($this->name, $this->version, $this->schemaUrl, $this->attributes); } - - /** - * @deprecated - * @phan-suppress PhanDeprecatedFunction - */ - public function eventLogger(): EventLoggerInterface - { - $eventLoggerProvider = Globals::eventLoggerProvider(); - - return $this->eventLoggers[$eventLoggerProvider] ??= $eventLoggerProvider->getEventLogger($this->name, $this->version, $this->schemaUrl, $this->attributes); - } } diff --git a/src/API/Instrumentation/Configurator.php b/src/API/Instrumentation/Configurator.php index 3ece0ace1..ccdcb8ea1 100644 --- a/src/API/Instrumentation/Configurator.php +++ b/src/API/Instrumentation/Configurator.php @@ -4,9 +4,7 @@ namespace OpenTelemetry\API\Instrumentation; -use OpenTelemetry\API\Logs\EventLoggerProviderInterface; use OpenTelemetry\API\Logs\LoggerProviderInterface; -use OpenTelemetry\API\Logs\NoopEventLoggerProvider; use OpenTelemetry\API\Logs\NoopLoggerProvider; use OpenTelemetry\API\Metrics\MeterProviderInterface; use OpenTelemetry\API\Metrics\Noop\NoopMeterProvider; @@ -30,7 +28,6 @@ final class Configurator implements ImplicitContextKeyedInterface private ?MeterProviderInterface $meterProvider = null; private ?TextMapPropagatorInterface $propagator = null; private ?LoggerProviderInterface $loggerProvider = null; - private ?EventLoggerProviderInterface $eventLoggerProvider = null; private function __construct() { @@ -55,7 +52,6 @@ public static function createNoop(): Configurator ->withMeterProvider(new NoopMeterProvider()) ->withPropagator(new NoopTextMapPropagator()) ->withLoggerProvider(NoopLoggerProvider::getInstance()) - ->withEventLoggerProvider(new NoopEventLoggerProvider()) ; } @@ -83,9 +79,6 @@ public function storeInContext(?ContextInterface $context = null): ContextInterf if ($this->loggerProvider !== null) { $context = $context->with(ContextKeys::loggerProvider(), $this->loggerProvider); } - if ($this->eventLoggerProvider !== null) { - $context = $context->with(ContextKeys::eventLoggerProvider(), $this->eventLoggerProvider); - } return $context; } @@ -121,15 +114,4 @@ public function withLoggerProvider(?LoggerProviderInterface $loggerProvider): Co return $self; } - - /** - * @deprecated - */ - public function withEventLoggerProvider(?EventLoggerProviderInterface $eventLoggerProvider): Configurator - { - $self = clone $this; - $self->eventLoggerProvider = $eventLoggerProvider; - - return $self; - } } diff --git a/src/API/Instrumentation/ContextKeys.php b/src/API/Instrumentation/ContextKeys.php index 7e69f95d6..ea1a66416 100644 --- a/src/API/Instrumentation/ContextKeys.php +++ b/src/API/Instrumentation/ContextKeys.php @@ -4,7 +4,6 @@ namespace OpenTelemetry\API\Instrumentation; -use OpenTelemetry\API\Logs\EventLoggerProviderInterface; use OpenTelemetry\API\Logs\LoggerProviderInterface; use OpenTelemetry\API\Metrics\MeterProviderInterface; use OpenTelemetry\API\Trace\TracerProviderInterface; @@ -56,15 +55,4 @@ public static function loggerProvider(): ContextKeyInterface return $instance ??= Context::createKey(LoggerProviderInterface::class); } - - /** - * @deprecated - * @return ContextKeyInterface - */ - public static function eventLoggerProvider(): ContextKeyInterface - { - static $instance; - - return $instance ??= Context::createKey(EventLoggerProviderInterface::class); - } } diff --git a/src/API/Logs/EventLoggerInterface.php b/src/API/Logs/EventLoggerInterface.php deleted file mode 100644 index f7c0890e9..000000000 --- a/src/API/Logs/EventLoggerInterface.php +++ /dev/null @@ -1,23 +0,0 @@ - $sdkBuilder->setTracerProvider($tracerProvider); $sdkBuilder->setMeterProvider($meterProvider); $sdkBuilder->setLoggerProvider($loggerProvider); - $sdkBuilder->setEventLoggerProvider($eventLoggerProvider); return $sdkBuilder; } diff --git a/src/Config/SDK/ComponentProvider/Propagator/TextMapPropagatorCloudTrace.php b/src/Config/SDK/ComponentProvider/Propagator/TextMapPropagatorCloudTrace.php index 5e69b10f3..b4a06db04 100644 --- a/src/Config/SDK/ComponentProvider/Propagator/TextMapPropagatorCloudTrace.php +++ b/src/Config/SDK/ComponentProvider/Propagator/TextMapPropagatorCloudTrace.php @@ -5,9 +5,9 @@ namespace OpenTelemetry\Config\SDK\ComponentProvider\Propagator; use Nevay\SPI\ServiceProviderDependency\PackageDependency; -use OpenTelemetry\Config\SDK\Configuration\ComponentProvider; -use OpenTelemetry\Config\SDK\Configuration\ComponentProviderRegistry; -use OpenTelemetry\Config\SDK\Configuration\Context; +use OpenTelemetry\API\Configuration\Config\ComponentProvider; +use OpenTelemetry\API\Configuration\Config\ComponentProviderRegistry; +use OpenTelemetry\API\Configuration\Context; use OpenTelemetry\Context\Propagation\TextMapPropagatorInterface; use OpenTelemetry\Extension\Propagator\CloudTrace\CloudTracePropagator; use Symfony\Component\Config\Definition\Builder\ArrayNodeDefinition; diff --git a/src/Config/SDK/_register.php b/src/Config/SDK/_register.php index 1971e75d2..98d625039 100644 --- a/src/Config/SDK/_register.php +++ b/src/Config/SDK/_register.php @@ -3,6 +3,7 @@ declare(strict_types=1); use Nevay\SPI\ServiceLoader; +use OpenTelemetry\API\Configuration\Config\ComponentProvider; use OpenTelemetry\Config\SDK\ComponentProvider\Instrumentation\General\HttpConfigProvider; use OpenTelemetry\Config\SDK\ComponentProvider\Instrumentation\General\PeerConfigProvider; use OpenTelemetry\Config\SDK\ComponentProvider\Logs\LogRecordExporterConsole; @@ -35,7 +36,6 @@ use OpenTelemetry\Config\SDK\ComponentProvider\Trace\SpanExporterZipkin; use OpenTelemetry\Config\SDK\ComponentProvider\Trace\SpanProcessorBatch; use OpenTelemetry\Config\SDK\ComponentProvider\Trace\SpanProcessorSimple; -use OpenTelemetry\API\Configuration\Config\ComponentProvider; ServiceLoader::register(ComponentProvider::class, TextMapPropagatorB3::class); ServiceLoader::register(ComponentProvider::class, TextMapPropagatorB3Multi::class); diff --git a/src/Contrib/Otlp/LogsConverter.php b/src/Contrib/Otlp/LogsConverter.php index 3cadf366f..80e0a00f2 100644 --- a/src/Contrib/Otlp/LogsConverter.php +++ b/src/Contrib/Otlp/LogsConverter.php @@ -78,6 +78,10 @@ public function convert(iterable $logs): ExportLogsServiceRequest private function convertLogRecord(ReadableLogRecord $record): LogRecord { $pLogRecord = new LogRecord(); + $eventName = $record->getEventName(); + if ($eventName !== null) { + $pLogRecord->setEventName($eventName); + } $pLogRecord->setBody(AttributesConverter::convertAnyValue($record->getBody())); $pLogRecord->setTimeUnixNano($record->getTimestamp() ?? 0); $pLogRecord->setObservedTimeUnixNano($record->getObservedTimestamp() ?? 0); diff --git a/src/Contrib/Otlp/composer.json b/src/Contrib/Otlp/composer.json index be792c76b..f502e48ea 100644 --- a/src/Contrib/Otlp/composer.json +++ b/src/Contrib/Otlp/composer.json @@ -19,7 +19,7 @@ "require": { "php": "^8.2", "php-http/discovery": "^1.14", - "open-telemetry/gen-otlp-protobuf": "^1.1", + "open-telemetry/gen-otlp-protobuf": "^1.5", "open-telemetry/api": "^1.0|^2.0", "open-telemetry/sdk": "^1.0|^2.0", "tbachert/spi": "^1.0.1" diff --git a/src/SDK/Logs/EventLogger.php b/src/SDK/Logs/EventLogger.php deleted file mode 100644 index 1e65c4d63..000000000 --- a/src/SDK/Logs/EventLogger.php +++ /dev/null @@ -1,52 +0,0 @@ -setAttribute('event.name', $name); - $logRecord->setAttributes($attributes); - $logRecord->setAttribute('event.name', $name); - $logRecord->setBody($body); - $logRecord->setTimestamp($timestamp ?? $this->clock->now()); - $logRecord->setContext($context ?? Context::getCurrent()); - $logRecord->setSeverityNumber($severityNumber ?? Severity::INFO); - - $this->logger->emit($logRecord); - } -} diff --git a/src/SDK/Logs/EventLoggerProvider.php b/src/SDK/Logs/EventLoggerProvider.php deleted file mode 100644 index fa7cdc1e4..000000000 --- a/src/SDK/Logs/EventLoggerProvider.php +++ /dev/null @@ -1,34 +0,0 @@ -loggerProvider->getLogger($name, $version, $schemaUrl, $attributes), - Clock::getDefault(), - ); - } - - public function forceFlush(): bool - { - return $this->loggerProvider->forceFlush(); - } -} diff --git a/src/SDK/Logs/EventLoggerProviderFactory.php b/src/SDK/Logs/EventLoggerProviderFactory.php deleted file mode 100644 index fbb0b5fda..000000000 --- a/src/SDK/Logs/EventLoggerProviderFactory.php +++ /dev/null @@ -1,22 +0,0 @@ - $record->getSeverityNumber(), 'severity_text' => $record->getSeverityText(), 'body' => $record->getBody(), + 'event_name' => $record->getEventName(), 'trace_id' => $spanContext !== null ? $spanContext->getTraceId() : '', 'span_id' => $spanContext !== null ? $spanContext->getSpanId() : '', 'trace_flags' => $spanContext !== null ? $spanContext->getTraceFlags() : null, diff --git a/src/SDK/Logs/NoopEventLoggerProvider.php b/src/SDK/Logs/NoopEventLoggerProvider.php deleted file mode 100644 index a7edd0d6a..000000000 --- a/src/SDK/Logs/NoopEventLoggerProvider.php +++ /dev/null @@ -1,25 +0,0 @@ -loggerProvider; } - /** - * @deprecated - */ - public function getEventLoggerProvider(): EventLoggerProviderInterface - { - return $this->eventLoggerProvider; - } - public function getPropagator(): TextMapPropagatorInterface { return $this->propagator; diff --git a/src/SDK/SdkAutoloader.php b/src/SDK/SdkAutoloader.php index fddcfc451..c30af65aa 100644 --- a/src/SDK/SdkAutoloader.php +++ b/src/SDK/SdkAutoloader.php @@ -34,7 +34,6 @@ use OpenTelemetry\SDK\Common\Configuration\EnvResolver; use OpenTelemetry\SDK\Common\Configuration\Variables; use OpenTelemetry\SDK\Common\Util\ShutdownHandler; -use OpenTelemetry\SDK\Logs\EventLoggerProviderFactory; use OpenTelemetry\SDK\Logs\LoggerProviderFactory; use OpenTelemetry\SDK\Metrics\MeterProviderFactory; use OpenTelemetry\SDK\Propagation\LateBindingTextMapPropagator; @@ -104,7 +103,6 @@ private static function environmentBasedInitializer(Configurator $configurator): ->build(); $loggerProvider = (new LoggerProviderFactory())->create($emitMetrics ? $meterProvider : null, $resource); - $eventLoggerProvider = (new EventLoggerProviderFactory())->create($loggerProvider); ShutdownHandler::register($tracerProvider->shutdown(...)); ShutdownHandler::register($meterProvider->shutdown(...)); @@ -114,7 +112,6 @@ private static function environmentBasedInitializer(Configurator $configurator): ->withTracerProvider($tracerProvider) ->withMeterProvider($meterProvider) ->withLoggerProvider($loggerProvider) - ->withEventLoggerProvider($eventLoggerProvider) ->withPropagator($propagator) ; } @@ -144,7 +141,6 @@ private static function fileBasedInitializer(Configurator $configurator): Config ->withMeterProvider($sdk->getMeterProvider()) ->withLoggerProvider($sdk->getLoggerProvider()) ->withPropagator($sdk->getPropagator()) - ->withEventLoggerProvider($sdk->getEventLoggerProvider()) ; } diff --git a/src/SDK/SdkBuilder.php b/src/SDK/SdkBuilder.php index f2624e947..91e97aeeb 100644 --- a/src/SDK/SdkBuilder.php +++ b/src/SDK/SdkBuilder.php @@ -5,8 +5,6 @@ namespace OpenTelemetry\SDK; use OpenTelemetry\API\Instrumentation\Configurator; -use OpenTelemetry\API\Logs\EventLoggerProviderInterface; -use OpenTelemetry\API\Logs\NoopEventLoggerProvider; use OpenTelemetry\Context\Context; use OpenTelemetry\Context\Propagation\NoopTextMapPropagator; use OpenTelemetry\Context\Propagation\TextMapPropagatorInterface; @@ -24,7 +22,6 @@ class SdkBuilder private ?TracerProviderInterface $tracerProvider = null; private ?MeterProviderInterface $meterProvider = null; private ?LoggerProviderInterface $loggerProvider = null; - private ?EventLoggerProviderInterface $eventLoggerProvider = null; private ?TextMapPropagatorInterface $propagator = null; private bool $autoShutdown = false; @@ -59,16 +56,6 @@ public function setLoggerProvider(LoggerProviderInterface $loggerProvider): self return $this; } - /** - * @deprecated - */ - public function setEventLoggerProvider(EventLoggerProviderInterface $eventLoggerProvider): self - { - $this->eventLoggerProvider = $eventLoggerProvider; - - return $this; - } - public function setPropagator(TextMapPropagatorInterface $propagator): self { $this->propagator = $propagator; @@ -81,7 +68,6 @@ public function build(): Sdk $tracerProvider = $this->tracerProvider ?? new NoopTracerProvider(); $meterProvider = $this->meterProvider ?? new NoopMeterProvider(); $loggerProvider = $this->loggerProvider ?? new NoopLoggerProvider(); - $eventLoggerProvider = $this->eventLoggerProvider ?? new NoopEventLoggerProvider(); if ($this->autoShutdown) { // rector rule disabled in config, because ShutdownHandler::register() does not keep a strong reference to $this ShutdownHandler::register($tracerProvider->shutdown(...)); @@ -93,7 +79,6 @@ public function build(): Sdk $tracerProvider, $meterProvider, $loggerProvider, - $eventLoggerProvider, $this->propagator ?? NoopTextMapPropagator::getInstance(), ); } @@ -109,7 +94,6 @@ public function buildAndRegisterGlobal(): ScopeInterface ->withTracerProvider($sdk->getTracerProvider()) ->withMeterProvider($sdk->getMeterProvider()) ->withLoggerProvider($sdk->getLoggerProvider()) - ->withEventLoggerProvider($sdk->getEventLoggerProvider()) ->storeInContext(); return Context::storage()->attach($context); diff --git a/tests/Unit/API/Instrumentation/InstrumentationTest.php b/tests/Unit/API/Instrumentation/InstrumentationTest.php index 847d0c1db..6609749b9 100644 --- a/tests/Unit/API/Instrumentation/InstrumentationTest.php +++ b/tests/Unit/API/Instrumentation/InstrumentationTest.php @@ -10,11 +10,8 @@ use OpenTelemetry\API\Instrumentation\CachedInstrumentation; use OpenTelemetry\API\Instrumentation\Configurator; use OpenTelemetry\API\Instrumentation\ContextKeys; -use OpenTelemetry\API\Logs\EventLoggerInterface; -use OpenTelemetry\API\Logs\EventLoggerProviderInterface; use OpenTelemetry\API\Logs\LoggerInterface; use OpenTelemetry\API\Logs\LoggerProviderInterface; -use OpenTelemetry\API\Logs\NoopEventLoggerProvider; use OpenTelemetry\API\Logs\NoopLoggerProvider; use OpenTelemetry\API\Metrics\MeterInterface; use OpenTelemetry\API\Metrics\MeterProviderInterface; @@ -56,7 +53,6 @@ public function test_globals_not_configured_returns_noop_instances(): void $this->assertInstanceOf(NoopMeterProvider::class, Globals::meterProvider()); $this->assertInstanceOf(NoopTextMapPropagator::class, Globals::propagator()); $this->assertInstanceOf(NoopLoggerProvider::class, Globals::loggerProvider()); - $this->assertInstanceOf(NoopEventLoggerProvider::class, Globals::eventLoggerProvider()); } public function test_globals_returns_configured_instances(): void @@ -65,14 +61,12 @@ public function test_globals_returns_configured_instances(): void $meterProvider = $this->createMock(MeterProviderInterface::class); $propagator = $this->createMock(TextMapPropagatorInterface::class); $loggerProvider = $this->createMock(LoggerProviderInterface::class); - $eventLoggerProvider = $this->createMock(EventLoggerProviderInterface::class); $scope = Configurator::create() ->withTracerProvider($tracerProvider) ->withMeterProvider($meterProvider) ->withPropagator($propagator) ->withLoggerProvider($loggerProvider) - ->withEventLoggerProvider($eventLoggerProvider) ->activate(); try { @@ -80,7 +74,6 @@ public function test_globals_returns_configured_instances(): void $this->assertSame($meterProvider, Globals::meterProvider()); $this->assertSame($propagator, Globals::propagator()); $this->assertSame($loggerProvider, Globals::loggerProvider()); - $this->assertSame($eventLoggerProvider, Globals::eventLoggerProvider()); } finally { $scope->detach(); } @@ -107,16 +100,12 @@ public function test_instrumentation_returns_configured_instances(): void $logger = $this->createMock(LoggerInterface::class); $loggerProvider = $this->createMock(LoggerProviderInterface::class); $loggerProvider->method('getLogger')->willReturn($logger); - $eventLogger = $this->createMock(EventLoggerInterface::class); - $eventLoggerProvider = $this->createMock(EventLoggerProviderInterface::class); - $eventLoggerProvider->method('getEventLogger')->willReturn($eventLogger); $propagator = $this->createMock(TextMapPropagatorInterface::class); $scope = Configurator::create() ->withTracerProvider($tracerProvider) ->withMeterProvider($meterProvider) ->withLoggerProvider($loggerProvider) - ->withEventLoggerProvider($eventLoggerProvider) ->withPropagator($propagator) ->activate(); @@ -124,7 +113,6 @@ public function test_instrumentation_returns_configured_instances(): void $this->assertSame($tracer, $instrumentation->tracer()); $this->assertSame($meter, $instrumentation->meter()); $this->assertSame($logger, $instrumentation->logger()); - $this->assertSame($eventLogger, $instrumentation->eventLogger()); } finally { $scope->detach(); } diff --git a/tests/Unit/Contrib/Otlp/LogsConverterTest.php b/tests/Unit/Contrib/Otlp/LogsConverterTest.php index 705746d75..948582bf7 100644 --- a/tests/Unit/Contrib/Otlp/LogsConverterTest.php +++ b/tests/Unit/Contrib/Otlp/LogsConverterTest.php @@ -6,8 +6,6 @@ use OpenTelemetry\API\Trace\SpanContext; use OpenTelemetry\API\Trace\SpanInterface; -use OpenTelemetry\Context\Context; -use OpenTelemetry\Context\ContextKeys; use OpenTelemetry\Contrib\Otlp\LogsConverter; use OpenTelemetry\SDK\Logs\ReadableLogRecord; use PHPUnit\Framework\Attributes\CoversClass; @@ -20,8 +18,7 @@ class LogsConverterTest extends TestCase private const TRACE_ID_BASE16 = 'ff000000000000000000000000000041'; private const SPAN_ID_BASE16 = 'ff00000000000041'; private const FLAGS = 12; - /** @var ReadableLogRecord&MockObject $record */ - private $record; + private ReadableLogRecord&MockObject $record; private LogsConverter $converter; public function setUp(): void @@ -34,17 +31,27 @@ public function test_convert(): void { $this->record->method('getBody')->willReturn('body'); - $request = $this->converter->convert([$this->record]); + $converted = $this->converter->convert([$this->record]); /** @psalm-suppress InvalidArgument */ - $row = $request->getResourceLogs()[0]->getScopeLogs()[0]->getLogRecords()[0]; - $this->assertSame('body', $row->getBody()->getStringValue()); + $logRecord = $converted->getResourceLogs()[0]->getScopeLogs()[0]->getLogRecords()[0]; + $this->assertSame('body', $logRecord->getBody()->getStringValue()); + } + + public function test_convert_event_name(): void + { + $this->record->method('getEventName')->willReturn('my.event'); + + $converted = $this->converter->convert([$this->record]); + /** @psalm-suppress InvalidArgument */ + $logRecord = $converted->getResourceLogs()[0]->getScopeLogs()[0]->getLogRecords()[0]; + + $this->assertSame('my.event', $logRecord->getEventName()); } public function test_convert_with_context(): void { $spanContext = SpanContext::create(self::TRACE_ID_BASE16, self::SPAN_ID_BASE16, self::FLAGS); $span = $this->createMock(SpanInterface::class); - $context = Context::getCurrent()->with(ContextKeys::span(), $span); $span->method('getContext')->willReturn($spanContext); $this->record->method('getSpanContext')->willReturn($spanContext); $request = $this->converter->convert([$this->record]); diff --git a/tests/Unit/SDK/Logs/EventLoggerProviderFactoryTest.php b/tests/Unit/SDK/Logs/EventLoggerProviderFactoryTest.php deleted file mode 100644 index 50930a366..000000000 --- a/tests/Unit/SDK/Logs/EventLoggerProviderFactoryTest.php +++ /dev/null @@ -1,48 +0,0 @@ -setEnvironmentVariable(Variables::OTEL_SDK_DISABLED, $disabled); - $loggerProvider = $this->createMock(LoggerProviderInterface::class); - $factory = new EventLoggerProviderFactory(); - $eventLoggerProvider = $factory->create($loggerProvider); - $this->assertInstanceOf($expected, $eventLoggerProvider); - } - - public static function createProvider(): array - { - return [ - 'sdk disabled' => [ - 'true', - NoopEventLoggerProvider::class, - ], - 'sdk enabled' => [ - 'false', - EventLoggerProvider::class, - ], - ]; - } -} diff --git a/tests/Unit/SDK/Logs/EventLoggerProviderTest.php b/tests/Unit/SDK/Logs/EventLoggerProviderTest.php deleted file mode 100644 index d0f890201..000000000 --- a/tests/Unit/SDK/Logs/EventLoggerProviderTest.php +++ /dev/null @@ -1,42 +0,0 @@ -loggerProvider = $this->createMock(LoggerProviderInterface::class); - $this->logger = $this->createMock(LoggerInterface::class); - $this->eventLoggerProvider = new EventLoggerProvider($this->loggerProvider); - } - - public function test_emit(): void - { - $this->loggerProvider->expects($this->once())->method('getLogger')->willReturn($this->logger); - - $this->eventLoggerProvider->getEventLogger('event.logger', '1.0', 'https://example.org/schema', ['foo' => 'foo']); - } - - public function test_force_flush(): void - { - $this->loggerProvider->expects($this->once())->method('forceFlush')->willReturn(true); - - $this->eventLoggerProvider->forceFlush(); - } -} diff --git a/tests/Unit/SDK/Logs/EventLoggerTest.php b/tests/Unit/SDK/Logs/EventLoggerTest.php deleted file mode 100644 index d7f65f5cc..000000000 --- a/tests/Unit/SDK/Logs/EventLoggerTest.php +++ /dev/null @@ -1,98 +0,0 @@ -clock = new TestClock(); - Clock::setDefault($this->clock); - $this->logger = $this->createMock(LoggerInterface::class); - $loggerProvider = $this->createMock(LoggerProviderInterface::class); - $loggerProvider->method('getLogger')->willReturn($this->logger); - $this->eventLoggerProvider = new EventLoggerProvider($loggerProvider); - } - - public function test_emit(): void - { - $this->logger->expects($this->once())->method('emit')->with($this->callback(function (LogRecord $logRecord) { - $expected = (new LogRecord('some.payload')) - ->setSeverityNumber(Severity::ERROR) - ->setTimestamp(123456) - ->setContext(Context::getCurrent()) - ->setAttributes([ - 'event.name' => 'my.event', - 'bar' => 'bar', - ]); - $this->assertEquals($expected, $logRecord); - - return true; - })); - - $eventLogger = $this->eventLoggerProvider->getEventLogger('event.logger', '1.0', 'https://example.org/schema', ['foo' => 'foo']); - $eventLogger->emit('my.event', 'some.payload', 123456, severityNumber: Severity::ERROR, attributes: ['bar' => 'bar']); - } - - public function test_default_values(): void - { - $this->logger->expects($this->once())->method('emit')->with($this->callback(function (LogRecord $logRecord) { - $expected = (new LogRecord()) - ->setSeverityNumber(Severity::INFO) - ->setTimestamp($this->clock->now()) - ->setContext(Context::getCurrent()) - ->setAttributes([ - 'event.name' => 'my.event', - ]); - $this->assertEquals($expected, $logRecord); - - return true; - })); - - $eventLogger = $this->eventLoggerProvider->getEventLogger('event.logger'); - $eventLogger->emit('my.event'); - } - - /** - * "The user provided Attributes MUST not take over the event.name attribute" - * @see https://github.com/open-telemetry/opentelemetry-specification/blob/v1.32.0/specification/logs/event-sdk.md#emit-event - */ - public function test_event_name_attribute_is_ignored(): void - { - $this->logger->expects($this->once())->method('emit')->with($this->callback(function (LogRecord $logRecord) { - $expected = (new LogRecord()) - ->setSeverityNumber(Severity::INFO) - ->setTimestamp($this->clock->now()) - ->setContext(Context::getCurrent()) - ->setAttributes([ - 'event.name' => 'my.event', - ]); - $this->assertEquals($expected, $logRecord); - - return true; - })); - - $eventLogger = $this->eventLoggerProvider->getEventLogger('event.logger'); - $eventLogger->emit('my.event', attributes: ['event.name' => 'not.my.event']); - } -} diff --git a/tests/Unit/SDK/Logs/NoopEventLoggerProviderTest.php b/tests/Unit/SDK/Logs/NoopEventLoggerProviderTest.php deleted file mode 100644 index 56400be0f..000000000 --- a/tests/Unit/SDK/Logs/NoopEventLoggerProviderTest.php +++ /dev/null @@ -1,23 +0,0 @@ -assertInstanceOf(NoopEventLoggerProvider::class, NoopEventLoggerProvider::getInstance()); - } - - public function test_force_flush(): void - { - $this->assertTrue(NoopEventLoggerProvider::getInstance()->forceFlush()); - } -} diff --git a/tests/Unit/SDK/SdkAutoloaderTest.php b/tests/Unit/SDK/SdkAutoloaderTest.php index 7cdded3e5..fa343b53b 100644 --- a/tests/Unit/SDK/SdkAutoloaderTest.php +++ b/tests/Unit/SDK/SdkAutoloaderTest.php @@ -7,7 +7,6 @@ use OpenTelemetry\API\Globals; use OpenTelemetry\API\Instrumentation\Configurator; use OpenTelemetry\API\LoggerHolder; -use OpenTelemetry\API\Logs\NoopEventLoggerProvider; use OpenTelemetry\API\Logs\NoopLoggerProvider; use OpenTelemetry\API\Metrics\Noop\NoopMeterProvider; use OpenTelemetry\API\Trace\NoopTracerProvider; @@ -70,7 +69,6 @@ public function test_noop_if_disabled(): void $this->assertInstanceOf(NoopMeterProvider::class, Globals::meterProvider()); $this->assertInstanceOf(NoopTracerProvider::class, Globals::tracerProvider()); $this->assertInstanceOf(NoopLoggerProvider::class, Globals::loggerProvider()); - $this->assertInstanceOf(NoopEventLoggerProvider::class, Globals::eventLoggerProvider()); $this->assertInstanceOf(NoopTextMapPropagator::class, Globals::propagator(), 'propagator not initialized by disabled autoloader'); } diff --git a/tests/Unit/SDK/SdkBuilderTest.php b/tests/Unit/SDK/SdkBuilderTest.php index d9e3fa6f6..3caf53209 100644 --- a/tests/Unit/SDK/SdkBuilderTest.php +++ b/tests/Unit/SDK/SdkBuilderTest.php @@ -5,7 +5,6 @@ namespace OpenTelemetry\Tests\Unit\SDK; use OpenTelemetry\API\Globals; -use OpenTelemetry\API\Logs\EventLoggerProviderInterface; use OpenTelemetry\Context\Propagation\TextMapPropagatorInterface; use OpenTelemetry\SDK\Logs\LoggerProviderInterface; use OpenTelemetry\SDK\Metrics\MeterProviderInterface; @@ -21,7 +20,6 @@ class SdkBuilderTest extends TestCase private TracerProviderInterface $tracerProvider; private MeterProviderInterface $meterProvider; private LoggerProviderInterface $loggerProvider; - private EventLoggerProviderInterface $eventLoggerProvider; private SdkBuilder $builder; public function setUp(): void @@ -30,11 +28,9 @@ public function setUp(): void $this->tracerProvider = $this->createMock(TracerProviderInterface::class); $this->meterProvider = $this->createMock(MeterProviderInterface::class); $this->loggerProvider = $this->createMock(LoggerProviderInterface::class); - $this->eventLoggerProvider = $this->createMock(EventLoggerProviderInterface::class); $this->builder = (new SdkBuilder()) ->setMeterProvider($this->meterProvider) ->setLoggerProvider($this->loggerProvider) - ->setEventLoggerProvider($this->eventLoggerProvider) ->setPropagator($this->propagator) ->setTracerProvider($this->tracerProvider) ->setAutoShutdown(true); @@ -47,7 +43,6 @@ public function test_build(): void $this->assertSame($this->propagator, $sdk->getPropagator()); $this->assertSame($this->tracerProvider, $sdk->getTracerProvider()); $this->assertSame($this->loggerProvider, $sdk->getLoggerProvider()); - $this->assertSame($this->eventLoggerProvider, $sdk->getEventLoggerProvider()); } public function test_build_and_register_global(): void @@ -57,7 +52,6 @@ public function test_build_and_register_global(): void $this->assertSame($this->propagator, Globals::propagator()); $this->assertSame($this->tracerProvider, Globals::tracerProvider()); $this->assertSame($this->loggerProvider, Globals::loggerProvider()); - $this->assertSame($this->eventLoggerProvider, Globals::eventLoggerProvider()); $scope->detach(); } } diff --git a/tests/Unit/SDK/SdkTest.php b/tests/Unit/SDK/SdkTest.php index 084be21bb..85acce35a 100644 --- a/tests/Unit/SDK/SdkTest.php +++ b/tests/Unit/SDK/SdkTest.php @@ -4,7 +4,6 @@ namespace OpenTelemetry\Tests\Unit\SDK; -use OpenTelemetry\API\Logs\EventLoggerProviderInterface; use OpenTelemetry\Context\Propagation\TextMapPropagatorInterface; use OpenTelemetry\SDK\Common\Configuration\Variables; use OpenTelemetry\SDK\Logs\LoggerProviderInterface; @@ -26,7 +25,6 @@ class SdkTest extends TestCase private MeterProviderInterface $meterProvider; private TracerProviderInterface $tracerProvider; private LoggerProviderInterface $loggerProvider; - private EventLoggerProviderInterface $eventLoggerProvider; public function setUp(): void { @@ -34,7 +32,6 @@ public function setUp(): void $this->meterProvider = $this->createMock(MeterProviderInterface::class); $this->tracerProvider = $this->createMock(TracerProviderInterface::class); $this->loggerProvider = $this->createMock(LoggerProviderInterface::class); - $this->eventLoggerProvider = $this->createMock(EventLoggerProviderInterface::class); } public function tearDown(): void @@ -90,11 +87,10 @@ public function test_builder(): void public function test_getters(): void { - $sdk = new Sdk($this->tracerProvider, $this->meterProvider, $this->loggerProvider, $this->eventLoggerProvider, $this->propagator); + $sdk = new Sdk($this->tracerProvider, $this->meterProvider, $this->loggerProvider, $this->propagator); $this->assertSame($this->propagator, $sdk->getPropagator()); $this->assertSame($this->meterProvider, $sdk->getMeterProvider()); $this->assertSame($this->tracerProvider, $sdk->getTracerProvider()); $this->assertSame($this->loggerProvider, $sdk->getLoggerProvider()); - $this->assertSame($this->eventLoggerProvider, $sdk->getEventLoggerProvider()); } }