| opendevreview | Dale Smith proposed openstack/magnum master: Add RPC call to initialize cluster before creation https://review.opendev.org/c/openstack/magnum/+/927555 | 02:15 |
|---|---|---|
| noonedeadpunk | hey folks! with upgrade to 2024.1 we've start getting issues with Magnum in terms of it not being able to maintain it's connections to DB, always exceeding pooling limits, regardless of how crazy we set them | 06:44 |
| noonedeadpunk | which led me to https://opendev.org/openstack/magnum/commit/d544698fae220549f68afa218dd252366fe90b27 | 06:44 |
| noonedeadpunk | and I'm not sure if you can just drop `autocommit=True` for db_session.EngineFacade? | 06:44 |
| noonedeadpunk | As I think what needs to be done, is to use `oslo_db.sqlalchemy.enginefacade` directly instead | 06:45 |
| jrosser | there is a bug in launchpad for this I think | 06:45 |
| noonedeadpunk | yeah, so if connections does not commit properly - that would totally explain the behaviour | 06:45 |
| noonedeadpunk | and I from vitrage and masakari do remember changing method to be used to create connections when autocommit is dropped | 06:46 |
| noonedeadpunk | let me find the diff for some more respectable project then waht I did for vitrage hehe | 06:48 |
| noonedeadpunk | but even if you check `db_session.EngineFacade` in codesearch - almost nobody does use it still https://codesearch.openstack.org/?q=db_session.EngineFacade&i=nope&literal=nope&files=&excludeFiles=&repos= | 06:49 |
| noonedeadpunk | ah, I found the bug report - https://bugs.launchpad.net/magnum/+bug/2067345 | 10:04 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!