14:00:21 <pdeore> #startmeeting glance 14:00:21 <opendevmeet> Meeting started Thu Aug 1 14:00:21 2024 UTC and is due to finish in 60 minutes. The chair is pdeore. Information about MeetBot at http://wiki.debian.org/MeetBot. 14:00:21 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 14:00:21 <opendevmeet> The meeting name has been set to 'glance' 14:00:21 <pdeore> #topic roll call 14:00:21 <pdeore> #link https://etherpad.openstack.org/p/glance-team-meeting-agenda 14:00:24 <pdeore> o/ 14:00:39 <mrjoshi> o/ 14:00:50 <dansmith> o/ 14:02:09 <pdeore> lets wait few minutes for others to join 14:03:01 <pdeore> abhishek_, croelandt joining ? 14:04:10 <abhishekk> o/ 14:04:24 <pdeore> let's get started 14:04:33 <pdeore> #topic release/periodic jobs updates 14:04:35 * abhishekk facing fluctuations in electricity, so might be disconnected in between 14:04:45 <pdeore> ack 14:04:54 <pdeore> M3 is 4 weeks from now 14:05:35 <pdeore> periodic jobs are all green except Failures on fips jobs recently, will check this in detail 14:05:43 <abhishekk> You need to connect with luzi and stephen for their respective work 14:06:50 <dansmith> so I'm hoping to have the inspector stuff merged in oslo before that and glance converted over to use it 14:06:50 <pdeore> yes, I think luzi is on PTO and someone from the team is working on it but will get in touch with the team and get the updates 14:06:56 <dansmith> but it's still a lot of work 14:07:08 <dansmith> (before M3 I mean) 14:07:40 <abhishekk> Is it possible to get all work done in time 14:08:07 <abhishekk> I mean I can help if required 14:08:24 <dansmith> sorry I jumped in, are you asking me or pdeore ? 14:08:42 <abhishekk> you, related to format inspector work 14:09:28 <dansmith> nothing specific, but you're welcome to review the oslo work if you want which might help 14:09:45 <abhishekk> cool, I will review it 14:09:47 <dansmith> otherwise, the glance change should be a pretty small patch to cut over 14:10:06 <dansmith> just wanted to highlight for pdeore that I'm hoping to get it done before M3 14:10:19 <dansmith> less important for glance I guess because glance already had the inspector, 14:10:39 <dansmith> but the other projects are carrying a lot of code I promised to move to oslo :) 14:11:06 <pdeore> ack, I think we still have enough time 14:11:07 <dansmith> I'm also working on tempest tests for the sample formats for glance 14:11:08 <abhishekk> SO right now it is just like removing glance inspector and use from oslo.utils, right? 14:11:28 <dansmith> abhishekk: pretty much.. a few small changes in how we use it, but pretty much just that yea 14:11:39 <dansmith> we will save the "defender" work for next cycle :) 14:11:48 <abhishekk> I thought you were talking about glance as a defender work :facepalm: 14:11:53 <dansmith> nono :P 14:12:09 <abhishekk> cool, I guess that will be doable 14:12:53 <abhishekk> I will review oslo work and then glance patches as well, thanks for the heads up 14:13:02 <dansmith> thanks 14:13:09 <dansmith> I'll cc you on the tempest patches as well 14:13:33 <dansmith> currently I have a test that throws all the formats at glance to convert and asserts that it catches things like datafile and refuses to convert, but works for others 14:13:33 <abhishekk> ++, thank you! 14:14:18 <abhishekk> Ok and where are you publishing the test images and providing it to glance? 14:14:50 <dansmith> yep, the repo is created as you know, and I have a patch to devstack that clones it, generates the images, configures tempest to use them 14:15:08 <abhishekk> makes sense 14:15:19 <dansmith> hopefully should be nice to have a strict set of tests that show what glance does and does not support going forward 14:15:39 <dansmith> and the next cve we can just add a generator case to the list and make sure it is caught before publication 14:16:06 <abhishekk> ++ 14:17:03 <dansmith> pdeore: anyway, that's all sorry for a bigger than expected hijack of the release section :) 14:17:29 <pdeore> dansmith, np :) thanks for the updates !! 14:17:39 <pdeore> let's move to next topic 14:17:43 <pdeore> #topic Importatnt Reviews 14:18:01 <pdeore> #link https://review.opendev.org/c/openstack/glance/+/923197 - Deprecate the "metadata_encryption_key" option - need to set +W 14:18:01 <pdeore> #link https://review.opendev.org/c/openstack/glance/+/923624 - Revert image state to queued if conversion fails 14:18:01 <pdeore> #link https://review.opendev.org/c/openstack/python-glanceclient/+/890634 - Add support for new location APIs 14:18:01 <pdeore> #link https://review.opendev.org/c/openstack/openstacksdk/+/883271/8 - Adding support for new add/get location apis 14:18:11 <dansmith> ohh, 14:18:29 <dansmith> I want that "conversion fails" one.. the test I'm writing currently checks for "importing" which is uncool 14:18:33 <dansmith> I'll try to hit that one 14:18:39 <pdeore> croelandt, it would be helpful if you could add your vote on these before your PTO 14:19:56 <pdeore> dansmith, ack 14:21:33 <pdeore> as per my last conversation with stephenfin he is going to update the configurable hashing algo patches 14:21:52 <pdeore> that's it from me for today 14:22:06 <pdeore> moving to open discussions 14:22:08 <pdeore> #topic Open Discussions 14:22:46 <pdeore> anyone has anything else to highlight ? 14:23:41 <abhishekk> Yes 14:23:58 <abhishekk> priority should be given to merge tempest tests for location API 14:24:39 <pdeore> yes, I will update it by addressing your recent comment 14:25:19 <abhishekk> yeah the wait for success is important to handle as it can be led to timeout if hash calculation fails due to some issue 14:26:05 <pdeore> yeah 14:26:08 <abhishekk> I think based on the task state you can decide what to check in test 14:26:20 <abhishekk> positive approach or negative approach 14:26:38 <pdeore> yes 14:26:54 <abhishekk> if success assert hash/checksum/location else assert everything is reverted 14:27:49 <abhishekk> that's it from me 14:28:51 <pdeore> ack thanks ! 14:29:24 <pdeore> let's wrap up then ... 14:29:38 <pdeore> Thanks everyone for joining !! 14:30:04 <pdeore> #endmeeting