Closed brettmc closed 1 day ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 74.23%. Comparing base (
eaba9e3
) to head (dd5ccd9
). Report is 1 commits behind head on main.
Converting back to draft, awaiting agreement in https://github.com/open-telemetry/semantic-conventions/issues/1031 on how to handle messaging.client.id
The recommendation for messaging.client_id -> messaging.client.id would be to drop the old attribute.
The recommendation from semconv maintainers is to clobber the old, which is what this PR does.
note that messaging.client_id was renamed to messaging.client.id, which causes a conflict in const name. It hasn't been resolved upstream yet, but manually removing the duplicate seems like the best way to go for us.