- Issue created by @fjgarlin
- @fjgarlin opened merge request.
- 🇪🇸Spain fjgarlin
Moving this to core. Recommended to create a new fork and branch from there instead.
A good starting point for the new MR is to uncomment the MariaDB options in the matrix and start testing/debugging within the MR.
- 🇪🇸Spain fjgarlin
I created a brand new fork and new MR as the other one was way outdated and coming from even another test project we initially used.
- New MR: https://git.drupalcode.org/project/drupal/-/merge_requests/5275
- Pipeline running: https://git.drupalcode.org/issue/drupal-3385621/-/pipelines/45352 (mariadb job is the first one in the "Test" column)
- MariaDB pipeline: https://git.drupalcode.org/issue/drupal-3385621/-/pipelines/45356We can wrong from here using the above MR.
- 🇬🇷Greece dimitriskr
The cause for this is written here
#3399858: MariaDB 10.3.22 setup.sql creates a user that already exists - 🇬🇧United Kingdom catch
This is one of the main blockers to switching off DrupalCI, so bumping to critical.
- 🇪🇸Spain fjgarlin
The related issue is now in "Needs review". Once there is an image available we can test here.
- First commit to issue fork.
- Status changed to Needs review
12 months ago 10:46pm 23 November 2023 - 🇫🇷France andypost
Pushed dev image and added extra WIP:commit to the MR to test new image, will remove after pipeline passed
- 🇫🇷France andypost
Reverted debugging commit and pushed production image
I'm sure it needs follow-up to use supported MariaDB version as 10.3 is outdated
- 🇬🇷Greece dimitriskr
@andypost I just pushed the 10.6 and 10.11 versions to the related issue's MR, I forgot to do it before
- 🇫🇷France andypost
I prefer to have consensus on which LTS versions we'll need in CI and figure out how it will be updated for CVEs and bugfix releases
- 🇪🇸Spain fjgarlin
Thanks for pushing the updated image. I can confirm that the MariaDB works now, as seen in the pipeline, which is great!!
I agree that any newer or LTS versions could go in a follow up as this issue was to make MariaDB work, and it works now 😀
I'm not marking RTBC as I created this MR but it's working as expected now. MariaDB pipeline: https://git.drupalcode.org/issue/drupal-3385621/-/pipelines/54691
- 🇬🇧United Kingdom catch
Let's open a follow-up for MariaDB versions, we'll at least need the newest LTS release so we've got some room to raise 11.x requirements in 📌 [11.x] [policy] Require MySQL 8.0 (increased from 5.7) or MariaDB 10.5 (increased from 10.3) for Drupal 11 Active but it doesn't need to block a commit here.
- Status changed to RTBC
12 months ago 12:06pm 24 November 2023 - 🇬🇧United Kingdom longwave UK
SIMPLETEST_DB = mysql://drupaltestbot:drupaltestbotpw@database/mysql?module=mysql
proves we are connecting to a mysql/mariadb instance, and _TARGET_DB is set correctly in the parent pipeline otherwise the database service wouldn't start at all, so this is RTBC.
- Status changed to Fixed
12 months ago 1:12pm 24 November 2023 - 🇬🇧United Kingdom catch
Committed/pushed to 11.x and 10.2.x, thanks!
#3401846: Add supported mariadb container images → is already open.
I opened 📌 [PP-1] Test against newer versions of MariaDB Active so we've got a core issue to add the environments once the containers are available.
Automatically closed - issue fixed for 2 weeks with no activity.
- Status changed to Fixed
11 months ago 10:36pm 17 December 2023