asv runners have the same machine name now #388

Merged
mngrif merged 6 commits from mngrif-asv-test into master 2026-02-24 21:51:41 +00:00
Contributor

previously the benchmark graphs were segmented by the name of the machine running the test, making it very difficult to interpret ASV's graphs.

this change should make our graphs readable, however we are still running these benchmarks on shared runners so there may be a high jitter factor between runs.

previously the benchmark graphs were segmented by the name of the machine running the test, making it very difficult to interpret ASV's graphs. this change should make our graphs readable, however we are still running these benchmarks on shared runners so there may be a high jitter factor between runs.
Author
Contributor

the artifacts now show the same machine name of "forgejo-runner" for any pod/container running it, as shown in the artifacts that allow this PR to be merged.
https://git.cleverthis.com/cleveragents/cleveragents-core/actions/runs/4654/artifacts/asv-results-pr

the artifacts now show the same machine name of "forgejo-runner" for any pod/container running it, as shown in the artifacts that allow this PR to be merged. https://git.cleverthis.com/cleveragents/cleveragents-core/actions/runs/4654/artifacts/asv-results-pr
freemo added the
Points
3
Priority
Medium
State
In Review
Type
Bug
labels 2026-02-24 03:36:44 +00:00
mngrif was assigned by freemo 2026-02-24 03:37:55 +00:00
brent.edwards approved these changes 2026-02-24 19:16:04 +00:00
brent.edwards left a comment
Member

Approved.

Approved.
mngrif scheduled this pull request to auto merge when all checks succeed 2026-02-24 19:52:11 +00:00
freemo force-pushed mngrif-asv-test from e04e6978af to 4e750b9b87 2026-02-24 20:48:43 +00:00 Compare
mngrif merged commit 7ec06d5db8 into master 2026-02-24 21:51:41 +00:00
Sign in to join this conversation.
No Reviewers
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: cleveragents/cleveragents-core#388