Skip to content

Commit 1f68a9a

Browse files
authored
Merge branch 'develop' into feat/monitor-trace
2 parents 8771d6f + 3b0728f commit 1f68a9a

File tree

453 files changed

+15687
-3255
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

453 files changed

+15687
-3255
lines changed

.craft.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -153,6 +153,7 @@ targets:
153153
- nodejs18.x
154154
- nodejs20.x
155155
- nodejs22.x
156+
- nodejs24.x
156157
license: MIT
157158

158159
# CDN Bundle Target
Lines changed: 32 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,32 @@
1+
# Bump OpenTelemetry instrumentations
2+
3+
1. Ensure you're on the `develop` branch with the latest changes:
4+
- If you have unsaved changes, stash them with `git stash -u`.
5+
- If you're on a different branch than `develop`, check out the develop branch using `git checkout develop`.
6+
- Pull the latest updates from the remote repository by running `git pull origin develop`.
7+
8+
2. Create a new branch `bump-otel-{yyyy-mm-dd}`, e.g. `bump-otel-2025-03-03`
9+
10+
3. Create a new empty commit with the commit message `feat(deps): Bump OpenTelemetry instrumentations`
11+
12+
4. Push the branch and create a draft PR, note down the PR number as {PR_NUMBER}
13+
14+
5. Create a changelog entry in `CHANGELOG.md` under
15+
`- "You miss 100 percent of the chances you don't take. — Wayne Gretzky" — Michael Scott` with the following format:
16+
`- feat(deps): Bump OpenTelemetry instrumentations ([#{PR_NUMBER}](https://github.com/getsentry/sentry-javascript/pull/{PR_NUMBER}))`
17+
18+
6. Find the "Upgrade OpenTelemetry instrumentations" rule in `.cursor/rules/upgrade_opentelemetry_instrumentations` and
19+
follow those complete instructions step by step.
20+
- Create one commit per package in `packages/**` with the commit message
21+
`Bump OpenTelemetry instrumentations for {SDK}`, e.g. `Bump OpenTelemetry instrumentation for @sentry/node`
22+
23+
- For each OpenTelemetry dependency bump, record an entry in the changelog with the format indented under the main
24+
entry created in step 5: `- Bump @opentelemetry/{instrumentation} from {previous_version} to {new_version}`, e.g.
25+
`- Bump @opentelemetry/instrumentation from 0.204.0 to 0.207.0` **CRITICAL**: Avoid duplicated entries, e.g. if we
26+
bump @opentelemetry/instrumentation in two packages, keep a single changelog entry.
27+
28+
7. Regenerate the yarn lockfile and run `yarn yarn-deduplicate`
29+
30+
8. Run `yarn fix` to fix all formatting issues
31+
32+
9. Finally update the PR description to list all dependency bumps
Lines changed: 33 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,33 @@
1+
---
2+
description: Use this rule if you are looking to grade OpenTelemetry instrumentations for the Sentry JavaScript SDKs
3+
globs: *
4+
alwaysApply: false
5+
---
6+
7+
# Upgrading OpenTelemetry instrumentations
8+
9+
1. For every package in packages/\*\*:
10+
- When upgrading dependencies for OpenTelemetry instrumentations we need to first upgrade `@opentelemetry/instrumentation` to the latest version.
11+
**CRITICAL**: `@opentelemetry/instrumentation` MUST NOT include any breaking changes.
12+
Read through the changelog of `@opentelemetry/instrumentation` to figure out if breaking changes are included and fail with the reason if it does include breaking changes.
13+
You can find the changelog at `https://github.com/open-telemetry/opentelemetry-js/blob/main/experimental/CHANGELOG.md`
14+
15+
- After successfully upgrading `@opentelemetry/instrumentation` upgrade all `@opentelemetry/instrumentation-{instrumentation}` packages, e.g. `@opentelemetry/instrumentation-pg`
16+
**CRITICAL**: `@opentelemetry/instrumentation-{instrumentation}` MUST NOT include any breaking changes.
17+
Read through the changelog of `@opentelemetry/instrumentation-{instrumentation}` to figure out if breaking changes are included and fail with the reason if it does including breaking changes.
18+
You can find the changelogs at `https://github.com/open-telemetry/opentelemetry-js-contrib/blob/main/packages/instrumentation-{instrumentation}/CHANGELOG.md`.
19+
20+
- Finally, upgrade third party instrumentations to their latest versions, these are currently:
21+
- @prisma/instrumentation
22+
23+
**CRITICAL**: Upgrades to third party instrumentations MUST NOT include breaking changes.
24+
Read through the changelog of each third party instrumentation to figure out if breaking changes are included and fail with the reason if it does include breaking changes.
25+
26+
2. For packages and apps in dev-packages/\*\*:
27+
- If an app depends on `@opentelemetry/instrumentation` >= 0.200.x upgrade it to the latest version.
28+
**CRITICAL**: `@opentelemetry/instrumentation` MUST NOT include any breaking changes.
29+
30+
- If an app depends on `@opentelemetry/instrumentation-http` >= 0.200.x upgrade it to the latest version.
31+
**CRITICAL**: `@opentelemetry/instrumentation-http` MUST NOT include any breaking changes.
32+
33+
3. Generate a new yarn lock file.

.github/ISSUE_TEMPLATE/bug.yml

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -136,13 +136,14 @@ body:
136136
id: additional
137137
attributes:
138138
label: Additional Context
139-
description:
140-
Add any other context here. Please keep the pre-filled text, which helps us manage issue prioritization.
141-
value: |-
142-
<sub>Tip: React with 👍 to help prioritize this issue. Please use comments to provide useful context, avoiding `+1` or `me too`, to help us triage it.</sub>
139+
description: Add any other context here.
143140
validations:
144141
required: false
145-
- type: markdown
142+
- type: dropdown
146143
attributes:
147-
value: |-
148-
## Thanks 🙏
144+
label: 'Priority'
145+
description: Please keep the pre-filled option, which helps us manage issue prioritization.
146+
default: 0
147+
options:
148+
- <sub>React with 👍 to help prioritize this issue. Please use comments to provide useful context, avoiding `+1`
149+
or `me too`, to help us triage it.</sub>

.github/ISSUE_TEMPLATE/feature.yml

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -27,14 +27,14 @@ body:
2727
id: additional
2828
attributes:
2929
label: Additional Context
30-
description:
31-
Add any other context here. Please keep the pre-filled text, which helps us manage feature prioritization.
32-
value: |-
33-
<sub>Tip: React with 👍 to help prioritize this improvement. Please use comments to provide useful context, avoiding `+1` or `me too`, to help us triage it.</sub>
30+
description: Add any other context here.
3431
validations:
3532
required: false
36-
- type: markdown
33+
- type: dropdown
3734
attributes:
38-
value: |-
39-
## Thanks 🙏
40-
Check our [triage docs](https://open.sentry.io/triage/) for what to expect next.
35+
label: 'Priority'
36+
description: Please keep the pre-filled option, which helps us manage issue prioritization.
37+
default: 0
38+
options:
39+
- <sub>React with 👍 to help prioritize this issue. Please use comments to provide useful context, avoiding `+1`
40+
or `me too`, to help us triage it.</sub>

.github/dependabot.yml

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,9 +14,15 @@ updates:
1414
interval: 'weekly'
1515
allow:
1616
- dependency-name: '@sentry/*'
17-
- dependency-name: '@opentelemetry/*'
18-
- dependency-name: '@prisma/instrumentation'
1917
- dependency-name: '@playwright/test'
18+
- dependency-name: '@opentelemetry/*'
19+
ignore:
20+
- dependency-name: '@opentelemetry/instrumentation'
21+
- dependency-name: '@opentelemetry/instrumentation-*'
22+
groups:
23+
opentelemetry:
24+
patterns:
25+
- '@opentelemetry/*'
2026
versioning-strategy: increase
2127
commit-message:
2228
prefix: feat

.github/dependency-review-config.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,3 +9,5 @@ allow-ghsas:
99
- GHSA-v784-fjjh-f8r4
1010
# Next.js Cache poisoning - We require a vulnerable version for E2E testing
1111
- GHSA-gp8f-8m3g-qvj9
12+
# devalue vulnerability - this is just used by nuxt & astro as transitive dependency
13+
- GHSA-vj54-72f3-p5jv

.github/workflows/auto-release.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ jobs:
1515
steps:
1616
- name: Get auth token
1717
id: token
18-
uses: actions/create-github-app-token@a8d616148505b5069dccd32f177bb87d7f39123b # v2.1.1
18+
uses: actions/create-github-app-token@67018539274d69449ef7c02e8e71183d1719ab42 # v2.1.4
1919
with:
2020
app-id: ${{ vars.SENTRY_RELEASE_BOT_CLIENT_ID }}
2121
private-key: ${{ secrets.SENTRY_RELEASE_BOT_PRIVATE_KEY }}
@@ -42,7 +42,7 @@ jobs:
4242
echo "version=$version" >> $GITHUB_OUTPUT
4343
4444
- name: Set up Node
45-
uses: actions/setup-node@v4
45+
uses: actions/setup-node@v6
4646
with:
4747
node-version-file: 'package.json'
4848

0 commit comments

Comments
 (0)