Skip to content

ci(stable-mir-ui): make manual UI workflow runnable#975

Open
Stevengre wants to merge 2 commits intocodex/import-stable-mir-ui-testsfrom
codex/stable-mir-ui-ci
Open

ci(stable-mir-ui): make manual UI workflow runnable#975
Stevengre wants to merge 2 commits intocodex/import-stable-mir-ui-testsfrom
codex/stable-mir-ui-ci

Conversation

@Stevengre
Copy link
Contributor

@Stevengre Stevengre commented Mar 5, 2026

Summary

  • make the manual test-stable-mir-ui workflow runnable on the self-hosted runner
  • keep the workflow's Rust toolchain aligned with rust-toolchain.toml
  • upload the real failure outputs from inside the Docker container, including updated skip.txt when requested

Context

Testing

  • Manual validation on zhaoji: rebuilt the workflow image after deriving RUST_TOOLCHAIN=nightly-2024-11-29 from rust-toolchain.toml
  • Manual validation on zhaoji: verified make stable-mir-json succeeds before the UI run
  • Manual validation on zhaoji: verified test-filter='foo or bar' is passed through correctly
  • Manual validation on zhaoji: verified proof artifacts and updated skip.txt are copied back from the container
  • Manual validation on zhaoji: ran a single-case smoke path (tests/ui/issues/issue-11205.rs) through make stable-mir-json, make build, and make test-stable-mir-ui; workflow plumbing succeeded and produced the expected failure artifacts, while the proof itself failed with AssertionError: Proof did not pass

@Stevengre Stevengre force-pushed the codex/import-stable-mir-ui-tests branch from 40658b4 to 2d4a181 Compare March 5, 2026 15:26
@Stevengre Stevengre force-pushed the codex/stable-mir-ui-ci branch from f8b82d0 to b985981 Compare March 5, 2026 15:27
@Stevengre Stevengre force-pushed the codex/import-stable-mir-ui-tests branch from 2d4a181 to a9c1f58 Compare March 5, 2026 15:29
Add workflow_dispatch-only GitHub Actions workflow that:
- Checks out rust-lang/rust at the pinned commit
- Builds kmir in Docker and runs test-stable-mir-ui
- Supports test-filter, timeout, and update-skip inputs
- Uploads proof show output on failure and updated skip.txt
@Stevengre Stevengre force-pushed the codex/stable-mir-ui-ci branch from b985981 to 3d00e67 Compare March 5, 2026 15:29
Build stable-mir-json before running the UI harness, derive the Rust toolchain from rust-toolchain.toml, and copy failure artifacts plus update-skip output back from the container before upload.
@Stevengre Stevengre changed the title ci(stable-mir-ui): add manual workflow for UI test runs ci(stable-mir-ui): fix workflow setup and artifacts Mar 6, 2026
@Stevengre Stevengre changed the title ci(stable-mir-ui): fix workflow setup and artifacts ci(stable-mir-ui): make manual UI workflow runnable Mar 6, 2026
@Stevengre Stevengre marked this pull request as ready for review March 6, 2026 04:55
@Stevengre Stevengre self-assigned this Mar 6, 2026
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