Skip to content

CI: Fix SNMP failures in Github CI on Windows#21924

Draft
LamentXU123 wants to merge 4 commits intophp:masterfrom
LamentXU123:fix-CI
Draft

CI: Fix SNMP failures in Github CI on Windows#21924
LamentXU123 wants to merge 4 commits intophp:masterfrom
LamentXU123:fix-CI

Conversation

@LamentXU123
Copy link
Copy Markdown
Contributor

@LamentXU123 LamentXU123 commented May 1, 2026

I think the cause of #21919 might be race condition. The test suite start when the SNMP service in 127.0.0.1 is not (fully) installed.

So I wrote a small script to check and wait before the service is usable and let the test start to run. This might be helpful in the remote to fix the CI :)

After this PR is tackled we can close #21919 , TiA.

@iluuu1994
Copy link
Copy Markdown
Member

This is not super plausible. From https://github.com/php/php-src/actions/runs/25172619354/job/73795997139, with timestamps enabled:

image

As you can see, the first failing test appears ~15 minutes after test have started running.

@LamentXU123
Copy link
Copy Markdown
Contributor Author

LamentXU123 commented May 1, 2026

This is not super plausible. From https://github.com/php/php-src/actions/runs/25172619354/job/73795997139, with timestamps enabled:

Ah yes, and also not all SNMP tests are failing. Some of them pass, some of them don't. And in each time the passed&failed tests are different. See https://github.com/php/php-src/actions/runs/25172619354/job/73795997139 and https://github.com/php/php-src/actions/runs/25187112602/job/73847232154

Would it be more reasonable to be a frequency issue...? The server couldn't handle many SNMP requests in short time ranges so some tests fails to reach the server.

(Idk I am just guessing, tests work fine locally in my Windows machine)

@iluuu1994
Copy link
Copy Markdown
Member

Not sure, the tests are already sequentialized using the CONFLICTS file.

@LamentXU123 LamentXU123 marked this pull request as draft May 2, 2026 08:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

SNMP tests fail on windows platforms in Github CI

2 participants