| *** mhen_ is now known as mhen | 02:22 | |
| opendevreview | Takashi Kajinami proposed openstack/nova master: Use native IntOpt for [libvirt] tb_cache_size https://review.opendev.org/c/openstack/nova/+/969014 | 04:54 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Remove tpm supoort detection for libvirt < 8.0.0 https://review.opendev.org/c/openstack/nova/+/952308 | 06:13 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: Use native IntOpt for [libvirt] tb_cache_size https://review.opendev.org/c/openstack/nova/+/969014 | 06:16 |
| opendevreview | zhou zhong proposed openstack/nova master: Create ephemeral and private secret when attaching encryptor https://review.opendev.org/c/openstack/nova/+/969019 | 07:58 |
| gokhan__ | Hi team, I encountered a critical issue during a Nova live migration. When migrating a VM that has a Cinder volume in the **`backing up`** status, the live migration fails, but the **target host then attempts to detach the wrong volume** as part of the cleanup process. It tries to detach last created instance volume. | 08:22 |
| gokhan__ | I suspect a bug in the rollback/cleanup logic. | 08:22 |
| gokhan__ | Source Host Log: https://paste.openstack.org/show/bpz0kwEkroSrCizM5C4L/ | 08:22 |
| gokhan__ | Target Host Log: https://paste.openstack.org/show/bv1KJ0ATIV9gX9RAIQi8/ | 08:22 |
| gokhan__ | Is this a known issue, maybe related to bug 2112187 or volume attachment cleanup on migration failure? Thanks! | 08:22 |
| gokhan__ | https://bugs.launchpad.net/nova/+bug/2112187 | 08:24 |
| *** sambork_ is now known as sambork | 09:19 | |
| opendevreview | Merged openstack/nova stable/2024.2: Update start_service() function in test https://review.opendev.org/c/openstack/nova/+/963848 | 09:27 |
| opendevreview | Merged openstack/nova stable/2024.2: Adds regression test for bug LP#2085135 https://review.opendev.org/c/openstack/nova/+/963849 | 09:27 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: Use consistent program name for wsgi scripts and entry points https://review.opendev.org/c/openstack/nova/+/942605 | 09:55 |
| opendevreview | Merged openstack/nova stable/2024.2: Reset the mapped field of nodes at service deletion https://review.opendev.org/c/openstack/nova/+/963850 | 10:29 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: [virt]Clarify the meaning of init_host https://review.opendev.org/c/openstack/nova/+/967398 | 12:06 |
| opendevreview | Konrad Gube proposed openstack/nova-specs master: Propose assisted-volume-extend API https://review.opendev.org/c/openstack/nova-specs/+/949504 | 12:58 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: Use python-native keyword-only arguments https://review.opendev.org/c/openstack/nova/+/969050 | 13:23 |
| opendevreview | sean mooney proposed openstack/nova-specs master: add spec for resouce notifications https://review.opendev.org/c/openstack/nova-specs/+/967712 | 13:41 |
| sean-k-mooney | gibi: im hoping ^ will be the final revsion of the resource notifcaion spec. im going to try and refocus on the cyborg spec and see if i cna get that into a workable shape to proceed with but tiem is obviously tight there. | 14:11 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Add basic xml generation for firmwar auto selection https://review.opendev.org/c/openstack/nova/+/969085 | 14:23 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Add capability to load loader and nvram from xml https://review.opendev.org/c/openstack/nova/+/969086 | 14:23 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Add basic xml generation for firmware auto selection https://review.opendev.org/c/openstack/nova/+/969085 | 14:24 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Add capability to load loader and nvram from xml https://review.opendev.org/c/openstack/nova/+/969086 | 14:24 |
| Uggla | Nova meeting in ~1h | 15:01 |
| *** haleyb|out is now known as haleyb | 15:54 | |
| Uggla | #startmeeting nova | 16:04 |
| opendevmeet | Meeting started Mon Dec 1 16:04:12 2025 UTC and is due to finish in 60 minutes. The chair is Uggla. Information about MeetBot at http://wiki.debian.org/MeetBot. | 16:04 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 16:04 |
| opendevmeet | The meeting name has been set to 'nova' | 16:04 |
| Uggla | Hello everyone | 16:04 |
| tkajinam | o/ | 16:04 |
| bauzas | o/ | 16:04 |
| fwiesel | o/ | 16:04 |
| kgube | o/ | 16:04 |
| gibi | o/ | 16:05 |
| opendevreview | sean mooney proposed openstack/nova master: [WIP] create ResourceSummaryNotification object https://review.opendev.org/c/openstack/nova/+/968558 | 16:06 |
| elodilles | o/ | 16:07 |
| Uggla | Let's go | 16:07 |
| Uggla | #topic Bugs (stuck/critical) | 16:08 |
| Uggla | #info No Critical bug | 16:08 |
| Uggla | #topic Gate status | 16:08 |
| Uggla | #link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs | 16:08 |
| Uggla | #link https://etherpad.opendev.org/p/nova-ci-failures-minimal | 16:08 |
| Uggla | #link https://zuul.openstack.org/builds?project=openstack%2Fnova&project=openstack%2Fplacement&branch=stable%2F*&branch=master&pipeline=periodic-weekly&skip=0 Nova&Placement periodic jobs status | 16:08 |
| Uggla | #info Please look at the gate failures and file a bug report with the gate-failure tag. | 16:08 |
| Uggla | #info Please try to provide a meaningful comment when you recheck | 16:08 |
| Uggla | i have not seen something special on the gate unless I miss something. | 16:09 |
| Uggla | Skipping next point as gmaan is not here. | 16:10 |
| Uggla | #topic Release Planning | 16:11 |
| Uggla | #link https://releases.openstack.org/gazpacho/schedule.html | 16:11 |
| Uggla | #info Second spec review day is tomorrow and FF is Thursday. | 16:12 |
| tkajinam | Do you mean spec freeze, right ? | 16:12 |
| Uggla | oops sorry my bad, yes I mean spec freeze (SF° | 16:12 |
| gibi | no no we do FF now and go for a long PTO :) | 16:13 |
| tkajinam | instead of feature freeze (if I understand what it represents) | 16:13 |
| tkajinam | lol | 16:13 |
| gibi | sorry :) | 16:13 |
| Uggla | gibi, hum maybe that's a good proposal. ;) | 16:13 |
| Uggla | #info I'll send a "warning" about SF on the mailing list. | 16:14 |
| Uggla | #info PTG etherpad for 2026.1 is available: https://etherpad.opendev.org/p/nova-2026.1-ptg | 16:16 |
| Uggla | #topic Review priorities | 16:16 |
| Uggla | #link https://etherpad.opendev.org/p/nova-2026.1-status | 16:16 |
| Uggla | #topic OpenAPI | 16:16 |
| Uggla | #link: https://review.opendev.org/q/topic:%22openapi%22+(project:openstack/nova+OR+project:openstack/placement)+-status:merged+-status:abandoned | 16:16 |
| Uggla | #info still 23 remaining atm. | 16:16 |
| sean-k-mooney | oh o/ | 16:17 |
| Uggla | #topic Stable Branches | 16:17 |
| * Uggla giving the mic to elodilles | 16:17 | |
| elodilles | thanks | 16:17 |
| elodilles | #info all three stable branches' gates seem to be in a good state again | 16:18 |
| elodilles | grenade jobs are passing again | 16:18 |
| elodilles | #info stable branch status / gate failures tracking etherpad: https://etherpad.opendev.org/p/nova-stable-branch-ci | 16:18 |
| elodilles | and that's all from me | 16:18 |
| elodilles | Uggla: back to you | 16:18 |
| Uggla | thanks elodilles | 16:21 |
| Uggla | #topic vmwareapi 3rd-party CI efforts Highlights | 16:21 |
| fwiesel | Hi, no updates from my side. | 16:21 |
| Uggla | fwiesel thanks | 16:21 |
| gibi | fwiesel: I have a question for you | 16:21 |
| fwiesel | gibi: Go ahead | 16:21 |
| gibi | what would be the way to trigger some vmware testing for this changes https://review.opendev.org/q/topic:%22eventlet-removal%22+project:openstack/oslo.vmware ? | 16:22 |
| gibi | the last one in the series is probably not a clean refactor and might cause troubles | 16:22 |
| gibi | https://review.opendev.org/c/openstack/oslo.vmware/+/967418 | 16:22 |
| fwiesel | gibi: A comment with "sap-openstack-ci recheck - some comment" should do the trick. But currently it is broken. | 16:25 |
| fwiesel | The neutron service cannot even start a simple thread, which I have trouble understanding. | 16:26 |
| gibi | OK thanks for the info | 16:26 |
| gibi | fwiesel: neutron when from eventlet to threading at the end of Flamingo | 16:27 |
| gibi | so probably that change is related to your trouble | 16:28 |
| gibi | anyhow I will circle back later to the oslo.vmware patch | 16:28 |
| fwiesel | gibi: Yeah, and at least in our VM, a simple "threading.Thread(target=func)" doesn't work in the place where the service should be started. the target function won't get started. | 16:28 |
| gibi | fwiesel: that line does not start a thread, you need .start() at the end to actually kick of the defined thread | 16:29 |
| Uggla | gibi, fwiesel is that ok, can we move on ? | 16:31 |
| fwiesel | Uggla: Sure, I don'T think we will debug that now :) | 16:31 |
| gibi | sure | 16:31 |
| gibi | :) | 16:31 |
| Uggla | #topic Gibi's news about eventlet removal | 16:32 |
| gibi | o/ | 16:32 |
| Uggla | gibi, something new to share about eventlet removal ? | 16:32 |
| gibi | unfortunately not much new on my side. The bottom 7 patches are ready for review in the compute series https://review.opendev.org/c/openstack/nova/+/965467/22 | 16:33 |
| gibi | and you already saw above my oslo.vmware series | 16:33 |
| gibi | which needs testing | 16:33 |
| gibi | sambork: anything on your side maybe? | 16:34 |
| gibi | I guess no, so we can move on | 16:36 |
| Uggla | thanks gibi | 16:36 |
| Uggla | #topic Open discussion | 16:37 |
| Uggla | I will cancel the next week meeting. | 16:37 |
| Uggla | I don't have a clear visibility on our agenda, so I'd rather cancel. | 16:38 |
| Uggla | I will write an email to openstack discuss too. | 16:38 |
| gibi | yeah some of us will travel next week so I agree to cancel | 16:38 |
| tkajinam | +1 | 16:39 |
| Uggla | Something you'd like to discuss ? | 16:39 |
| tkajinam | I'm just wondering if we can quickly discuss https://review.opendev.org/c/openstack/nova-specs/+/966584 . | 16:40 |
| Uggla | sure | 16:41 |
| gibi | I'm pretty +2 on it :) | 16:41 |
| tkajinam | :-) | 16:41 |
| Uggla | bauzas as well. | 16:41 |
| bauzas | yup, I can send it to the gate | 16:42 |
| tkajinam | ah, ok | 16:42 |
| tkajinam | we can agree how complicated the logic to select the existing firmware is, based on the actual code logic | 16:42 |
| tkajinam | I mean leaving it for follow-up and keep the original direction so far | 16:42 |
| gibi | tkajinam: ^^ good startegy | 16:42 |
| tkajinam | I saw sean-k-mooney also voted his +2 :-) | 16:43 |
| sean-k-mooney | i did | 16:44 |
| sean-k-mooney | but i dont agree with everything | 16:44 |
| sean-k-mooney | i belvie we shoudl be regenergatin the xml without lookign at the exsitng xml | 16:44 |
| bauzas | can we still send it to the gate then ? | 16:45 |
| sean-k-mooney | for both hard-reboot and live migrate | 16:45 |
| sean-k-mooney | i see the current propoal to pin it as a an issue for day2 operations and posisbel security updates if the device path ever change | 16:45 |
| tkajinam | ok so you are mostly on the same side as Uggla. | 16:46 |
| sean-k-mooney | so to me that part is compliecating the propsoal by requiring extra code to maintain the definaiton and complicating upgrades | 16:46 |
| sean-k-mooney | yes | 16:46 |
| sean-k-mooney | i suggested we move forward and revisit in the code review | 16:46 |
| sean-k-mooney | im not going to block on this point either way | 16:46 |
| sean-k-mooney | btu i dont think the curernt proposal s the right longterm choice | 16:46 |
| tkajinam | yeah | 16:46 |
| tkajinam | let's continue the discussion in the actual code review. | 16:47 |
| Uggla | yep I think we can progress anyway. | 16:47 |
| gibi | I don't see dansmith around but based on his review comment he would disagree with always triggering firmware re-select at hard reboot | 16:48 |
| bauzas | yup and I agree with him | 16:48 |
| bauzas | that's why tkajinam said this for both of the actions | 16:48 |
| dansmith | sorry, still catching up | 16:49 |
| * dansmith reads back | 16:49 | |
| dansmith | I don't understand the day2 maintenance issue -- we're currently hard-coding the value into the xml today, correct? | 16:50 |
| dansmith | so all I'm proposing is we don't suddenly change that for any instances due to an operation that a user or operator would not expect could fundamentally and invisibly change the configuration/behavior of the instance | 16:51 |
| sean-k-mooney | we do yes which mean we somethime dont find the correct firmware blob and or need to symlink files if they move | 16:51 |
| dansmith | right, but if the guest has booted, then it's right.. I just don't think changing it on a working guest without some understanding of what's going on makes sense | 16:51 |
| sean-k-mooney | we had a regression in rhel 9 which they fixed but we were broken by a chagne like that | 16:51 |
| dansmith | sure, and letting libvirt pick going forward is the solution there | 16:52 |
| sean-k-mooney | right but the curernt propaly mean that does nto happen until the vm is mvoed | 16:52 |
| sean-k-mooney | which is a porblem if you do inplace upgrades | 16:52 |
| dansmith | yep, understood | 16:52 |
| sean-k-mooney | as i said my prefernce is to regenreated it | 16:52 |
| sean-k-mooney | but i dont object to the current proposal | 16:52 |
| sean-k-mooney | in most case peopel are going to cold/live migrate for disto changes so i think for the most part we are ok | 16:53 |
| dansmith | I'm not majorly opposed to regen all the time either, I just think it's the less-safe option | 16:53 |
| dansmith | I thought the discussion at PTG landed on just doing the safe thing | 16:53 |
| sean-k-mooney | so for now the proposal say "we check the old xml and keep it the same if the xml exist" | 16:54 |
| sean-k-mooney | so im ok with moving forward with that and we can disucss more in the impelation if that what we want ot keep | 16:54 |
| sean-k-mooney | i dont think i was able to attend the ptg session so i dont know the geneal outcome of that | 16:55 |
| sean-k-mooney | but if that was the general feeling cool | 16:55 |
| Uggla | ok | 16:56 |
| Uggla | something else to discuss ? | 16:57 |
| kgube | Hi, just wanted to mention again that I rewrote my spec to address the input from the the PTG, and I would be happy to get any feedback on it: https://review.opendev.org/c/openstack/nova-specs/+/949504 | 16:57 |
| sean-k-mooney | kgube: its been a while but all the cinder bits are in placean right and you need the nova aprt to be added with teh call back to cinder before the end to end fucntionality works | 16:58 |
| sean-k-mooney | kgube: i belive there was a quition on the cidner side about how long to wiat for nova is that still an open? | 16:59 |
| sean-k-mooney | ill admin i have not looked at your spec slicne last cycle so perhaps that already captured. | 17:00 |
| kgube | sean-k-mooney: i have had to rewrite it significantly | 17:01 |
| sean-k-mooney | ok so we shoudl read it fresh rather then a repoposal | 17:01 |
| kgube | sean-k-mooney: there was some fundamental critique on using the external server event to trigger the extend operatoion | 17:02 |
| kgube | so it now it proposes a new API | 17:02 |
| kgube | for that purpose. So yes, it's basically a new spec. | 17:03 |
| sean-k-mooney | ok soyour mirroing the exsitng os-assisted-volume-snapshots aprpoch aby adding /assisted-volume-extend | 17:04 |
| kgube | yes | 17:04 |
| sean-k-mooney | i tought we dicsued that last cycle by maybe i imgangied that | 17:04 |
| sean-k-mooney | i dont know if other have had a change to look at it or have any questions? | 17:05 |
| sean-k-mooney | we are over time for the meeting | 17:05 |
| Uggla | yep, we need to move on. | 17:06 |
| sean-k-mooney | skiming it the new spec seam sto elimianate a rondtrip to cinder and a race condtion so over all it sound liek an improvmeent | 17:06 |
| Uggla | #topic Bug scrubbing | 17:06 |
| Uggla | #info up to 199 (+1) | 17:06 |
| Uggla | #link: https://etherpad.opendev.org/p/nova-bug-triage-roster | 17:06 |
| Uggla | #info 2 bugs assigned to each volunteer. | 17:06 |
| Uggla | so we can start to deal again with all these open bugs. | 17:07 |
| Uggla | Thanks for the volunteers btw. | 17:07 |
| Uggla | As mentioned we are over time, so time to close. | 17:08 |
| Uggla | Thanks for joining this meeting. Have a nice day/evening. | 17:08 |
| Uggla | #endmeeting | 17:08 |
| opendevmeet | Meeting ended Mon Dec 1 17:08:28 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 17:08 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2025/nova.2025-12-01-16.04.html | 17:08 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2025/nova.2025-12-01-16.04.txt | 17:08 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2025/nova.2025-12-01-16.04.log.html | 17:08 |
| elodilles | thanks o/ | 17:08 |
| tkajinam | thanks ! | 17:10 |
| gibi | Uggla: thanks. I will check the bugs assigned for triage o/ | 17:30 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Add capability to load ssm feature from existing xml https://review.opendev.org/c/openstack/nova/+/969131 | 17:45 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Use firmeare auto-selection by libvirt https://review.opendev.org/c/openstack/nova/+/969132 | 17:45 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: Use consistent program name for wsgi scripts and entry points https://review.opendev.org/c/openstack/nova/+/942605 | 17:58 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!