| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-11-10 | |||
| 19:43:44 | sean-k-mooney | i can check but i woudl suspect not | |
| 19:44:11 | sean-k-mooney | am not it does nto look liek it does | |
| 19:44:12 | haleyb | https://opendev.org/openstack/requirements/src/branch/master/.zuul.d/cross-jobs.yaml#L170 | |
| 19:44:13 | sean-k-mooney | https://zuul.opendev.org/t/openstack/job/cross-nova-functional | |
| 19:44:43 | sean-k-mooney | ya so either we could rbing it in line with nova but then we woudl loose the test coverage for this | |
| 19:44:59 | sean-k-mooney | or we can leave it as is and do a release of placement | |
| 19:45:04 | sean-k-mooney | Uggla: ^ | |
| 19:45:09 | sean-k-mooney | gibi: ^ | |
| 19:45:42 | sean-k-mooney | haleyb: its late for them so they proably wont see that until tomorow, you coudl propsoe addign placement to the requried project in the requiremet repo as a test | |
| 19:46:06 | haleyb | i'm assuming there isn't a chicken/egg here, i.e. can that nova-cross job succeed on a patch updating placement requirement | |
| 19:46:14 | sean-k-mooney | that proably ok as the short term fix and it might become the long term one | |
| 19:46:58 | sean-k-mooney | ya there kind of is | |
| 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": "..."}]} | |