-
Notifications
You must be signed in to change notification settings - Fork 66
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 MariaDB 10.11 #223
Conversation
[test] |
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]>
268709b
to
d752c16
Compare
[test] |
2 similar comments
[test] |
[test] |
f40 mariadb10.11 is already in bodhi: https://bodhi.fedoraproject.org/updates/FEDORA-2024-fd6c145765 |
It will no longer be modular, so this PR would need to be adjusted accordingly, when the rpms are out. |
[test] |
Because of mariadb10.11 is not yet in branch f40. Do not test mariadb10.11 on Fedora. |
[test] |
RHEL8 and RHEL9 tests are failing:
Failed test is here:
|
[test] |
There is a missing entry for 10.11 at https://github.com/sclorg/mariadb-container/blob/master/root-common/usr/share/container-scripts/mysql/init/40-datadir-action.sh#L8 -- it should likely include links to all versions between 10.5 and 10.11 |
6b549a5
to
f16ea18
Compare
I've pushed an additional commit that should help fixing the upgrade. |
f16ea18
to
447d6f4
Compare
[test] |
447d6f4
to
8603271
Compare
[test] |
Could you please squash the fix up commits before merging? |
Definitely. But I would like to merge it as soon as all tests are passing. |
1a9aff3
to
c6244f3
Compare
Do not use modular based mariadb:10.11 Use proper version for 10.11/Dockerfile Enable module for 10.11/Dockerfile.rhel9 and 10.11/Dockerfile.c9s Version 10.11 is in scope for CentOS 7 and RHEL7. Update 10.11/root/usr/share/container-scripts/mysql/README.md
c6244f3
to
3b4e42a
Compare
Rebased |
The failure of 10.5 on RHEL-9 was likely caused by a wrong compose (mariadb 10.5 was missing), it seems to be fixed now, so re-running the tests. |
Signed-off-by: Petr "Stone" Hracek <[email protected]>
[test] |
[test-openshift] |
[test] |
Signed-off-by: Petr "Stone" Hracek <[email protected]>
7970af1
to
2d92a6f
Compare
[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.
LGTM, thanks.
Support MariaDB 10.11 for building, testing, and pushing to Quay.io.
The pull request is separated to more commits.