| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-07-10 | |||
| 13:15:09 | opendevreview | Merged openstack/nova master: api: Address issues with keypairs APIs https://review.opendev.org/c/openstack/nova/+/952214 | |
| 13:15:21 | opendevreview | Merged openstack/nova stable/2025.1: FUP Update pci-passthrough and virtual-gpu documentation https://review.opendev.org/c/openstack/nova/+/948254 | |
| 13:24:37 | opendevreview | Markus Hentsch proposed openstack/nova master: Adapt to Cinder and Glance changes for image encryption https://review.opendev.org/c/openstack/nova/+/926326 | |
| 13:37:56 | opendevreview | Balazs Gibizer proposed openstack/nova master: [pci]Keep used dev in Placement regardless of dev_spec https://review.opendev.org/c/openstack/nova/+/954149 | |
| 13:37:57 | opendevreview | Balazs Gibizer proposed openstack/nova master: [PCI tracker]Remove non configured devs when freed https://review.opendev.org/c/openstack/nova/+/954613 | |
| 13:57:42 | gibi | ^^ this was a tough one | |
| 14:18:54 | sean-k-mooney | ah i see the edge case this adresed | |
| 14:35:01 | gibi | sambork: FYI https://review.opendev.org/c/openstack/nova/+/954324/comment/41b996aa_19440791/ | |
| 14:44:46 | sambork | gibi, thanks I will take a look | |
| 14:50:41 | sambork | "It generally looks like the timeout work needs to be done in the same patch to ensure that we raise the error at the same level and update _InstanceEvent accordingly | |
| 14:53:55 | gibi | OK I can live with that :) | |
| 14:57:26 | opendevreview | Takashi Kajinami proposed openstack/nova master: sqlalchemy: Use built-in declarative https://review.opendev.org/c/openstack/nova/+/953751 | |
| 15:36:30 | sean-k-mooney | sambork: kamil is this your upstream nic? | |
| 17:22:39 | gmaan | sean-k-mooney[m]: thanks for review on manager role. I agree to merge both change (policy granularity and default change to manager). I replied to this comment, if that make sense - https://review.opendev.org/c/openstack/nova/+/953063/7/nova/policies/migrate_server.py#101 | |
| 17:23:34 | gmaan | oslo.policy implementation does not cover the policy granularity which is something we can implement in future but I used the deprecated_rule way so that we can handle the operator use case | |
| 17:25:39 | gmaan | I am mentioning it in deprecation_reason that policy is not deprecated and used to handle the custom policy case | |
| 17:32:19 | gmaan | sean-k-mooney[m]: I notice that we still have time for oslo lib freeze for this cycle, so let me try implementing the granular policy case in oslo.policy and then our implementation will looks good. | |
| 17:32:29 | gmaan | and that will handle the future case of policy granularity also | |
| 18:00:41 | opendevreview | Merged openstack/nova master: Add spawn_on https://review.opendev.org/c/openstack/nova/+/948079 | |
| #openstack-nova - 2025-07-11 | |||
| 04:17:22 | opendevreview | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.opendev.org/c/openstack/nova/+/954688 | |
| 06:54:18 | sambork | sean-k-mooney[m], yes thats my nick | |
| 08:59:43 | opendevreview | benlei proposed openstack/nova master: FIX: instance live migration failed after swap volume https://review.opendev.org/c/openstack/nova/+/954210 | |
| 09:02:58 | opendevreview | benlei proposed openstack/nova master: FIX: instance live migration failed after swap volume https://review.opendev.org/c/openstack/nova/+/954210 | |
| 09:07:31 | opendevreview | benlei proposed openstack/nova master: FIX: instance live migration failed after swap volume https://review.opendev.org/c/openstack/nova/+/954210 | |
| 09:15:04 | jssfr | `VIR_DOMAIN_UNDEFINE_KEEP_TPM` is passed, which isn't passed anywhere in nova. | |
| 09:15:04 | jssfr | Fun question. I'm looking at a Nova 2023.1 installation with vTPM enabled. As far as I can tell, the vTPM state gets completely deleted on VM power on, e.g. after poweroff from within the VM. The code path leading up to this is apparently power_on -> driver._hard_reboot -> driver.destroy -> driver.cleanup -> driver._cleanup -> libvirt undefine, and libvirt drops the swtpm directory on undefine unless | |
| 09:15:12 | jssfr | am I holding this wrong? | |
| 09:17:11 | jssfr | meanwhile I'm going to try to reproduce this on 2024.1, but given that I traced back the codepath on 2024.1, I'm not hopeful... | |
| 11:55:14 | jssfr | I reproduced the same issue in 2024.1 :( | |
| 12:55:17 | opendevreview | Balazs Gibizer proposed openstack/nova master: FUP: Translate scatter-gather to futurist https://review.opendev.org/c/openstack/nova/+/953338 | |
| 13:48:58 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make nova.utils.pass_context private https://review.opendev.org/c/openstack/nova/+/948188 | |
| 13:48:58 | opendevreview | Balazs Gibizer proposed openstack/nova master: Move ConductorManager to use spawn_on https://review.opendev.org/c/openstack/nova/+/948187 | |
| 13:48:59 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make the default executor configurable https://review.opendev.org/c/openstack/nova/+/948087 | |
| 13:48:59 | opendevreview | Balazs Gibizer proposed openstack/nova master: Rename DEFAULT_GREEN_POOL to DEFAULT_EXECUTOR https://review.opendev.org/c/openstack/nova/+/948086 | |
| 13:49:00 | opendevreview | Balazs Gibizer proposed openstack/nova master: Document native threading mode and tuneables https://review.opendev.org/c/openstack/nova/+/949364 | |
| 13:49:00 | opendevreview | Balazs Gibizer proposed openstack/nova master: Print ThreadPool statistics https://review.opendev.org/c/openstack/nova/+/948340 | |
| 13:49:01 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-next with n-sch in threading mode https://review.opendev.org/c/openstack/nova/+/948450 | |
| 13:49:01 | opendevreview | Balazs Gibizer proposed openstack/nova master: Allow services to start with threading https://review.opendev.org/c/openstack/nova/+/948311 | |
| 13:49:02 | opendevreview | Balazs Gibizer proposed openstack/nova master: Do not yield in threading mode https://review.opendev.org/c/openstack/nova/+/950994 | |
| 13:49:03 | opendevreview | Balazs Gibizer proposed openstack/nova master: Warn on long task wait time for executor https://review.opendev.org/c/openstack/nova/+/952666 | |
| 13:49:03 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-api and -metadata in threaded mode https://review.opendev.org/c/openstack/nova/+/951957 | |
| 13:49:05 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run unit test with threading mode https://review.opendev.org/c/openstack/nova/+/953475 | |
| 13:49:05 | opendevreview | Balazs Gibizer proposed openstack/nova master: Allow to start unit test without eventlet https://review.opendev.org/c/openstack/nova/+/953436 | |
| 13:49:07 | opendevreview | Balazs Gibizer proposed openstack/nova master: [test]RPC using threading or eventlet selectively https://review.opendev.org/c/openstack/nova/+/953815 | |
| #openstack-nova - 2025-07-12 | |||
| 00:39:48 | melwitt | jssfr: it looks like that flag may be relatively new i.e. newer than when vTPM was added to nova https://patchew.org/Libvirt/20220822120554.3529999-1-stefanb@linux.ibm.com/20220822120554.3529999-7-stefanb@linux.ibm.com/ so it seems like it's something we need to add | |
| 06:48:12 | jssfr | melwitt, thanks for the feedback. so this fell on our feet because we have users with Windows 11 who enabled bitlocker and ... well, they're a bit sad now after they powered their VMs off ;). | |
| 06:48:25 | jssfr | we can't be the first with this type of usage, can we? | |
| #openstack-nova - 2025-07-14 | |||
| 07:04:31 | opendevreview | Takashi Kajinami proposed openstack/nova master: Remove unused utilities https://review.opendev.org/c/openstack/nova/+/954174 | |
| 07:04:32 | opendevreview | Takashi Kajinami proposed openstack/nova master: Migrate ExceptionHelper to test utilities https://review.opendev.org/c/openstack/nova/+/954176 | |
| 08:01:18 | gibi | The nova-ceph-multistore job is broken and blocking the gate since 2025-07-11 08:33:54 I reported the bug ttps://bugs.launchpad.net/nova/+bug/2116852 | |
| 08:43:38 | mikal | Uggla: what is your preferred way for me to solicit a second set of +2's on https://review.opendev.org/c/openstack/nova/+/926126 onwards? sean-k-mooney[m] has already +2'ed all four patches. | |
| 11:49:03 | opendevreview | Merged openstack/placement master: Replace deprecated assertItemsEqual https://review.opendev.org/c/openstack/placement/+/953582 | |
| 14:54:46 | opendevreview | Balazs Gibizer proposed openstack/nova master: DNM: Force librbd1 19.2.0 to fix ceph job https://review.opendev.org/c/openstack/nova/+/954956 | |
| 15:23:25 | fungi | looks like the recent breaking change in cloud-init has reawakened interest in https://launchpad.net/nova/+bug/1663304 | |
| 15:38:09 | opendevreview | Balazs Gibizer proposed openstack/nova master: DNM: Force librbd1 19.2.0 to fix ceph job https://review.opendev.org/c/openstack/nova/+/954956 | |
| 16:15:59 | dansmith | gibi: I think maybe adding a global monotonic deadline to the RecordSortContext that we can check and use to limit how long we run would probably be all we need there | |
| 16:16:29 | dansmith | the timer is mostly just to avoid the threads querying another batch if the whole process is taking too long | |
| 16:16:39 | dansmith | I can try to propose something if you want | |
| 16:17:41 | gibi | feel free to propose, I'm mostly done for the day anyhow. And I will start with running some local trials with this batching and shorting scatter gather locally hopefully tomorrow to understand it more | |
| 16:19:38 | gibi | I agree if we can check a global deadline before yielding the next item and exit instead that would workh | |
| 16:20:32 | dansmith | cool | |
| 17:16:01 | sean-k-mooney | fungi: ya.. the problem with that is they interface they are suign is kind of a private implation detail | |
| 17:16:15 | sean-k-mooney | fungi: so it snot clear that we shoudl do that | |
| 18:09:21 | opendevreview | Dan Smith proposed openstack/nova master: Remove eventlet timer from multi_cell_list https://review.opendev.org/c/openstack/nova/+/954990 | |
| 18:16:09 | opendevreview | Dan Smith proposed openstack/nova master: Remove eventlet timer from multi_cell_list https://review.opendev.org/c/openstack/nova/+/954990 | |
| 18:17:53 | melwitt | jssfr: perhaps not but I have personally not heard about the problem before and did not find any ticket about it when I did a quick search in bugs.launchpad.net/nova | |
| 18:29:25 | opendevreview | Balazs Gibizer proposed openstack/nova master: DNM: Force librbd1 19.2.0 to fix ceph job https://review.opendev.org/c/openstack/nova/+/954956 | |
| 19:18:38 | opendevreview | Merged openstack/nova master: Imported Translations from Zanata https://review.opendev.org/c/openstack/nova/+/954688 | |
| 21:55:41 | opendevreview | melanie witt proposed openstack/nova master: WIP libvirt: Get info with abs path, rebase with rel path https://review.opendev.org/c/openstack/nova/+/955039 | |
| #openstack-nova - 2025-07-15 | |||
| 00:38:44 | opendevreview | Ghanshyam proposed openstack/nova master: Add project manager role in Nova API policy rule https://review.opendev.org/c/openstack/nova/+/953063 | |
| 00:58:39 | opendevreview | Ghanshyam proposed openstack/nova master: Add project manager role in Nova API policy rule https://review.opendev.org/c/openstack/nova/+/953063 | |
| 12:16:19 | opendevreview | sean mooney proposed openstack/nova master: add detailed logging to get_updated_guest_xml https://review.opendev.org/c/openstack/nova/+/955056 | |
| 13:07:03 | opendevreview | Balazs Gibizer proposed openstack/nova master: DNM: Force librbd1 19.2.0 to fix ceph job https://review.opendev.org/c/openstack/nova/+/954956 | |
| 15:29:37 | bauzas | nova meeting in 30 mins now | |
| 16:00:25 | opendevmeet | The meeting name has been set to 'nova' | |
| 16:00:25 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 16:00:25 | opendevmeet | Meeting started Tue Jul 15 16:00:25 2025 UTC and is due to finish in 60 minutes. The chair is bauzas. Information about MeetBot at http://wiki.debian.org/MeetBot. | |
| 16:00:25 | bauzas | #startmeeting nova | |
| 16:00:46 | bauzas | #link https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting | |
| 16:01:05 | bauzas | howdy folks, I'll be your humble chair for today's meeting | |
| 16:01:08 | bauzas | who's around ? | |
| 16:01:18 | elodilles | o/ | |
| 16:01:31 | fwiesel | o/ | |
| 16:01:36 | ratailor_ | o/ | |
| 16:01:56 | gmaan | o/ | |
| 16:02:30 | gibi | o/ | |
| 16:03:00 | bauzas | okay, let's start | |
| 16:03:08 | bauzas | #topic Bugs (stuck/critical) | |
| 16:03:10 | bauzas | doh | |
| 16:03:15 | bauzas | #topic Bugs (stuck/critical) | |
| 16:03:40 | bauzas | #info One Critical bug | |
| 16:03:42 | gibi | https://bugs.launchpad.net/nova/+bug/2116852 | |
| 16:03:46 | gibi | nova-ceph-multistore job fails constantly with memory allocation error Edit | |
| 16:03:47 | bauzas | #link https://bugs.launchpad.net/nova/+bug/2116852 | |
| 16:03:54 | bauzas | yeah | |
| 16:03:58 | bauzas | just saw that | |
| 16:04:06 | bauzas | any proposed patch ? | |
| 16:04:09 | gibi | patch to sidable the single tempest test to unblock the gate: https://review.opendev.org/c/openstack/tempest/+/954949 | |
| 16:04:21 | bauzas | ack | |
| 16:04:29 | gibi | patch that proves that it is caused by ceph package update https://review.opendev.org/c/openstack/nova/+/954956 | |