| *** mhen_ is now known as mhen | 01:08 | |
| *** mhen_ is now known as mhen | 02:06 | |
| opendevreview | Markus Hentsch proposed openstack/glance master: Standardization of encrypted images https://review.opendev.org/c/openstack/glance/+/926295 | 09:56 |
|---|---|---|
| opendevreview | Cyril Roelandt proposed openstack/glance_store master: [WIP] s3: add option to disable checksum calculation/validation https://review.opendev.org/c/openstack/glance_store/+/959201 | 14:25 |
| abhishekk | @masahito have you confirmed that the issue still persists with 1.40.1 as well or just with 1.36.? | 14:28 |
| croelandt | According to the Changelog it should be with any version >= 1.36.0, but yes, it would be nice to have confirmation | 14:39 |
| croelandt | tkajinam: I merged https://review.opendev.org/c/openstack/glance_store/+/954192 , didn't you have a similar patch for glance itself? | 14:39 |
| tkajinam | croelandt, I think this is the one https://review.opendev.org/c/openstack/glance/+/954191 | 14:41 |
| masahito | i'm not sure the same problem happens with 1.40.1 because we use epoxy version and its constraint is 1.36.26. | 14:41 |
| masahito | yes, according to boto3 changelog any version >=1.36.0 has same issue. | 14:42 |
| tkajinam | croelandt, this was quite simplified due to a different change merged first ( https://review.opendev.org/c/openstack/glance/+/926560 ) | 14:42 |
| masahito | let me test epoxy and boto3 1.40.1 combination. | 14:45 |
| abhishekk | masahito: thank you, that will be very helpful | 15:06 |
| abhishekk | also I think it works with swift because of this, https://review.opendev.org/c/openstack/swift/+/944073 (its also backported to epoxy - https://review.opendev.org/c/openstack/swift/+/956194/2) | 15:07 |
| croelandt | abhishekk: ok, I think there might be issues with Ceph/Cinder though | 15:22 |
| abhishekk | croelandt: ack, good to hear from cinder side as well | 15:23 |
| croelandt | tkajinam: ah, but I guess our tests use a release of glance_store, so this will have to wait a bit before being merged :) | 15:24 |
| tkajinam | croelandt, yeah | 15:24 |
| croelandt | abhishekk: I'll ping someone directly if I don't get answers to my email | 15:24 |
| abhishekk | sounds good | 15:24 |
| tkajinam | croelandt, we probably have to "backport" the changes to replace Ubuntu Jammy in CI because 2025.2 no longer supports it. I can take care of it once 2025.2 is branch is cut from the latest release | 15:24 |
| croelandt | tkajinam: sure, please do | 15:40 |
| croelandt | abhishekk: I'm looking at your glance_store refactoring patches | 15:41 |
| croelandt | I think I can approve them myself, bcause I doubt you'll get more reviews for them | 15:41 |
| croelandt | how does that sound? | 15:41 |
| masahito | croelandt: tested with boto3 1.40.1. 1.40.1 still hit the same issue. | 15:50 |
| opendevreview | Merged openstack/glance stable/2025.1: Fix inaccurate usage reporting when delayed_delete is enabled https://review.opendev.org/c/openstack/glance/+/958886 | 16:45 |
| abhishekk | croelandt: yes you can | 18:50 |
| abhishekk | masahito: thanks for the confirmation | 18:50 |
| abhishekk | I think you can test with croelandt's approach [WIP] s3: add option to disable checksum calculation/validation https://review.opendev.org/c/openstack/glance_store/+/959201 | 18:51 |
| opendevreview | Merged openstack/glance master: glance-multistore-cinder-import-fips: move to CentOS 10 https://review.opendev.org/c/openstack/glance/+/958713 | 23:39 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!