| stephenfin | croelandt: rosmaita: Would you folks be able to look at https://review.opendev.org/c/openstack/glance/+/932701 today so we can merge the DevStack change? | 12:16 |
|---|---|---|
| rosmaita | stephenfin: ack | 12:44 |
| croelandt | stephenfin: I +2ed, but I think Abhishek has some concerns | 15:49 |
| croelandt | he's on PTO today though :/ | 15:49 |
| stephenfin | croelandt: Indeed. I've relied to his comment. Given that this is effectively persisting the current status quo, I wonder if we might merge as-is and revisit if we feel the need to | 15:50 |
| stephenfin | Also, we've merged this same code in nova, placement, cinder and neutron (plus others I didn't author) already | 15:51 |
| croelandt | Yeah I think this is the same code in all projects and it seems to work fine | 16:09 |
| croelandt | You tested Devstack with this? | 16:09 |
| stephenfin | croelandt: yup here https://review.opendev.org/c/openstack/devstack/+/932702 | 16:40 |
| *** atmark_ is now known as atmark | 19:37 | |
| opendevreview | Merged openstack/glance master: Add new glance.wsgi module https://review.opendev.org/c/openstack/glance/+/932701 | 22:01 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!