| opendevreview | Merged openstack/nova master: Avoid reading 4GiB of reason text from RFB https://review.opendev.org/c/openstack/nova/+/991220 | 01:42 |
|---|---|---|
| opendevreview | Merged openstack/nova stable/2025.1: [CI][stable-only] nova-grenade-multinode fix https://review.opendev.org/c/openstack/nova/+/989579 | 03:18 |
| *** benj_5 is now known as benj_ | 04:45 | |
| opendevreview | Merged openstack/nova master: Fix swap disk creation skipped on NFS during cold migration https://review.opendev.org/c/openstack/nova/+/988547 | 07:54 |
| Uggla | Reminder: Today is spec review day | 09:04 |
| opendevreview | Kamil Sambor proposed openstack/nova master: Change nova-alt-configurations job https://review.opendev.org/c/openstack/nova/+/983179 | 09:48 |
| opendevreview | Merged openstack/nova-specs master: Intel TDX support in libvirt driver https://review.opendev.org/c/openstack/nova-specs/+/979608 | 10:07 |
| gibi | Uggla: both SNP and TDX specs are approved now. So we made good progress :) | 10:19 |
| opendevreview | Merged openstack/nova-specs master: libvirt: AMD SEV-SNP support https://review.opendev.org/c/openstack/nova-specs/+/983376 | 10:24 |
| gibi | bauzas: dansmith: I reviewed the bidirectional RPC liveness spec and I have one RPC compat questions that one of you probably can answer easily https://review.opendev.org/c/openstack/nova-specs/+/984384 | 10:48 |
| bauzas | gibi: I can try to look at this afternoon | 10:49 |
| gibi | thanks | 10:55 |
| * bauzas will have to invoke his very old chamanic knowledge about ServiceGroups and RPC compat :) | 10:57 | |
| opendevreview | dalekseev proposed openstack/nova master: Restrict machine type check to QEMU instances https://review.opendev.org/c/openstack/nova/+/991137 | 12:10 |
| opendevreview | Takashi Kajinami proposed openstack/osc-placement master: Update support python versions https://review.opendev.org/c/openstack/osc-placement/+/991603 | 12:18 |
| gibi | sambork, Uggla filed a new CLI UX bug for eventlet + py3.14 https://bugs.launchpad.net/nova/+bug/2155245 | 12:24 |
| Uggla | gibi ok | 12:27 |
| Uggla | and cool for the CC spec | 12:28 |
| sambork | @gibi, ack, I can take a look at that. At first glance, it looks like a similar issue to the one you described yesterday with oslo and pickling | 12:28 |
| gibi | yeah someting wrong happens when the python interpreter exits | 12:35 |
| gibi | it only uggly but causing no issue as far as I see | 12:35 |
| opendevreview | Kamil Sambor proposed openstack/nova master: Restore 'fork' start method for daemon mode on Python 3.14 https://review.opendev.org/c/openstack/nova/+/987815 | 12:35 |
| opendevreview | Kamil Sambor proposed openstack/nova master: [dnm] testing python 3.14 with restored fork https://review.opendev.org/c/openstack/nova/+/987821 | 12:35 |
| Uggla | gibi, if you have time to review 471815: Expose vlan trunking in metadata/configdrive | https://review.opendev.org/c/openstack/nova-specs/+/471815 | 12:50 |
| Uggla | sorry to bother all of you with this one, but I would like not to miss it another time. | 12:51 |
| opendevreview | Thibaut Démaret proposed openstack/nova master: libvirt: add disk rotation_rate support for local disks https://review.opendev.org/c/openstack/nova/+/979693 | 13:15 |
| Uggla | gmaan, there is this new bug entered by jlejeune that might interest you: https://bugs.launchpad.net/nova/+bug/2155153 | 13:25 |
| opendevreview | ChungWon Lee proposed openstack/nova-specs master: Spec for reduce failed builds value by time https://review.opendev.org/c/openstack/nova-specs/+/991636 | 13:31 |
| gibi | Uggla: I feel like https://bugs.launchpad.net/nova/+bug/2155153 is a duplicate of an older bug of mine. Looking for it... | 13:33 |
| Uggla | gibi, :) I was just thinnking about it | 13:33 |
| gibi | found https://bugs.launchpad.net/nova/+bug/2147554 and did the paperwork | 13:35 |
| opendevreview | ChungWon Lee proposed openstack/nova-specs master: Spec for reduce failed builds value by time https://review.opendev.org/c/openstack/nova-specs/+/991636 | 13:37 |
| opendevreview | ribaudr proposed openstack/nova master: Add reproducer for bug #2115798 https://review.opendev.org/c/openstack/nova/+/991642 | 13:46 |
| opendevreview | ribaudr proposed openstack/nova master: Fix race condition in security group add/remove https://review.opendev.org/c/openstack/nova/+/991643 | 13:46 |
| opendevreview | ribaudr proposed openstack/nova master: Fix misleading description of assisted volume snapshots https://review.opendev.org/c/openstack/nova/+/991649 | 13:57 |
| sean-k-mooney | bauzas: is this on your review list https://review.opendev.org/c/openstack/nova-specs/+/990259 | 14:02 |
| bauzas | no | 14:02 |
| sean-k-mooney | i woudl assume if we are goint to do the virtio fs cold migrtion of host attach stuff | 14:02 |
| sean-k-mooney | we would want that to be ocmplete first | 14:02 |
| bauzas | I won't have time for reviewing the implementation | 14:02 |
| sean-k-mooney | ack | 14:03 |
| sean-k-mooney | it feel strang not to adress the hugepage/file backed memory pain point with manilla share attachment so that the feature "just works" with all vms | 14:04 |
| sean-k-mooney | before inveting in the lifecyle operations but ok the memfd work is much smaller then the move opertions | 14:04 |
| sean-k-mooney | or hot attahce/detach | 14:04 |
| sean-k-mooney | tis technically not a hard requriement | 14:05 |
| sean-k-mooney | but it would also make testing this in ci simpler | 14:05 |
| sean-k-mooney | anyway im happe to review the memfd sutff if someone else is | 14:05 |
| opendevreview | Kamil Sambor proposed openstack/nova master: Fix resize error notifications using wrong event type https://review.opendev.org/c/openstack/nova/+/991657 | 14:05 |
| sean-k-mooney | i probaly wont have tiem for the other virtio fs features however | 14:05 |
| cw0306-lee[m] | Hi, Uggla! I tried to write spec for reducing failed_builds value. https://review.opendev.org/c/openstack/nova-specs/+/991636 | 14:06 |
| Uggla | cw0306-lee[m] cool thanks putting it on the list | 14:54 |
| sean-k-mooney | cw0306-lee[m]: i provcided some feedback on wordign ectra | 15:20 |
| gouthamr | sean-k-mooney: Uggla: my 0.02 agreement for your sentiment.. i'm okay continuing to work on hot attach/live migration with virtiofs and let them slip into I. Would really like the memfd and cold migration enablement to land in hibiscus | 15:20 |
| gouthamr | and can help push this as much as possible :) | 15:21 |
| sean-k-mooney | gouthamr: dont think tis stricly a bloceker for any of that work | 15:21 |
| sean-k-mooney | but if we are going ot invest int impvoing the ux i think the memfd work si certally one of the more valuabel aspects | 15:21 |
| sean-k-mooney | not that i really have an issue with any of the virtiofs enchment in genreal | 15:22 |
| sean-k-mooney | we can test with file backed memory or hugepages in ci today if we need too but for the maniall share move operaiton we will need to do that and have tempest testing | 15:23 |
| sean-k-mooney | functional testing will prove out most of the flow but i woudl really like to see tempest actuly do the moves as well | 15:23 |
| gouthamr | yes.. am working on that part | 15:23 |
| gouthamr | not memfd, but working on the cold migration tempest tests right now, i see memfd as making things a whole lot easier | 15:24 |
| gibi | bauzas: thanks for confirming my RPC compat issue in the bidirectional RPC hearthbeat spec | 15:24 |
| sean-k-mooney | Uggla: how are we currently testing manilla shares in our jobs | 15:25 |
| sean-k-mooney | i dont see manilla enabeld in any of them | 15:26 |
| sean-k-mooney | that defintly somethign we will have to corect if it is a gap today before building on more features | 15:27 |
| Uggla | sean-k-mooney, there are tests against devstack in the sdk. | 15:44 |
| sean-k-mooney | oh ok so its the sdk job that givign use oru current test coverage | 15:51 |
| sean-k-mooney | we probaly shoudl fix that as that pretty light and add new temptet test for this into one of our jobs | 15:51 |
| sean-k-mooney | but ok that better then nothing | 15:52 |
| gouthamr | Uggla: sean-k-mooney: We'll have a test job running against manila repos soon.. https://review.opendev.org/c/openstack/manila-tempest-plugin/+/984884 | 15:56 |
| gouthamr | i can add that non-voting to nova so we can see things work on the cold migration code changes | 15:56 |
| sean-k-mooney | gouthamr: we can figure out where to run it but we can probaly add it or a subset of its test into one of our exsting jobs | 15:57 |
| gouthamr | ack, especially the multinode one, because i don't want to reinvent the setup into a new job elsewhere | 15:57 |
| sean-k-mooney | well i was thinkign nova-alt-config | 15:57 |
| * gouthamr can look | 15:57 | |
| sean-k-mooney | but we have a couple of multi node ones we coudl use | 15:57 |
| sean-k-mooney | do we need ceph? | 15:58 |
| sean-k-mooney | or will it work with lvm | 15:58 |
| gouthamr | yes, either works | 15:58 |
| sean-k-mooney | then ya we can look at the job run time and eitehr add a new one or extend one of our shorter jobs | 15:58 |
| gouthamr | we'll cover both in manila land.. but we can pick one for nova | 15:58 |
| sean-k-mooney | nova-live-migration and the ceph version are pretty small | 15:58 |
| gouthamr | ack | 15:59 |
| sean-k-mooney | if the test are short to run and we dont have issues with memory when deploying another serivce we coudl have ti in re then one | 15:59 |
| sean-k-mooney | we norally try to keep our josb below 2 hours if we can | 16:00 |
| sean-k-mooney | the live migration jobs are about an hour at the moment | 16:00 |
| sean-k-mooney | so those woudl be fine. the alt config job is a littel longer but its still below 90 mins typically | 16:01 |
| sean-k-mooney | speaking of job lenght we may want os see fi some of the things tested in nova-next shoudl move to some of the other jobs | 16:03 |
| sean-k-mooney | thats our longest job by far and some of those "next" itmes shoud proably be the default by now | 16:04 |
| gmaan | Uggla: ack, that was same bug which gibi reported. it is when service is stopped before RPC is completely initialized. I planned to propose the fix but got distracted, I will propose fix today. | 16:13 |
| gmaan | let me do that before i forget again | 16:14 |
| Uggla | gmaan 👍 | 16:16 |
| gibi | :) | 16:36 |
| *** EugenMayer4401808 is now known as EugenMayer440180 | 16:44 | |
| opendevreview | Ghanshyam Maan proposed openstack/nova master: Fix AttributeError in Service.stop() during early startup shutdown https://review.opendev.org/c/openstack/nova/+/991702 | 17:40 |
| opendevreview | Rodolfo Alonso proposed openstack/os-vif master: Remove workaround for oslo.i18n ``translate`` method https://review.opendev.org/c/openstack/os-vif/+/991720 | 18:51 |
| opendevreview | Rodolfo Alonso proposed openstack/os-vif master: Make ``PyPyroute2._lookup_interface`` private https://review.opendev.org/c/openstack/os-vif/+/991292 | 18:52 |
| *** elodilles is now known as elodilles_OoO | 20:17 | |
| gmaan | gibi: Uggla graceful shutdown bug fix https://review.opendev.org/c/openstack/nova/+/991702 | 20:56 |
| opendevreview | Ghanshyam Maan proposed openstack/nova master: Add functional tests for compute service graceful shutdown https://review.opendev.org/c/openstack/nova/+/991743 | 21:25 |
| gmaan | sean-k-mooney: ^^ as you asked about the graceful shutdown functional tests. It is easy to create the scenario with event set/wait and timeouts | 21:26 |
| opendevreview | Merged openstack/os-vif master: Remove workaround for oslo.i18n ``translate`` method https://review.opendev.org/c/openstack/os-vif/+/991720 | 23:21 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!