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