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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for Opentelemetry #2344

Merged
merged 17 commits into from Nov 28, 2022
Merged

Add support for Opentelemetry #2344

merged 17 commits into from Nov 28, 2022

Conversation

adinauer
Copy link
Member

@adinauer adinauer commented Nov 8, 2022

馃摐 Description

Integration branch for collecting parts of our Opentelemetry support

馃挕 Motivation and Context

Partially F ixes #2327

馃挌 How did you test it?

馃摑 Checklist

  • I reviewed the submitted code
  • I added tests to verify the changes
  • I updated the docs if needed
  • No breaking changes

馃敭 Next steps

@github-actions
Copy link
Contributor

github-actions bot commented Nov 8, 2022

Messages
馃摉 Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Generated by 馃毇 dangerJS against b137776

@adinauer adinauer mentioned this pull request Nov 9, 2022
21 tasks
@github-actions
Copy link
Contributor

github-actions bot commented Nov 14, 2022

Performance metrics 馃殌

Plain With Sentry Diff
Startup time 300.86 ms 390.44 ms 89.58 ms
Size 1.73 MiB 2.32 MiB 612.47 KiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
507f924 342.51 ms 402.65 ms 60.14 ms
b85d8aa 289.35 ms 335.92 ms 46.56 ms
4a9c176 320.62 ms 334.68 ms 14.06 ms
7597ded 289.60 ms 339.69 ms 50.09 ms
3695453 299.25 ms 360.04 ms 60.79 ms
f809aac 301.51 ms 346.60 ms 45.09 ms
3695453 314.63 ms 353.10 ms 38.47 ms
3695453 301.78 ms 371.14 ms 69.36 ms
4a9c176 319.77 ms 363.20 ms 43.43 ms
90e9745 314.68 ms 357.28 ms 42.60 ms

App size

Revision Plain With Sentry Diff
507f924 1.73 MiB 2.32 MiB 609.95 KiB
b85d8aa 1.73 MiB 2.32 MiB 611.62 KiB
4a9c176 1.73 MiB 2.33 MiB 612.69 KiB
7597ded 1.73 MiB 2.32 MiB 609.88 KiB
3695453 1.73 MiB 2.32 MiB 611.62 KiB
f809aac 1.73 MiB 2.32 MiB 608.63 KiB
3695453 1.73 MiB 2.32 MiB 611.62 KiB
3695453 1.73 MiB 2.32 MiB 611.62 KiB
4a9c176 1.73 MiB 2.33 MiB 612.69 KiB
90e9745 1.73 MiB 2.32 MiB 608.63 KiB

Previous results on branch: feat/otel

Startup times

Revision Plain With Sentry Diff
e8ec152 315.22 ms 347.42 ms 32.20 ms
21f8f05 316.04 ms 333.04 ms 17.00 ms
ea122bf 338.47 ms 360.12 ms 21.65 ms
65dc4aa 384.68 ms 437.84 ms 53.16 ms
a7480d5 355.94 ms 396.98 ms 41.04 ms
e6c648a 280.58 ms 305.35 ms 24.76 ms
0f8ea2d 301.90 ms 354.44 ms 52.54 ms

App size

Revision Plain With Sentry Diff
e8ec152 1.73 MiB 2.33 MiB 612.70 KiB
21f8f05 1.73 MiB 2.33 MiB 612.70 KiB
ea122bf 1.73 MiB 2.33 MiB 612.70 KiB
65dc4aa 1.73 MiB 2.32 MiB 612.47 KiB
a7480d5 1.73 MiB 2.33 MiB 612.70 KiB
e6c648a 1.73 MiB 2.32 MiB 612.26 KiB
0f8ea2d 1.73 MiB 2.33 MiB 612.70 KiB

@adinauer adinauer mentioned this pull request Nov 21, 2022
4 tasks
CHANGELOG.md Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
@adinauer adinauer marked this pull request as ready for review November 28, 2022 15:47
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.

None yet

3 participants