Skip to content

[ticket/17555] Another attempt at fixing Windows test runs#6996

Open
iMattPro wants to merge 3 commits into
phpbb:masterfrom
iMattPro:ticket/17555
Open

[ticket/17555] Another attempt at fixing Windows test runs#6996
iMattPro wants to merge 3 commits into
phpbb:masterfrom
iMattPro:ticket/17555

Conversation

@iMattPro

@iMattPro iMattPro commented Jul 20, 2026

Copy link
Copy Markdown
Member

PHPBB-17555

Checklist:

  • Correct branch: master for new features; 3.3.x for fixes
  • Tests pass
  • Code follows coding guidelines: master and 3.3.x
  • Commit follows commit message format

Tracker ticket:

https://tracker.phpbb.com/browse/PHPBB-17555

@iMattPro

iMattPro commented Jul 20, 2026

Copy link
Copy Markdown
Member Author

Based on these experiments—four parallel Windows jobs create about a 68% chance of at least one stall, versus roughly 25% with one job. So testing Windows on just one PHP version instead of four reduces exposure. These failures are always random and due to GitHub’s underlying runner instability, which we can't fix.

Changes include:

  • Removed IIS, FastCGI, URL Rewrite, Node, and unnecessary ACL setup (no longer needed since not running functional tests).
  • Removed the ineffective Windows pcntl request.
  • Replaced three same-runner retries with a native 10-minute test timeout (all this did was make stalled tests run for 50 minutes).
  • Added a 20-minute job timeout.
  • Added PostgreSQL 60-second statement and 30-second lock timeouts.
  • Reduced Windows testing to just PHP 8.2, the minimum requirement for phpBB (all the other mysql tests handle the later PHP versions)
  • Isolated functional tests from unit tests and added a PHP-FPM safeguard to recycle after 100 HTTP requests, clearing accumulated state and resources throughout the functional suite.

@iMattPro
iMattPro force-pushed the ticket/17555 branch 3 times, most recently from f6d4a43 to cc2a6e6 Compare July 21, 2026 22:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant