Skip to content

ci: fix remaining master CI failures#15123

Merged
grandizzy merged 2 commits into
masterfrom
centaur/green-master-ci
Jun 10, 2026
Merged

ci: fix remaining master CI failures#15123
grandizzy merged 2 commits into
masterfrom
centaur/green-master-ci

Conversation

@decofe

@decofe decofe commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Summary

  • keep sccache enabled in CI MPP, but remove Swatinem/rust-cache from that workflow because the action itself fails before build/test starts
  • extend the slow nextest override to cover the remaining slow CI tests seen on master: chisel live logs and wallet session wrapper
  • update slow nextest filters to match module-qualified names such as cmd::clone::tests::test_clone_*, bind::..., cmd::can_bind_e2e, and script::...
  • make the wallet session project-root assertion tolerate Windows-rendered temp paths while still checking root forwarding
  • fix the Tempo dependency bump workflow to create PRs against master, not main

Why

  • master already includes ci: increase timeouts for slow CLI integration tests #15117, but the latest master CI still failed on additional timeout-prone tests, a Windows-only rendered path assertion, and CI MPP rust-cache setup
  • the previous timeout filters were anchored to bare test names, but nextest reports these tests with module-qualified names, so the override did not apply and tests still timed out at the default 90s
  • the PR run showed CI MPP still failing inside Swatinem/rust-cache even with save-if: false, so CI MPP now relies on sccache only
  • the scheduled Tempo dependency bump job successfully pushed its branch, then failed PR creation because this repo's base branch is master

Validation

  • git diff --check
  • cargo metadata --no-deps --format-version 1
  • rustfmt crates/script/src/wallet_session.rs --edition 2024 --check
  • parsed .config/nextest.toml with Python tomllib
  • regex sanity check matched latest failed/flaky nextest names, including cmd::clone::tests::test_clone_*, bind tests, can_bind_e2e, script concurrency, chisel live logs, wallet session, and vaddr_e2e::...

Note: targeted cargo tests were started but stopped during cold dependency compilation before test execution.

Prompted by: @grandizzy

@decofe decofe force-pushed the centaur/green-master-ci branch from c96eb7f to f970e29 Compare June 10, 2026 05:45
@grandizzy grandizzy marked this pull request as ready for review June 10, 2026 06:43
@grandizzy grandizzy merged commit d51d76e into master Jun 10, 2026
19 checks passed
@grandizzy grandizzy deleted the centaur/green-master-ci branch June 10, 2026 06:44
@github-project-automation github-project-automation Bot moved this to Done in Foundry Jun 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants