| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-06-04 | |||
| 16:41:42 | sean-k-mooney | marlinc: i would proably add it to the lvm driver personally, i think it woudl be relvitly simple to do | |
| 16:41:58 | bauzas | before doing anything with version checks, please check with the cinder folks about populating the rotation rate into the connection info details | |
| 16:42:14 | bauzas | yeah, lvm seems the easiast approach | |
| 16:42:25 | marlinc | (Also actually the functional test for live migration, honestly I have never implemented something so complex as I have no experience at all with the testing framework and how it works in Nova) | |
| 16:42:26 | bauzas | but that's not my own garden :) | |
| 16:42:48 | marlinc | Especially since there is no existing volume based live migration functional I could see | |
| 16:42:54 | bauzas | marlinc: once you get a go from the cinder folks about exposing the rotation rate, come to me | |
| 16:43:03 | sean-k-mooney | marlinc: if cinder accpet the enhancement we can revisit that and or help you with that | |
| 16:43:07 | marlinc | Alright | |
| 16:43:20 | bauzas | cool | |
| 16:43:23 | bauzas | moving on then | |
| 16:43:26 | marlinc | Thank you, I'll update our internal ticket | |
| 16:43:44 | bauzas | marlinc: I assume you know how to reach the cinder folks? | |
| 16:44:36 | marlinc | Well, I'm going to assume #openstack-cinder and I'll check their contribution documentation | |
| 16:44:39 | bauzas | I have no potential design disapproval on using a specific connection info detail for setting the right value in the xml | |
| 16:45:00 | sean-k-mooney | marlinc: yep thats the correct channel | |
| 16:45:05 | bauzas | but you'll need to write a nova spec for explaining that it'll require a recent enough cinder and new computes | |
| 16:45:19 | sean-k-mooney | by the way there may be a usecase here for nova local storage too. | |
| 16:45:22 | marlinc | I was maybe a bit afraid though about the special rotation_rate 1 that is libvirt specific | |
| 16:45:23 | bauzas | and testing will be interesting to be discussed in the spec proocess | |
| 16:45:40 | marlinc | But that is probably something for the spec | |
| 16:45:52 | bauzas | the rotation rate is storage specific | |
| 16:46:07 | bauzas | how you tune it for the guest is libvirt specific indeed | |
| 16:46:34 | marlinc | That is why, right now we return rotation_rate 1 for SSDs, however that might not be smart from a cinder -> nova integration perspective | |
| 16:46:42 | marlinc | It is a magic value | |
| 16:46:56 | sean-k-mooney | well really your tryign to say "this is an ssd" or not | |
| 16:46:58 | bauzas | we're overtime and we still have two topics to discuss | |
| 16:47:05 | sean-k-mooney | afnd there may be better way to express that | |
| 16:47:15 | bauzas | but I think we may need to discuss that from a cross-project perspective | |
| 16:47:19 | sean-k-mooney | ya lets continue this converatoin after the meeting | |
| 16:47:23 | marlinc | Yes that is right now the primary use case though we also use it to set it to 7200 for HDDs | |
| 16:47:32 | marlinc | Alright thank you | |
| 16:47:38 | bauzas | next one then (3 of 4) | |
| 16:47:45 | bauzas | Luzi: floating ip behavior (assign fip: https://bugs.launchpad.net/neutron/+bug/2060808 , remove fip: https://bugs.launchpad.net/nova/+bug/2060812) | |
| 16:47:47 | Luzi | these two bugs describe changes with the neutron handling of floating ips (compared to the deprecated nova code). 1. one allows allocating a floating ip to a vm even if it is attached to another vm ("stealing" it). The 2. one does not check the VM when removing a floating ip, resulting in always removing the ip (even if the vm-name was correct, and the ip a mistake) | |
| 16:48:17 | bauzas | are you asking for reviews on the nova patch ? | |
| 16:48:24 | Luzi | I raised that at the PTG with Neutron, but they wanted Nova poeple to look over it, and tell, whether you want to change these bahaviors or not | |
| 16:48:33 | bauzas | oh my bad | |
| 16:49:08 | Luzi | (i normally don't have time to attend the Nova meetings) | |
| 16:49:31 | Luzi | I would be glad if you could check out these bugs again and give your input on it | |
| 16:49:56 | bauzas | Luzi: well, it's hard to comment those bugs on a limited timeframe | |
| 16:50:09 | Luzi | yeah, we can discuss this on launchpad also | |
| 16:50:11 | bauzas | could you please go back to the nova channel on another time ? | |
| 16:50:15 | Luzi | sure | |
| 16:50:31 | bauzas | Luzi: well, what's your timezone, please remind me ? | |
| 16:50:39 | Luzi | UTC-1 | |
| 16:51:09 | bauzas | okay, would that work if you would ping us again on tomorrow UTC afternoon ? | |
| 16:51:34 | Luzi | yeah i can do taht | |
| 16:51:43 | bauzas | thanks | |
| 16:52:01 | bauzas | last one | |
| 16:52:05 | bauzas | kgube: re-proposed extend volume completion spec: https://review.opendev.org/c/openstack/nova-specs/+/917133 | |
| 16:52:12 | kgube | Hi, this is just a review request | |
| 16:52:27 | bauzas | cool, we'll review all the proposed specs indeed | |
| 16:52:40 | kgube | The spec has been reproposed from last cycle and not much has changed | |
| 16:52:49 | bauzas | I think I need to remember the exact situation we had last cycle | |
| 16:53:06 | bauzas | and why this didn't get enough traction | |
| 16:53:18 | bauzas | I think we were basically awaiting cinder's feedback, right? | |
| 16:53:21 | kgube | The cinder dependencies had to get merged | |
| 16:53:31 | bauzas | yah that | |
| 16:53:42 | bauzas | and the cinder spec was accepted ? | |
| 16:53:47 | kgube | but cinderclient now supports the feature | |
| 16:54:05 | bauzas | okay, so where are we exactly on the cinder side ? | |
| 16:54:17 | bauzas | everything eventually got merged ? | |
| 16:54:46 | kgube | well, there are some cinder changes left, but they depend on the nova change again | |
| 16:54:55 | bauzas | yeah | |
| 16:54:58 | bauzas | I remember that | |
| 16:55:09 | bauzas | but okay, that's on our plate now | |
| 16:55:21 | bauzas | I'll then review the spec reproposal | |
| 16:55:27 | sean-k-mooney | they depend on https://review.opendev.org/c/openstack/nova/+/873560 right | |
| 16:55:28 | kgube | thanks! | |
| 16:55:38 | sean-k-mooney | which was waiting for the cinder client release | |
| 16:55:42 | sean-k-mooney | which has now happened | |
| 16:55:42 | bauzas | yeah | |
| 16:55:46 | bauzas | exactly | |
| 16:56:00 | sean-k-mooney | ok then if that can be rebased | |
| 16:56:05 | bauzas | I don't see anything controversial but I'll just doublecheck | |
| 16:56:09 | kgube | yeah, i still need to fix the build for this | |
| 16:56:12 | sean-k-mooney | we can review and then cidner can complete the rest | |
| 16:56:14 | bauzas | before blindly reapproving | |
| 16:56:36 | bauzas | cool | |
| 16:56:49 | bauzas | I think the dust has settled then | |
| 16:57:03 | bauzas | are we good with wrapping up the meeting then ? | |
| 16:58:12 | bauzas | looks so | |
| 16:58:16 | bauzas | thanks all | |
| 16:58:20 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2024/nova.2024-06-04-16.00.log.html | |
| 16:58:20 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2024/nova.2024-06-04-16.00.txt | |
| 16:58:20 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2024/nova.2024-06-04-16.00.html | |
| 16:58:20 | opendevmeet | Meeting ended Tue Jun 4 16:58:20 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 16:58:20 | bauzas | #endmeeting | |
| 23:17:56 | opendevreview | melanie witt proposed openstack/nova master: Consolidate vTPM and ephemeral encryption secret creation https://review.opendev.org/c/openstack/nova/+/912094 | |
| 23:17:57 | opendevreview | melanie witt proposed openstack/nova master: Support create with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/870932 | |
| 23:17:57 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Add optional 'description' kwarg to create_secret() https://review.opendev.org/c/openstack/nova/+/919986 | |
| 23:17:58 | opendevreview | melanie witt proposed openstack/nova master: Clean up unused ephemeral encryption libvirt secrets https://review.opendev.org/c/openstack/nova/+/919990 | |
| 23:17:58 | opendevreview | melanie witt proposed openstack/nova master: Validate key manager create access in API https://review.opendev.org/c/openstack/nova/+/919989 | |
| 23:17:59 | opendevreview | melanie witt proposed openstack/nova master: Support (resize|cold migration) with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/870933 | |
| 23:17:59 | opendevreview | melanie witt proposed openstack/nova master: libvirt: "Auto heal" ephemeral encryption secrets on guest launch https://review.opendev.org/c/openstack/nova/+/919991 | |
| 23:18:00 | opendevreview | melanie witt proposed openstack/nova master: Support rebuild with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/870939 | |
| 23:18:00 | opendevreview | melanie witt proposed openstack/nova master: Support live migration with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/905512 | |
| 23:18:02 | opendevreview | melanie witt proposed openstack/nova master: Add encryption support to qemu-img rebase https://review.opendev.org/c/openstack/nova/+/870936 | |
| 23:18:02 | opendevreview | melanie witt proposed openstack/nova master: Support encrypted source images for qcow2 https://review.opendev.org/c/openstack/nova/+/919992 | |