- Issue created by @simonbaese
- Merge request !10648Issue #3495554: Add regression test for transaction manager → (Open) created by simonbaese
- 🇩🇪Germany simonbaese Berlin
If this is an error in the provided test, please change the issue type to "Support Request". In that case, we should add a change note to 🐛 Ensure post transaction callbacks are only at the end of the root Drupal transaction Fixed because it is unclear what needs to be adjusted.
- 🇩🇪Germany simonbaese Berlin
Please also let me know how to set up the test to observe the error in the CI pipelines.
- 🇩🇪Germany simonbaese Berlin
Ok, apparently the kernel tests pass in the pipeline. Maybe this is a problem with DDEV? Could someone give a pointer to what the issue could be? I will investigate a little more and put this issue on postponed.
- 🇮🇹Italy mondrake 🇮🇹
Could this be just another manifestation of 🐛 Transaction autocommit during shutdown relies on unreliable object destruction order Active & friends?
- 🇩🇪Germany simonbaese Berlin
@mondrake Thank you. That is a nice pointer. Indeed, turning off XDebug or using
xdebug.mode=debugprevents the error. Do you think there is value in keeping this issue open? Maybe we can cross-reference and close it if there are other open issues. - 🇮🇹Italy mondrake 🇮🇹
@simonbaese we have some issues open like ✨ Allow explicit COMMIT in Transaction objects Active and 📌 [PP-1] Convert all transactions to explicit COMMIT Postponed , but we probably need a meta plan issue. I'll try to open one and reparent this issue to that.