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

feat(v8/deps): Upgrade all OpenTelemetry dependencies #15098

Merged
merged 5 commits into from
Jan 31, 2025

Conversation

nwalters512
Copy link
Contributor

@nwalters512 nwalters512 commented Jan 20, 2025

Backport of #14967 to v8. The diff here will shrink a lot when #15094 is merged.

@nwalters512
Copy link
Contributor Author

The tests here are expected to fail until #15094 lands.

@nwalters512
Copy link
Contributor Author

@AbhiPrasad hello again! Wondering if you can offer any insight into why the suites/tracing/tedious/test.ts integration test is failing. It fails for me locally too, but only when I run the suite as a whole. If I run yarn test suites/tracing/tedious/test.ts to run it in isolation, it fails.

I added some logging of expected and envelope here:

if (expectedType !== envelopeItemType) {
throw new Error(`Expected envelope item type '${expectedType}' but got '${envelopeItemType}'`);
}

It looks like the connection to the mssql server is failing:

              breadcrumbs: [
                {
                  timestamp: 1737481330.83,
                  category: 'console',
                  level: 'error',
                  message: "ConnectionError: Login failed for user 'sa'.\n" +
                    '    at Login7TokenHandler.onErrorMessage (/Users/nathan/git/sentry-javascript/node_modules/tedious/lib/token/handler.js:186:19)\n' +
                    '    at Readable.<anonymous> (/Users/nathan/git/sentry-javascript/node_modules/tedious/lib/token/token-stream-parser.js:19:33)\n' +
                    '    at Readable.emit (node:events:519:28)\n' +
                    '    at addChunk (node:internal/streams/readable:559:12)\n' +
                    '    at readableAddChunkPushObjectMode (node:internal/streams/readable:536:3)\n' +
                    '    at Readable.push (node:internal/streams/readable:391:5)\n' +
                    '    at nextAsync (node:internal/streams/from:194:22)\n' +
                    '    at process.processTicksAndRejections (node:internal/process/task_queues:95:5) {\n' +
                    "  code: 'ELOGIN'\n" +
                    '}'
                }
              ],

@AbhiPrasad
Copy link
Member

closing up some of my other tasks for the day - this is top of my todo alongside getting #15119 to a mergeable state tomorrow morning.

Thanks again for all the PRs and effort @nwalters512! If you want some Sentry swag, please shoot me an email at a + prasad + @ + sentry.io (remove the pluses) and we'll get something to you 😄

@AbhiPrasad AbhiPrasad self-assigned this Jan 21, 2025
@klippx
Copy link

klippx commented Jan 31, 2025

Thanks @nwalters512 you are a champion! 🏆

@andreiborza andreiborza marked this pull request as ready for review January 31, 2025 15:35
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.

Tests are now passing, this backport LGTM.

Thanks so much for this @nwalters512!

@andreiborza andreiborza merged commit 3673689 into getsentry:v8 Jan 31, 2025
147 checks passed
@nwalters512 nwalters512 deleted the v8-upgrade-otel-deps branch January 31, 2025 15:56
alexandresoro pushed a commit to alexandresoro/ouca-web that referenced this pull request Feb 4, 2025
This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [@sentry/react](https://github.com/getsentry/sentry-javascript/tree/master/packages/react) ([source](https://github.com/getsentry/sentry-javascript)) | dependencies | minor | [`8.53.0` -> `8.54.0`](https://renovatebot.com/diffs/npm/@sentry%2freact/8.53.0/8.54.0) |

---

### Release Notes

<details>
<summary>getsentry/sentry-javascript (@&#8203;sentry/react)</summary>

### [`v8.54.0`](https://github.com/getsentry/sentry-javascript/releases/tag/8.54.0)

[Compare Source](getsentry/sentry-javascript@8.53.0...8.54.0)

-   feat(v8/deps): Upgrade all OpenTelemetry dependencies ([#&#8203;15098](getsentry/sentry-javascript#15098))
-   fix(node/v8): Add compatibility layer for Prisma v5 ([#&#8203;15210](getsentry/sentry-javascript#15210))

Work in this release was contributed by [@&#8203;nwalters512](https://github.com/nwalters512). Thank you for your contribution!

#### Bundle size 📦

| Path                                                             | Size              |
| ---------------------------------------------------------------- | ----------------- |
| [@&#8203;sentry/browser](https://github.com/sentry/browser)                                                  | 23.3 KB   |
| [@&#8203;sentry/browser](https://github.com/sentry/browser) - with treeshaking flags                         | 23.17 KB  |
| [@&#8203;sentry/browser](https://github.com/sentry/browser) (incl. Tracing)                                  | 35.9 KB   |
| [@&#8203;sentry/browser](https://github.com/sentry/browser) (incl. Tracing, Replay)                          | 73.27 KB  |
| [@&#8203;sentry/browser](https://github.com/sentry/browser) (incl. Tracing, Replay) - with treeshaking flags | 66.71 KB  |
| [@&#8203;sentry/browser](https://github.com/sentry/browser) (incl. Tracing, Replay with Canvas)              | 77.57 KB  |
| [@&#8203;sentry/browser](https://github.com/sentry/browser) (incl. Tracing, Replay, Feedback)                | 89.5 KB   |
| [@&#8203;sentry/browser](https://github.com/sentry/browser) (incl. Feedback)                                 | 39.51 KB  |
| [@&#8203;sentry/browser](https://github.com/sentry/browser) (incl. sendFeedback)                             | 27.91 KB  |
| [@&#8203;sentry/browser](https://github.com/sentry/browser) (incl. FeedbackAsync)                            | 32.71 KB  |
| [@&#8203;sentry/react](https://github.com/sentry/react)                                                    | 25.98 KB  |
| [@&#8203;sentry/react](https://github.com/sentry/react) (incl. Tracing)                                    | 38.71 KB  |
| [@&#8203;sentry/vue](https://github.com/sentry/vue)                                                      | 27.58 KB  |
| [@&#8203;sentry/vue](https://github.com/sentry/vue) (incl. Tracing)                                      | 37.75 KB  |
| [@&#8203;sentry/svelte](https://github.com/sentry/svelte)                                                   | 23.46 KB  |
| CDN Bundle                                                       | 24.49 KB  |
| CDN Bundle (incl. Tracing)                                       | 37.6 KB   |
| CDN Bundle (incl. Tracing, Replay)                               | 72.9 KB   |
| CDN Bundle (incl. Tracing, Replay, Feedback)                     | 78.23 KB  |
| CDN Bundle - uncompressed                                        | 71.92 KB  |
| CDN Bundle (incl. Tracing) - uncompressed                        | 111.52 KB |
| CDN Bundle (incl. Tracing, Replay) - uncompressed                | 225.78 KB |
| CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed      | 238.88 KB |
| [@&#8203;sentry/nextjs](https://github.com/sentry/nextjs) (client)                                          | 38.96 KB  |
| [@&#8203;sentry/sveltekit](https://github.com/sentry/sveltekit) (client)                                       | 36.4 KB   |
| [@&#8203;sentry/node](https://github.com/sentry/node)                                                     | 162.85 KB |
| [@&#8203;sentry/node](https://github.com/sentry/node) - without tracing                                   | 99.14 KB  |
| [@&#8203;sentry/aws-serverless](https://github.com/sentry/aws-serverless)                                           | 131.23 KB |

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box

---

This PR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOS4xNTYuMSIsInVwZGF0ZWRJblZlciI6IjM5LjE1Ni4xIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJkZXBlbmRlbmNpZXMiXX0=-->

Reviewed-on: https://git.tristess.app/alexandresoro/ouca-web/pulls/75
Reviewed-by: Alexandre Soro <code@soro.dev>
Co-authored-by: renovate <renovate@git.tristess.app>
Co-committed-by: renovate <renovate@git.tristess.app>
alexandresoro pushed a commit to alexandresoro/ouca-backend that referenced this pull request Feb 4, 2025
This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [@sentry/node](https://github.com/getsentry/sentry-javascript/tree/master/packages/node) ([source](https://github.com/getsentry/sentry-javascript)) | dependencies | minor | [`8.53.0` -> `8.54.0`](https://renovatebot.com/diffs/npm/@sentry%2fnode/8.53.0/8.54.0) |

---

### Release Notes

<details>
<summary>getsentry/sentry-javascript (@&#8203;sentry/node)</summary>

### [`v8.54.0`](https://github.com/getsentry/sentry-javascript/releases/tag/8.54.0)

[Compare Source](getsentry/sentry-javascript@8.53.0...8.54.0)

-   feat(v8/deps): Upgrade all OpenTelemetry dependencies ([#&#8203;15098](getsentry/sentry-javascript#15098))
-   fix(node/v8): Add compatibility layer for Prisma v5 ([#&#8203;15210](getsentry/sentry-javascript#15210))

Work in this release was contributed by [@&#8203;nwalters512](https://github.com/nwalters512). Thank you for your contribution!

#### Bundle size 📦

| Path                                                             | Size              |
| ---------------------------------------------------------------- | ----------------- |
| [@&#8203;sentry/browser](https://github.com/sentry/browser)                                                  | 23.3 KB   |
| [@&#8203;sentry/browser](https://github.com/sentry/browser) - with treeshaking flags                         | 23.17 KB  |
| [@&#8203;sentry/browser](https://github.com/sentry/browser) (incl. Tracing)                                  | 35.9 KB   |
| [@&#8203;sentry/browser](https://github.com/sentry/browser) (incl. Tracing, Replay)                          | 73.27 KB  |
| [@&#8203;sentry/browser](https://github.com/sentry/browser) (incl. Tracing, Replay) - with treeshaking flags | 66.71 KB  |
| [@&#8203;sentry/browser](https://github.com/sentry/browser) (incl. Tracing, Replay with Canvas)              | 77.57 KB  |
| [@&#8203;sentry/browser](https://github.com/sentry/browser) (incl. Tracing, Replay, Feedback)                | 89.5 KB   |
| [@&#8203;sentry/browser](https://github.com/sentry/browser) (incl. Feedback)                                 | 39.51 KB  |
| [@&#8203;sentry/browser](https://github.com/sentry/browser) (incl. sendFeedback)                             | 27.91 KB  |
| [@&#8203;sentry/browser](https://github.com/sentry/browser) (incl. FeedbackAsync)                            | 32.71 KB  |
| [@&#8203;sentry/react](https://github.com/sentry/react)                                                    | 25.98 KB  |
| [@&#8203;sentry/react](https://github.com/sentry/react) (incl. Tracing)                                    | 38.71 KB  |
| [@&#8203;sentry/vue](https://github.com/sentry/vue)                                                      | 27.58 KB  |
| [@&#8203;sentry/vue](https://github.com/sentry/vue) (incl. Tracing)                                      | 37.75 KB  |
| [@&#8203;sentry/svelte](https://github.com/sentry/svelte)                                                   | 23.46 KB  |
| CDN Bundle                                                       | 24.49 KB  |
| CDN Bundle (incl. Tracing)                                       | 37.6 KB   |
| CDN Bundle (incl. Tracing, Replay)                               | 72.9 KB   |
| CDN Bundle (incl. Tracing, Replay, Feedback)                     | 78.23 KB  |
| CDN Bundle - uncompressed                                        | 71.92 KB  |
| CDN Bundle (incl. Tracing) - uncompressed                        | 111.52 KB |
| CDN Bundle (incl. Tracing, Replay) - uncompressed                | 225.78 KB |
| CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed      | 238.88 KB |
| [@&#8203;sentry/nextjs](https://github.com/sentry/nextjs) (client)                                          | 38.96 KB  |
| [@&#8203;sentry/sveltekit](https://github.com/sentry/sveltekit) (client)                                       | 36.4 KB   |
| [@&#8203;sentry/node](https://github.com/sentry/node)                                                     | 162.85 KB |
| [@&#8203;sentry/node](https://github.com/sentry/node) - without tracing                                   | 99.14 KB  |
| [@&#8203;sentry/aws-serverless](https://github.com/sentry/aws-serverless)                                           | 131.23 KB |

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box

---

This PR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOS4xNTYuMSIsInVwZGF0ZWRJblZlciI6IjM5LjE1Ni4xIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJkZXBlbmRlbmNpZXMiXX0=-->

Reviewed-on: https://git.tristess.app/alexandresoro/ouca-backend/pulls/527
Reviewed-by: Alexandre Soro <code@soro.dev>
Co-authored-by: renovate <renovate@git.tristess.app>
Co-committed-by: renovate <renovate@git.tristess.app>
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.

4 participants