stabilize logger.enabled (#1580)

- update Logger.isEnabled signature to include context and severity number, per spec
- accept event_name on a LogRecord
- add event name to otlp logs payload
- update or remove some comments that were based on the old logs-bridge specification

Author: brettmc

Logs/LateBindingLogger.php

@@ -5,6 +5,7 @@
 namespace OpenTelemetry\API\Logs;
 
 use Closure;
+use OpenTelemetry\Context\ContextInterface;
 
 class LateBindingLogger implements LoggerInterface
 {
@@ -21,7 +22,7 @@ public function emit(LogRecord $logRecord): void
         ($this->logger ??= ($this->factory)())->emit($logRecord);
     }
 
-    public function isEnabled(): bool
+    public function isEnabled(?ContextInterface $context = null, ?int $severityNumber = null): bool
     {
         return true;
     }

Logs/LogRecord.php

@@ -16,6 +16,7 @@ class LogRecord
     protected int $severityNumber = 0;
     protected ?string $severityText = null;
     protected array $attributes = [];
+    protected ?string $eventName = null;
 
     public function __construct(protected mixed $body = null)
     {
@@ -90,6 +91,13 @@ public function setBody(mixed $body = null): self
         return $this;
     }
 
+    public function setEventName(string $eventName): self
+    {
+        $this->eventName = $eventName;
+
+        return $this;
+    }
+
     /**
      * @param int|null $observedTimestamp Time, in nanoseconds since the unix epoch, when the event was observed by the collection system.
      */

Logs/LoggerInterface.php

@@ -4,18 +4,15 @@
 
 namespace OpenTelemetry\API\Logs;
 
+use OpenTelemetry\Context\ContextInterface;
+
 interface LoggerInterface
 {
-    /**
-     * This method should only be used when implementing a `log appender`
-     * @see https://github.com/open-telemetry/opentelemetry-specification/blob/v1.32.0/specification/logs/bridge-api.md#artifact-naming
-     */
     public function emit(LogRecord $logRecord): void;
 
     /**
-     * Determine if the logger is enabled. Logs bridge API authors SHOULD call this method each time they
-     * are about to generate a LogRecord, to avoid performing computationally expensive work.
-     * @experimental
+     * Determine if the logger is enabled. Instrumentation authors SHOULD call this method each time they
+     * emit a LogRecord, to ensure they have the most up-to-date response.
      */
-    public function isEnabled(): bool;
+    public function isEnabled(?ContextInterface $context = null, ?int $severityNumber = null): bool;
 }

Logs/NoopLogger.php

@@ -4,6 +4,7 @@
 
 namespace OpenTelemetry\API\Logs;
 
+use OpenTelemetry\Context\ContextInterface;
 use Psr\Log\LoggerTrait;
 
 class NoopLogger implements LoggerInterface
@@ -31,7 +32,7 @@ public function log($level, $message, array $context = []): void
     {
     }
 
-    public function isEnabled(): bool
+    public function isEnabled(?ContextInterface $context = null, ?int $severityNumber = null): bool
     {
         return false;
     }