Skip to content

feat(java): enable OTEL by default in onboarding options #13006

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

Merged
merged 1 commit into from
Mar 17, 2025

Conversation

lcian
Copy link
Member

@lcian lcian commented Mar 14, 2025

DESCRIBE YOUR PR

The OTEL option will now be checked by default on the getting started guides for Java backend platforms.
OTEL is fully integrated with the SDK by now and improves scopes and trace propagation.
It can be enabled independently from tracing, which we still leave unchecked by default.

IS YOUR CHANGE URGENT?

Help us prioritize incoming PRs by letting us know when the change needs to go live.

  • Urgent deadline (GA date, etc.):
  • Other deadline:
  • None: Not urgent, can wait up to 1 week+

Copy link

vercel bot commented Mar 14, 2025

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

Name Status Preview Comments Updated (UTC)
sentry-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 14, 2025 1:44pm
2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
changelog ⬜️ Ignored (Inspect) Mar 14, 2025 1:44pm
develop-docs ⬜️ Ignored (Inspect) Mar 14, 2025 1:44pm

@lcian lcian requested a review from adinauer March 14, 2025 13:33
Copy link

codecov bot commented Mar 14, 2025

Bundle Report

Changes will decrease total bundle size by 15 bytes (-0.0%) ⬇️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
sentry-docs-server-cjs 10.54MB -9 bytes (-0.0%) ⬇️
sentry-docs-client-array-push 9.39MB -6 bytes (-0.0%) ⬇️

Affected Assets, Files, and Routes:

view changes for bundle: sentry-docs-client-array-push

Assets Changed:

Asset Name Size Change Total Size Change (%)
static/chunks/pages/_app-*.js -3 bytes 868.16kB -0.0%
static/chunks/3319-*.js -3 bytes 393.41kB -0.0%
static/iB6z0_McWfL5O7IszBhzb/_buildManifest.js (New) 578 bytes 578 bytes 100.0% 🚀
static/iB6z0_McWfL5O7IszBhzb/_ssgManifest.js (New) 77 bytes 77 bytes 100.0% 🚀
static/YbQJPIeimdxTfYCnld3eg/_buildManifest.js (Deleted) -578 bytes 0 bytes -100.0% 🗑️
static/YbQJPIeimdxTfYCnld3eg/_ssgManifest.js (Deleted) -77 bytes 0 bytes -100.0% 🗑️
view changes for bundle: sentry-docs-server-cjs

Assets Changed:

Asset Name Size Change Total Size Change (%)
1729.js -3 bytes 1.57MB -0.0%
../instrumentation.js -3 bytes 904.32kB -0.0%
9523.js -3 bytes 880.37kB -0.0%

@priscilawebdev
Copy link
Member

Recently, we decided to have only errors selected by default in our SaaS product (see) and it seems that the docs reflect that. Are we sure we want to have OpenTelemetry checked?

@adinauer
Copy link
Member

@priscilawebdev yeah, we won't enable Performance by default but would prefer to have OpenTelemetry on by default as it adds support for lots of frameworks. It's not just for creating spans but also Scopes (used to be hub) propagation as well as distributed tracing.

@priscilawebdev
Copy link
Member

@adinauer I understand. I just saw that in the onboarding, the Open Telemetry is checked by default, but there we use a different component.

image

@adinauer
Copy link
Member

Yes, we want it enabled by default everywhere.

@lcian lcian merged commit aadacba into master Mar 17, 2025
11 checks passed
@lcian lcian deleted the lcian/feat/java-otel-on-by-default branch March 17, 2025 09:23
@github-actions github-actions bot locked and limited conversation to collaborators Apr 2, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants