add rabbitmq to semantic conventions yaml

[blumamir]

In trace semantic conventions markdown there is a section about rabbitmq with attribute messaging.rabbitmq.routing_key.
This attribute is not reflected in the messaging.yaml file here.

opentelemetry-js auto-generate the attributes from this yaml, thus it is currently missing

[blumamir]

I think the docfx check failure is not related to this PR:

docfx not installed. An error occurred during installation:
 The remote server returned an error: (503) Server Unavailable. Service Unavailable
docfx package files install completed. Performing other installation steps.

[arminru]

Good catch, thank you!

Please make sure to reference this new group in https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/trace/semantic_conventions/messaging.md#rabbitmq and run the markdown generator as documented here: https://github.com/open-telemetry/opentelemetry-specification/tree/main/semantic_conventions.
This way, we can also remove the sentence "The routing key MUST be provided to the attribute messaging.rabbitmq.routing_key, unless it is empty." from the prose.

[arminru]

I think the docfx check failure is not related to this PR:

docfx not installed. An error occurred during installation:
 The remote server returned an error: (503) Server Unavailable. Service Unavailable
docfx package files install completed. Performing other installation steps.

Indeed. I re-ran the job and now it's passing. Apparently Chocolatey was down before.

[blumamir]

Good catch, thank you!

Please make sure to reference this new group in https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/trace/semantic_conventions/messaging.md#rabbitmq and run the markdown generator as documented here: https://github.com/open-telemetry/opentelemetry-specification/tree/main/semantic_conventions.
This way, we can also remove the sentence "The routing key MUST be provided to the attribute messaging.rabbitmq.routing_key, unless it is empty." from the prose.

Done. Thank you for the review and for pointing me to the relevant documentation.
Great automation 👍

[arminru]

Please also add an entry in CHANGELOG.md that this already specified key is now also defined in the respective YAML file.

[blumamir]

Please also add an entry in CHANGELOG.md that this already specified key is now also defined in the respective YAML file.

done

[SergeyKanzhelev]

this PR is uncontroversial as it only changes from text to yaml, not the semantic. And got a lot of approvals fast. Merging