| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-08-23 | |||
| 16:22:40 | gibi | I see that https://review.opendev.org/q/topic:bp/volume-backed-server-rebuild+status:open is green but lacking core review | |
| 16:23:14 | dansmith | I initially did a bunch of review on it, | |
| 16:23:25 | dansmith | but was hoping to get another core to have fresh eyes on it before we merge | |
| 16:23:38 | sean-k-mooney | the tempet test is red is that related to teh testcases they are adding | |
| 16:23:43 | dansmith | so I'm good to +W that if someone else looks it over, but I had my hands all in it, so I think that'd be better | |
| 16:23:57 | dansmith | sean-k-mooney: is it? it was passing at one point | |
| 16:24:00 | opendevreview | Merged openstack/nova stable/ussuri: [ironic] Minimize window for a resource provider to be lost https://review.opendev.org/c/openstack/nova/+/853540 | |
| 16:24:09 | gibi | it is behind https://review.opendev.org/q/topic:bp/update-userdata+status:open where I need to go back and re-review. after that I might be able to look at the rebuild sets but no hard promises | |
| 16:24:27 | sean-k-mooney | well some of the jobs on the tempest repo are red but i dont know if the new ones worked | |
| 16:24:55 | dansmith | the runs logs are too old, so it needs a recheck | |
| 16:25:22 | sean-k-mooney | ack shall i do it or have you already done it | |
| 16:25:29 | dansmith | I done did it | |
| 16:25:34 | sean-k-mooney | cool | |
| 16:25:52 | dansmith | seems unlikely to merge at this late date if it's behind something else for a microversion, | |
| 16:25:55 | dansmith | but that kinda sucks because a lot was done on it | |
| 16:26:08 | dansmith | so if someone has time to look that'd be nice, maybe we can FFE it if we need | |
| 16:26:20 | sean-k-mooney | well it coudl be merge ahead of the other patch | |
| 16:26:41 | sean-k-mooney | we might want to swap them since there is an openquestion on the update user data pathc | |
| 16:26:54 | gibi | I have the intention to go back the the update user_data patch (that is the next microversion) and also melwitt reviewe that before | |
| 16:26:58 | dansmith | it's a cross-project thing where cinder has done a bunch of work to enable it, so it definitely sucks to cause them to miss again | |
| 16:27:01 | gibi | so we might be able to unblock the it | |
| 16:27:04 | dansmith | already missed last cycle, IIRC | |
| 16:27:35 | sean-k-mooney | dansmith: there is only one patch in the other fature to merge | |
| 16:27:45 | sean-k-mooney | but yes it woudl be shame if it missed | |
| 16:28:05 | dansmith | okay well if we can get someone to look it over while we decide or work on the other, that'd be cool | |
| 16:28:10 | dansmith | maybe melwitt would want to look at it | |
| 16:28:40 | gibi | ack | |
| 16:29:16 | gibi | any other bp to raise? | |
| 16:30:15 | sean-k-mooney | am jsut i think the teanat id one is likely not going to land | |
| 16:30:38 | gibi | it is in merge conflict since we merged the unittest.mock change ~3weeks ago | |
| 16:30:39 | sean-k-mooney | dansmith: i kind of lost track of the SRBAC stuff are there any reverts that still need to land for that | |
| 16:30:49 | dansmith | um | |
| 16:31:08 | sean-k-mooney | its not really a blue print but basically is there any work we need to have in zed | |
| 16:31:14 | dansmith | I think it merged underneath something from gmann | |
| 16:31:22 | sean-k-mooney | ok | |
| 16:32:01 | dansmith | but I'm not seeing it actually | |
| 16:32:17 | dansmith | nope: https://review.opendev.org/c/openstack/nova/+/848021 | |
| 16:32:32 | dansmith | that one and the one above | |
| 16:32:48 | sean-k-mooney | ack so those should be set as a review priorty to land for zed | |
| 16:32:56 | dansmith | maybe we can get gmann to +2 that and someone else to ack it, yeah | |
| 16:33:34 | sean-k-mooney | is gmann on pto i have not spoken to them in a few days | |
| 16:33:57 | dansmith | I think something is keeping him busy, but he was around the last few days a little | |
| 16:34:04 | gibi | I added that rbac stuff to the tracking etherpad | |
| 16:34:10 | dansmith | gibi: thanks | |
| 16:34:51 | sean-k-mooney | i think thats the main set that we needed to talk about | |
| 16:35:18 | gibi | ack then moving on | |
| 16:35:28 | gibi | #topic Review priorities | |
| 16:35:33 | gibi | #link https://review.opendev.org/q/status:open+(project:openstack/nova+OR+project:openstack/placement+OR+project:openstack/os-traits+OR+project:openstack/os-resource-classes+OR+project:openstack/os-vif+OR+project:openstack/python-novaclient+OR+project:openstack/osc-placement)+(label:Review-Priority%252B1+OR+label:Review-Priority%252B2) | |
| 16:35:56 | gibi | looks fairly relevant | |
| 16:36:06 | gibi | #topic Stable Branches | |
| 16:36:14 | gibi | elodilles: any news from stable/ | |
| 16:36:15 | gibi | ? | |
| 16:36:20 | elodilles | yepp | |
| 16:36:24 | elodilles | #info stable/train was unblocked last week (with the backport of the patch that migrated jobs to zuul v3) | |
| 16:36:40 | elodilles | #info now only stable/stein (and older) are blocked: grenade and other devstack based jobs fail with the same timeout issue as stable/train was previously | |
| 16:36:50 | elodilles | #info stable branch status / gate failures tracking etherpad: https://etherpad.opendev.org/p/nova-stable-branch-ci | |
| 16:37:19 | sean-k-mooney | elodilles i think there is an issue with cliff on yoga and older by the way | |
| 16:37:30 | sean-k-mooney | more for cilometer then us | |
| 16:37:32 | elodilles | sean-k-mooney: is it? :-o | |
| 16:37:49 | sean-k-mooney | but cliff 4. something droped python 3.6 support | |
| 16:37:59 | sean-k-mooney | so centos 8 stream jobs are failing | |
| 16:38:13 | sean-k-mooney | i did see it on stable/victoria on nova however if i am rememebring correctly | |
| 16:38:17 | elodilles | isn't that non-voting? | |
| 16:38:35 | elodilles | i mean in that case it's not blocking the gate | |
| 16:38:47 | sean-k-mooney | not for nova | |
| 16:39:09 | sean-k-mooney | i work with peopel that work on ceilometer that are trigng to fix it for there jobs | |
| 16:39:23 | sean-k-mooney | just mentioning it as an fyi incase you see stabel review related to it | |
| 16:39:41 | elodilles | sean-k-mooney: ack, thanks, i'll look into that | |
| 16:40:09 | sean-k-mooney | https://review.opendev.org/c/openstack/requirements/+/854130 | |
| 16:40:34 | elodilles | sean-k-mooney: ++ | |
| 16:40:59 | elodilles | i've noted that patch | |
| 16:41:29 | gibi | anything else about stable? | |
| 16:41:42 | elodilles | nothing else from me | |
| 16:41:45 | gibi | thanks elodilles | |
| 16:41:49 | elodilles | np | |
| 16:42:05 | gibi | #topic Open discussion | |
| 16:42:10 | gibi | nothing on the agenda. | |
| 16:42:25 | gibi | does anybody has a topic still for today? | |
| 16:43:27 | gibi | I take the silence as no | |
| 16:43:33 | gibi | so thanks for joining | |
| 16:43:35 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2022/nova.2022-08-23-16.00.log.html | |
| 16:43:35 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2022/nova.2022-08-23-16.00.txt | |
| 16:43:35 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2022/nova.2022-08-23-16.00.html | |
| 16:43:35 | opendevmeet | Meeting ended Tue Aug 23 16:43:35 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 16:43:35 | gibi | #endmeeting | |
| 16:43:58 | elodilles | thanks o/ | |
| 16:47:48 | sean-k-mooney | elodilles: i put the details in the etherpad | |
| 16:47:55 | sean-k-mooney | https://etherpad.opendev.org/p/nova-stable-branch-ci#L22 | |
| 16:50:16 | JayF | btw; thanks to you all for the help with the stable reviews, trying to get the ironic driver as good as it can be in the stable branch. https://review.opendev.org/c/openstack/nova/+/821350 still needs another core vote in victoria | |
| 16:50:36 | JayF | I know you rarely need us, but I'm happy to help if you all run into Ironic shenanigans in the future :D | |
| 16:51:41 | gibi | sean-k-mooney: fyi I created a test case and now I can confirm that different PFs are tracked as separate PCI pools even if they have same kind of VFs. Fortunately both the apply and the consume logic in stats able to consume from two pools for a single pci request. So everything works \o/ even with splitted pools | |
| 16:51:46 | sean-k-mooney | well there is one open spec that we really need your import on | |
| 16:52:18 | sean-k-mooney | gibi: nice | |
| 16:52:37 | sean-k-mooney | JayF: if you have not read https://review.opendev.org/c/openstack/nova-specs/+/842015 | |
| 16:52:51 | sean-k-mooney | JayF: we dont currently have a good way forward on that | |
| 16:53:33 | JayF | I've absolutely had backchannel chats with Julia about that (and felt the pain it is trying to solve). I will scratch my brain for it again but I'm not sure I know how to break out of the logjam | |
| 16:54:32 | sean-k-mooney | we were hopign to have a face to face session on this at the ptg before it went virtual | |
| 16:54:52 | sean-k-mooney | it really feels like it needs a white board and some time to talk thing through | |
| 16:54:59 | JayF | I have $800 of Alaska Airlines credit proving my intention to have showed up to that :( | |
| 16:55:19 | JayF | hopefully we can find time in virtual summit, anyway, although you're right it's often easier to solve things irl | |