| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-11-10 | |||
| 19:33:06 | haleyb | not sure if anyone has cycles to look at a new bug i just filed, but cross-nova-functional job is blocking the requirements repo CI - https://bugs.launchpad.net/nova/+bug/2131032 | |
| 19:34:22 | sean-k-mooney | haleyb: that looks unrelated to bumping os-ken | |
| 19:34:48 | haleyb | sean-k-mooney: yes, that is just a trigger that is running that job, other changes have same failure | |
| 19:35:17 | sean-k-mooney | https://zuul.opendev.org/t/openstack/builds?job_name=cross-nova-functional&project=openstack/requirements | |
| 19:35:27 | sean-k-mooney | so its failing on other change to the requiremetn repo too | |
| 19:35:35 | haleyb | yup | |
| 19:37:09 | sean-k-mooney | im running the funcitonal test locally with master quickly to ssse if i see the same | |
| 19:37:36 | haleyb | anything since october 31 is broken, which lines-up with the change i cited, https://review.opendev.org/c/openstack/nova/+/963036 | |
| 19:38:37 | sean-k-mooney | so you think tis related to https://review.opendev.org/c/openstack/nova/+/963036/5/nova/tests/functional/libvirt/test_pci_in_placement.py | |
| 19:38:39 | haleyb | sean-k-mooney: anyway, thanks for looking and running tests | |
| 19:38:44 | sean-k-mooney | the other change is identical | |
| 19:39:00 | sean-k-mooney | oh | |
| 19:39:01 | haleyb | sean-k-mooney: well that is the line that is failing | |
| 19:39:07 | sean-k-mooney | ok i think i know what happening | |
| 19:39:48 | sean-k-mooney | this uses a new config option added as a bug fix | |
| 19:39:56 | sean-k-mooney | but i dont knwo if that is released | |
| 19:40:22 | sean-k-mooney | or rather if that release does exsit is it allowed by UC | |
| 19:40:51 | sean-k-mooney | its enabling https://github.com/openstack/placement/commit/5b73b980d0f022de2855480f57c9d5f04a7a4712 | |
| 19:41:29 | sean-k-mooney | ya https://review.opendev.org/c/openstack/placement/+/962776 | |
| 19:41:33 | sean-k-mooney | is only on master | |
| 19:42:34 | haleyb | ah, that seems to be it. does that cross-nova job need to require the placement project? just thinking of fixes | |
| 19:42:57 | sean-k-mooney | so nova does https://zuul.opendev.org/t/openstack/job/nova-tox-functional-py312 | |
| 19:43:06 | sean-k-mooney | because we use the placement fixture form master | |
| 19:43:19 | sean-k-mooney | but that means we can nit issues like this | |
| 19:43:35 | sean-k-mooney | the correct fix woudl likely be to release placment with the bug fix | |
| 19:43:35 | haleyb | cross-nova-functional doesn't do that | |
| 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 | |