[API] Support for Logger::Enabled() is incomplete

[marcalff]

  nostd::shared_ptr<opentelemetry::logs::Logger> logger = ...;

  if (logger->Enabled(severity)) {
  }

Logger::Enabled() always return false.

This is due to:

  inline bool Enabled(Severity severity) const noexcept
  {
    return static_cast<uint8_t>(severity) >= OPENTELEMETRY_ATOMIC_READ_8(&minimum_severity_);
  }

  mutable uint8_t minimum_severity_{kMaxSeverity};

The severity is kMaxSeverity by default, meaning all events are disabled by default.

There is a method to change the logger severity:

  void SetMinimumSeverity(uint8_t severity_or_max) noexcept
  {
    OPENTELEMETRY_ATOMIC_WRITE_8(&minimum_severity_, severity_or_max);
  }

but is it protected, and never called in the code base.

Any user code that checks for Enabled() can not emit logs.

Note that EmitLogRecord() helpers never honor the Enabled() flag either.

[marcalff]

To revisit once spec is merged:

• https://github.com/open-telemetry/opentelemetry-specification/pull/4020

[marcalff]

Spec is now merged.

Adding a Enabled() method in the API is an ABI breaking change, to implement in ABI_VERSION_2 only.

[marcalff]

See related:

• https://github.com/open-telemetry/opentelemetry-specification/pull/4063

[github-actions[bot]]

This issue was marked as stale due to lack of activity.

[github-actions[bot]]

This issue is available for anyone to work on. Make sure to reference this issue in your pull request. :sparkles: Thank you for your contribution! :sparkles:

[psx95]

Since there have been no takers for this, I can take up this issue.