| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-08-27 | |||
| 16:28:38 | fwiesel | Yeah, that is its own bug an discussion | |
| 16:28:40 | sean-k-mooney | if there is a followup for the rebuild case i can upgrade to +2 | |
| 16:29:07 | sean-k-mooney | ok upgraded to +2 based on the latest ci results | |
| 16:29:09 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/910627 | |
| 16:29:14 | fwiesel | \o/ | |
| 16:29:27 | fwiesel | That would be then from my side for this week, unless there are questions. | |
| 16:29:34 | sean-k-mooney | im tempeted to +w since other have not reviewd in 6 months | |
| 16:29:36 | bauzas | not from me | |
| 16:29:53 | bauzas | I can review the patch | |
| 16:30:03 | bauzas | CI is green | |
| 16:30:27 | bauzas | and we have some kind of gentlemen's agreement on 'other virt drivers' patches we may not know by heart | |
| 16:30:30 | sean-k-mooney | ok, but general question since cores are not really maintianing the vmware stuff do we want ot require 2 +2s if ci is green and fwiesel's team are the autors of the patch? | |
| 16:30:49 | sean-k-mooney | im stictly talking about chagne that only impact vmware like this | |
| 16:30:51 | bauzas | that's the whole point of running a stable CI | |
| 16:31:03 | dansmith | I still want good review (i.e. two +2s) | |
| 16:31:19 | bauzas | if we can trust the vmware job, then we can get some good regressing insight | |
| 16:31:33 | bauzas | but I second dansmith on not release our two cores approval | |
| 16:31:56 | sean-k-mooney | ok then unless i see more people reviewin this i think we need to reopen the deprecation pathces | |
| 16:31:58 | bauzas | previously, we had the same concerns when other drivers were asking for reviews and we had that agreement | |
| 16:32:22 | bauzas | I don't see the need of threatening the driver | |
| 16:32:26 | sean-k-mooney | im not seeing other actully take the time to review but bauzas if your happy too review then cool | |
| 16:32:31 | gibi | I would be OK to drop down to 1x +2 for a driver only code with a good CI | |
| 16:32:48 | dansmith | I'm mostly concerned about drivers doing things they shouldn't and not so much the review of the actual vmware-specific bits | |
| 16:32:49 | bauzas | fwiesel makes a solid effort of keeping the job paceful, so we just need reviews | |
| 16:32:59 | dansmith | and I think if we drop down the review requirement we're just asking for rubber stamps | |
| 16:33:16 | bauzas | yeah, exactly why I think we need to continue the agreement we had | |
| 16:33:43 | sean-k-mooney | bauzas: that agreement goes too ways assuming more cores then just me look at the patch :) | |
| 16:33:49 | bauzas | two cores need to lookup the patches and somehow be "enlighted" by the result of the CI job | |
| 16:34:00 | sean-k-mooney | im happy for the work fwiesel has been doing | |
| 16:34:13 | bauzas | sean-k-mooney: correct and that's why I'm saying this is a normal situation | |
| 16:34:26 | bauzas | I got the signal and I'll do my homework | |
| 16:34:53 | bauzas | anything else about vmware ? | |
| 16:35:39 | fwiesel | Not from me. | |
| 16:35:41 | bauzas | #topic Open discussion | |
| 16:35:48 | bauzas | I have two procedural items | |
| 16:35:52 | bauzas | (bauzas) Sep 3 meeting | |
| 16:36:05 | bauzas | I'll be traveling to the OIF Asia Summit | |
| 16:36:19 | bauzas | only that week | |
| 16:36:27 | bauzas | so I won't be able to run the meeting | |
| 16:36:35 | bauzas | so either we skip or someone runs it | |
| 16:37:17 | bauzas | the most critical thing is to say next week that FeatureFreeze is coming in 2 days | |
| 16:37:36 | bauzas | so, anyone fancy runnning it ? | |
| 16:37:50 | bauzas | I can notify the deadline async using the ML | |
| 16:38:26 | bauzas | okay, I don't see any head popping up | |
| 16:38:29 | bauzas | so, | |
| 16:38:37 | bauzas | #info Sep 3 meeting is CANCELLED | |
| 16:38:45 | bauzas | #action bauzas to notify by email | |
| 16:38:51 | bauzas | second poit | |
| 16:38:53 | bauzas | point* | |
| 16:39:00 | bauzas | (bauzas) Epoxy PTL seat | |
| 16:39:19 | bauzas | if you contributed to nova recently, you received some email | |
| 16:39:38 | bauzas | maybe you don't know but I'm running for a TC seat this cycle | |
| 16:40:11 | bauzas | I think I can both sit on two chairs (TC and PTL) but I wanted to let people consider the PTL position | |
| 16:41:05 | bauzas | either way, my personal take is that IMHO nova is that important to me that I'd rather stick on that seat than leave the project to be leaderless (procedurarely) | |
| 16:41:26 | bauzas | but if you consider running for that position, my DM is open | |
| 16:41:45 | bauzas | that's it from me | |
| 16:42:02 | bauzas | I don't see any other item in the agenda | |
| 16:42:43 | bauzas | so unless someone disagrees loudly, let's end this meeting earlier and lemme give you 15 mins back of your time | |
| 16:43:35 | bauzas | okay, thanks folks | |
| 16:43:44 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2024/nova.2024-08-27-16.01.log.html | |
| 16:43:44 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2024/nova.2024-08-27-16.01.txt | |
| 16:43:44 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2024/nova.2024-08-27-16.01.html | |
| 16:43:44 | opendevmeet | Meeting ended Tue Aug 27 16:43:44 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 16:43:44 | bauzas | #endmeeting | |
| 16:43:52 | gibi | thanks | |
| 16:43:52 | fwiesel | Thanks, everyone | |
| 16:44:03 | elodilles | thanks o/ | |
| 16:44:42 | JayF | sean-k-mooney: https://review.opendev.org/c/openstack/nova/+/923910 what do you need from me for this to land? | |
| 16:45:14 | JayF | sean-k-mooney: I know FF is in two days, I've been asking for reviews but with the image issues flipping everything upside down, they haven't been coming :) | |
| 16:46:57 | bauzas | JayF: no, FF is next week | |
| 16:47:14 | JayF | oh, cool, I thought it was literally EOW | |
| 16:47:27 | JayF | either way doesn't matter unless I get reviews and/or know what actions I need to take :D | |
| 16:47:34 | bauzas | oh shit | |
| 16:47:44 | bauzas | https://releases.openstack.org/dalmatian/schedule.html#d-ff | |
| 16:47:48 | bauzas | this is this week | |
| 16:47:54 | JayF | Maybe I'll find time to do a video demo, one of my junior folks did that for a patch stuck in Ironic and it helped get reviews | |
| 16:47:56 | bauzas | my brain fucked it up | |
| 16:48:24 | JayF | I think every core on any OpenStack project has a get-out-of-mistake-free card after the major CVE handling in the middle of the cycle | |
| 16:51:04 | opendevreview | Jay Faulkner proposed openstack/nova master: [ironic] Factor out metadata and send to ironic https://review.opendev.org/c/openstack/nova/+/923910 | |
| 16:56:06 | dansmith | bauzas: did you see my comments on that vmware patch? | |
| 16:56:08 | dansmith | I didn't -1 because they were questions | |
| 16:56:13 | sean-k-mooney | JayF: i guess i just need to back and review agian | |
| 16:56:34 | JayF | sean-k-mooney: would a video demo of it in devstack help you review it? | |
| 16:56:35 | sean-k-mooney | JayF: i think it was largly fine the last time i looked but i wanted to see it end to end | |
| 16:56:51 | sean-k-mooney | JayF: proably not but ill watch it if you ahve one | |
| 16:57:19 | JayF | I mean, I won't make one if the primary audience doesn't want it :D | |
| 17:07:32 | dansmith | man the gate is effing busy | |
| 18:01:39 | sean-k-mooney | i havnt looked recently but we are coming to FF | |
| 18:01:45 | sean-k-mooney | so ya... | |
| 18:53:16 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Handle empty imageRef alongside null for local BDM check https://review.opendev.org/c/openstack/nova/+/927311 | |
| 18:54:31 | opendevreview | Merged openstack/nova master: Use format_inspector from oslo https://review.opendev.org/c/openstack/nova/+/925025 | |
| 18:55:00 | dansmith | hoo bioy | |
| 19:06:20 | sean-k-mooney | ah cool so the tempest test change also merged | |
| #openstack-nova - 2024-08-28 | |||
| 01:07:15 | opendevreview | melanie witt proposed openstack/nova master: nova-manage: Add --max_rows option to 'db purge' command https://review.opendev.org/c/openstack/nova/+/927339 | |
| 07:50:30 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add extra specs for USB redirection. https://review.opendev.org/c/openstack/nova/+/927354 | |
| 07:50:51 | mikal | sean-k-mooney: that's the extra specs for USB device redirection finished I hope. | |
| 08:15:56 | jayaanand_ | https://www.irccloud.com/pastebin/WNKStFUI/ | |
| 08:16:17 | jayaanand_ | i am using multi-node devstack setup to troubleshoot customer issue. Devstack is deployed with latest code. during live migration i am facing below issue. your suggestion is deeply appreciate. | |
| 08:17:18 | opendevreview | Fabian Wiesel proposed openstack/nova master: Add more password generation options https://review.opendev.org/c/openstack/nova/+/865669 | |
| 08:20:38 | frickler | jayaanand_: are you using ovn or ovs? you'll likely need to check neutron logs for the cause of that port binding failure | |