| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-02-02 | |||
| 16:38:55 | Uggla | erlon to my mind that sounds reasonable. I will add this patch to the review list. | |
| 16:39:57 | erlon | right, thanks. I will do a full review on it as well since I believe there's a few bits missing. And I'll propose something for nova-manage | |
| 16:40:26 | Uggla | ok | |
| 16:40:27 | erlon | nova-manage volume_attachment refresh --ephemeral [INSTANCE_UUID] [VOLUME_ID] [CONNECTOR_PATH] | |
| 16:40:27 | erlon | or | |
| 16:40:27 | erlon | nova-manage ephemeral_attachments refresh [INSTANCE_UUID] | |
| 16:40:45 | tkajinam | I'm wondering if a few other operations can resolve it. I feel like only providing live-migration to fix bdm would be incomplete from operator's perspective | |
| 16:41:56 | erlon | The first, would mix concepts (volume vs ephemeral), but in my opinion would make a smoother for the user experience. The second is more consistent, what would require more code and a new command | |
| 16:41:59 | tkajinam | (and we may want a similar functionality in cold migration path for example | |
| 16:43:11 | Uggla | erlon I would rather the second, but that's just a feeling. | |
| 16:43:44 | erlon | tkajinam: right, today, for volume we only have the functionality on migrate, resize and re-attach (shelve), are you thinking on something else? | |
| 16:45:25 | tkajinam | erlon, no the list looks good for start. if we extend it we also have to extend the refresh mechanism for cinder volumes, likely | |
| 16:46:55 | tkajinam | and I personally tend to support the 2nd approach given ephemeral volumes are handled slightly different from cinder volumes and I don't want to confuse users to find out which options can be used for resources | |
| 16:47:01 | tkajinam | for example volume id is not needed for ephemeral volumes | |
| 16:47:29 | Uggla | tkajinam ++ | |
| 16:47:51 | erlon | yeah, I also believe that the current operations should be okay, since it is mostly a point in time operation. As long as the operator have a way to update the VMs they should be fine. | |
| 16:47:51 | Uggla | erlon, something else ? | |
| 16:47:54 | erlon | ok | |
| 16:47:58 | gibi | sounds sane | |
| 16:47:58 | erlon | thats all from me | |
| 16:48:09 | Uggla | thanks erlon | |
| 16:48:28 | Uggla | I guess lajoskatona wanted to share someting ? | |
| 16:48:39 | lajoskatona | Uggla: thanks, I have :-) | |
| 16:49:07 | lajoskatona | o/ | |
| 16:49:28 | lajoskatona | I started to work on the migration away from neutronclient calls to SDK ones, and I have few patches up for review: | |
| 16:49:34 | lajoskatona | https://review.opendev.org/q/topic:%22sdk_for_neutron%22 | |
| 16:49:59 | lajoskatona | The last one for ports is red currently, I have to revisit it when I will have time | |
| 16:50:57 | lajoskatona | I tried to do small patches, each one covers one neutron API element, like networks or subnets | |
| 16:51:44 | lajoskatona | and it would be really helpful if I have feedback if the overall direction is good or any suggestion in review | |
| 16:51:55 | gibi | i will try to start reviewing them | |
| 16:52:31 | Uggla | me too. | |
| 16:52:42 | lajoskatona | thanks. I believe teh logic under remained the same, I just changed from what SDK returns instead of the dicts returned by neutronclient | |
| 16:53:27 | gibi | can we put a recurring topic about this on the agenda? | |
| 16:53:33 | lajoskatona | the other "big" difference are the exceptions as Neutronclient has some real exotic ones, and I tried to cover those with exceptions from SDK | |
| 16:53:39 | gibi | (just to help me remember) | |
| 16:54:02 | Uggla | gibi sure. | |
| 16:54:03 | lajoskatona | that can work, we do the same in Neutron meeting for this topic when I have more for review | |
| 16:54:16 | gibi | Uggla: thanks | |
| 16:54:38 | lajoskatona | thanks for the time | |
| 16:54:39 | Uggla | lajoskatona did you checked what was done for manila regarding sdk ? | |
| 16:55:14 | lajoskatona | yes, I checked in the past but not deeply | |
| 16:55:31 | MaxLamprecht[m] | o/ Uggla I have also one additional topic | |
| 16:56:09 | Uggla | MaxLamprecht[m] do you think you can cover it in 5mn ? | |
| 16:56:11 | lajoskatona | that's it from me for SDK | |
| 16:56:19 | Uggla | thanks lajoskatona | |
| 16:56:23 | MaxLamprecht[m] | sure | |
| 16:56:34 | Uggla | ok MaxLamprecht[m], so please go ahead | |
| 16:56:39 | MaxLamprecht[m] | I am working on resolving some inconsistencies between bdms/cinder-attachments that are happening regulary in our environment. | |
| 16:56:48 | MaxLamprecht[m] | I have these MRs incl. regression tests. Would be great to get review :) https://review.opendev.org/c/openstack/nova/+/972045 https://review.opendev.org/c/openstack/nova/+/974832 https://review.opendev.org/c/openstack/nova/+/934984 | |
| 16:58:21 | Uggla | MaxLamprecht[m], I have put them on the review list. | |
| 16:58:54 | Uggla | it looks like you have reproducer so it is cool | |
| 16:58:54 | MaxLamprecht[m] | I am currently investigating every single cause of these "out of sync" states in our env and try to fix the root causes. Maybe I will come up with more MRs in the next weeks/months. | |
| 16:58:56 | MaxLamprecht[m] | thanks | |
| 16:59:40 | Uggla | MaxLamprecht[m] something else to add ? | |
| 16:59:54 | MaxLamprecht[m] | nop thats everything :) thx | |
| 17:00:23 | Uggla | good thanks. It is time to close because we are at the top of the hour. | |
| 17:01:01 | Uggla | Thanks for joining this meeting. Have a nice day/evening and see you next week. | |
| 17:01:09 | Uggla | #endmeeting | |
| 17:01:09 | opendevmeet | Meeting ended Mon Feb 2 17:01:09 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 17:01:09 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2026/nova.2026-02-02-16.01.html | |
| 17:01:09 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2026/nova.2026-02-02-16.01.txt | |
| 17:01:09 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2026/nova.2026-02-02-16.01.log.html | |
| 17:01:09 | gibi | thanks | |
| 17:01:15 | lajoskatona | Bye, o/ | |
| 17:01:39 | bauzas | thanks | |
| 17:02:04 | elodilles | thanks o/ | |
| 17:05:52 | elodilles | Uggla: :S i hope she gets well soon | |
| 17:16:39 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Remove tpm support detection for libvirt < 8.0.0 https://review.opendev.org/c/openstack/nova/+/952308 | |
| 17:17:01 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Drop redundant chown of tpm data directory https://review.opendev.org/c/openstack/nova/+/962446 | |
| 17:33:38 | opendevreview | melanie witt proposed openstack/nova master: DNM test volume snapshot NFS https://review.opendev.org/c/openstack/nova/+/956282 | |
| 17:36:19 | opendevreview | Merged openstack/nova stable/2025.1: Init virt driver before use https://review.opendev.org/c/openstack/nova/+/975165 | |
| 17:38:35 | gmaan | gibi: on graceful shutdown, this is ready . I have abandon the DB, object change for the new topic field https://review.opendev.org/c/openstack/nova/+/975242 | |
| 17:38:38 | gmaan | bauzas: ^^ | |
| 17:41:44 | gibi | gmaan: ack, I will circle back | |
| 17:41:49 | gmaan | thanks | |
| 18:09:04 | opendevreview | Balazs Gibizer proposed openstack/nova master: Remove eventlet from CacheConcurrencyTestCase https://review.opendev.org/c/openstack/nova/+/970069 | |
| 18:09:04 | opendevreview | Balazs Gibizer proposed openstack/nova master: Remove eventlet from libvirt/test_driver https://review.opendev.org/c/openstack/nova/+/970070 | |
| 18:09:05 | opendevreview | Balazs Gibizer proposed openstack/nova master: Remove eventlet from libvirt/volume/test_mount https://review.opendev.org/c/openstack/nova/+/970071 | |
| 18:09:05 | opendevreview | Balazs Gibizer proposed openstack/nova master: Speed up RetryDecorator in unit test https://review.opendev.org/c/openstack/nova/+/975443 | |
| 18:12:10 | opendevreview | Balazs Gibizer proposed openstack/nova master: Speed up RetryDecorator in unit test https://review.opendev.org/c/openstack/nova/+/975443 | |
| 18:19:42 | gibi | gmaan: the first graceful patch looks good to me I just have a question about RPC minor version / service version https://review.opendev.org/c/openstack/nova/+/975242/4#message-96014581a53a1bfc9a6dbc419832eb7dd61dac46 | |
| 18:21:38 | gmaan | gibi: yes, I am planning to bump that when new RPC server/new topic will be in use in compute service | |
| 18:22:10 | gmaan | because that is the time, we need to handle the old vs new compute via versioning or fallback etc | |
| 18:23:56 | gibi | gmaan: OK make sense | |
| 18:24:03 | gibi | I just wanted to be sure we will do it | |
| 18:24:09 | gibi | my +2 stands :) | |
| 18:24:16 | gmaan | yeah, thanks | |
| 18:52:51 | gmaan | gibi: checked the STOPPED events change, its looks nice now. I left one comment about calculating the remaining_delay while it wait for shutdown condition https://review.opendev.org/c/openstack/nova/+/974445/comment/597be334_afaa6a5f/ | |
| 19:26:09 | gibi | gmaan: thanks you have a good point. I will fix it tomorrow | |
| 19:28:02 | sean-k-mooney | gibi: would you mind adding https://review.opendev.org/c/openstack/nova/+/975171/1 and https://review.opendev.org/c/openstack/nova/+/973149 to your review list for tomorrow | |
| 19:30:38 | gibi | sean-k-mooney: done :) | |
| 19:31:24 | gibi | I mean the first one is done | |
| 19:31:53 | gibi | the second will need my fresh brain tomorrow | |
| 19:33:49 | sean-k-mooney | thanks :) | |
| 19:38:14 | dansmith | gibi: can I have you for a sec? | |
| 19:38:20 | dansmith | fresh brain probably not required | |
| 19:51:30 | gmaan | gibi: ack | |
| 19:53:17 | dansmith | nevermind I think | |
| 20:07:49 | opendevreview | Merged openstack/nova master: Add regression test to repoduce bug 2139351 https://review.opendev.org/c/openstack/nova/+/975171 | |
| #openstack-nova - 2026-02-03 | |||
| 01:26:53 | opendevreview | melanie witt proposed openstack/nova master: testing: Run functional tests under [testenv:cover] https://review.opendev.org/c/openstack/nova/+/975346 | |