| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-08-11 | |||
| 08:37:06 | jssfr | we plan on resuming that work in the G cycle | |
| 08:40:59 | jssfr | *get it merged for Flamingo | |
| 09:44:49 | sean-k-mooney | Uggla: https://blueprints.launchpad.net/nova/+spec/xml-image-meta is now merged if you want to mark it as implemnted | |
| 09:53:52 | jayaanand | can we consider https://bugs.launchpad.net/cinder/+bug/1878913 and https://review.opendev.org/c/openstack/nova/+/680648 as stop gap arrangement. right now, customer is not able extend NFS volumes attached to Nova VM | |
| 09:59:38 | sean-k-mooney | jayaanand: if i recall correctly the spec for that was nto accpated this cycle but ill check | |
| 10:00:16 | sean-k-mooney | https://review.opendev.org/c/openstack/nova-specs/+/949504 | |
| 10:00:24 | sean-k-mooney | jayaanand: ^ | |
| 10:01:14 | sean-k-mooney | jayaanand: extended for nfs i think is techinally not supproted today | |
| 10:02:28 | sean-k-mooney | """ | |
| 10:02:30 | sean-k-mooney | Many remotefs-based volume drivers in Cinder use the ``qemu-img resize`` | |
| 10:02:32 | sean-k-mooney | command to extend volume files. | |
| 10:02:34 | sean-k-mooney | However, when the volume is attached to a guest, QEMU will lock the file and | |
| 10:02:36 | sean-k-mooney | ``qemu-img`` will be unable to resize it. | |
| 10:02:38 | sean-k-mooney | In this case, only the QEMU process holding the lock can resize the volume, | |
| 10:02:40 | sean-k-mooney | which can be triggered through the QEMU monitor command ``block-resize``. | |
| 10:02:42 | sean-k-mooney | There is currently no adequate way for Cinder to use this feature, so the NFS, | |
| 10:02:44 | sean-k-mooney | NetApp NFS, Powerstore NFS, and Quobyte volume drivers all disable extending | |
| 10:02:46 | sean-k-mooney | attached volumes.""" | |
| 10:02:48 | sean-k-mooney | from teh spec | |
| 10:03:38 | sean-k-mooney | jayaanand: we dont consdier that to be bug becasue it has never work and there is a proposal to add it as a feature the problem is that the folks workign on it have done so very intermitently | |
| 10:04:44 | sean-k-mooney | jayaanand: https://review.opendev.org/c/openstack/nova/+/873560 that is the poc | |
| 10:05:15 | sean-k-mooney | jayaanand: part of the problem is the cider side is not done yet https://review.opendev.org/q/topic:%22bp/extend-volume-completion-action%22 | |
| 10:06:13 | sean-k-mooney | although part of it merged in epocy i belive so we may be at the poitn where the nova side can progress but that will be 2026.1 at this rate | |
| 10:08:56 | jayaanand | we tested https://review.opendev.org/c/openstack/nova/+/680648 with customer NFS volumes attached to Nova VM. except encrypted volumes other scenarios NFS volumes expanded. This is ok with most of the customers. Without this changes. user need expand volume in NetApp filer and reboot VM to reflect size change. | |
| 10:09:14 | kgube | sean-k-mooney, the cinder dependency is actually merged | |
| 10:09:36 | jayaanand | this changes https://review.opendev.org/c/openstack/nova/+/680648 has extra error scenarios no logic changers will help customer to get information about errors. | |
| 10:09:59 | kgube | sean-k-mooney, at least the first part, that the nova changes depend on | |
| 10:10:30 | sean-k-mooney | kgube: the new api exits but is not hooked up right | |
| 10:10:51 | sean-k-mooney | i.e. cinder is not actully waiting for the completion callback correct? | |
| 10:11:27 | kgube | well, it has to be enabled in the drivers, but that depends on nova support again | |
| 10:12:28 | sean-k-mooney | jayaanand: we can consdier improving the error handeling but from a nova perspective extending an nfs voluem wont be supproted until kgube's feateure is complete | |
| 10:12:54 | kgube | sean-k-mooney, it's a cross-dependency, so we have to do a sort of a ping-pong of merges | |
| 10:13:20 | kgube | currently the ball is on the nova side | |
| 10:13:20 | sean-k-mooney | kgube: ack, i assume your going to reporpose the nova sepc for 2026.1 | |
| 10:13:30 | kgube | yes | |
| 10:14:05 | jayaanand | thank you! sean-k-mooney for considering error handling. is it possible to merge https://review.opendev.org/c/openstack/nova/+/680648 for this release. | |
| 10:14:17 | sean-k-mooney | kgube: have the cidner folks reviewd the cinder parts? if we are tat the critical poing we proably need to corrdiate the revew on both side to get it finally landed | |
| 10:16:42 | sean-k-mooney | jayaanand: i have not looked at it in detail but maybe. it likely will need a release note and moer testing. ideally a functional test to repoduce the incorrect event state in a preceedign patch then this patch woudl update the functional test to show that the instance action event was properly set to error | |
| 10:17:15 | sean-k-mooney | the curret unit test does not actully show it fixes the reporte bug in reporting | |
| 10:23:13 | kgube | sean-k-mooney, cinder folks have merged the initial patch: https://review.opendev.org/c/openstack/cinder-specs/+/866718, the next cinder patch is a minor change that will depend on a new nova microversion to enable the new behavior: https://review.opendev.org/c/openstack/cinder/+/891602 | |
| 10:23:31 | kgube | so this will have to be merged after the nova change | |
| 10:24:06 | kgube | I can ask them to already review it, though | |
| 10:27:52 | jayaanand | thank you! sean-k-mooney and kgube. i really appreciate your attenction | |
| 10:29:08 | opendevreview | Thomas Rotsaert proposed openstack/nova master: feat(hypervisors) - Adding new hypervisor property `hypervisor_rack`. https://review.opendev.org/c/openstack/nova/+/948276 | |
| 11:12:22 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Update to use '/v2.1' prefix, avoid project ID https://review.opendev.org/c/openstack/nova/+/952267 | |
| 11:12:23 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Use v2.1 API in OSAPIFixture https://review.opendev.org/c/openstack/nova/+/952269 | |
| 11:12:23 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Remove unnecessary API version overrides https://review.opendev.org/c/openstack/nova/+/952268 | |
| 11:12:24 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Remove deprecated pipeline_factory https://review.opendev.org/c/openstack/nova/+/952271 | |
| 11:12:24 | opendevreview | Stephen Finucane proposed openstack/nova master: conf: Rename '[api] neutron_default_tenant_id' https://review.opendev.org/c/openstack/nova/+/952270 | |
| 11:12:25 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Deprecate v2 API https://review.opendev.org/c/openstack/nova/+/954102 | |
| 11:12:25 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Remove '[api] auth_strategy', NoAuthMiddlware https://review.opendev.org/c/openstack/nova/+/952272 | |
| 11:12:26 | opendevreview | Stephen Finucane proposed openstack/nova master: WIP: api: Remove version controller split https://review.opendev.org/c/openstack/nova/+/954103 | |
| 13:28:03 | opendevreview | Merged openstack/nova master: sqlalchemy: Use built-in declarative https://review.opendev.org/c/openstack/nova/+/953751 | |
| 13:28:15 | opendevreview | Merged openstack/nova master: Enable E711 consistently https://review.opendev.org/c/openstack/nova/+/956436 | |
| 13:28:28 | opendevreview | Merged openstack/nova master: db: Fix api_db_opts reading main_db_opts https://review.opendev.org/c/openstack/nova/+/956869 | |
| 14:10:48 | opendevreview | Lajos Katona proposed openstack/nova master: WIP Use SDK for Neutron https://review.opendev.org/c/openstack/nova/+/928022 | |
| 14:35:24 | opendevreview | Merged openstack/nova master: Fix live migration error logging https://review.opendev.org/c/openstack/nova/+/951009 | |
| 18:10:11 | opendevreview | sean mooney proposed openstack/nova master: Remove Linux bridge VIF support from Nova https://review.opendev.org/c/openstack/nova/+/957040 | |
| 18:15:16 | opendevreview | sean mooney proposed openstack/os-vif master: remove linux bridge plugin https://review.opendev.org/c/openstack/os-vif/+/941586 | |
| 20:14:35 | opendevreview | Merged openstack/nova master: Remove unused utilities https://review.opendev.org/c/openstack/nova/+/954174 | |
| 22:19:41 | Callum027 | sean-k-mooney: Screwed up the upgrade note for the image meta patch, so a quick fix: https://review.opendev.org/c/openstack/nova/+/956988 | |
| #openstack-nova - 2025-08-12 | |||
| 02:08:12 | opendevreview | Masanori Kuroha proposed openstack/nova master: Copy applied provider config https://review.opendev.org/c/openstack/nova/+/948304 | |
| 06:36:31 | opendevreview | Takashi Kajinami proposed openstack/nova master: Remove logic for unsupported old libvirt/qemu https://review.opendev.org/c/openstack/nova/+/952318 | |
| 06:46:28 | opendevreview | Kamil Sambor proposed openstack/nova master: Switch nova-conductor to use ThreadPoolExecutor https://review.opendev.org/c/openstack/nova/+/957088 | |
| 07:42:11 | opendevreview | Lajos Katona proposed openstack/nova master: WIP Use SDK for Neutron https://review.opendev.org/c/openstack/nova/+/928022 | |
| 09:48:13 | opendevreview | Merged openstack/nova master: Fix libvirt metadata upgrade release note https://review.opendev.org/c/openstack/nova/+/956988 | |
| 11:30:53 | opendevreview | Merged openstack/nova master: mypy: fix type annotations for PciDevicePoolList and NUMATopologyLimits https://review.opendev.org/c/openstack/nova/+/955792 | |
| 11:31:05 | opendevreview | Merged openstack/nova master: tests: Update to use '/v2.1' prefix, avoid project ID https://review.opendev.org/c/openstack/nova/+/952267 | |
| 11:31:18 | opendevreview | Merged openstack/nova master: tests: Remove unnecessary API version overrides https://review.opendev.org/c/openstack/nova/+/952268 | |
| 11:31:31 | opendevreview | Merged openstack/nova master: tests: Use v2.1 API in OSAPIFixture https://review.opendev.org/c/openstack/nova/+/952269 | |
| 11:50:10 | opendevreview | Biser Milanov proposed openstack/nova master: Ensure StorPool volumes are attached to a single compute node when starting an instance https://review.opendev.org/c/openstack/nova/+/957119 | |
| 11:51:58 | opendevreview | Biser Milanov proposed openstack/nova master: Ensure StorPool volumes are attached to a single compute node when starting an instance https://review.opendev.org/c/openstack/nova/+/957119 | |
| 11:54:38 | opendevreview | Biser Milanov proposed openstack/nova stable/2024.1: Ensure StorPool volumes are attached to a single compute node when starting an instance https://review.opendev.org/c/openstack/nova/+/957121 | |
| 12:45:26 | opendevreview | Merged openstack/nova master: conf: Rename '[api] neutron_default_tenant_id' https://review.opendev.org/c/openstack/nova/+/952270 | |
| 13:00:26 | mkuroha | Hi, gibi, sean. Please review this patch when you have time! https://review.opendev.org/c/openstack/nova/+/948304 This is an implementation patch for provider.yaml, which was discussed in the nova-spec. | |
| 13:09:48 | sean-k-mooney | mkuroha: gibi is on pto until tomorrow just an fyi | |
| 13:10:14 | sean-k-mooney | mkuroha: i saw some update come in via my email for that but have not looked yet | |
| 13:11:11 | sean-k-mooney | mkuroha: just looking at the file list we will proably want to add a release note to annouch the change of behavior i see you have already update the admin docs which is good | |
| 13:12:33 | sean-k-mooney | you do have a test to assert that if the traits are removed the provider are updated but i have not read that in detail | |
| 13:12:50 | sean-k-mooney | mkuroha: so over all this is lookign promising given the ci is also green | |
| 13:29:53 | mkuroha | sean-k-mooney: Thank you for your reply! I'll add the release notes. Yes, I have added tests for patterns where traits are deleted or updated. (As I mentioned in the comments in the patch, I have not yet implemented cleanup process for patterns where provider config file is deleted or renamed.) | |
| 14:51:04 | opendevreview | Dan Smith proposed openstack/nova master: Run nova-next with n-sch in threading mode https://review.opendev.org/c/openstack/nova/+/948450 | |
| 14:51:04 | opendevreview | Dan Smith proposed openstack/nova master: Allow services to start with threading https://review.opendev.org/c/openstack/nova/+/948311 | |
| 15:01:59 | Uggla | Nova meeting in ~1 hour | |
| 15:03:23 | gmaan | Uggla: I might not join as board meetings in parallel to nova meeting | |
| 15:03:48 | Uggla | gmaan, no pb, thanks to let me know. | |
| 15:07:44 | dansmith | I have an appointment so I'll not be here | |
| 15:08:04 | Uggla | ok dansmith | |
| 15:40:58 | Uggla | Nova meeting in 20mn | |
| 15:57:45 | sean-k-mooney | Uggla: just an fyi. i filed https://bugs.launchpad.net/nova/+bug/2120323 to track compleing the removal of linux bridge supprot this cycle. | |
| 15:58:24 | sean-k-mooney | neutron remvoed it last cycle so you would have to use neutron form 2024.2 to use it today | |
| 15:58:49 | sean-k-mooney | i have to update the nova and os-vif patches but they are mostly ready for review so it would be nice to complete that this cycle | |
| 15:59:25 | Uggla | sean-k-mooney, I agree | |
| 15:59:46 | tkajinam | having less (unused) code is always better :-) | |
| 16:00:00 | opendevmeet | The meeting name has been set to 'nova' | |
| 16:00:00 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 16:00:00 | opendevmeet | Meeting started Tue Aug 12 16:00:00 2025 UTC and is due to finish in 60 minutes. The chair is Uggla. Information about MeetBot at http://wiki.debian.org/MeetBot. | |
| 16:00:00 | Uggla | #startmeeting nova | |
| 16:00:14 | Uggla | Hello everyone | |
| 16:00:19 | sean-k-mooney | o/ | |