| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-04-26 | |||
| 16:35:05 | bauzas | I'm sure stephenfin said he was okay to leave it for others to continue :) | |
| 16:35:27 | bauzas | hence the gerrit hammer | |
| 16:35:47 | bauzas | this may help people to react and assign some time for this | |
| 16:36:43 | bauzas | I'll also drop the review-prio flag which is meaningless in this case as we can't merge it as it is | |
| 16:37:43 | bauzas | ok, moving on if nobody yells | |
| 16:38:10 | bauzas | #topic Stable Branches | |
| 16:38:14 | bauzas | elodilles: your turn | |
| 16:38:20 | elodilles | #info ussuri and older branches are blocked until 'l-c drop' patches merge - https://review.opendev.org/q/I514f6b337ffefef90a0ce9ab0b4afd083caa277e | |
| 16:38:30 | elodilles | #info other branches should be OK | |
| 16:38:36 | elodilles | #info nova projects' stable/victoria transitioned to Extended Maintenance - no further releases will be produced from victoria, but branch remains open to accept bug fixes | |
| 16:38:44 | elodilles | and that's all I think ^^^ | |
| 16:39:15 | melwitt | we need second core wink wink | |
| 16:39:24 | elodilles | :] | |
| 16:39:41 | gibi | I would be happy to approve... ;) | |
| 16:39:52 | gibi | (has some blocked train backports :D) | |
| 16:40:03 | bauzas | I can do things | |
| 16:40:12 | gibi | bauzas: ask elodilles to add me into the stable-core group | |
| 16:40:12 | bauzas | my brain fsck'd me | |
| 16:40:20 | gibi | then I can help | |
| 16:40:53 | bauzas | gibi: I think we said at the PTG I should propose your name against the stable team | |
| 16:41:01 | gibi | yeah I think so | |
| 16:41:05 | bauzas | so, | |
| 16:41:05 | gibi | so lets do it :D | |
| 16:41:19 | bauzas | #1 I'll do my homework and review such l-c patches | |
| 16:41:53 | bauzas | #2 I'll do my duty and engage discussions about reconciling the nova team and the nova-stable team in some intelligent manner | |
| 16:43:18 | bauzas | last topic in the agenda, | |
| 16:43:21 | bauzas | #topic Open discussion | |
| 16:43:27 | bauzas | (gibi) Allow claiming PCI PF if child VF is unavailable https://review.opendev.org/c/openstack/nova/+/838555 | |
| 16:43:33 | bauzas | gibi: take the mic | |
| 16:43:38 | gibi | thanks | |
| 16:43:45 | gibi | so | |
| 16:43:51 | gibi | it is a bug | |
| 16:43:58 | gibi | we saw DB inconsistencies at customers | |
| 16:44:23 | gibi | the pci_devices table contains available PF and unavailable children VF rows | |
| 16:44:39 | gibi | this basically an impossible situation | |
| 16:44:54 | gibi | the VF should be available if the PF is available | |
| 16:45:04 | gibi | or the VF should be allocated | |
| 16:45:12 | gibi | anyhow | |
| 16:45:26 | gibi | I propsed a fix https://review.opendev.org/c/openstack/nova/+/838555 | |
| 16:45:40 | gibi | it removes some of the strictness of the state check during the PCI claim | |
| 16:45:56 | gibi | basically allows allocating the available PF if the children VFs are unavailable | |
| 16:46:06 | gibi | this would heal the inconsistent DB state | |
| 16:46:13 | gibi | artom had a good point in the review | |
| 16:46:32 | gibi | that we tend to handle DB healing via nova-manage CLI instead | |
| 16:46:45 | bauzas | not true | |
| 16:46:59 | sean-k-mooney | we heal some things on object load form the db | |
| 16:47:02 | bauzas | we had db healing made thru data migrations | |
| 16:47:51 | artom | Right, but those fixes are from "we had thing X a long time ago, and it might still be in the DB, so now when we load it we convert to thing X" | |
| 16:47:59 | sean-k-mooney | we do have the heal allcoation type commands but i think healing this on agent start is the right thing | |
| 16:48:01 | bauzas | we only heal things thru the CLI if this is for example something due to some relationship between two DBs | |
| 16:48:08 | artom | vs examples like placement-audit and Lee's connection_info update | |
| 16:48:09 | sean-k-mooney | as we are also fixing the in memroy representation | |
| 16:48:18 | bauzas | sean-k-mooney: correct, because two DBs were involved | |
| 16:48:52 | artom | My other point was - the DB somehow got into an inconsistent state, wouldn't it be wiser to at least let the operator know, vs siltently fixing it? | |
| 16:48:58 | bauzas | in the case of placement audit, this was about reconciling two datastores kept from two different projects | |
| 16:49:05 | sean-k-mooney | artom: i dont think so | |
| 16:49:17 | bauzas | I agree with sean-k-mooney | |
| 16:49:25 | bauzas | we could log such thing | |
| 16:49:32 | bauzas | but no need to claim it loud | |
| 16:49:32 | sean-k-mooney | artom: i think this happened beacue of how the custoemr recreated the compute node after the hdd died | |
| 16:49:34 | artom | OK :) Not a hill I want to die on, but wanted to at least raise the question | |
| 16:49:35 | gibi | I've added a WARNING log in the patch | |
| 16:49:52 | artom | Seems like I'm outnumbered :) | |
| 16:49:52 | sean-k-mooney | i dont think this is something that most operators would hit | |
| 16:50:12 | bauzas | artom: I won't ask you how many divisions you have | |
| 16:50:19 | gibi | sean-k-mooney: one more thing, you said it should be fixed at agent restart. Now my patch fixes it during PCI claim | |
| 16:50:31 | artom | bauzas, divisions o_O? | |
| 16:50:51 | bauzas | gibi: I like the pci claim approach | |
| 16:51:05 | sean-k-mooney | i have not reviewd the third patch yet | |
| 16:51:08 | gibi | sean-k-mooney: and I have a separate patch for the agent restart + remove VF + inconsistent state case | |
| 16:51:24 | gibi | sean-k-mooney: ahh, OK, let me know you opininon once you reviewed it | |
| 16:51:27 | bauzas | artom: sorry, I'll DM you the reference :) | |
| 16:51:40 | sean-k-mooney | ack we can proceed on the gerrit review | |
| 16:51:47 | artom | It's going to be an obscure French thing, isn't it :P | |
| 16:52:28 | gibi | artom, sean-k-mooney, bauzas: thanks, we can move on | |
| 16:52:37 | bauzas | ++ | |
| 16:52:56 | gibi | I will comment on the patch linking to the meeting logs | |
| 16:52:56 | bauzas | sounds we got a consensus : review gibi's patch | |
| 16:53:28 | bauzas | #agreed let's continue to review gibi's work with pci claims fixing the inconstitency | |
| 16:53:45 | bauzas | that's all we had for the meeting | |
| 16:53:55 | bauzas | any last minute item people wanna raise ? | |
| 16:54:32 | bauzas | looks not, | |
| 16:54:37 | bauzas | thanks all ! | |
| 16:54:37 | gibi | - | |
| 16:54:39 | gibi | thanks! | |
| 16:54:46 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2022/nova.2022-04-26-16.00.log.html | |
| 16:54:46 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2022/nova.2022-04-26-16.00.txt | |
| 16:54:46 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2022/nova.2022-04-26-16.00.html | |
| 16:54:46 | opendevmeet | Meeting ended Tue Apr 26 16:54:46 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 16:54:46 | bauzas | #endmeeting | |
| 16:54:48 | elodilles | thanks o/ | |
| 16:54:59 | gibi | bauzas: I hope you did not forget to leave around 18:30 :) | |
| 16:55:38 | bauzas | gibi: well, I chose to stay :) | |
| 16:55:45 | artom | He replaced himself with a very small shellscript | |
| 16:55:56 | bauzas | Sophie will stare at me when I go off the office room | |
| 16:56:00 | artom | Turns out we was getting bash'ed this whole time | |
| 16:56:09 | gibi | artom: lol | |
| 16:56:17 | bauzas | artom: I wish I could | |
| 16:57:11 | bauzas | artom: but I've been told any IA bot turns crazy after you let people discussing with it | |
| 16:58:02 | gibi | it is our, humans, fault to make AI bots crazy | |