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

MySql 5.7 service and path not found anymore on "Visual Studio 2022" image #3895

Closed
fredericDelaporte opened this issue Nov 19, 2023 · 4 comments

Comments

@fredericDelaporte
Copy link

Since a few days, NHibernate AppVeyor builds are failing their MySql run.

Our yaml uses a database matrix and starts only the required database for the run.

For MySql, that is:

        Start-Service 'MySQL57'
        # Create nhibernate database (not handled by NHibernate.TestDatabaseSetup.dll)
        $env:MYSQL_PWD = 'Password12!'
        & 'C:\Program Files\MySQL\MySQL Server 5.7\bin\mysql' -e 'CREATE DATABASE nhibernate CHARACTER SET utf8 COLLATE utf8_general_ci;' --user=root

But this has started to fail six days ago. We use the "Visual Studio 2022" image.

The build fails to find the MySql service and its path.

Start-Service : Cannot find any service with service name 'MySQL57'.
...
Command executed with exception: The term 'C:\Program Files\MySQL\MySQL Server 5.7\bin\mysql' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path was included, verify that the path is correct and try again.

An example failure is here.
A previous successful build with same configuration is here.

@dennis-gr
Copy link

According to this Visual Studio 2022 now includes MySQL 8.0 (which makes sense since 5.7 has reached EOL).

@fredericDelaporte
Copy link
Author

fredericDelaporte commented Nov 19, 2023

Thanks for the link,
But the documentation still states that is the 5.7 version which is included, and there is no announcement about this change in the update page.

So, it looks like there was no way to plan for this apparent breaking change. And we cannot even be certain of the right course of action to take since the change is not yet actually documented.

@OwenMcDonnell
Copy link

Apologies, merging of the PR for the documentation somehow slipped through the cracks, although the updates had occurred a couple weeks prior.

@fredericDelaporte
Copy link
Author

Ok, thanks.

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

No branches or pull requests

3 participants