Skip to content
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

Install SQLServer Express directly instead of using Chocolatey #17583

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

cmb69
Copy link
Member

@cmb69 cmb69 commented Jan 26, 2025

They had quite some outages recently, and we don't gain much of using Chocolatey for this setup anyway. Thus we install the server directly.


This is bit fiddly, but Chocolatey might not be that reliable anymore, and https://github.com/php/php-src/actions/runs/12974386954/job/36184205778 installs the full SQLServer, which is total overkill (and takes a few minutes longer). I think we should use this for master, and see how it goes.

@cmb69
Copy link
Member Author

cmb69 commented Jan 26, 2025

Works fine with https://github.com/ankane/setup-sqlserver, but SQLServer installation takes 5 min; let's try without installing the tools (requires manual installation).

@cmb69
Copy link
Member Author

cmb69 commented Jan 26, 2025

Still super slow; try SQLServer Express instead (LocalDB might be an even lighter alternative, but might need adaption).

@cmb69
Copy link
Member Author

cmb69 commented Jan 26, 2025

Now setup time is okay, but so far tests didn't run. Need to fix.

They had quite some outages recently, and we don't gain much of using
Chocolatey for this setup anyway.  Thus we install the server directly.
@cmb69 cmb69 changed the title Don't install SQLServer via Chocolatey Install SQLServer Express directly instead of using Chocolatey Jan 26, 2025
@cmb69 cmb69 marked this pull request as ready for review January 26, 2025 18:34
@cmb69 cmb69 requested a review from TimWolla as a code owner January 26, 2025 18:34
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