You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
test(cli): forward tests flake when the port is taken after the CLI's port check #4797
I'm landing the #2912 audit-event follow-ups. #4777 failed Branch Checks because of an unrelated flaky CLI forward test, so the PR needed a rerun before review.
Description
openshell-cli::sandbox_create_lifecycle_integration::sandbox_forward_background_tracks_owned_child_when_pid_discovery_fails failed on x86_64-linux with:
fake forward must bind: Os { code: 98, kind: AddrInUse, message: "Address already in use" }
...
× ssh process started but local forward listener was not reachable
╰─▶ ssh exited with status exit status: 101 before local forward listener opened on 127.0.0.1:39481
#4570 made the forward tests retry on a new port when another process takes theirs, but only when the CLI's pre-check reports "is already in use". The port can also be taken after that check passes and before the fake ssh binds it. The fake then panics with exit status 101, which the retry does not recognize, so the test fails.
Activity
krishicks commentedon Oct 21, 2026
Closing: a flaky-test fix doesn't need an issue. The fix is in #4799.