-
Notifications
You must be signed in to change notification settings - Fork 7.8k
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
backup location for mysql-8.0.31-winx64.zip #17563
Conversation
Are we sure this is actually caused by rate-limiting? This error appeared for the very first build today. https://github.com/php/php-src/actions/runs/12950383228/job/36123045302 |
i don't have a better theory, and I can download it from my own IP just fine 🤔 |
I wonder why it's stuck on
|
in case of rate-limiting on dev.mysql.com
I suspect this is a chocolatey or dev.mysql.com problem. Possibly, GH runs into rate-limiting because GH actions worker share a public IP. If that's the case, we should report the error rather than trying to fix it ourselves. |
It's a bit ridiculous to download the full MySQL (200MB?) for each and every PR/commit (see #16303 for an attempted alternative). |
@cmb69 I don't object to changing the installation process, but one thing that I dislike is the ever growing list of version numbers that we need to bump, especially if it includes the patch version. Is there a way we can avoid that? |
attempt to replicate choco setup
the problem seems to have vanished by itself (?) |
@iluuu1994, for MariaDB we could likely use their REST API; basically lock in to an LTS, and just detect whether there is a new point release.
I had expected this. We're likely not the only one installing MySQL via GH actions. :) |
Eventually (https://github.com/php/php-src/actions/runs/12949436197/job/36132329873) |
in case of rate-limiting on dev.mysql.com. if the WINDOWS_X64_ZTS CI pass on this PR, that means it's probably working.
ref #17561