Description
The test sends reload, waits for TCP port-up, then fires a single HTTP request with no retry. Port-up does not mean worker-ready after a reload, so the request can land mid-restart and die with cURL error 56: Recv failure: Connection reset by peer.
Flaked twice during the #588 cycle (PR #806) with the identical signature on different legs, green on re-run / other legs both times:
- run 34405095115,
Tests (8.2, 6.4.*) leg
- run 34405583432,
Tests (8.4, 8.0.*) leg
Where
tests/WorkermanCommandTest.php:81-93
Suggested fix
Retry the post-reload request (a few attempts with a short sleep) instead of firing once, or wait for an actual HTTP 200 rather than TCP port-up before asserting.
Description
The test sends
reload, waits for TCP port-up, then fires a single HTTP request with no retry. Port-up does not mean worker-ready after a reload, so the request can land mid-restart and die withcURL error 56: Recv failure: Connection reset by peer.Flaked twice during the #588 cycle (PR #806) with the identical signature on different legs, green on re-run / other legs both times:
Tests (8.2, 6.4.*)legTests (8.4, 8.0.*)legWhere
tests/WorkermanCommandTest.php:81-93Suggested fix
Retry the post-reload request (a few attempts with a short sleep) instead of firing once, or wait for an actual HTTP 200 rather than TCP port-up before asserting.