Problem/Motivation

Add testing with Maria DB 10.6 which is required by Drupal 11./

Steps to reproduce

Proposed resolution

Like it says, starting using an Maria DB 10.6test environment

Remaining tasks

Postponed on #3420973: Add MariaDB 10.6 to CI

User interface changes

API changes

Data model changes

Release notes snippet

Issue fork drupal-3420974

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

quietone created an issue. See original summary.

andypost made their first commit to this issue’s fork.

andypost’s picture

Second commit can be removed after testing of the image

andypost’s picture

Status: Postponed » Needs review

image pushed so removing testing commit and using new image https://git.drupalcode.org/project/drupalci_environments/-/jobs/808648

maybe the new job should be added to daily ones?

dimitriskr’s picture

smustgrave’s picture

Status: Needs review » Needs work

Reviewed 6586, had test failures.

andypost changed the visibility of the branch 3420974-mariadb-10.6-testing to hidden.

andypost’s picture

Status: Needs work » Needs review

It was testing MR

smustgrave’s picture

Status: Needs review » Reviewed & tested by the community

Tests are green for 10.6

  • catch committed 702fcb09 on 10.2.x
    Issue #3420974 by andypost, quietone: Add testing wtih Maria DB 10.6
    
    (...

  • catch committed db681552 on 11.x
    Issue #3420974 by andypost, quietone: Add testing wtih Maria DB 10.6
    
catch’s picture

Version: 11.x-dev » 10.2.x-dev
Status: Reviewed & tested by the community » Fixed
Issue tags: -Major version only

Committed/pushed to 11.x and cherry-picked to 10.2.x, thanks!

Removing the 'major version only' tag because it's fine to test with this combination on 10.x too.

dimitriskr’s picture

Version: 10.2.x-dev » 11.x-dev
Status: Fixed » Reviewed & tested by the community

MariaDB 10.6: https://git.drupalcode.org/issue/drupal-2015121/-/pipelines/95023
MySQL 8: https://git.drupalcode.org/issue/drupal-2015121/-/pipelines/95019

There's something going on with Drupal\FunctionalTests\Bootstrap\UncaughtExceptionTest test

dimitriskr’s picture

Version: 11.x-dev » 10.2.x-dev
Status: Reviewed & tested by the community » Fixed

  • catch committed ef82260b on 10.2.x
    Revert "Issue #3420974 by andypost, quietone: Add testing wtih Maria DB...

  • catch committed 4f01149e on 11.x
    Revert "Issue #3420974 by andypost, quietone: Add testing wtih Maria DB...
catch’s picture

Version: 10.2.x-dev » 11.x-dev
Status: Fixed » Needs work

Those look like real failures. Reverted from both branches.

dimitriskr’s picture

Well, after the 3rd consecutive retry, the previously failed test passed.. Random? Connected with infrastructure issues?
https://git.drupalcode.org/issue/drupal-2015121/-/jobs/818007

andypost’s picture

dimitriskr’s picture

I ran again the jobs and got the same error as before

Original phpunit error: https://git.drupalcode.org/issue/drupal-2015121/-/jobs/817769
Latest phpunit error: https://git.drupalcode.org/issue/drupal-2015121/-/jobs/918319

I re-ran the job only once (comparing to last time which passed at the 4th try) and it passed. Upon checking the browser output, I noticed it throwed different exception for the same reason. Is it something in the mysql driver?

Failing:

PDOException: SQLSTATE[HY000] [1698] Access denied for user 'cij44wcj6ozuljp5'@'127.0.0.1' in Drupal\Component\DependencyInjection\PhpArrayContainer->createService() (line 79 of /builds/issue/drupal-2015121/core/lib/Drupal/Component/DependencyInjection/PhpArrayContainer.php).

Drupal\Component\DependencyInjection\PhpArrayContainer->createService(Array, 'database') (Line: 177)
Drupal\Component\DependencyInjection\Container->get('database', 1) (Line: 219)
Drupal\Component\DependencyInjection\PhpArrayContainer->resolveServicesAndParameters(Array) (Line: 62)
Drupal\Component\DependencyInjection\PhpArrayContainer->createService(Array, 'cache.container') (Line: 177)
Drupal\Component\DependencyInjection\Container->get('cache.container') (Line: 574)
Drupal\Core\DrupalKernel->getCachedContainerDefinition() (Line: 960)
Drupal\Core\DrupalKernel->initializeContainer() (Line: 514)
Drupal\Core\DrupalKernel->boot() (Line: 734)
Drupal\Core\DrupalKernel->handle(Object) (Line: 19)

Passing:

Drupal\Core\Database\DatabaseAccessDeniedException: SQLSTATE[HY000] [1045] Access denied for user 'w7oihw6z5qupbfy5'@'127.0.0.1' (using password: YES) in Drupal\mysql\Driver\Database\mysql\Connection::open() (line 207 of /builds/issue/drupal-2015121/core/modules/mysql/src/Driver/Database/mysql/Connection.php).

Drupal\mysql\Driver\Database\mysql\Connection::open(Array) (Line: 460)
Drupal\Core\Database\Database::openConnection('default', 'default') (Line: 191)
Drupal\Core\Database\Database::getConnection('default')
call_user_func_array('Drupal\Core\Database\Database::getConnection', Array) (Line: 79)
Drupal\Component\DependencyInjection\PhpArrayContainer->createService(Array, 'database') (Line: 177)
Drupal\Component\DependencyInjection\Container->get('database', 1) (Line: 219)
Drupal\Component\DependencyInjection\PhpArrayContainer->resolveServicesAndParameters(Array) (Line: 62)
Drupal\Component\DependencyInjection\PhpArrayContainer->createService(Array, 'cache.container') (Line: 177)
Drupal\Component\DependencyInjection\Container->get('cache.container') (Line: 574)
Drupal\Core\DrupalKernel->getCachedContainerDefinition() (Line: 960)
Drupal\Core\DrupalKernel->initializeContainer() (Line: 514)
Drupal\Core\DrupalKernel->boot() (Line: 734)
Drupal\Core\DrupalKernel->handle(Object) (Line: 19)
catch’s picture

What's the status of #3401846: Add supported mariadb container images - are we missing something that was fixed in there maybe?

dimitriskr’s picture

The differences in my.cnf is that #3401846: Add supported mariadb container images has extra transaction_isolation = READ-COMMITTED and innodb_thread_concurrency = 0, while the committed my.cnf only has innodb_use_native_aio = 0

quietone’s picture

What is the next step here?

quietone’s picture

Status: Needs work » Needs review

I am not sure if there is work to be done here. Tests are passing on maria 10.6 so I am setting to NR.

catch’s picture

Status: Needs review » Reviewed & tested by the community

Yeah it looks fine to me too :)

We'll need another 11.x-specific issue to remove environments on the older database versions.

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed f1515fb and pushed to 11.x. Thanks!

Didn't backport cleanly to 10.3 and I don;t think we need it there as long as we have in 11.x

  • alexpott committed f1515fb7 on 11.x
    Issue #3420974 by andypost, quietone, dimitriskr, catch: Add testing...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.