Skip to content

Conversation

pull[bot]
Copy link

@pull pull bot commented Oct 17, 2025

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

iluuu1994 and others added 13 commits October 17, 2025 13:04
There have been misterious macOS failures with the following error for a long
time, which would usually happen 1-2x per nightly run:

> The hosted runner lost communication with the server. Anything in your
> workflow that terminates the runner process, starves it for CPU/Memory, or
> blocks its network access can cause this error.

After way too much debugging, it looks like this is caused by the two fpm tests
skipped in this commit. When there's a failure, the responsible test will run
for at least 120 seconds until the job is eventually killed by GitHub Actions.
It's not clear yet why the tests stall.

Debugging this is a PITA because the GitHub Actions log gets partially lost for
killed jobs. It took an external log server to actually figure out where the job
fails. Let's disable the tests first to be sure this actually solves the issue.
* PHP-8.1:
  [skip ci] Skip suspicious fpm tests on GH actions & macOS
* PHP-8.2:
  [skip ci] Skip suspicious fpm tests on GH actions & macOS
* PHP-8.3:
  [skip ci] Skip suspicious fpm tests on GH actions & macOS
* PHP-8.4:
  [skip ci] Skip suspicious fpm tests on GH actions & macOS
* PHP-8.5:
  [skip ci] Skip suspicious fpm tests on GH actions & macOS
Should've been skip, not flaky...
* PHP-8.1:
  [skip ci] Fix typo
* PHP-8.2:
  [skip ci] Fix typo
* PHP-8.3:
  [skip ci] Fix typo
* PHP-8.4:
  [skip ci] Fix typo
* PHP-8.5:
  [skip ci] Fix typo
@pull pull bot locked and limited conversation to collaborators Oct 17, 2025
@pull pull bot added the ⤵️ pull label Oct 17, 2025
@pull pull bot merged commit 26c96d3 into wudi:master Oct 17, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants