Support type bytes for OTLP AnyValue encoding

[lzchen]

Is your feature request related to a problem?

Proto definition of AnyValue supports bytes values. Our current encoding logic doesn't support this yet.

Describe the solution you'd like

Add support in encoding logic for bytes type.

Describe alternatives you've considered

No response

Additional Context

No response

Would you like to implement a fix?

None

[wasup-yash]

we just need a check right in OTLP as we have for other types in proto ?

[lzchen]

@wasup-yash

Yeah should be the only thing that is needed.

[wasup-yash]

@wasup-yash

Yeah should be the only thing that is needed.

ok creating a PR for it as a draft first

[dimaqq]

Unfortunately, the #4218 alone is not enough.

The bytes-valued attributes are coerced to strings way before they reach the exporter.

It happens here:

opentelemetry-python/opentelemetry-api/src/opentelemetry/attributes/__init__.py

Lines 110 to 121 in afec2dd

	def _clean_attribute_value(
	value: types.AttributeValue, limit: Optional[int]
	) -> Optional[types.AttributeValue]:
	if value is None:
	return None
	if isinstance(value, bytes):
	try:
	value = value.decode()
	except UnicodeDecodeError:
	_logger.warning("Byte attribute could not be decoded.")
	return None

What would be better to reopen this issue or create a new one?

[xrmx]

@dimaqq please open a new one, referencing this

[dimaqq]

Sure, #4460