Skip to content

Conversation

saviogl
Copy link

@saviogl saviogl commented Aug 25, 2025

Adds parsing logic for OTel spans with messaging.system attributes to properly categorize queue operations. Producer spans get 'queue.publish' operation and consumer spans get 'queue.process' operation. The span description is extracted from the span name (e.g., "Sidekiq::Worker" from "Sidekiq::Worker publish").

Thanks for your Pull Request 🎉

Please keep these instructions in mind so we can review it more efficiently:

  • Add the references of all the related issues/PRs in the description
  • Whether it's a new feature or a bug fix, make sure they're covered by new test cases
  • If this PR contains any refactoring work, please give it its own commit(s)
  • Finally, please add an entry to the corresponding changelog

Other Notes

  • We squash all commits before merging
  • We generally review new PRs within a week
  • If you have any question, you can ask for feedback in our discord community first

Description

Describe your changes:

cursor[bot]

This comment was marked as outdated.

cursor[bot]

This comment was marked as outdated.

@saviogl saviogl force-pushed the feature-correctly-parse-queue-systems branch from 86187f0 to cae5645 Compare August 25, 2025 16:08
cursor[bot]

This comment was marked as outdated.

Adds parsing logic for OTel spans with messaging.system attributes to properly
categorize queue operations. Producer spans get 'queue.publish' operation and
consumer spans get 'queue.process' operation. The span description is extracted
from the span name (e.g., "Sidekiq::Worker" from "Sidekiq::Worker publish").
@saviogl saviogl force-pushed the feature-correctly-parse-queue-systems branch from cae5645 to a985a2b Compare August 25, 2025 16:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant