| tkajinam | abhishekk, dansmith the fix has been merged in oslo.utils but we need a new release and upper-constraints change so that the fix is consumed by tests. I hope these complete soon but in case you want to unblock the gate asap then make it non-voting. | 09:48 |
|---|---|---|
| tkajinam | make the sqla master test non-voting. I mean | 09:49 |
| stephenfin | abhishekk: what tkajinam said. This is not a sqlalchemy issue but rather an oslo.utils issue | 10:34 |
| tkajinam | We may want to have a job with latest "release" voting in addition to another non-voting job with master. | 10:38 |
| tkajinam | though adding more jobs is not ideal in general | 10:38 |
| abhishekk | ack, any issues in moving current one in periodic pipeline? | 10:38 |
| abhishekk | pranali, could you please update your patch to make that job non-voting instead of moving it to periodic? | 10:40 |
| tkajinam | it's the project decision but you probably have to have one in non-periodic line unless you are careful enough to catch any issue detected in periodic pipeline. | 10:41 |
| tkajinam | If you leave problems there then these would kill you when we finally bump sqlalchemy to 2.x | 10:41 |
| tkajinam | ... which will be coming soon. | 10:41 |
| abhishekk | ack | 10:42 |
| abhishekk | better to make it non voting now | 10:43 |
| pranali | abhishekk, ack | 10:46 |
| pranali | tkajinam, could you please share the link for the oslo.utils fix? | 10:58 |
| pranali | ohh got it | 11:00 |
| opendevreview | Pranali Deore proposed openstack/glance master: Make glance-tox-functional-py39-sqlalchemy-tips job non-voting https://review.opendev.org/c/openstack/glance/+/900276 | 11:12 |
| opendevreview | Merged openstack/glance master: Make glance-tox-functional-py39-sqlalchemy-tips job non-voting https://review.opendev.org/c/openstack/glance/+/900276 | 15:16 |
| *** tosky_ is now known as tosky | 18:43 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!