| opendevreview | OpenStack Proposal Bot proposed openstack/glance master: Imported Translations from Zanata https://review.opendev.org/c/openstack/glance/+/972617 | 04:12 |
|---|---|---|
| opendevreview | Max proposed openstack/glance master: feat: return size in get_image_data_iter https://review.opendev.org/c/openstack/glance/+/976627 | 14:03 |
| *** croeland1 is now known as croelandt | 15:25 | |
| croelandt | pdeore_: I think we can remove https://review.opendev.org/c/openstack/glance_store/+/879940 , right? | 15:42 |
| croelandt | rosmaita: dansmith: I would like to push https://review.opendev.org/c/openstack/glance_store/+/959201 before we release glance_store this week, so if you have a minute to review it :) | 15:44 |
| rosmaita | ack | 15:45 |
| dansmith | today is very busy.. I will try this afternoon but can you hit me tomorrow? | 15:45 |
| dansmith | (if you still need it) | 15:45 |
| croelandt | ok! | 15:46 |
| croelandt | I'm kind of reluctant to push my own patches :/ | 15:47 |
| croelandt | btw, any patch under review you care about, you might want to point out to me *now* so it makes the cut | 16:03 |
| croelandt | abhishekk: also true for you if you got anything @ | 16:03 |
| croelandt | ^ | 16:03 |
| opendevreview | Cyril Roelandt proposed openstack/glance_store master: Return a warning if cinder_store_user_name is undefined https://review.opendev.org/c/openstack/glance_store/+/947143 | 16:18 |
| croelandt | rosmaita: also if you can confirm that David Hill's understanding of the bug makes sense, that would help merge this bug fix as well ^ | 16:23 |
| rosmaita | ok, will put it on my list | 16:23 |
| croelandt | sorry for bothering you with that, I know you're busy :-( | 16:24 |
| tkajinam | that check looks wired to me | 16:46 |
| croelandt | tkajinam: leave a comment on the bug please :) | 16:47 |
| tkajinam | croelandt, already done | 16:47 |
| tkajinam | :-) | 16:47 |
| tkajinam | oh, not in bug but in the patch | 16:48 |
| croelandt | sorry | 16:49 |
| croelandt | meant in the patch | 16:50 |
| croelandt | all good | 16:50 |
| tkajinam | I also left a comment in the bug | 16:51 |
| opendevreview | Takashi Kajinami proposed openstack/glance master: Fix missing context when replacing image location https://review.opendev.org/c/openstack/glance/+/973275 | 16:51 |
| opendevreview | Takashi Kajinami proposed openstack/glance_store master: Migrate setup configuration to pyproject.toml https://review.opendev.org/c/openstack/glance_store/+/974096 | 16:53 |
| abhishekk | @croelandt no glance_store patches but I have few glance patches which can certainly needs reviews | 16:57 |
| croelandt | abhishekk: https://review.opendev.org/c/openstack/glance_store/+/971515 can you add a release note here? | 16:57 |
| croelandt | yeah I'm focusing on the store patches right now but we can discuss glance patches tomorrow | 16:58 |
| abhishekk | looking | 16:58 |
| opendevreview | Merged openstack/glance_store master: Replace remaining use of md5 method from oslo.utils https://review.opendev.org/c/openstack/glance_store/+/947713 | 17:02 |
| opendevreview | Abhishek Kekane proposed openstack/glance_store master: Fix Swift driver size validation timing in chunked uploads https://review.opendev.org/c/openstack/glance_store/+/971515 | 17:08 |
| abhishekk | croelandt: ^^ | 17:09 |
| croelandt | I don't think we have anyone else usually reviewing our swift patches | 17:11 |
| croelandt | OK it should land in a few hours now :) | 17:19 |
| croelandt | Going out, will check IRC in a few hours also :) | 17:19 |
| opendevreview | Merged openstack/glance_store master: Fix Swift driver size validation timing in chunked uploads https://review.opendev.org/c/openstack/glance_store/+/971515 | 20:03 |
| opendevreview | Merged openstack/glance_store master: s3: add options to control checksum calculation/validation https://review.opendev.org/c/openstack/glance_store/+/959201 | 22:48 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!