Skip to content

ci(nix): record flake, build, and smoke timings in job summaries - #311

Merged
davidabram merged 1 commit into
perf/ci-stack-2-macos-smoke-reusefrom
perf/ci-stack-3-timing-diagnostics
Oct 11, 2026
Merged

davidabram merged 1 commit into
perf/ci-stack-2-macos-smoke-reusefrom
perf/ci-stack-3-timing-diagnostics

Conversation

@davidabram

@davidabram davidabram commented Oct 9, 2026 •

Copy link
Copy Markdown
Member

Pull request stack

Review and merge bottom-up. Each PR targets the previous PR's branch, so Files changed shows only that step's incremental changes.

Step Pull request Change
1/4 #308 — Faster Nix checks Rust test-profile cache; deduplicate default MBT check
2/4 #309 — Quint workflow cleanup Stop recompiling Rust MBT separately
3/4 #310 — macOS smoke-build reuse Reuse release artifact for macOS smoke tests
4/4 #311 — CI timing summaries Record Nix phase durations in job summaries

You are reviewing step 4/4: #311. Parent: #310. Next: #315 — Crane test dependency preparation. Four follow-up PRs continue through #318.


Measure each successful Nix phase with Bash SECONDS and print its duration to the GitHub Actions job summary: flake checks, Linux native build/smoke, per-platform release validation, and macOS release smoke. This makes future cold/warm CI comparisons discoverable without downloading large logs, while preserving the original commands and failure behavior (run steps already use shell fail-fast).

Validation: workflow lint via nix flake check and Linux/macOS GitHub Actions. Recording uses integer-second precision, not a performance benchmark by itself.

@davidabram
davidabram force-pushed the perf/ci-stack-3-timing-diagnostics branch from f19cf72 to b8cf4de Compare October 10, 2026 14:24
@davidabram
davidabram removed this pull request from stack #313 October 10, 2026 14:25
@davidabram
davidabram added this pull request to stack #312 October 10, 2026 14:27
@davidabram
davidabram removed this pull request from stack #312 October 10, 2026 22:50
@davidabram
davidabram added this pull request to stack #320 October 10, 2026 22:50
@davidabram
davidabram merged commit ba8bacd into remove-tokio-db-runtimes Oct 11, 2026
17 checks passed
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.

1 participant