Fail benchmarks on cache action errors #2

Merged
stephen merged 1 commit from fix-bench-cache-failures into main 2026-06-08 05:12:47 +00:00
Owner

The benchmark scripts were ignoring cache-action failures in their restore and save helpers. A broken restore or save could still produce benchmark rows, which can make a warm run look valid when the cache path failed.

This makes the Rust, Node, and Go benchmark helpers fail the script when the cache action exits nonzero, while still printing the captured cache-action log for diagnosis.

Tests run:

  • bash -n benchmarks/scripts/bench-rust.sh benchmarks/scripts/bench-node.sh benchmarks/scripts/bench-go.sh
The benchmark scripts were ignoring cache-action failures in their restore and save helpers. A broken restore or save could still produce benchmark rows, which can make a warm run look valid when the cache path failed. This makes the Rust, Node, and Go benchmark helpers fail the script when the cache action exits nonzero, while still printing the captured cache-action log for diagnosis. Tests run: - bash -n benchmarks/scripts/bench-rust.sh benchmarks/scripts/bench-node.sh benchmarks/scripts/bench-go.sh
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
rasterstate/fjord-actions!2
No description provided.