| opendevreview | Rajat Dhasmana proposed openstack/glance master: Don't allow hash calculation by default https://review.opendev.org/c/openstack/glance/+/949668 | 06:04 |
|---|---|---|
| opendevreview | Rajat Dhasmana proposed openstack/glance master: Docs: Add warning about ``do_secure_hash`` config https://review.opendev.org/c/openstack/glance/+/949672 | 06:04 |
| opendevreview | Abhishek Kekane proposed openstack/glance_store master: [swift] Add size validation for image uploads https://review.opendev.org/c/openstack/glance_store/+/949626 | 06:59 |
| opendevreview | Abhishek Kekane proposed openstack/glance_store master: Run ceph-tempest job specifically for RBD https://review.opendev.org/c/openstack/glance_store/+/949702 | 07:29 |
| opendevreview | Merged openstack/glance master: reno: Update master for unmaintained/2023.1 https://review.opendev.org/c/openstack/glance/+/936132 | 12:58 |
| opendevreview | Abhishek Kekane proposed openstack/glance-specs master: Add spec to provide image size in advance https://review.opendev.org/c/openstack/glance-specs/+/947423 | 14:44 |
| opendevreview | Cyril Roelandt proposed openstack/glance master: Remove unused method iso8601_from_timestamp https://review.opendev.org/c/openstack/glance/+/939648 | 15:18 |
| croelandt | abhishekk: I was looking at https://review.opendev.org/c/openstack/glance/+/947637 and I think we're gonna have to be fine with one +2 per eventlet patch | 15:21 |
| abhishekk | Ack, so far we don't have any other option | 15:21 |
| croelandt | the +1 from Francesco is also reassuring :) | 15:24 |
| abhishekk | agreee | 15:26 |
| abhishekk | croelandt: have we released store? | 15:28 |
| abhishekk | else I guess you can have a look at s3 eventlet removal patch of store | 15:28 |
| croelandt | abhishekk: https://review.opendev.org/c/openstack/releases/+/949545 Elod and Jens seem against it | 15:55 |
| croelandt | I think we might skip the release? | 15:55 |
| croelandt | I wanted to discuss it during tomorrow's meeting | 15:55 |
| abhishekk | IF we approve s3 patch then we can do the release else we need to skip as nothing is merged there than disabling the hash calculation for job | 15:56 |
| croelandt | so https://review.opendev.org/c/openstack/glance_store/+/949631 | 15:57 |
| croelandt | abhishekk: looks pretty good except for the comment at line 729 | 16:02 |
| croelandt | in 3 years I'm not gonna understand why wereference eventlet there :D | 16:03 |
| croelandt | It feels like a leftover | 16:03 |
| abhishekk | Nah, I kept it there for purpose | 16:03 |
| abhishekk | If you want to remove it then I can go ahead and remove it | 16:04 |
| croelandt | oh ok | 16:05 |
| croelandt | ok I'm not sure I understand the purpose but let's merge as is | 16:06 |
| croelandt | and I'll update the release patch later | 16:06 |
| abhishekk | sounds good | 16:12 |
| opendevreview | Abhishek Kekane proposed openstack/python-glanceclient master: Add support for passing image size to Glance API https://review.opendev.org/c/openstack/python-glanceclient/+/949787 | 17:36 |
| opendevreview | Merged openstack/glance_store master: Replace Eventlet with concurrent.futures https://review.opendev.org/c/openstack/glance_store/+/949631 | 17:48 |
| -opendevstatus- NOTICE: Setuptools 80.7.0 broke python package installs for many affecting CI jobs. That release has been yanked and it should be safe to recheck failed changes. | 22:00 | |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!