-
Notifications
You must be signed in to change notification settings - Fork 200
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
Support mysql 8.4 on CentOS Stream 10 #376
Conversation
Signed-off-by: Petr "Stone" Hracek <[email protected]>
Signed-off-by: Petr "Stone" Hracek <[email protected]>
Signed-off-by: Petr "Stone" Hracek <[email protected]>
Signed-off-by: Petr "Stone" Hracek <[email protected]>
Signed-off-by: Petr "Stone" Hracek <[email protected]>
adcf054
to
c1352ad
Compare
Signed-off-by: Petr "Stone" Hracek <[email protected]>
Dockerfile diffs. Difference Dockerfile.c9s against Dockerfile.c10s:
Differences between Dockerfile.fedora against Dockerfile.c10s
Differences between Dockerfile.rhel9 and Dockerfile.c10s
|
@hhorak @FaramosCZ I have caught some errors. What could be a problem?
|
Weird. The basic setup & usage works without any issue in VM. (1) Is the problem reproducible? (2) Can I get a result of (or an equivalent of) |
There seems to be something different when it comes to datadir initialization and steps we do after it in the container, as it starts to fail pretty soon:
The issues after are symptoms of not having the daemon started properly. |
For the reproducer, this is something that might get to the same error:
|
There are more incompatible changes in 8.4 (https://dev.mysql.com/doc/refman/8.4/en/mysql-nutshell.html) that we'll need to address in the container, so far:
|
…n_plugin variable In order to maintain strong authentication by default, unless a user changes the setting by setting MYSQL_AUTHENTICATION_POLICY or MYSQL_DEFAULT_AUTHENTICATION_PLUGIN, value 'caching_sha2_password,,' will be used, which means caching_sha2_password will be required as the first factor and 2nd and 3rd factor would be optional. If MYSQL_DEFAULT_AUTHENTICATION_PLUGIN is set and MYSQL_AUTHENTICATION_POLICY is not, the authentication_policy will be set to 'MYSQL_DEFAULT_AUTHENTICATION_PLUGIN,,'
procps-ng ships pgrep tool that is used to monitor processes when waiting for mysqld to shutdown properly
the reason is that starting MySQL 8.4, mysql_native_password is not available unless explicitly enabled in the server configuration.
…anyway Starting 8.0.16, mysql_upgrade became NooP and server executes all actions automatically when necessary. After mysql_upgrade being labeled as deprecated since this version, it is missing in 8.4.x entirely. In addition to that, the file mysql_upgrade_info file has been deprecated as well and removed in 8.4.x as well, which makes all the tests done around upgrade and mysql_upgrade_info invalid. Since we have a recent enough version in production for enough time, it seems ok to remove the logic about upgrading the datadir entirely from the container image and leave everything on the server itself. The only datadir actions that seem to make sense even in 8.4.x are analyze and optimize, so those are kept untouched.
Many cases were not guarded by ct_check_testcase_result call. Also removes warnings like: egrep: warning: egrep is obsolescent; using grep -E Some mysql_cmd calls and its output is not relevant for tests, do not print it.
[test] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As I am not an expert with MySQL, please address my comments on the base of your better knowledge.
Othewise. LGTM
@FaramosCZ PTAL. Your review is mandatory |
Otherwise LGTM |
Signed-off-by: Petr "Stone" Hracek <[email protected]>
Signed-off-by: Petr "Stone" Hracek <[email protected]>
Signed-off-by: Petr "Stone" Hracek <[email protected]>
Let's get test it. As soon as tests PASSWED we can merge it. |
[test-openshift][test-openshift-pytest] |
@phracek thanks for taking care for the left-over changes, I was just about to finish it and saw you did it for me, thanks a lot!! |
This pull request adds support for testing Mysql 8.4 on CentOS Stream 10.
The rest of operation systems is skipped for now.
The pull request is separated into several commit for better review.