Skip to content

Conversation

@JPeer264
Copy link
Member

This disables the creation of dependabot updates for opentelemetry. Based on the image below there was not a real benefit of having this, except noise.

Screenshot 2025-11-19 at 11 21 02
@andreiborza
Copy link
Member

For reference: The reason these are low-value is that bumping OpenTelemetry instrumentations needs to happen in lockstep across the entire codebase and often requires additional hand-tuning to make it work.

I created a PR to add a cursor command to make this a bit easier for us: #18253

@github-actions
Copy link
Contributor

github-actions bot commented Nov 19, 2025

node-overhead report 🧳

Note: This is a synthetic benchmark with a minimal express app and does not necessarily reflect the real-world performance impact in an application.

Scenario Requests/s % of Baseline Prev. Requests/s Change %
GET Baseline 11,191 - 8,783 +27%
GET With Sentry 1,652 15% 1,354 +22%
GET With Sentry (error only) 7,788 70% 5,981 +30%
POST Baseline 1,198 - 1,200 -0%
POST With Sentry 539 45% 494 +9%
POST With Sentry (error only) 1,065 89% 1,036 +3%
MYSQL Baseline 4,079 - 3,258 +25%
MYSQL With Sentry 672 16% 464 +45%
MYSQL With Sentry (error only) 3,342 82% 2,673 +25%

View base workflow run

@JPeer264 JPeer264 requested a review from andreiborza November 20, 2025 12:15
Copy link
Member

@andreiborza andreiborza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ty!

@JPeer264 JPeer264 enabled auto-merge (squash) November 20, 2025 16:34
@JPeer264 JPeer264 merged commit 28e9cc6 into develop Nov 20, 2025
107 of 109 checks passed
@JPeer264 JPeer264 deleted the jp/ignore-opentelemetry-updates branch November 20, 2025 16:35
This was referenced Nov 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants