| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-08-23 | |||
| 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 | gibi | #endmeeting | |
| 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 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2022/nova.2022-08-23-16.00.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 | Log: https://meetings.opendev.org/meetings/nova/2022/nova.2022-08-23-16.00.log.html | |
| 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 | |
| 16:56:36 | sean-k-mooney | +2w on that victoria backport | |
| 16:57:03 | JayF | wonderful, thanks. I'll have probably three fresh in the morning as I pull everything down another level. | |
| 16:57:30 | JayF | sean-k-mooney: https://etherpad.opendev.org/p/JayFTODO I put all the branches on this so I'd stop doing that | |
| 16:57:48 | JayF | sean-k-mooney: it's worse for Ironic, we have mid-stream bugfix releases that are numbered, and they might as well be named after colors for all the meaning the names have for me lolsob | |
| 16:58:28 | sean-k-mooney | i see :) | |
| 17:01:07 | sean-k-mooney | ok im gong to more or less call it a day. | |
| 17:01:34 | sean-k-mooney | ill likely do some code review on my ipad while i cook/order dinner but im going to drop off irc ffor the day | |
| 17:01:45 | gibi | sean-k-mooney: have a nice evening o/ | |
| 17:15:02 | opendevreview | Elod Illes proposed openstack/nova stable/train: [ironic] Minimize window for a resource provider to be lost https://review.opendev.org/c/openstack/nova/+/853546 | |
| 17:16:05 | JayF | elodilles: thanks, I had that on my todo list but I appreciate it \o/ | |
| 17:22:33 | elodilles | JayF: np, just re-applied the cherry-pick to have the right format of the commit message, otherwise it seems right, so +2'd it already | |
| 17:22:46 | JayF | So, what you want in the commit is | |
| 17:23:05 | elodilles | it's not the commit message in general o:) | |
| 17:23:06 | JayF | cherry-picked-from (master SHA)\ncherry-picked-from (master-1 sha)\n etc etc | |
| 17:23:17 | JayF | which is achieved by a clean cherry pick from stable N -> stable N-1 | |
| 17:23:23 | elodilles | but to do the cherry-picking branch-by-branch, | |
| 17:24:22 | elodilles | so that one can see that the cherry picking was in the right order, from the right (latest) patch sets, etc. | |
| 17:25:09 | JayF | so for my most recent master change, I'd cherry-pick it to stable/yoga, then cherry pick *the yoga change* onto xena, and so on | |
| 17:25:40 | JayF | and if at any point one patch in the cycle is changed or rebased (whoops), the ones after it have to be re-cherry-picked | |
| 17:26:26 | elodilles | JayF: exactly :) | |
| 17:26:44 | JayF | is this a nova-ism? Or something generally that most OpenStack projects do that Ironic has never really enforced? | |
| 17:27:54 | elodilles | the backports should be done branch-by-branch according to stable policy | |
| 17:28:08 | JayF | ack, so I'll update how I do it in Ironic world too | |
| 17:28:37 | elodilles | but yes, probably only nova has this check as (we) stable cores were picky about this one o:) | |
| 17:29:00 | JayF | Yeah, I've been focused on stable reviews in Ironic for a long time, and this is *never* something I've checked for | |
| 17:29:03 | elodilles | to avoid wrong/not latest/missing backports | |
| 17:29:11 | JayF | frankly, I doubt I'd -1 a change for this now -- but I do always manually check the earlier branches | |
| 17:29:18 | JayF | and I'll start doing it right for me ;D | |