The recommendation is to:

1. Keep supporting standard Attributes - see Attributes "hell" opentelemetry-specification#4201 for the context
2. Add log-specific attributes type that can take anyvalue and accept it as well

We already have AnyValue here

IIUC, we should just use that in place of Attributes in the Logs API/SDK?

we can't change Attributes definition for spans and metrics - those don't support AnyValue on the API level.
For logs, we should have a way to add AnyValue attribute, but we should make sure it's easy to keep using the same Attributes type as for spans and metrics.

I'm not an expert in Python API design and there could be much nicer ways to achieve it, but I'd create a LogAttributes union (with AnyValue supported) and accepted Union[Attributes | LogAttributes] in LogRecord ctor.

👍 I'm just proposing to change it for the Logging API only and leave other signals with the existing Attributes type.

I'm not an expert in Python API design and there could be much nicer ways to achieve it, but I'd create a LogAttributes union (with AnyValue supported) and accepted Union[Attributes | LogAttributes] in LogRecord ctor.

Since Attributes is a subset of AnyValue, there shouldn't be any functional different vs using AnyValue directly. Is this more of a readability concern?

IIUC, we should just use that in place of Attributes in the Logs API/SDK?

I believe we would replace AttributeValue with AnyValue. Isn't that all that is needed?

I think it's also important to allow standard Attributes to be populated on logs - see open-telemetry/opentelemetry-specification#4201

I.e. I should be able to write

attributes = Attributes(...)
counter.add(1, attributes=attributes)
logger.emit(LogRecord(..., attributes=attributes)

if LogRecord takes LogAttributes only, users would need to do manual conversion between types (which is inefficient and increases complexity).

PS: please ignore my concern if pythonic implicit type conversions make it irrelevant. In any case, I suggest to write a test that uses standard attributes on a log record and make sure it works and does not raise type-checker errors

PS: please ignore my concern if pythonic implicit type conversions make it irrelevant.

Ah yes that's the case, Attributes just describes a structural type which is implicit implemented, not a named class.

attributes = {"foo": "bar"}
counter.add(1, attributes=attributes)
logger.emit(LogRecord(..., attributes=attributes)

should work fine

I believe we would replace AttributeValue with AnyValue. Isn't that all that is needed?

We should probably still do this no?

Did a PR to check this, it's in draft because I don't want this in the next release.