Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2025-11-10
19:46:59 haleyb i can propose a patch and rebase the os-ken change
19:47:40 sean-k-mooney in this case we proably shoudl not have merge the nova change until after plascement was released
19:47:54 sean-k-mooney so we might want to revert that or do the release to resolve it
19:49:41 sean-k-mooney https://review.opendev.org/c/openstack/nova/+/963036 is purlly a testing patch so reverting and reapplying it would not impafct anything but let see what gibi thinks in the morning
19:51:32 haleyb sean-k-mooney: that seems to be the original patch, i'm proposing something in the requirements repo as well
19:52:16 sean-k-mooney yep im just saying we may end up doing a revert of that isntead
19:52:27 sean-k-mooney so i htink the cross job is actully doign the right thing
19:52:31 sean-k-mooney and the bug is in nova
19:52:46 sean-k-mooney our functional test shoudl not fail with the released version of placement
19:52:57 sean-k-mooney haleyb: its failign for me locally as well
19:53:14 sean-k-mooney its only passign in ci beasue that is usign master
19:54:07 sean-k-mooney so the correct fix i think is to revert the nova patch and either update it ot degrade gracfully if the option is not aviable or wait to merge it until we have released a new version fo placement
19:54:40 sean-k-mooney ill push a patch quickly to catch the NoSuchGroupError error quickly
19:57:47 haleyb sean-k-mooney: ack, thanks, i pushed more of a test patch in requirements, but will -W it, https://review.opendev.org/c/openstack/requirements/+/966630
20:03:39 opendevreview sean mooney proposed openstack/nova master: allow funtional test to run with released placment https://review.opendev.org/c/openstack/nova/+/966631
20:03:56 sean-k-mooney haleyb: i think ^ would fix it properly
20:04:44 sean-k-mooney im testing that locally now
20:05:22 haleyb sean-k-mooney: thanks, if that works guess you can add a Closes-bug there
20:05:44 sean-k-mooney yep is there one fileed
20:06:00 haleyb https://bugs.launchpad.net/nova/+bug/2131032
20:06:36 sean-k-mooney ah right you linked that before
20:07:11 opendevreview sean mooney proposed openstack/nova master: allow funtional test to run with released placment https://review.opendev.org/c/openstack/nova/+/966631
20:07:26 haleyb i tried to be thorough :)
20:08:04 sean-k-mooney as this is a test only change it jsut runs the tox jobs so it will be pretty quick to report back
20:08:35 sean-k-mooney well quick for nova :)
20:10:51 haleyb yeah, the one i proposed to requirements is already 30% done, can propose one that just relies on your change later as a second test
20:15:46 sean-k-mooney functional: OK (639.38=setup[32.16]+cmd[605.77,1.46] seconds)
20:16:21 sean-k-mooney cool so that worked im going go go eat but melwitt dansmith if your about could you take a look at ^ to ublock the requiremetns gate
20:17:08 sean-k-mooney or gmaan ^
20:24:24 gmaan checking
20:31:27 gmaan sean-k-mooney: haleyb did we test it in requirement repo? i mean anything else not failing in nova cross job in requirement gate?
20:34:17 haleyb gmaan: i was going to update my test patch to depend on it, just waiting for results from my original change, but all the failures seem to have the same backtrace
20:34:39 gmaan haleyb: cool
20:38:47 haleyb ok, jobs queueing in https://review.opendev.org/c/openstack/requirements/+/966630 should know in 45-ish minutes
20:40:05 gmaan thanks
21:00:59 haleyb gmaan: seemed to work, jobs still running but cross-nova-functional has passed
21:01:01 gmaan sean-k-mooney: haleyb: job passing on requirement gate so +2
21:01:03 gmaan yeah
21:01:35 gmaan sean-k-mooney: i think we should add a new job nova-tox-functional-py312-master-placement' to test the placement from source and make nova-tox-functional-py312 to test the released version
21:44:25 opendevreview Merged openstack/nova master: allow funtional test to run with released placment https://review.opendev.org/c/openstack/nova/+/966631
23:30:00 opendevreview melanie witt proposed openstack/nova master: TPM: add documentation and reno for live migration https://review.opendev.org/c/openstack/nova/+/962889
23:32:00 opendevreview melanie witt proposed openstack/nova master: DNM vtpm tempest https://review.opendev.org/c/openstack/nova/+/957477
#openstack-nova - 2025-11-11
00:19:37 mikal sean-k-mooney: huh, that's news to me. I haven't played with trixie yet.
02:28:22 opendevreview melanie witt proposed openstack/nova-specs master: Re-propose vTPM live migration https://review.opendev.org/c/openstack/nova-specs/+/961564
11:19:51 gibi sean-k-mooney: haleyb: I'm going to revert https://review.opendev.org/c/openstack/nova/+/963036 wait for a placement release and then re-apply
11:20:40 sean-k-mooney gibi: we merged a workaround last night
11:20:52 sean-k-mooney basiclalyi just caught the excpetion
11:21:04 sean-k-mooney so we can just revert that after the release
11:21:51 sean-k-mooney gibi: https://review.opendev.org/c/openstack/nova/+/966631
11:22:25 sean-k-mooney gibi: if you want to revert out the whole thign we can also do that but its not blocking anything anymore
12:46:28 gibi sean-k-mooney: I'm fine with the exception handling merged last night. thanks for the pointer
12:49:46 sean-k-mooney gibi: as i said on the bug techinally it will test lightly different thigns depending on the palcement version installed. but other then time/performance the externally visable result shoudl be the same
12:49:54 sean-k-mooney so i think that perfectly fine
13:04:52 gibi it is fine, I agree
13:05:01 gibi thanks for fixing it
14:43:54 opendevreview Takashi Kajinami proposed openstack/nova-specs master: libvirt: Use built-in firmware auto-selection for UEFI firmware https://review.opendev.org/c/openstack/nova-specs/+/966584
17:00:58 vsaienko sean-k-money: maybe you remember about any performance issues of VDPA on connect6-dx, I can not get more that 5.6Gbs on 10Gb interface with hwardware VDPA
17:15:45 sean-k-mooney mellox connectx-6 do not support vdpa fully in hardware nor does bluefiled2 so neigher can accive full line rate for 64b small packets if i recall but its alo been like 3 years since i looked at this
17:16:28 sean-k-mooney with a connectx6-dx the traslation form the native ring format to virtio format is happening in software in the nics firmware
17:30:23 melwitt dansmith, bauzas: just a heads up that the vtpm spec and patch series is updated to remove the addition of a new instance system metadata key -- so it's all ready for review again
17:31:57 melwitt I spent some extra time to beef up functional testing in the series by adding checking of key manager service secrets ownership and I'm happy it seems to be working well
17:38:19 dansmith ack I have a bit stack of review notifications, I've just been consumed with something else recently
17:38:52 vsaienko sean-k-mooney: interesting two VMs with normal ports on same computes, using same connect6dx can get near 9Gbs
17:39:27 vsaienko maybe it does not work with vxlan encapsulation, will try vlans
17:41:06 sean-k-mooney the vxlan encap is also not entirly offloded to hardware or at leat has some overhead
17:41:16 sean-k-mooney i dont recall how much but its not free
17:42:14 vsaienko with asap2 it is fast
17:45:36 sean-k-mooney and technially unsupproted
17:46:02 sean-k-mooney ml2/ovs and ml2/ovn offcilly only supprot upstream openvsiwch
17:46:11 sean-k-mooney they may work with nvidias fork
17:46:18 sean-k-mooney but that not officlly supproted or tested.
17:47:11 sean-k-mooney so any bugs or feautre that are not part of upstream kernel ovs with the tcflower offlowad path are not openstack related bugs and shoudl be reported to nvidia
17:47:48 vsaienko ok, thanks sean-k-mononey, I'll continue digging if find anything I'll let you know
18:39:46 opendevreview Balazs Gibizer proposed openstack/nova master: Make RBD Tpool usage conditional https://review.opendev.org/c/openstack/nova/+/956089
18:39:46 opendevreview Balazs Gibizer proposed openstack/nova master: Make libvirt Tpool proxying conditional https://review.opendev.org/c/openstack/nova/+/956090
18:39:47 opendevreview Balazs Gibizer proposed openstack/nova master: Make guestfs' Tpool usage optional https://review.opendev.org/c/openstack/nova/+/965821
18:39:47 opendevreview Balazs Gibizer proposed openstack/nova master: Make image upload tpool usage conditional https://review.opendev.org/c/openstack/nova/+/966505
18:39:48 opendevreview Balazs Gibizer proposed openstack/nova master: Remove tpool_execute as it is unused https://review.opendev.org/c/openstack/nova/+/966552
18:39:48 opendevreview Balazs Gibizer proposed openstack/nova master: Fix ProviderTree copying with threading Lock https://review.opendev.org/c/openstack/nova/+/956091
18:39:49 opendevreview Balazs Gibizer proposed openstack/nova master: [test]Further categorization of disabled unit tests https://review.opendev.org/c/openstack/nova/+/956092
18:39:49 opendevreview Balazs Gibizer proposed openstack/nova master: Do not fork compute workers in native threading mode https://review.opendev.org/c/openstack/nova/+/965466
18:39:50 opendevreview Balazs Gibizer proposed openstack/nova master: Compute manager to use thread pools selectively https://review.opendev.org/c/openstack/nova/+/966016
18:39:51 opendevreview Balazs Gibizer proposed openstack/nova master: Libvirt event handling without eventlet https://review.opendev.org/c/openstack/nova/+/965949
18:39:51 opendevreview Balazs Gibizer proposed openstack/nova master: Run nova-compute in native threading mode https://review.opendev.org/c/openstack/nova/+/965467
20:10:31 opendevreview Merged openstack/nova master: api: Add response body schemas for volumes APIs https://review.opendev.org/c/openstack/nova/+/952348
21:24:32 jizaymes_ hey all. Having lots of issues getting nova-compute to start after upgrading from 2025.1 to 2025.2 -- get errors about 'Resource provider data migration failed fatally during startup'
21:26:20 jizaymes_ AMD SEV support detected \n kernel doesn't support AMD SEV-ES \n Requesting provider tree reshape in order to move memory encryption context inventory from the root compute node provider sc01 to a child provider.
21:26:20 jizaymes_ Resource provider data migration failed fatally during startup for node sc01.: nova.exception.ReshapeFailed: Resource provider inventory and allocation data migration failed: {"errors": [{"status": 403, "title": "Forbidden", "detail": "Access was denied to this resource.\n\n Policy does not allow placement:reshaper:reshape to be performed. ", "code": "placement.undefined_code", "request_id": "..."}]}
21:27:35 jizaymes_ if the kernel says its not enabled i'm not sure why it would try to move stuff under it, but in any case, adding that policy to placement yields a "this is already in the default policy" message so lost as to how where to look next.
#openstack-nova - 2025-11-12
01:48:07 opendevreview Steve Baker proposed openstack/nova master: Add VNC console support for the Ironic driver https://review.opendev.org/c/openstack/nova/+/942528
07:39:55 opendevreview Nicolai Ruckel proposed openstack/nova master: Preserve UEFI NVRAM variable store https://review.opendev.org/c/openstack/nova/+/959682
07:41:55 opendevreview Rajesh Tailor proposed openstack/nova-specs master: Show finish_time field in instance action show https://review.opendev.org/c/openstack/nova-specs/+/929780
08:44:04 nicolairuckel sean-k-mooney, sorry for the delay. I didn't realize that my responses were just drafts until today. But I think I'm finally getting the hang of Gerrit.
08:44:29 sean-k-mooney nicolairuckel: :)
08:45:06 sean-k-mooney ya the fact they are nto sent until you hit reply is slightly difefnt then gitlab and github which send them by defualt unless you explcitly tell it your doign a review
08:45:57 nicolairuckel It's funny because I used to complain about their default because it creates so many notifications. :D
08:46:16 sean-k-mooney nicolairuckel: sound like at least localy you not whave a mostly working version.
08:46:30 sean-k-mooney nicolairuckel: ya i dnot use notifcation on github basically ever
08:46:54 sean-k-mooney its an entirly broken feature given the signal to noise ratio
08:48:17 sean-k-mooney nicolairuckel: so what im thinking currently is we fix nvram preservation for rebooo. we do the saem for vtpm

Earlier   Later