Skip to content

ci: execute Cloud host identity tests on relevant PRs #30420

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
teamleaderleo merged 1 commit into from
Nov 1, 2026
Merged

ci: execute Cloud host identity tests on relevant PRs #30420

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
23 changes: 23 additions & 0 deletions .github/workflows/cmux-tui-spec.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -464,6 +464,29 @@ jobs:
BASE_SHA: ${{ steps.base.outputs.sha }}
run: python3 "$RUNNER_TEMP/cmux-tui-clippy-changed.py" --repo "$GITHUB_WORKSPACE" --base "$BASE_SHA"

# The host's clone/enrollment and root-peer IPC invariants must execute,
# not only compile. Reuse this job's Rust setup and warm target directory.
- name: Cloud host identity tests
if: ${{ !cancelled() }}
env:
BASE_SHA: ${{ steps.base.outputs.sha }}
run: |
set -o pipefail
if [[ "$BASE_SHA" =~ ^[0-9a-f]{40}$ ]] \
&& changed=$(git diff --no-renames --name-only "$BASE_SHA" HEAD); then
if ! python3 scripts/ci/cargo_test_verdict.py touches cmux-tui/crates/cmux-host <<<"$changed"; then
echo "No Cloud host test input changed; skipping identity tests." >> "$GITHUB_STEP_SUMMARY"
exit 0
fi
fi
cd cmux-tui
log="$RUNNER_TEMP/cmux-host-identity-tests.log"
cargo test --locked -p cmux-host --lib identity 2>&1 | tee "$log"
if ! grep -qE 'test result: ok\. [1-9][0-9]* passed' "$log"; then
echo "::error title=Cloud host identity tests::identity selected no test"
exit 1
fi

# The panic ratchet (plans/cmux-next/rust-panic-free.md) on the same packages:
# their production targets with the ratchet's lints, judged against the baseline
# at HEAD. Only cmux-tui.yml's dispatch run checked it before, so #27935 landed two
Expand Down
Loading