| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-02-02 | |||
| 16:35:22 | tkajinam | ah, ok | |
| 16:35:49 | erlon | tkajinam: so, that behaves like you're saying for the volume attachments. But if the virtual machine has ephemeral devices back it by RBD, those are not refreshed. The ask is to, make them behave like volumes. | |
| 16:37:43 | erlon | so, when you shelve, or migrate, you would get all volume mappings (cinder and ephemeral) updated (today only cinders are) | |
| 16:38:54 | tkajinam | sounds fair | |
| 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 | |