- Issue created by @dimitriskr
- @dimitriskr opened merge request.
- @dimitriskr opened merge request.
- ๐บ๐ธUnited States nicxvan
HEAD is broken due to ๐ Set default transaction isolation level to READ-COMMITTED in MySQL/MariaDB Active
I think this is the issue to resolve it.
- ๐ซ๐ทFrance andypost
Yes it needs to fix
Installer Isolation Level Existing Settings (Drupal\Tests\mysql\Functional\InstallerIsolationLevelExistingSettings) โ Installer โ โ Failed asserting that two strings are equal. โ ---ยทExpected โ +++ยทActual โ @@ @@ โ -'REPEATABLE-READ' โ +'READ-COMMITTED' โ โ /builds/issue/drupal-3539366/core/modules/mysql/tests/src/Functional/InstallerIsolationLevelExistingSettingsTest.php:58 โด
- ๐ซ๐ทFrance andypost
- First commit to issue fork.
- @godotislate opened merge request.
- @andypost opened merge request.
- ๐ซ๐ทFrance andypost
Replaced comparison with session vs global isolation level
- ๐ฌ๐ทGreece dimitriskr
dimitriskr โ changed the visibility of the branch 3539366-11.x-tx-isolation-mysql to hidden.
- ๐ฌ๐ทGreece dimitriskr
dimitriskr โ changed the visibility of the branch 3539366-10.5.x-tx-isolation-mysql to hidden.
- ๐ซ๐ทFrance andypost
I think it's fine for me to RTBC !13038 as not an author
Moreover it needs backport to 10.x
- ๐ฆ๐บAustralia acbramley
This is affecting MRs now https://git.drupalcode.org/issue/drupal-3015152/-/jobs/6259156
- ๐ฎ๐นItaly mondrake ๐ฎ๐น
mondrake โ changed the visibility of the branch 3539366-fix-tx-isolation to hidden.
- ๐ฌ๐งUnited Kingdom catch
Committed/pushed to 11.x and cherry-picked to 11.2,x, thanks!
This doesn't apply against 10.6.x, do we need a backport?
- @godotislate opened merge request.
MR 13052 Port to 10.6.x. It also applies clean to 10.5.x.
The Needs Review Queue Bot โ tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".
This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.
Consult the Drupal Contributor Guide โ to find step-by-step guides for working with issues.
- ๐บ๐ธUnited States nicxvan
I think the bot is confused. Updated metadata.
I also reviewed and it like the same as the original fix.
- ๐ฌ๐งUnited Kingdom catch
Committed/pushed to 10.6.x and cherry-picked to 10.5.x, thanks!
Automatically closed - issue fixed for 2 weeks with no activity.