| opendevreview | Rajat Dhasmana proposed openstack/glance_store master: DNM: Check failures by enabling secure hash https://review.opendev.org/c/openstack/glance_store/+/949562 | 06:40 |
|---|---|---|
| opendevreview | Abhishek Kekane proposed openstack/glance_store master: [cinder] Add size validation for image uploads https://review.opendev.org/c/openstack/glance_store/+/949137 | 06:57 |
| opendevreview | Abhishek Kekane proposed openstack/glance_store master: [rbd] Add size validation for image uploads https://review.opendev.org/c/openstack/glance_store/+/949362 | 07:19 |
| opendevreview | Rajat Dhasmana proposed openstack/glance_store master: DNM: Check failures by enabling secure hash https://review.opendev.org/c/openstack/glance_store/+/949562 | 11:27 |
| opendevreview | Stephen Finucane proposed openstack/glance master: Add pyproject.toml to support pip 23.1 https://review.opendev.org/c/openstack/glance/+/899951 | 11:48 |
| opendevreview | Stephen Finucane proposed openstack/glance master: Add pyproject.toml to support pip 23.1 https://review.opendev.org/c/openstack/glance/+/899951 | 11:48 |
| opendevreview | Stephen Finucane proposed openstack/glance stable/2025.1: Add pyproject.toml to support pip 23.1 https://review.opendev.org/c/openstack/glance/+/949614 | 11:49 |
| opendevreview | Stephen Finucane proposed openstack/glance stable/2024.2: Add pyproject.toml to support pip 23.1 https://review.opendev.org/c/openstack/glance/+/949615 | 11:50 |
| opendevreview | Stephen Finucane proposed openstack/glance stable/2024.1: Add pyproject.toml to support pip 23.1 https://review.opendev.org/c/openstack/glance/+/949616 | 11:50 |
| abhishek_ | dansmith: good morning (when you are here) I guess if we pass image size to swift backend then implementation is broken as it is implemented in a way to only write data to backend which will be matched to image size and then break from that point | 12:08 |
| abhishek_ | https://github.com/openstack/glance_store/blob/master/glance_store/_drivers/swift/store.py#L975-L982 | 12:08 |
| *** tosky_ is now known as tosky | 12:27 | |
| opendevreview | Abhishek Kekane proposed openstack/glance_store master: [swift] Add size validation for image uploads https://review.opendev.org/c/openstack/glance_store/+/949626 | 12:56 |
| abhishek_ | Here I have made expected changes ^^ | 12:57 |
| opendevreview | Abhishek Kekane proposed openstack/glance_store master: Replace Eventlet with concurrent.futures https://review.opendev.org/c/openstack/glance_store/+/949631 | 13:50 |
| opendevreview | Cyril Roelandt proposed openstack/glance master: reno: Update master for unmaintained/2023.1 https://review.opendev.org/c/openstack/glance/+/936132 | 14:07 |
| croelandt | rosmaita: ^ I'll push in 24 hours unless you have more objections | 14:08 |
| opendevreview | Abhishek Kekane proposed openstack/glance_store master: [filesystem] Add size validation for image uploads https://review.opendev.org/c/openstack/glance_store/+/949124 | 14:40 |
| abhishek_ | dansmith: do you know whether we can run particular job based on change in file? | 14:41 |
| dansmith | abhishek_: you can exclude files from triggering a job.. so you can exclude everything other than the file you want to trigger on | 14:41 |
| abhishek_ | currently even if I change file,swift or cinder backend in glance_store then it runs ceph job for it which I think should not happen | 14:42 |
| abhishek_ | ack I will look for it, for cinder I guess we should run cinder multistore job, for swift it should be tempest-integrated-import-storage, for ceph it should be nova-ceph-multistore, we have job for s3 as well, just thinking what should be the case for file backend | 14:44 |
| dansmith | yeah, so add file,swift to the exclusion list for the ceph job so changes to those files don't cause the ceph job to run, but changes to core glance_store stuff does, and changes to ceph.py does | 14:44 |
| abhishek_ | ack, sounds good | 14:48 |
| opendevreview | Rajat Dhasmana proposed openstack/glance master: Don't allow hash calculation by default https://review.opendev.org/c/openstack/glance/+/949668 | 19:04 |
| opendevreview | Rajat Dhasmana proposed openstack/glance master: Docs: Add warning about ``do_secure_hash`` config https://review.opendev.org/c/openstack/glance/+/949672 | 19:50 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!