| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-03-08 | |||
| 22:03:19 | artom | Whoever thought that making the compute_rpcapi.rollback_live_migration_at_destination() a conditional *from the source* deserves... bad things. | |
| 22:05:25 | artom | Although apparently it's safe to add conditions, since vpmem and mdevs has been added | |
| 23:05:59 | opendevreview | Artom Lifshitz proposed openstack/nova master: Add cpuset_reserved helper to instance NUMA topology https://review.opendev.org/c/openstack/nova/+/910020 | |
| 23:06:00 | opendevreview | Artom Lifshitz proposed openstack/nova master: Power on cores for isolated emulator threads https://review.opendev.org/c/openstack/nova/+/909795 | |
| 23:06:00 | opendevreview | Artom Lifshitz proposed openstack/nova master: Reproducer for not powering on isolated emulator threads cores https://review.opendev.org/c/openstack/nova/+/910021 | |
| 23:06:01 | opendevreview | Artom Lifshitz proposed openstack/nova master: pwr mgmt: handle live migrations correctly https://review.opendev.org/c/openstack/nova/+/909806 | |
| 23:06:01 | opendevreview | Artom Lifshitz proposed openstack/nova master: Reproducer test for live migration with power management https://review.opendev.org/c/openstack/nova/+/910022 | |
| 23:06:02 | opendevreview | Artom Lifshitz proposed openstack/nova master: pwr mgmt: make API into a per-driver object https://review.opendev.org/c/openstack/nova/+/912320 | |
| #openstack-nova - 2024-03-09 | |||
| 01:40:00 | opendevreview | Artom Lifshitz proposed openstack/nova master: Add cpuset_reserved helper to instance NUMA topology https://review.opendev.org/c/openstack/nova/+/910020 | |
| 01:40:01 | opendevreview | Artom Lifshitz proposed openstack/nova master: Power on cores for isolated emulator threads https://review.opendev.org/c/openstack/nova/+/909795 | |
| 01:40:01 | opendevreview | Artom Lifshitz proposed openstack/nova master: Reproducer for not powering on isolated emulator threads cores https://review.opendev.org/c/openstack/nova/+/910021 | |
| 01:40:02 | opendevreview | Artom Lifshitz proposed openstack/nova master: Reproducer test for live migration with power management https://review.opendev.org/c/openstack/nova/+/910022 | |
| 01:40:02 | opendevreview | Artom Lifshitz proposed openstack/nova master: pwr mgmt: make API into a per-driver object https://review.opendev.org/c/openstack/nova/+/912320 | |
| 01:40:03 | opendevreview | Artom Lifshitz proposed openstack/nova master: pwr mgmt: handle live migrations correctly https://review.opendev.org/c/openstack/nova/+/909806 | |
| 03:37:28 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Create persistent mdevs https://review.opendev.org/c/openstack/nova/+/910041 | |
| 05:27:03 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Create persistent mdevs https://review.opendev.org/c/openstack/nova/+/910041 | |
| 09:34:08 | opendevreview | Vasyl Saienko proposed openstack/nova master: Fix calling nova-manage image_property get|set https://review.opendev.org/c/openstack/nova/+/912329 | |
| 09:34:29 | opendevreview | Vasyl Saienko proposed openstack/nova master: Fix calling nova-manage image_property get|set https://review.opendev.org/c/openstack/nova/+/912329 | |
| 10:05:56 | opendevreview | Vasyl Saienko proposed openstack/nova master: Fix calling nova-manage image_property show https://review.opendev.org/c/openstack/nova/+/912329 | |
| 11:30:04 | opendevreview | Pavlo Shchelokovskyy proposed openstack/nova master: Fix nova-manage image_property show unexpected keyword https://review.opendev.org/c/openstack/nova/+/880557 | |
| #openstack-nova - 2024-03-11 | |||
| 12:07:57 | artom_ | sean-k-mooney, bauzas, gibi, I updated my stack to fix libvirt CPU power management: https://review.opendev.org/c/openstack/nova/+/909806 | |
| 12:08:28 | artom_ | Uggla will also need to update his patch at https://review.opendev.org/c/openstack/nova/+/877773, as it's also necessary | |
| 12:57:20 | gibi | artom_: nice patches. I have one question about test coverage in the last one | |
| 13:39:01 | Uggla | artom_, I'll do that ASAP. | |
| 14:20:47 | opendevreview | Merged openstack/nova master: Reproducers for bug 1869804 https://review.opendev.org/c/openstack/nova/+/877772 | |
| 15:53:04 | opendevreview | Merged openstack/nova master: Add cpuset_reserved helper to instance NUMA topology https://review.opendev.org/c/openstack/nova/+/910020 | |
| 15:53:12 | opendevreview | Merged openstack/nova master: Reproducer for not powering on isolated emulator threads cores https://review.opendev.org/c/openstack/nova/+/910021 | |
| 16:55:05 | opendevreview | Gaudenz Steinlin proposed openstack/nova master: Fix attaching port with different port security https://review.opendev.org/c/openstack/nova/+/912478 | |
| 18:24:37 | opendevreview | Artom Lifshitz proposed openstack/nova master: pwr mgmt: handle live migrations correctly https://review.opendev.org/c/openstack/nova/+/909806 | |
| 18:24:37 | opendevreview | Artom Lifshitz proposed openstack/nova master: Reproducer test for live migration with power management https://review.opendev.org/c/openstack/nova/+/910022 | |
| #openstack-nova - 2024-03-12 | |||
| 02:51:46 | opendevreview | Youngjun Yoo proposed openstack/nova master: refectory: simplifying if condition expressions https://review.opendev.org/c/openstack/nova/+/912502 | |
| 06:10:48 | opendevreview | Amit Uniyal proposed openstack/nova stable/2023.2: Disconnecting volume from the compute host https://review.opendev.org/c/openstack/nova/+/911396 | |
| 08:34:27 | opendevreview | Amit Uniyal proposed openstack/nova stable/2023.1: Added context manager for instance lock https://review.opendev.org/c/openstack/nova/+/912511 | |
| 08:34:28 | opendevreview | Amit Uniyal proposed openstack/nova stable/2023.1: Separate OSError with ValueError https://review.opendev.org/c/openstack/nova/+/912512 | |
| 08:34:29 | opendevreview | Amit Uniyal proposed openstack/nova stable/2023.1: Disconnecting volume from the compute host https://review.opendev.org/c/openstack/nova/+/912513 | |
| 10:08:48 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (manila abstraction) https://review.opendev.org/c/openstack/nova/+/831194 | |
| 10:08:48 | opendevreview | ribaudr proposed openstack/nova master: Amend ShareMappingStatus due to asynchronous call https://review.opendev.org/c/openstack/nova/+/908864 | |
| 10:08:49 | opendevreview | ribaudr proposed openstack/nova master: Mounting the shares as part of the initialization process https://review.opendev.org/c/openstack/nova/+/880075 | |
| 10:08:49 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (drivers and compute manager part) https://review.opendev.org/c/openstack/nova/+/833090 | |
| 10:08:50 | opendevreview | ribaudr proposed openstack/nova master: Add metadata for shares https://review.opendev.org/c/openstack/nova/+/850500 | |
| 10:08:50 | opendevreview | ribaudr proposed openstack/nova master: Deletion of associated share mappings on instance deletion https://review.opendev.org/c/openstack/nova/+/881472 | |
| 10:08:51 | opendevreview | ribaudr proposed openstack/nova master: Add share_info parameter to reboot method for each driver (driver part) https://review.opendev.org/c/openstack/nova/+/854823 | |
| 10:08:52 | opendevreview | ribaudr proposed openstack/nova master: Add share_info parameter to resume method for each driver (driver part) https://review.opendev.org/c/openstack/nova/+/860284 | |
| 10:08:52 | opendevreview | ribaudr proposed openstack/nova master: Support rebooting an instance with shares (compute manager part) https://review.opendev.org/c/openstack/nova/+/854824 | |
| 10:08:54 | opendevreview | ribaudr proposed openstack/nova master: Add helper methods to rescue/unrescue shares https://review.opendev.org/c/openstack/nova/+/860286 | |
| 10:08:54 | opendevreview | ribaudr proposed openstack/nova master: Support resuming an instance with shares (compute manager part) https://review.opendev.org/c/openstack/nova/+/860285 | |
| 10:08:56 | opendevreview | ribaudr proposed openstack/nova master: Support rescuing an instance with shares (compute manager part) https://review.opendev.org/c/openstack/nova/+/860288 | |
| 10:08:56 | opendevreview | ribaudr proposed openstack/nova master: Support rescuing an instance with shares (driver part) https://review.opendev.org/c/openstack/nova/+/860287 | |
| 10:08:58 | opendevreview | ribaudr proposed openstack/nova master: Check shares support (compute manager) https://review.opendev.org/c/openstack/nova/+/885751 | |
| 10:08:58 | opendevreview | ribaudr proposed openstack/nova master: Allow to mount manila share using Cephfs protocol https://review.opendev.org/c/openstack/nova/+/883862 | |
| 10:09:00 | opendevreview | ribaudr proposed openstack/nova master: Check shares support (only API exception) https://review.opendev.org/c/openstack/nova/+/885752 | |
| 10:09:00 | opendevreview | ribaudr proposed openstack/nova master: Add share lock/unlock and restrict visibility https://review.opendev.org/c/openstack/nova/+/890340 | |
| 10:09:02 | opendevreview | ribaudr proposed openstack/nova master: Add helper methods to attach/detach shares https://review.opendev.org/c/openstack/nova/+/885753 | |
| 10:09:02 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (API) https://review.opendev.org/c/openstack/nova/+/836830 | |
| 10:09:04 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach notification https://review.opendev.org/c/openstack/nova/+/851028 | |
| 10:09:04 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach notification https://review.opendev.org/c/openstack/nova/+/850501 | |
| 10:09:06 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach_error notification https://review.opendev.org/c/openstack/nova/+/860282 | |
| 10:09:06 | opendevreview | ribaudr proposed openstack/nova master: Add shares to InstancePayload https://review.opendev.org/c/openstack/nova/+/851029 | |
| 10:09:08 | opendevreview | ribaudr proposed openstack/nova master: Add libvirt test to ensure metadata are working. https://review.opendev.org/c/openstack/nova/+/852086 | |
| 10:09:08 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach_error notification https://review.opendev.org/c/openstack/nova/+/860283 | |
| 10:09:10 | opendevreview | ribaudr proposed openstack/nova master: Docs about Manila shares API usage https://review.opendev.org/c/openstack/nova/+/871642 | |
| 10:09:10 | opendevreview | ribaudr proposed openstack/nova master: Add virt/libvirt error test cases https://review.opendev.org/c/openstack/nova/+/852087 | |
| 10:09:12 | opendevreview | ribaudr proposed openstack/nova master: Amend DB model add a unique constraint. https://review.opendev.org/c/openstack/nova/+/912518 | |
| 13:58:12 | opendevreview | ribaudr proposed openstack/nova master: Fix: Live migrating to a host with cpu_shared_set configured will now update the VM's configuration accordingly. https://review.opendev.org/c/openstack/nova/+/903706 | |
| 13:58:12 | opendevreview | ribaudr proposed openstack/nova master: Fix [backport]: Live migrating to a host with cpu_shared_set configured will now update the VM's configuration accordingly. https://review.opendev.org/c/openstack/nova/+/877773 | |
| 15:34:39 | fwiesel | Hi, regarding the bug report (https://bugs.launchpad.net/nova/+bug/2055700) (ServerActionsV293TestJSON.test_rebuild_volume_backed_server). I think, I do now why I get the exception. What I don't know is, how that is ever suppose to work with other hypervisors. | |
| 15:36:21 | fwiesel | ComputeManager._rebuild_default_impl is calling driver.destroy on the instance, and *then* calls a volume detach. That raises an InstanceNotFound in the vmwareapi driver, and I don't know how it should not. | |
| 15:37:04 | fwiesel | Here is the destroy: https://opendev.org/openstack/nova/src/branch/master/nova/compute/manager.py#L3695-L3701 | |
| 15:37:35 | fwiesel | Then the call to _rebuild_volume_backed_instance (https://opendev.org/openstack/nova/src/branch/master/nova/compute/manager.py#L3714-L3715) | |
| 15:38:12 | fwiesel | Which in turn calls _detach_root_volume (https://opendev.org/openstack/nova/src/branch/master/nova/compute/manager.py#L3591) | |
| 15:38:55 | fwiesel | And that unsurprisingly calls driver.detach_volume (https://opendev.org/openstack/nova/src/branch/master/nova/compute/manager.py#L3556-L3557) | |
| 15:49:08 | fwiesel | I am unclear on how that is supposed to work. If you destroy an instance, my expectation would be that the driver does report InstanceNotFound when you do further operations on it. Apparently that is not the case with libvirt. What path will get triggered there? | |
| 15:51:26 | bauzas | reminder: nova meeting in 10 mins | |
| 15:51:58 | bauzas | fwiesel: I'm currently working hard but we could discuss this in the meeting | |
| 15:56:10 | fwiesel | Sure, no hurry from my side. I am also fine doing it via the bug-report. | |
| 16:01:09 | opendevmeet | The meeting name has been set to 'nova' | |
| 16:01:09 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 16:01:09 | opendevmeet | Meeting started Tue Mar 12 16:01:09 2024 UTC and is due to finish in 60 minutes. The chair is bauzas. Information about MeetBot at http://wiki.debian.org/MeetBot. | |
| 16:01:09 | bauzas | #startmeeting nova | |
| 16:01:12 | elodilles | o/ | |
| 16:01:13 | bauzas | hey folks | |
| 16:01:16 | dansmith | o/ | |
| 16:01:17 | bauzas | #link https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting | |
| 16:01:28 | fwiesel | o/ | |
| 16:02:13 | auniyal | o/ | |
| 16:02:22 | gibi | o/ | |
| 16:02:39 | bauzas | okay, let's start | |
| 16:03:00 | bauzas | #topic Bugs (stuck/critical) | |
| 16:03:06 | bauzas | #info Two Critical bugs | |
| 16:03:11 | bauzas | #link https://bugs.launchpad.net/nova/+bug/2052915 | |
| 16:03:15 | bauzas | #link https://bugs.launchpad.net/nova/+bug/2056613 | |
| 16:03:45 | bauzas | for the first one, AFAIK, we no longer have a problem | |
| 16:03:52 | bauzas | so we could close it | |
| 16:04:13 | bauzas | elodilles: ^ | |
| 16:04:54 | elodilles | well nova-grenade-multinode is not failing | |
| 16:05:21 | elodilles | but that's not neutron-ovs-grenade-* | |
| 16:06:05 | bauzas | are we running those jobs in Nova ? I don't find so | |
| 16:06:15 | elodilles | so if nova was linked due to nova-grenade-multinode then we can close | |
| 16:06:24 | elodilles | otherwise we don't even run those jobs | |
| 16:06:32 | bauzas | OK, I'll close this for nova | |