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

Pin CI to Ubuntu 22.04 #566

Merged
merged 1 commit into from
Oct 22, 2024
Merged

Conversation

andyundso
Copy link
Member

Microsoft SQL Server 2017 will not work on Ubuntu 24.04.

Microsoft SQL Server 2017 will not work on Ubuntu 24.04.
@aharpervc
Copy link
Contributor

For the nth time, remind me why we're unable to run tests on the stock container images?

@andyundso
Copy link
Member Author

we are using the Stock container images, but something with the SQL Server 2017 image is broken on Ubuntu 24.04 (I assume the old 18.04 Kernel of SQL Server 2017 image does not work with newer distros).

@aidanharan
Copy link
Contributor

It may be related to microsoft/mssql-docker#899

@aharpervc
Copy link
Contributor

we are using the Stock container images, but something with the SQL Server 2017 image is broken on Ubuntu 24.04 (I assume the old 18.04 Kernel of SQL Server 2017 image does not work with newer distros).

No I mean like, mcr.microsoft.com/mssql/server:2019-latest, etc rather than installing

@andyundso
Copy link
Member Author

we only install on Windows, as there we do not have container images available. on Linux, we use containers (see e.g. here, just need to open the part which says "Setup MSSQL" and there you see the docker pull logs).

@aharpervc
Copy link
Contributor

Ah, I see potatoqualitee/mssqlsuite is running the linux image as you said, when possible

@andyundso andyundso merged commit f971cce into rails-sqlserver:master Oct 22, 2024
44 checks passed
@andyundso andyundso deleted the pin-ubuntu-2204 branch October 22, 2024 19:03
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.

3 participants