| *** mhen_ is now known as mhen | 02:45 | |
| vsaienko | Hello nova team, I'm checking VDPA on Mellanox connect6-dx card, and found that it works only when mac address on VDPA device matches VM mac address. Currently nova does not update mac address of neutron port (like its done with sriov). Am I missing something or how its intended to work? | 07:15 |
|---|---|---|
| gibi | aand we have a full green nova-next run with native threaded nova-compute. https://review.opendev.org/c/openstack/nova/+/965467/11 (There is a lot of cleanup work left but the direction seems good) | 08:53 |
| gibi | sean-k-mooney: dansmith: ^^ | 08:54 |
| Uggla | gibi \o/ | 09:03 |
| razoff | Hi there ! | 09:53 |
| razoff | At my company I developed a backup solution (through a proxy instance) for openstack instances and I have one question for you. | 09:53 |
| razoff | I've been struggling to run efficient incremental backups. | 09:53 |
| razoff | Is it possible in any way to access dirty blocks of volume mounted on an instance through openstack API ? | 09:53 |
| razoff | I am asking this on the Nova channel because I noticed that you are using libvirt for virtualization and I know that it is possible to get dirty blocks from that lib. | 09:53 |
| razoff | Thank you in advance ! | 09:53 |
| gibi | razoff: OpenStack does not provide such APIS | 11:59 |
| razoff | @gibi Thanks for the answer is there any plans to add such capabilities in the future ? | 12:04 |
| gibi | I haven't heard about such plans. Also this sounds like a pretty low level API so it is not really fits into nova's high level APIs i.e. starting and stopping VMs. | 12:09 |
| gibi | razoff: if you can integrate with libvirt to get it and such interaction does not affect the state of the domain, then my suggestion is to try to integrate with libvirt | 12:10 |
| gibi | if you need information or action from nova to help such integration then that can be raise | 12:10 |
| sean-k-mooney | razoff: change block tracking is more or less unplanned to be supported in the future | 12:28 |
| sean-k-mooney | razoff: we have discussed the idea of have a proxy server to explore it and apis but its unlikely to proceed | 12:29 |
| razoff | sean-k-mooney: Alright thanls a lot for the answers | 13:01 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Make image upload tpool usage conditional https://review.opendev.org/c/openstack/nova/+/966505 | 13:28 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Fix ProviderTree copying with threading Lock https://review.opendev.org/c/openstack/nova/+/956091 | 13:28 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: [test]Further categorization of disabled unit tests https://review.opendev.org/c/openstack/nova/+/956092 | 13:28 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Do not fork compute workers in native threading mode https://review.opendev.org/c/openstack/nova/+/965466 | 13:28 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Compute manager to use native thread pools https://review.opendev.org/c/openstack/nova/+/966016 | 13:28 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Libvirt event handling without eventlet https://review.opendev.org/c/openstack/nova/+/965949 | 13:28 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-compute in native threading mode https://review.opendev.org/c/openstack/nova/+/965467 | 13:28 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Remove tpool_execute as it is unused https://review.opendev.org/c/openstack/nova/+/966552 | 13:28 |
| vsaienko | sean-k-mooney: hello, I've seen you was involved in VDPA implementation maybe you can help? I'm checking VDPA on Mellanox connect6-dx card, and found that it works only when mac address on VDPA device matches VM mac address. Currently nova does not update mac address of neutron port (like its done with sriov). Am I missing something or how its intended to work? | 13:36 |
| sean-k-mooney | vsaienko: correc tthat is becasue updating the mac is ment ot be done by qemu | 13:38 |
| sean-k-mooney | this used to be done automaticaly and memonox broke the kernel driver compatiablity at some ponit when they added the ablity to set the mac viat the vdpa tool | 13:39 |
| sean-k-mooney | https://lore.kernel.org/lkml/20240617094314-mutt-send-email-mst@kernel.org/T/ | 13:40 |
| sean-k-mooney | is related but nova intentionally does not use that as its libvirts/qemus role to set that. we set the mac in the xml | 13:40 |
| sean-k-mooney | https://issues.redhat.com/browse/RHEL-7298 may be the rhle tracker for this https://issues.redhat.com/browse/RHEL-7298 | 13:42 |
| vsaienko | sean-k-mooney: hm.. it does not look like qemu/libvirt can set mac address on vdpa device by itsown. At least when I create VDPA device without specifying MAC and later when starting VM (with mac from openstack) traffic simply no passing. And vdpa device config shows not mac on vdpa device. | 13:45 |
| opendevreview | Max proposed openstack/nova master: performance: reduce calls to libvirt / add caching https://review.opendev.org/c/openstack/nova/+/922855 | 13:46 |
| vsaienko | there are some open patches to qemu but they not yet merged https://patchwork.ozlabs.org/project/qemu-devel/list/?series=453198 | 13:46 |
| sean-k-mooney | vsaienko: right so as i said this was a kernel driver regression where they broke userspace | 13:49 |
| sean-k-mooney | https://issues.redhat.com/browse/OSP-31614 | 13:49 |
| sean-k-mooney | orginally when we intergated vdpa it 3as not possibel to set the mac or queue via anwty8in other then qemu | 13:50 |
| sean-k-mooney | and it "just worked" at the time | 13:51 |
| vsaienko | many thanks sean-k-mooney | 14:16 |
| Uggla | Hi elodilles, could you have a look at 966518: Add Nova deadlines to the official schedule | https://review.opendev.org/c/openstack/releases/+/966518 | 15:05 |
| 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 | 15:13 |
| yuval_ | yo nova wizards | 15:28 |
| yuval_ | whats up | 15:28 |
| yuval_ | I have a question which is simple but may have a complicate answer | 15:29 |
| yuval_ | I want to edit my vm xml to have some custom properties, then create 5 more vm's with the same xml - is that possible? | 15:29 |
| yuval_ | bauzas: ? | 15:30 |
| yuval_ | eharney: what you think? | 15:30 |
| dansmith | yuval_: no, specifically disallowed and nova will overwrite your properties the next time it has a chance | 15:33 |
| yuval_ | dansmith: what my options? | 15:34 |
| yuval_ | I need fork nova and patch the driver.py? | 15:34 |
| Uggla | Info: Nova meeting in ~30mn | 15:35 |
| elodilles | Uggla: hi o/ yepp, will review it ASAP :) | 15:41 |
| Uggla | elodilles thx | 15:43 |
| elodilles | Uggla: there are some yaml issue in the patch, i've commented it where | 15:50 |
| Uggla | elodilles ok | 15:51 |
| Uggla | elodilles, should be ok now | 15:59 |
| elodilles | will review it after the meeting :] | 16:02 |
| Uggla | elodilles, no pb | 16:02 |
| Uggla | #startmeeting nova | 16:02 |
| opendevmeet | Meeting started Mon Nov 10 16:02:57 2025 UTC and is due to finish in 60 minutes. The chair is Uggla. Information about MeetBot at http://wiki.debian.org/MeetBot. | 16:02 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 16:02 |
| opendevmeet | The meeting name has been set to 'nova' | 16:02 |
| Uggla | Hello everyone | 16:03 |
| gibi | o/ | 16:03 |
| gmaan | o/ | 16:04 |
| elodilles | o/ | 16:05 |
| Luzi | o/ | 16:05 |
| Uggla | #topic Bugs (stuck/critical) | 16:05 |
| Uggla | #info No Critical bug | 16:06 |
| Uggla | #topic Gate status | 16:06 |
| Uggla | #link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs | 16:06 |
| Uggla | #link https://etherpad.opendev.org/p/nova-ci-failures-minimal | 16:06 |
| 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:06 |
| Uggla | #info Please look at the gate failures and file a bug report with the gate-failure tag. | 16:06 |
| Uggla | #info Please try to provide a meaningful comment when you recheck | 16:06 |
| Uggla | I have not seen something special with the gate, someone wants to share something ? | 16:07 |
| Uggla | moving on | 16:08 |
| Uggla | #topic tempest-with-latest-microversion job status | 16:08 |
| Uggla | #link https://zuul.opendev.org/t/openstack/builds?job_name=tempest-with-latest-microversion&skip=0 | 16:08 |
| Uggla | gmaan, do you want to share something about it ? | 16:08 |
| gmaan | i have not looked into it since long time. but it is in my list sometime next week | 16:08 |
| Uggla | ok | 16:09 |
| Uggla | #topic Release Planning | 16:09 |
| Uggla | #link https://releases.openstack.org/gazpacho/schedule.html | 16:09 |
| Uggla | #info Soft spec freeze November 20th ! | 16:10 |
| Uggla | I warned ^ on the openstack-discuss as this is coming super quick. | 16:10 |
| Uggla | Thanks bauzas for suggesting it | 16:10 |
| Uggla | #info PTG etherpad for 2026.1 is available: https://etherpad.opendev.org/p/nova-2026.1-ptg | 16:10 |
| sean-k-mooney | i have updated the weigher spec as best i coudl based on what we dicussed at the ptg | 16:12 |
| sean-k-mooney | https://review.opendev.org/c/openstack/nova-specs/+/951222/6/specs/2026.1/approved/resource-provider-weigher.rst | 16:12 |
| sean-k-mooney | since most of the question revoed arboudn how this coudl tie into cpu teiring most of the update focused on that and the ohter usercase dicussed | 16:12 |
| Uggla | sean-k-mooney cool thx | 16:12 |
| sean-k-mooney | as a resutl its now longer but much of the impelemation detail have been remvoed so the over all contet shoudl hopefully be more userful | 16:13 |
| sean-k-mooney | thats just an fyi we can move on | 16:14 |
| Uggla | #topic Review priorities | 16:15 |
| Uggla | #link https://etherpad.opendev.org/p/nova-2026.1-status | 16:15 |
| Uggla | ^ fyi i have created the above doc, I'm currently updating it. Not yet finished. | 16:15 |
| 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 28 remaining atm. | 16:16 |
| Uggla | gmaan, are you ok if we restart the review of this ? | 16:17 |
| gmaan | yes, that is in list. stephenfin pinged last week on this and I planned time on wed to do re-start it | 16:17 |
| Uggla | I will try to review as well, so maybe a first filter. | 16:18 |
| Uggla | moving on | 16:18 |
| Uggla | #topic Stable Branches | 16:19 |
| gmaan | first 9 are +2 from me, so they are ready for 2nd core review I think sean-k-mooney is planning that | 16:19 |
| Uggla | gmaan 👍 | 16:19 |
| Uggla | elodilles are you around ? | 16:19 |
| elodilles | yepp | 16:19 |
| elodilles | #info stable/2024.1 branch is deleted and unmaintained/2024.1 was cut from 2024.1-eom tag | 16:19 |
| elodilles | #info 2024.2 Dalmation (stable/2024.2) will move to End of Life in ~half a year (given it's not a SLURP release) | 16:20 |
| elodilles | just a heads up ^^^ | 16:20 |
| elodilles | #info stable branches (stable/2025.* and stable/2024.2) seem to be in OK state | 16:20 |
| elodilles | #info stable branch status / gate failures tracking etherpad: https://etherpad.opendev.org/p/nova-stable-branch-ci | 16:20 |
| elodilles | and that's all from me for now about stable branches | 16:20 |
| elodilles | Uggla: back to you | 16:21 |
| Uggla | thx elodilles | 16:21 |
| Uggla | btw have you seen 3 patches from ovh to the unmaintained branch ? | 16:21 |
| Uggla | elodilles ^ ? | 16:22 |
| elodilles | Uggla: i haven't got there yet, but will check them ASAP o:) | 16:22 |
| gmaan | just a note, py3.9 jobs (centos steam and rocky linux if we are running in nova) on 2024.1 is still broken more details are in below bug | 16:22 |
| gmaan | #link https://bugs.launchpad.net/devstack/+bug/2129913 | 16:22 |
| Uggla | elodilles no worries, just a note, I don't think there is urgency. | 16:23 |
| Uggla | #topic vmwareapi 3rd-party CI efforts Highlights | 16:23 |
| elodilles | gmaan: oh :-o so there ARE some stable gate issue on stable/2024.2 then! :-o | 16:23 |
| sean-k-mooney | gmaan: yes its on my todo list | 16:24 |
| gmaan | elodilles: yeah, we need to wait for this fix to merge and then gate should should ok there | 16:24 |
| gmaan | #link https://review.opendev.org/c/openstack/devstack/+/965601 | 16:24 |
| gmaan | i sw you just approved that | 16:24 |
| gmaan | sean-k-mooney: ++ | 16:24 |
| elodilles | gmaan: oh, good :) so that is also fixing stable/2024.2 o:) cool. | 16:25 |
| Uggla | are we good ? | 16:26 |
| Uggla | #topic vmwareapi 3rd-party CI efforts Highlights | 16:26 |
| Uggla | fwiesel something to share ? | 16:26 |
| fwiesel | no, nothing from my side | 16:27 |
| Uggla | thx Fabian | 16:27 |
| Uggla | #topic Gibi's news about eventlet removal | 16:27 |
| gibi | o/ | 16:27 |
| * Uggla giving the mic to gibi | 16:27 | |
| gibi | I saw the first nova-next full green run with nova-compute using native threads https://review.opendev.org/c/openstack/nova/+/965467/12#message-1d09a711d4c712a949e22c40ec4c41f274eb472a I'm not sure how stable it is and there is definitely a lot of cleanup work in that patch series. | 16:28 |
| gibi | Also we landed the patch that changed the default for nova-api, nova-metadata, and nova-scheduler. Those services are now running with native threading by default | 16:29 |
| gibi | all over the gate :) | 16:29 |
| gibi | so if you see anything strange with those services then let me know | 16:30 |
| sean-k-mooney | so far that has been uneventful | 16:30 |
| sean-k-mooney | hopefully it stays taht away | 16:30 |
| gibi | yepp | 16:30 |
| gibi | and that is it from me. | 16:30 |
| Uggla | \o/ | 16:30 |
| * gibi passes the mic back to Uggla | 16:30 | |
| Uggla | #topic Open discussion | 16:31 |
| Uggla | I think we have something from Luzi | 16:31 |
| Uggla | (Luzi) I added a blueprint for the Image Encryption: https://blueprints.launchpad.net/nova/+spec/luks-image-encryption | 16:32 |
| Luzi | yeah, for the image encryption: I updated the glance and cinder specs and proposed a blueprint for nova | 16:32 |
| Luzi | #link https://blueprints.launchpad.net/nova/+spec/luks-image-encryption | 16:32 |
| Luzi | I just link the glance spec in the blueprint | 16:32 |
| Uggla | dansmith fyi ^ | 16:33 |
| dansmith | ack | 16:34 |
| Luzi | that is all I think, or do you need more for nova? | 16:35 |
| Uggla | Luzi, I think that's ok of the moment, I'll ping you if something is needed. | 16:36 |
| Luzi | ok | 16:36 |
| Uggla | There is another topic, from stevebaker, but he can't attend the meeting | 16:37 |
| Uggla | Just CC here: Following up from PTG Ironic Graphical Console session the nova change is ready to review (https://review.opendev.org/c/openstack/nova/+/942528) but needs some way of tracking the work, such as making the existing approved https://blueprints.launchpad.net/nova/+spec/ironic-vnc-console specless. | 16:37 |
| *** ykarel is now known as ykarel|pto | 16:37 | |
| Uggla | the BP was already approved so I guess it is ok | 16:38 |
| Uggla | sean-k-mooney if you can have a look to the spec ^ that would be great. | 16:40 |
| sean-k-mooney | yes | 16:41 |
| Uggla | does someone want to discuess anything else ? | 16:43 |
| gibi | - | 16:45 |
| Uggla | ok last quick topic | 16:45 |
| Uggla | #topic Bug scrubbing | 16:45 |
| Uggla | #info up to 189 | 16:45 |
| Uggla | #link: https://etherpad.opendev.org/p/nova-bug-triage-roster | 16:45 |
| Uggla | if you are volunteer for triaging the open bugs, please add your name to ^ | 16:46 |
| Uggla | I have shared this as well on openstack discuss: https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/4AAJGCXNQ2BYY5K2SPSRSBNZP37AYUQC/ | 16:46 |
| Uggla | that's all folks. | 16:47 |
| Uggla | Thanks for joining this meeting. Have a nice day/evening. | 16:48 |
| * gibi added his name to the roster | 16:48 | |
| sean-k-mooney | Uggla: are we ok with comemting that https://blueprints.launchpad.net/nova/+spec/ironic-vnc-console is reappoved for this cycle? | 16:48 |
| gibi | Uggla: thanks for running it | 16:48 |
| sean-k-mooney | shoudl i jsut update the serise goal ot 2026.1 | 16:48 |
| Uggla | sean-k-mooney yes it was already approved so for me it is ok | 16:49 |
| Uggla | unless you tell me it is not. :) | 16:49 |
| sean-k-mooney | ok i just updated it to 2026.1 to relect that | 16:49 |
| Uggla | gibi thanks for adding your name to the roster ! I hope we will be more than 2. | 16:50 |
| sean-k-mooney | i was just confirming since it was approved for antelope in launch pad | 16:50 |
| Uggla | yep | 16:50 |
| Uggla | sean-k-mooney, I will to the update in launchpad asap. | 16:50 |
| sean-k-mooney | i just did it for this one | 16:50 |
| Uggla | sean-k-mooney 👍 | 16:50 |
| Uggla | thc | 16:51 |
| sean-k-mooney | but feel free to correct anyting if needed | 16:51 |
| Uggla | *thx | 16:51 |
| Uggla | are we done for today ? | 16:51 |
| sean-k-mooney | yep, nothing form me anyway | 16:51 |
| Uggla | Then, thanks everyone | 16:52 |
| Uggla | #endmeeting | 16:52 |
| opendevmeet | Meeting ended Mon Nov 10 16:52:08 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:52 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2025/nova.2025-11-10-16.02.html | 16:52 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2025/nova.2025-11-10-16.02.txt | 16:52 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2025/nova.2025-11-10-16.02.log.html | 16:52 |
| elodilles | thanks o/ | 16: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:02 |
| opendevreview | sean mooney proposed openstack/nova master: move nova-alt-config to debian-13 https://review.opendev.org/c/openstack/nova/+/966479 | 18:09 |
| opendevreview | sean mooney proposed openstack/nova master: move nova-alt-config to debian-13 https://review.opendev.org/c/openstack/nova/+/966479 | 18:10 |
| sean-k-mooney | mikal: ... libvirt.libvirtError: unsupported configuration: spice graphics are not supported with this QEMU | 18:13 |
| sean-k-mooney | mikal: i need to check if i just need to install an addtional package | 18:13 |
| sean-k-mooney | but just spotted that when trying tup update the nova job to debian 13 | 18:13 |
| sean-k-mooney | its still supproted in sid | 18:13 |
| sean-k-mooney | so my guess is its just not installed by default | 18:14 |
| sean-k-mooney | mikal: qemu-system-modules-spice | 18:17 |
| sean-k-mooney | https://packages.debian.org/trixie/i386/qemu-system-modules-spice | 18:17 |
| sean-k-mooney | ya so its an optional addtional package | 18:17 |
| sean-k-mooney | ill fix that in devstack when runing on debian its a recommended depenency in treixe but we intentiolly dont install those | 18:19 |
| sean-k-mooney | https://packages.debian.org/trixie/qemu-system-x86 | 18:19 |
| 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:33 |
| sean-k-mooney | haleyb: that looks unrelated to bumping os-ken | 19:34 |
| haleyb | sean-k-mooney: yes, that is just a trigger that is running that job, other changes have same failure | 19:34 |
| sean-k-mooney | https://zuul.opendev.org/t/openstack/builds?job_name=cross-nova-functional&project=openstack/requirements | 19:35 |
| sean-k-mooney | so its failing on other change to the requiremetn repo too | 19:35 |
| haleyb | yup | 19:35 |
| sean-k-mooney | im running the funcitonal test locally with master quickly to ssse if i see the same | 19:37 |
| haleyb | anything since october 31 is broken, which lines-up with the change i cited, https://review.opendev.org/c/openstack/nova/+/963036 | 19: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 |
| haleyb | sean-k-mooney: anyway, thanks for looking and running tests | 19:38 |
| sean-k-mooney | the other change is identical | 19:38 |
| sean-k-mooney | oh | 19:39 |
| haleyb | sean-k-mooney: well that is the line that is failing | 19:39 |
| sean-k-mooney | ok i think i know what happening | 19:39 |
| sean-k-mooney | this uses a new config option added as a bug fix | 19:39 |
| sean-k-mooney | but i dont knwo if that is released | 19:39 |
| sean-k-mooney | or rather if that release does exsit is it allowed by UC | 19:40 |
| sean-k-mooney | its enabling https://github.com/openstack/placement/commit/5b73b980d0f022de2855480f57c9d5f04a7a4712 | 19:40 |
| sean-k-mooney | ya https://review.opendev.org/c/openstack/placement/+/962776 | 19:41 |
| sean-k-mooney | is only on master | 19:41 |
| haleyb | ah, that seems to be it. does that cross-nova job need to require the placement project? just thinking of fixes | 19:42 |
| sean-k-mooney | so nova does https://zuul.opendev.org/t/openstack/job/nova-tox-functional-py312 | 19:42 |
| sean-k-mooney | because we use the placement fixture form master | 19:43 |
| sean-k-mooney | but that means we can nit issues like this | 19:43 |
| sean-k-mooney | the correct fix woudl likely be to release placment with the bug fix | 19:43 |
| haleyb | cross-nova-functional doesn't do that | 19:43 |
| sean-k-mooney | i can check but i woudl suspect not | 19:43 |
| sean-k-mooney | am not it does nto look liek it does | 19:44 |
| haleyb | https://opendev.org/openstack/requirements/src/branch/master/.zuul.d/cross-jobs.yaml#L170 | 19:44 |
| sean-k-mooney | https://zuul.opendev.org/t/openstack/job/cross-nova-functional | 19:44 |
| 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 |
| sean-k-mooney | or we can leave it as is and do a release of placement | 19:44 |
| sean-k-mooney | Uggla: ^ | 19:45 |
| sean-k-mooney | gibi: ^ | 19:45 |
| 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:45 |
| 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 |
| sean-k-mooney | that proably ok as the short term fix and it might become the long term one | 19:46 |
| sean-k-mooney | ya there kind of is | 19:46 |
| haleyb | i can propose a patch and rebase the os-ken change | 19:46 |
| sean-k-mooney | in this case we proably shoudl not have merge the nova change until after plascement was released | 19:47 |
| sean-k-mooney | so we might want to revert that or do the release to resolve it | 19:47 |
| 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:49 |
| haleyb | sean-k-mooney: that seems to be the original patch, i'm proposing something in the requirements repo as well | 19:51 |
| sean-k-mooney | yep im just saying we may end up doing a revert of that isntead | 19:52 |
| sean-k-mooney | so i htink the cross job is actully doign the right thing | 19:52 |
| sean-k-mooney | and the bug is in nova | 19:52 |
| sean-k-mooney | our functional test shoudl not fail with the released version of placement | 19:52 |
| sean-k-mooney | haleyb: its failign for me locally as well | 19:52 |
| sean-k-mooney | its only passign in ci beasue that is usign master | 19:53 |
| 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 |
| sean-k-mooney | ill push a patch quickly to catch the NoSuchGroupError error quickly | 19:54 |
| 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 | 19:57 |
| 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 |
| sean-k-mooney | haleyb: i think ^ would fix it properly | 20:03 |
| sean-k-mooney | im testing that locally now | 20:04 |
| haleyb | sean-k-mooney: thanks, if that works guess you can add a Closes-bug there | 20:05 |
| sean-k-mooney | yep is there one fileed | 20:05 |
| haleyb | https://bugs.launchpad.net/nova/+bug/2131032 | 20:06 |
| sean-k-mooney | ah right you linked that before | 20:06 |
| 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 |
| haleyb | i tried to be thorough :) | 20:07 |
| 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 |
| sean-k-mooney | well quick for nova :) | 20:08 |
| 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:10 |
| sean-k-mooney | functional: OK (639.38=setup[32.16]+cmd[605.77,1.46] seconds) | 20:15 |
| 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:16 |
| sean-k-mooney | or gmaan ^ | 20:17 |
| gmaan | checking | 20:24 |
| 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:31 |
| 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 |
| gmaan | haleyb: cool | 20:34 |
| haleyb | ok, jobs queueing in https://review.opendev.org/c/openstack/requirements/+/966630 should know in 45-ish minutes | 20:38 |
| gmaan | thanks | 20:40 |
| haleyb | gmaan: seemed to work, jobs still running but cross-nova-functional has passed | 21:00 |
| gmaan | sean-k-mooney: haleyb: job passing on requirement gate so +2 | 21:01 |
| gmaan | yeah | 21:01 |
| 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:01 |
| opendevreview | Merged openstack/nova master: allow funtional test to run with released placment https://review.opendev.org/c/openstack/nova/+/966631 | 21:44 |
| opendevreview | melanie witt proposed openstack/nova master: TPM: add documentation and reno for live migration https://review.opendev.org/c/openstack/nova/+/962889 | 23:30 |
| opendevreview | melanie witt proposed openstack/nova master: DNM vtpm tempest https://review.opendev.org/c/openstack/nova/+/957477 | 23:32 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!