Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(otel): add bullmq open telemetry instrumentation #4066

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

dillonstreator
Copy link
Contributor

Followup of #3920

@opentelemetry/auto-instrumentations-node does not include bullmq. As it is a core component of the system, it should be instrumented.
https://www.npmjs.com/package/@opentelemetry/auto-instrumentations-node#supported-instrumentations

https://github.com/jenniferplusplus/opentelemetry-instrumentation-bullmq looks like a good option and supports the bullmq v5.

@dillonstreator dillonstreator requested a review from a team as a code owner February 28, 2024 23:31
Copy link

vercel bot commented Feb 28, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
medplum-provider ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 29, 2024 5:19pm
3 Ignored Deployments
Name Status Preview Comments Updated (UTC)
medplum-app ⬜️ Ignored (Inspect) Visit Preview Feb 29, 2024 5:19pm
medplum-storybook ⬜️ Ignored (Inspect) Visit Preview Feb 29, 2024 5:19pm
medplum-www ⬜️ Ignored (Inspect) Visit Preview Feb 29, 2024 5:19pm

Copy link

vercel bot commented Feb 28, 2024

@dillonstreator is attempting to deploy a commit to the Medplum Team on Vercel.

A member of the Team first needs to authorize it.

Copy link

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/@jenniferplusplus/opentelemetry-instrumentation-bullmq@0.5.0 Transitive: unsafe +4 2.13 MB jenniferplusplus

View full report↗︎

@reshmakh reshmakh added the audit-logging Commits related to audit logging and observability label Feb 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
audit-logging Commits related to audit logging and observability
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

None yet

2 participants