| *** mhen_ is now known as mhen | 01:55 | |
| croelandt | #startmeeting glance | 14:01 |
|---|---|---|
| opendevmeet | Meeting started Thu May 15 14:01:14 2025 UTC and is due to finish in 60 minutes. The chair is croelandt. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:01 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:01 |
| opendevmeet | The meeting name has been set to 'glance' | 14:01 |
| croelandt | #topic roll call | 14:01 |
| abhishekk | o/ | 14:01 |
| croelandt | #link https://etherpad.openstack.org/p/glance-team-meeting-agenda | 14:01 |
| croelandt | It's probably juste going to be the two of us :) | 14:01 |
| croelandt | well, the *3* of us \o/ | 14:02 |
| croelandt | #topic Release/periodic job updates | 14:02 |
| croelandt | I only see a failure for glance-multistore-cinder-import-fips | 14:02 |
| abhishekk | that is related to dependency between taskflow and one other library | 14:02 |
| croelandt | ok I see the Python version for Nova is not the right one | 14:03 |
| croelandt | so it's out of our hands? | 14:03 |
| abhishekk | I think so, we can wait and keep watch on it | 14:03 |
| croelandt | yes | 14:03 |
| croelandt | #topic Important stable patches | 14:03 |
| croelandt | #link https://review.opendev.org/c/openstack/glance/+/948810 | 14:04 |
| croelandt | #link https://review.opendev.org/c/openstack/glance/+/948812 | 14:04 |
| croelandt | So these are backports of Dan's fix for a bug that was holding the gate in master | 14:04 |
| croelandt | the bug is also present in stable/2024.* | 14:04 |
| croelandt | I think we should backport the fix so we can easily merge backports in stable/2024.* (that is, without hitting "recheck" 12 times) | 14:04 |
| croelandt | any thoughts? | 14:04 |
| abhishekk | sounds good | 14:04 |
| abhishekk | I will ninja approve it one by one | 14:05 |
| croelandt | ok, thanks a lot! | 14:05 |
| abhishekk | np! | 14:05 |
| croelandt | #topic glance-gaps patches | 14:05 |
| mrjoshi | o/ | 14:05 |
| croelandt | I will take over the SDK/CLI patches next week since mrjoshi is going to leave us :-( | 14:05 |
| croelandt | mrjoshi: anything we need to know regarding the work that is left to be done? Anything important you want to highlight? | 14:05 |
| abhishekk | croelandt: all the best ;) | 14:06 |
| mrjoshi | Thank you so croelandt , I have addressed all the comments and the patches are up, they are failing on zuul jobs, intermittent failure. | 14:07 |
| abhishekk | are those related to glance or your patches? | 14:07 |
| mrjoshi | No it's not related to my patch | 14:09 |
| mrjoshi | it's been occuring in other sdk patches aswell | 14:09 |
| croelandt | ok so that's fine :) | 14:09 |
| abhishekk | cool | 14:10 |
| croelandt | Well thanks a lot for all your work on the SDK/CLI, we made huge progress thanks to you | 14:10 |
| abhishekk | ++, thank you for all your efforts mrjoshi!! | 14:10 |
| croelandt | #topic Issues with secure hash | 14:11 |
| croelandt | #link https://review.opendev.org/c/openstack/glance/+/949672 | 14:11 |
| croelandt | #link https://review.opendev.org/c/openstack/glance/+/949668 | 14:11 |
| croelandt | so I'm a bit out of the loop here | 14:11 |
| croelandt | First of all, I think the doc patch should be merged with the code patch | 14:11 |
| croelandt | so e don't risk having inaccurate docs | 14:11 |
| croelandt | and then, I think we want to decide whether it's ok to rely on the "old" behaviour like some Tempest do | 14:12 |
| croelandt | am I right? | 14:12 |
| abhishekk | First of all, is it a good idea to flip the config value? rosmaita wdyt? | 14:12 |
| abhishekk | This feature is still in early phase, and I am sure no one is still using it (talking about hash calculation in new location API) | 14:13 |
| croelandt | :/ | 14:14 |
| abhishekk | We have enabled it by default, so that all the images/snapshot etc will have hash/checksum calculated | 14:14 |
| croelandt | Maybe Tempest could use the right config value instead of us changing the default, right? | 14:14 |
| abhishekk | that is why I have disabled it in the job, we don't need to calculate the hashes in tests IMO | 14:15 |
| abhishekk | But, we need to disable it for all the jobs which are using ceph as backend | 14:16 |
| abhishekk | Right now I have disabled it only for glance_store job, but we need to do it for nova-ceph-multistore job as well | 14:16 |
| croelandt | so we'd drop Rajat's patches? | 14:16 |
| abhishekk | We definitely need some input whether it is really good to flip the config value | 14:17 |
| abhishekk | may be dansmith, rosmaita can comment here about the same | 14:18 |
| croelandt | yeah I think it would solve our testing issues, but it might be worse for real users | 14:18 |
| croelandt | ok so I think we need to get Dan, Brian, Rajat and the two of us in a 15 minute IRC meeting at some point | 14:18 |
| croelandt | and figure this out | 14:18 |
| abhishekk | ALSO you are the boss now, so whatever you say!!! | 14:18 |
| dansmith | I haven't looked at this so I need time to digest | 14:18 |
| croelandt | If I had it my way, we'd remove all features | 14:19 |
| abhishekk | bang!! | 14:19 |
| croelandt | I don't think it's urgent, so maybe we can try and find a time slot next week maybe? | 14:19 |
| abhishekk | dansmith: ack | 14:19 |
| croelandt | unless this gets resolved in Gerrit comments by then :) | 14:19 |
| croelandt | ok moving on, then | 14:20 |
| croelandt | #topic One easy patch per core dev | 14:21 |
| croelandt | #link https://review.opendev.org/q/topic:%22bug/2110185%22 | 14:21 |
| croelandt | ok so I guess we'd want to have all these patches merged in the same release, so the behaviour is consistent across all drivers, right? | 14:21 |
| abhishekk | yes, we need it for our main feature, pass image size in advance to glance | 14:22 |
| croelandt | ok so I'll start reviewing those but I'll only push +W once they all seem good | 14:23 |
| croelandt | ideally we would have them all in M2 | 14:23 |
| abhishekk | sounds good, we also need a release for glance store after that, so it will be good if we get it early as well | 14:23 |
| Luzi | o/ | 14:23 |
| abhishekk | Luzi: hi | 14:24 |
| croelandt | yes | 14:24 |
| abhishekk | croelandt: I have approved your patch related to unused method | 14:24 |
| croelandt | by the way we have a release for glacne_store M1 | 14:24 |
| croelandt | is there anything else M1-related I need to do as PTL? | 14:25 |
| abhishekk | cool! I think that is enough | 14:25 |
| croelandt | nice | 14:25 |
| croelandt | #topic Open Discussion | 14:25 |
| croelandt | Anything else to discuss? | 14:25 |
| Luzi | hi | 14:25 |
| Luzi | we do have some time to work on the image encryption again starting next week | 14:26 |
| abhishekk | great! | 14:26 |
| abhishekk | we are here if you guys need anything, just ping us! | 14:27 |
| croelandt | yes don't hesitate to ping on #openstack-glance | 14:27 |
| Luzi | were there any other points except for the copy image and glance_direct still missing in the patch? | 14:27 |
| Luzi | i think markus answered a few open review questions - it would be good if someone could get over the patch again | 14:28 |
| abhishekk | I think those are important, I will again have a look tomorrow and comment if something else is missing! | 14:28 |
| Luzi | thank you :) | 14:28 |
| abhishekk | np!! | 14:28 |
| croelandt | ok, I think we can wrap up | 14:29 |
| abhishekk | thank you!! | 14:31 |
| croelandt | #endmeeting | 14:31 |
| opendevmeet | Meeting ended Thu May 15 14:31:15 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:31 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/glance/2025/glance.2025-05-15-14.01.html | 14:31 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/glance/2025/glance.2025-05-15-14.01.txt | 14:31 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/glance/2025/glance.2025-05-15-14.01.log.html | 14:31 |
| *** kevko6 is now known as kevko | 15:49 | |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!