Skip to content

test: add SDK benchmarks e2e workflow#1201

Draft
giortzisg wants to merge 3 commits intomasterfrom
test/sdk-benchmarks
Draft

test: add SDK benchmarks e2e workflow#1201
giortzisg wants to merge 3 commits intomasterfrom
test/sdk-benchmarks

Conversation

@giortzisg
Copy link
Contributor

Summary

  • Test PR to validate the reusable e2e benchmark workflow from getsentry/sdk-benchmarks
  • Calls e2e.yml@feat/generic-e2e-workflow which auto-discovers Go apps (net-http, gin, echo) and runs benchmarks

Test plan

  • Verify the benchmark workflow triggers on this PR
  • Verify benchmark results are posted as a PR comment

Note: This is a test PR — do not merge.

🤖 Generated with Claude Code

@github-actions
Copy link

github-actions bot commented Feb 18, 2026

Semver Impact of This PR

None (no version bump detected)

📋 Changelog Preview

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


Breaking Changes 🛠

  • Add support for go 1.26 by giortzisg in #1193
    • bump minimum supported go version to 1.24

New Features ✨

  • Log specific message for RequestEntityTooLarge by giortzisg in #1185

Bug Fixes 🐛

  • Ensure correct signal delivery on multi-client setups by giortzisg in #1190

Internal Changes 🔧

Deps

  • Bump golang.org/x/crypto to 0.48.0 by giortzisg in #1196
  • Use go1.24.0 by giortzisg in #1195
  • Bump github.com/gofiber/fiber/v2 from 2.52.9 to 2.52.11 in /fiber by dependabot in #1191
  • Bump getsentry/craft from 2.19.0 to 2.20.1 by dependabot in #1187

Other

  • Add omitzero and remove custom serialization by giortzisg in #1197
  • Rename Telemetry Processor components by giortzisg in #1186

Other

  • test: add SDK benchmarks e2e workflow by giortzisg in #1201

🤖 This preview updates automatically when you update the PR.

@github-actions
Copy link

github-actions bot commented Feb 18, 2026

SDK Benchmark Results

App SDK Version Status p95 Overhead p99 Overhead
go/echo 0.42.0 ✅ No regression +5.07% +6.03%
go/gin 0.42.0 ❔ Inconclusive (not converged) +4.74% +5.94%
go/net-http 0.42.0 ✅ No regression +3.28% +4.16%

go/echo — No regression

Current branch: 0.42.0 | Latest release: 0.42.0

Metric Latest Release Current Branch Diff p-value
p95 +3.79% [+2.62%, +4.97%] +5.07% [+3.30%, +6.85%] +1.28pp 0.0001
p99 +5.23% [+3.74%, +6.72%] +6.03% [+4.35%, +7.72%] +0.80pp 0.0000
Details
  • Iterations used: 10
  • Converged: yes
  • RPS: N/A
  • Duration: N/A

go/gin — Inconclusive (not converged)

Current branch: 0.42.0 | Latest release: 0.42.0

Metric Latest Release Current Branch Diff p-value
p95 +4.52% [+2.07%, +6.97%] +4.74% [+0.42%, +9.06%] +0.22pp 0.0350
p99 +6.40% [+3.70%, +9.09%] +5.94% [+2.05%, +9.83%] -0.46pp 0.0072
Details
  • Iterations used: 10
  • Converged: no
  • RPS: N/A
  • Duration: N/A

go/net-http — No regression

Current branch: 0.42.0 | Latest release: 0.42.0

Metric Latest Release Current Branch Diff p-value
p95 +4.26% [+2.70%, +5.81%] +3.28% [+2.20%, +4.35%] -0.98pp 0.0001
p99 +4.57% [+2.74%, +6.40%] +4.16% [+0.44%, +7.88%] -0.41pp 0.0321
Details
  • Iterations used: 10
  • Converged: yes
  • RPS: N/A
  • Duration: N/A

@getsentry getsentry deleted a comment from github-actions bot Feb 18, 2026
lcian and others added 2 commits February 19, 2026 15:03
With df=4 (5 iterations), the t-critical value drops from 4.303 to
2.776, making convergence 2x more likely.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
With df=9 (10 iterations), t_crit drops to 2.262 and the convergence
std threshold rises to 2.80pp, making convergence reliably achievable
on shared GitHub Actions runners.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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.

2 participants

Comments