Skip to content

test(astro): Re-enable server island tracing e2e test in Astro 6#19872

Merged
Lms24 merged 2 commits intodevelopfrom
lms/test-astro-6-serverIslands
Mar 19, 2026
Merged

test(astro): Re-enable server island tracing e2e test in Astro 6#19872
Lms24 merged 2 commits intodevelopfrom
lms/test-astro-6-serverIslands

Conversation

@Lms24
Copy link
Member

@Lms24 Lms24 commented Mar 19, 2026

withastro/astro#15753 was fixed, so our server islands e2e test should work again in Astro 6. This PR bumps the Astro version to the most recent one and re-enables the test

@github-actions
Copy link
Contributor

github-actions bot commented Mar 19, 2026

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


Bug Fixes 🐛

  • (serverless) Add node to metadata by nicohrubec in #19878

Internal Changes 🔧

  • (astro) Re-enable server island tracing e2e test in Astro 6 by Lms24 in #19872

🤖 This preview updates automatically when you update the PR.

@Lms24 Lms24 self-assigned this Mar 19, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Mar 19, 2026

size-limit report 📦

⚠️ Warning: Base artifact is not the latest one, because the latest workflow run is not done yet. This may lead to incorrect results. Try to re-run all tests to get up to date results.

Path Size % Change Change
@sentry/browser 25.52 kB -0.44% -112 B 🔽
@sentry/browser - with treeshaking flags 24.04 kB -0.43% -102 B 🔽
@sentry/browser (incl. Tracing) 42.5 kB -0.27% -115 B 🔽
@sentry/browser (incl. Tracing, Profiling) 47.17 kB -0.23% -105 B 🔽
@sentry/browser (incl. Tracing, Replay) 81.31 kB -0.14% -109 B 🔽
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 70.88 kB -0.17% -116 B 🔽
@sentry/browser (incl. Tracing, Replay with Canvas) 86.01 kB -0.13% -110 B 🔽
@sentry/browser (incl. Tracing, Replay, Feedback) 98.27 kB -0.11% -104 B 🔽
@sentry/browser (incl. Feedback) 42.35 kB -0.24% -99 B 🔽
@sentry/browser (incl. sendFeedback) 30.2 kB -0.37% -110 B 🔽
@sentry/browser (incl. FeedbackAsync) 35.25 kB -0.32% -110 B 🔽
@sentry/browser (incl. Metrics) 26.81 kB -0.43% -115 B 🔽
@sentry/browser (incl. Logs) 26.95 kB -0.43% -116 B 🔽
@sentry/browser (incl. Metrics & Logs) 27.64 kB -0.38% -103 B 🔽
@sentry/react 27.3 kB -0.34% -93 B 🔽
@sentry/react (incl. Tracing) 44.86 kB -0.21% -91 B 🔽
@sentry/vue 29.99 kB -0.32% -94 B 🔽
@sentry/vue (incl. Tracing) 44.38 kB -0.25% -107 B 🔽
@sentry/svelte 25.55 kB -0.45% -115 B 🔽
CDN Bundle 28.2 kB -0.27% -74 B 🔽
CDN Bundle (incl. Tracing) 43.39 kB -0.27% -117 B 🔽
CDN Bundle (incl. Logs, Metrics) 29.07 kB -0.26% -75 B 🔽
CDN Bundle (incl. Tracing, Logs, Metrics) 44.26 kB -0.23% -99 B 🔽
CDN Bundle (incl. Replay, Logs, Metrics) 68.12 kB -0.14% -90 B 🔽
CDN Bundle (incl. Tracing, Replay) 80.23 kB -0.13% -101 B 🔽
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) 81.14 kB -0.12% -91 B 🔽
CDN Bundle (incl. Tracing, Replay, Feedback) 85.78 kB -0.11% -91 B 🔽
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) 86.67 kB -0.12% -101 B 🔽
CDN Bundle - uncompressed 82.32 kB -0.37% -304 B 🔽
CDN Bundle (incl. Tracing) - uncompressed 128.25 kB -0.25% -315 B 🔽
CDN Bundle (incl. Logs, Metrics) - uncompressed 85.19 kB -0.36% -304 B 🔽
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed 131.11 kB -0.24% -315 B 🔽
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed 208.82 kB -0.15% -304 B 🔽
CDN Bundle (incl. Tracing, Replay) - uncompressed 245.1 kB -0.13% -315 B 🔽
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed 247.95 kB -0.13% -315 B 🔽
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 258.01 kB -0.13% -315 B 🔽
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed 260.85 kB -0.13% -315 B 🔽
@sentry/nextjs (client) 47.28 kB -0.19% -86 B 🔽
@sentry/sveltekit (client) 42.95 kB -0.28% -119 B 🔽
@sentry/node-core 56.28 kB -0.12% -62 B 🔽
@sentry/node 173.21 kB +0.04% +52 B 🔺
@sentry/node - without tracing 96.27 kB -0.08% -76 B 🔽
@sentry/aws-serverless 113.28 kB -0.06% -61 B 🔽

View base workflow run

….serverIslands.test.ts

Co-authored-by: Charly Gomez <charly.gomez@sentry.io>
@Lms24 Lms24 enabled auto-merge March 19, 2026 14:57
@Lms24 Lms24 disabled auto-merge March 19, 2026 14:57
@Lms24 Lms24 enabled auto-merge (squash) March 19, 2026 14:59
@github-actions
Copy link
Contributor

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 8,938 - 11,512 -22%
GET With Sentry 1,662 19% 1,991 -17%
GET With Sentry (error only) 6,091 68% 7,302 -17%
POST Baseline 1,187 - 1,265 -6%
POST With Sentry 574 48% 633 -9%
POST With Sentry (error only) 1,047 88% 1,136 -8%
MYSQL Baseline 3,257 - 3,457 -6%
MYSQL With Sentry 447 14% 431 +4%
MYSQL With Sentry (error only) 2,630 81% 2,862 -8%

View base workflow run

@Lms24 Lms24 merged commit 399df7c into develop Mar 19, 2026
236 checks passed
@Lms24 Lms24 deleted the lms/test-astro-6-serverIslands branch March 19, 2026 15:16
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.

3 participants