Logs SDK: Logging Handler bridge populates unstable `code` attributes

[lmolkova]

opentelemetry-python/opentelemetry-sdk/src/opentelemetry/sdk/_logs/_internal/__init__.py

Lines 482 to 484 in 59d6754

	attributes[SpanAttributes.CODE_FILEPATH] = record.pathname
	attributes[SpanAttributes.CODE_FUNCTION] = record.funcName
	attributes[SpanAttributes.CODE_LINENO] = record.lineno

There is no formal conventions defining if logs SDK/bridges should populate code attributes. Also, those attributes are not stable (yet).

The recommendation is to

• remove or put them behind experimental feature-flag
• stop using deprecated SpanAttributes and use https://github.com/open-telemetry/opentelemetry-python/blob/main/opentelemetry-semantic-conventions/src/opentelemetry/semconv/_incubating/attributes/code_attributes.py to make the status obvious via the import path

Part of open-telemetry/community#1751

[xrmx]

Code attributes have become stable and we'll switch to the stable version for the next release. Also we removed usage of SpanAttributes. @lmolkova is it fine to close for you?

[lmolkova]

Thanks @xrmx ! It's awesome, the only concern I have is that they should be opt-in. I.e. I think there are no logs conventions that mention them as recommended/required.
Added a comment on a related open-telemetry/semantic-conventions#1981