| *** tosky_ is now known as tosky | 11:48 | |
| pranali | #startmeeting glance | 14:00 |
|---|---|---|
| opendevmeet | Meeting started Thu Feb 29 14:00:14 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 |
| pranali | o/ | 14:00 |
| mrjoshi | o/ | 14:00 |
| croelandt | o/ | 14:00 |
| abhishekk | o/ | 14:00 |
| rosmaita | o/ | 14:00 |
| * croelandt only half-paying attention | 14:01 | |
| pranali | Let's start | 14:01 |
| pranali | #topic release/periodic jobs updates | 14:01 |
| pranali | We are in M3 week, since we are going to tag m3, we need to get these patches in | 14:01 |
| pranali | releasenotes for m3 - #link https://review.opendev.org/c/openstack/glance/+/910202 | 14:01 |
| pranali | Refresh Config - #link https://review.opendev.org/c/openstack/glance/+/910203/1 | 14:01 |
| pranali | before merging releasenotes patch we need to wait for glance-cache-manage deprecation patch to confirm if deprecation note is rendering properly or not | 14:01 |
| pranali | but i think it's again failing on functional-py39 after recheck | 14:02 |
| pranali | we can wait for some more time for this | 14:03 |
| pranali | once these 2 patches merged I will submit the release patch | 14:04 |
| pranali | Regarding New Location APIs, Can we raise FFE ? so that we can have this feature in this cycle? | 14:04 |
| rosmaita | is the holdup with new location apis getting reviews? | 14:05 |
| rosmaita | i mean, is the holdup that we don't have enough reviews | 14:05 |
| pranali | well, few of them are having +2s but the base patch is still waiting for the +2 | 14:05 |
| pranali | #link https://review.opendev.org/q/topic:%22New-Location-Apis%22 | 14:06 |
| rosmaita | thanks! | 14:06 |
| pranali | this is been tested with nova poc patch and cinder poc as well, though the newly added cinder job is failing on this similar to old loc api, locally it's working | 14:07 |
| pranali | abhishekk, croelandt what would you suggest? | 14:07 |
| abhishekk | my concern is it is failing in gate for cinder | 14:08 |
| abhishekk | SO unless we fixed it on priority it will be a problematic | 14:08 |
| pranali | it's working locally, so we need to figure out the problem with that failing job | 14:09 |
| pranali | and we can fix it later as well | 14:09 |
| abhishekk | what if it needs changes in your code | 14:10 |
| pranali | i think is, this feature will get delayed by one more cycle | 14:10 |
| rosmaita | well, we can aim to have it finished before M-1 and do a preliminary release | 14:10 |
| pranali | i don't think the issue is with new loc api code because same is failing for existing old loc api as well | 14:10 |
| abhishekk | I am not saying there is issue in your code | 14:11 |
| abhishekk | I am saying that what if we need to make changes in your code as well to make it work | 14:12 |
| pranali | I mean if there are bugs then we can fix those later as well and backport those | 14:12 |
| abhishekk | So I agree with rosmaita to have fix it till m1 or else move it to next cycle | 14:12 |
| pranali | m1 means next cycle, right? | 14:13 |
| abhishekk | sorry rc1 | 14:13 |
| rosmaita | i actually meant m-1 of dalmation | 14:13 |
| abhishekk | ohh | 14:13 |
| rosmaita | but if we think we can have everything worked out before rc-1, then we could do a FFE | 14:14 |
| pranali | and for having it in rc1, we need FFE, right? | 14:14 |
| abhishekk | FFE needs favor from 2 Cores | 14:14 |
| rosmaita | it just sounds like we will have to work on this and only this before rc-1 | 14:14 |
| rosmaita | so if we are ok with that, i am not against a FFE | 14:14 |
| abhishekk | I can vote in favor | 14:14 |
| abhishekk | I need FFE for centralized db as well | 14:15 |
| abhishekk | but again it is ok to be get merged next cycle | 14:15 |
| rosmaita | so, here is my thought ... | 14:15 |
| pranali | yeah, till rc1 we have 2 weeks in hands so we can work on that i think | 14:16 |
| rosmaita | this is a SLURP release, so for something like location API that impacts 2 other services, might be better to release in Dalmation (non-slurp) so we have some time to work out issues | 14:16 |
| rosmaita | because many consumers won't upgrade to dalmation | 14:16 |
| rosmaita | so i suggest concentrate on the centralized DB for rc-1 | 14:17 |
| rosmaita | and then we prioritize getting location API finished before M-1 of Dalmatian | 14:17 |
| pranali | hmm ok makes sense | 14:17 |
| abhishekk | ack | 14:17 |
| rosmaita | and croelandt will point out that "Dalmation" is not how you spell "Dalmatian" | 14:18 |
| pranali | ok, then let's raise FFE for centralized DB | 14:18 |
| abhishekk | ack | 14:18 |
| abhishekk | So do i need to raise it today or tomorrow? | 14:19 |
| pranali | abhishekk, we need to submit FFE mail today itself | 14:19 |
| abhishekk | ack | 14:19 |
| pranali | that's it from me for today | 14:19 |
| pranali | moving to open discussions | 14:20 |
| pranali | #topic Open Discussion | 14:20 |
| pranali | deprecating glance-scrubber patch is merged | 14:20 |
| abhishekk | mail sent | 14:20 |
| pranali | I just have one request, since we have decided to move New LoC API to m1 please review the patches on priority | 14:21 |
| pranali | We had decided the same in last cycle as well, though we got some other unexpected issues like ceph deletion issue, we didn't get reviews on the patches on time :( | 14:22 |
| pranali | abhishekk, ack Thanks | 14:22 |
| pranali | so this one needs 2 cores approval right? | 14:23 |
| abhishekk | yes or PTL | 14:23 |
| pranali | ohk | 14:23 |
| rosmaita | pranali: you can announce that no features will be merged in D until after location API has been merged | 14:24 |
| abhishekk | :D | 14:24 |
| pranali | :D | 14:24 |
| rosmaita | i think the core team will agree to that | 14:24 |
| pranali | yeah at least accepting specs for new RFEs will be in my hand :P | 14:25 |
| rosmaita | you know, as PTL you should have at least -W powers | 14:25 |
| pranali | I just have one question regarding release, if we raise FFE the we can tag m3 right ? | 14:25 |
| abhishekk | :P | 14:25 |
| rosmaita | i'm pretty sure we can configure gerrit that way | 14:25 |
| pranali | :D | 14:26 |
| abhishekk | can we skip it and tag rc1 directly? | 14:26 |
| pranali | ohh ok | 14:26 |
| rosmaita | yeah, there's no requirement for an m-3 release | 14:26 |
| pranali | Thanks ! | 14:26 |
| abhishekk | cool | 14:26 |
| abhishekk | by the way pranali if you run tempest locally then you get smae error as you are getting in gate | 14:27 |
| abhishekk | so may be this will help you to debug it locally and see what is happening | 14:27 |
| pranali | ohh ok, I will test that locally then | 14:28 |
| rosmaita | just replied to abhishekk's FFE email | 14:28 |
| pranali | I still don't see that mail in my inbox :/ | 14:29 |
| abhishekk | thank you rosmaita o/ | 14:29 |
| rosmaita | pranali: https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/message/MNVHK22LUBXPL5X37U4NEVQOCZSEV3BK/ | 14:30 |
| pranali | anyone has anything else to highlight? | 14:30 |
| abhishekk | Nope | 14:30 |
| pranali | rosmaita, Thanks ! | 14:31 |
| pranali | ok, let's conclude for the day then | 14:31 |
| rosmaita | works for me! | 14:31 |
| pranali | Thanks everyone for joining ! | 14:31 |
| pranali | #endmeeting | 14:31 |
| opendevmeet | Meeting ended Thu Feb 29 14:31:52 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:31 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/glance/2024/glance.2024-02-29-14.00.html | 14:31 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/glance/2024/glance.2024-02-29-14.00.txt | 14:31 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/glance/2024/glance.2024-02-29-14.00.log.html | 14:31 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!