| *** mklejn_ is now known as mklejn | 08:34 | |
| croelandt | o/ | 14:00 |
|---|---|---|
| pranali | #startmeeting glance | 14:00 |
| opendevmeet | Meeting started Thu Feb 22 14:00:22 2024 UTC and is due to finish in 60 minutes. The chair is pranali. 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 |
| pranali | #topic roll call | 14:00 |
| pranali | #link https://etherpad.openstack.org/p/glance-team-meeting-agenda | 14:00 |
| mklejn | o/ | 14:00 |
| pranali | o/ | 14:00 |
| mrjoshi | o/ | 14:00 |
| pranali | let's start, abhishek will not be able to join today | 14:01 |
| pranali | #topic release/periodic jobs updates | 14:01 |
| pranali | As we all know, M3 is in next week, so we need reviews on priority for Centralized cache DB & New location API changes | 14:01 |
| pranali | and considering the random gate failures we might need to be prepare for FFE for the same if all the patches are approved till next Wednesday | 14:02 |
| pranali | glance_store 4.7.0 release patch is submitted #link https://review.opendev.org/c/openstack/releases/+/909436 | 14:02 |
| pranali | Periodic jobs are all green except intermittent TIME_OUTs on fips jobs | 14:03 |
| pranali | moving to next topic | 14:03 |
| pranali | #topic Important Reviews | 14:03 |
| pranali | Centralized cache DB - #link https://review.opendev.org/q/topic:%22centralized-cache-db%22 | 14:03 |
| pranali | New Location APIS - #link https://review.opendev.org/q/topic:%22New-Location-Apis%22 | 14:03 |
| pranali | Kindly please review these patches on priority so that we can get those in before m3 | 14:03 |
| pranali | For new location apis, patches are updated by fixing the review comments and nova & cinder side PoCs are working as expected | 14:04 |
| pranali | whoami-rajat, has tested this from cinder side and it's working for New location api, there were issue with existing old location api reagrding incompatible location url which is also fixed in this patch | 14:04 |
| pranali | #link https://review.opendev.org/c/openstack/glance/+/906359/8 | 14:04 |
| pranali | dansmith, we need your vote as well on these new location api patches, I have addressed all the suggestion given by you on location-import change | 14:05 |
| pranali | that's it from me | 14:06 |
| pranali | moving to next | 14:06 |
| pranali | #topic Shelved instance image protection | 14:06 |
| pranali | #link https://review.opendev.org/q/topic:%22shelved-image-protection%22 | 14:06 |
| pranali | There is a change proposed to add protection for removing images of shelved instances along with some nova side changes. | 14:07 |
| croelandt | I'm a bit worried about that | 14:07 |
| pranali | As abhishekk suggested I think we need a design spec first for this enhancement | 14:07 |
| croelandt | It adds a config option, which is not "a simple fix" | 14:07 |
| croelandt | and do we know whether they expect some kind of backport? Is it a feature or a bug in their mind? | 14:08 |
| pranali | yeah right | 14:08 |
| croelandt | especially since this config option must then be handled by ooo/NG | 14:08 |
| croelandt | so yeah can they discuss it with us at PTG? | 14:08 |
| pranali | I think mklejn is here | 14:09 |
| mklejn | croelandt: I can join any discussion necessary. Please bare in mind that it's my first contribution, so I'm slightly lost | 14:09 |
| croelandt | mklejn: oh ok :) | 14:10 |
| croelandt | mklejn: so I don't think the patch is bad, at first glance (pun intended) it looks fine | 14:10 |
| croelandt | but the way we do things, there is a lot of brueaucracy (sigh) and also real concerns about backporting & deployment | 14:10 |
| croelandt | We have a meeting between April 8 - 12 that you could join | 14:10 |
| croelandt | pranali: do we have an Etherpad with a list of topics already? | 14:11 |
| pranali | croelandt, no not yet :) once it's ready I will make to add this topic there | 14:11 |
| mklejn | croelandt: I've also made integration tests for this, but I didn't know which tempest is the right place to push it :) | 14:11 |
| croelandt | ok :) | 14:11 |
| croelandt | mklejn: how urgent would you say this is? Can this wait for the next cycle? | 14:12 |
| mklejn | croelandt: yes, it can. don't see a problem with that | 14:13 |
| croelandt | ok so probably we should discuss it during PTG in April and merge it during the next cycle | 14:14 |
| pranali | mklejn, cool.. we are already at the end of this current cycle so we can discuss this during PTG and have it in next cycle along with spce | 14:14 |
| mklejn | croelandt, pranali: sure, I'll join PTG then | 14:15 |
| pranali | mklejn, gr8 | 14:15 |
| pranali | let's move to open discussions then | 14:15 |
| pranali | #topic Open Discussion | 14:15 |
| mklejn | and I'll do the spec in the meantime. about tempest: should I push tests to tempest repo or glance-tempest-plugin? | 14:15 |
| croelandt | Ideally I'd say the plugin | 14:17 |
| pranali | I think tempest would be ok, or you can add scenario test in the plugin | 14:17 |
| mklejn | plugin it is then. thanks | 14:17 |
| pranali | croelandt, could you please have a look at the location api patches again ? I have updated few things today | 14:18 |
| croelandt | yeah I remember I had a few comments | 14:19 |
| croelandt | I'll go throug them again | 14:19 |
| pranali | tomorrow is Recharge Day, so I will submit the release related patches like, releasenotes & config refresh on monday | 14:20 |
| pranali | croelandt, Thanks ! | 14:20 |
| pranali | so that's it from me, anyone has anything else to discuss ? | 14:20 |
| croelandt | Can't wait to be on Monday to review that! | 14:20 |
| pranali | :) | 14:21 |
| pranali | shall we conclude for the day then ? | 14:22 |
| croelandt | fine with me | 14:22 |
| pranali | cool | 14:22 |
| pranali | Thanks everyone for joining !! | 14:22 |
| pranali | #endmeeting | 14:22 |
| opendevmeet | Meeting ended Thu Feb 22 14:22:50 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:22 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/glance/2024/glance.2024-02-22-14.00.html | 14:22 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/glance/2024/glance.2024-02-22-14.00.txt | 14:22 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/glance/2024/glance.2024-02-22-14.00.log.html | 14:22 |
| *** haleyb|out is now known as haleyb | 14:52 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!