-
-
Notifications
You must be signed in to change notification settings - Fork 32
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Action randomly fail on windows #79
Comments
Can confirm, also experiencing this issue randomly in the PR pact-foundation/pact-php#194 |
A recent change might have fixed this or at least made it less likely to occur: 370d521 Please keep trying it out and let me know if you see these errors again. My hypothesis right now is that, since the cache keys were the same across operating systems, if (for example) the job for Windows tried to run with PHP 7.4 and lowest dependencies at the same time as the job for Linux with PHP 7.4 and lowest dependencies, then maybe there was some kind of read-access blocking to the device, since I assume it's sharing the same mounted filesystem for the caches. Anyway, I have no confirmation this has fixed it, but I haven't seen the issue since I made this change. |
I just got a failure on Windows in another repo: https://github.com/ramsey/php-library-starter-kit/runs/2070621484
|
Another error in a different repo. Different error, but I think it's the same root cause. https://github.com/ramsey/devtools-lib/runs/2078491516?check_suite_focus=true
|
Yet another in a different repo. This is the "access is denied" error: https://github.com/ramsey/conventional-commits/runs/2108197091?check_suite_focus=true |
https://github.com/LastDragon-ru/lara-asp-formatter/runs/2108870196?check_suite_focus=true but seems it occurs less than before |
Also another error:
|
…ts still randomly fails (ramsey/composer-install#79)
…ts still randomly fails (ramsey/composer-install#79)
…ts still randomly fails (ramsey/composer-install#79)
…ts still randomly fails (ramsey/composer-install#79)
…ts still randomly fails (ramsey/composer-install#79)
…ts still randomly fails (ramsey/composer-install#79)
…ts still randomly fails (ramsey/composer-install#79)
…ts still randomly fails (ramsey/composer-install#79)
…ts still randomly fails (ramsey/composer-install#79)
I can confirm this issue is still there. we do have the same issue in https://github.com/phpDocumentor/phpDocumentor |
If you look at the order of the restore messages this seems to be odd... Linux does this:
it looks like the cache restore is taking to long on windows, and it continues before it was fully restored? |
The issue still occurs (https://github.com/phpDocumentor/phpDocumentor/runs/3630487891?check_suite_focus=true); maybe something as simple as a retry option would be useful? This could also help in cases of flaky networks when downloading/installing composer packages |
In ramsey/composer-install#79 we see the same behaviour as in our own builds; because the package doesn't natively supports a retry feature I have added 2 'backup' tasks doing the same thing. I hope this will make builds more stable
This is fixed in v2, please upgrade using Thanks! |
Action: https://github.com/LastDragon-ru/lara-asp/blob/f990092be38daceb9e7323ccd7683c8d92c974a3/.github/workflows/phpunit.yml
The text was updated successfully, but these errors were encountered: