Skip to content
This repository was archived by the owner on May 5, 2025. It is now read-only.

Conversation

@Swatinem
Copy link
Contributor

No description provided.

@Swatinem Swatinem requested a review from a team February 20, 2025 15:36
@Swatinem Swatinem self-assigned this Feb 20, 2025
@codspeed-hq
Copy link

codspeed-hq bot commented Feb 20, 2025

CodSpeed Performance Report

Congrats! CodSpeed is installed 🎉

🆕 0 new benchmarks were detected.

You will start to see performance impacts in the reports once the benchmarks are run from your default branch.

Detected benchmarks

@Swatinem
Copy link
Contributor Author

Interesting.

The first run in valgrind / instrumentation mode was unbearably slow, ~17 minutes vs <2 min locally.
"Downgrading" to the walltime based measurements makes the CI run quite a bit faster, taking "only" 3 minutes, on par with the normal testsuite.

I see both runs in https://codspeed.io/codecov/shared/runs, but the second "walltime" run does not properly load 🤔

@Swatinem Swatinem added this pull request to the merge queue Feb 21, 2025
Merged via the queue into main with commit ae50c42 Feb 21, 2025
8 checks passed
@Swatinem Swatinem deleted the swatinem/ci-benchmarks branch February 21, 2025 08:19
@JerrySentry JerrySentry restored the swatinem/ci-benchmarks branch March 11, 2025 21:01
JerrySentry added a commit that referenced this pull request Mar 11, 2025
@overwatch-beta
Copy link

overwatch-beta bot commented Mar 11, 2025

✅ Sentry found no issues in your recent changes ✅

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants