| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-06-06 | |||
| 16:48:10 | gibi | dansmith: ahh true, we can do it pre project then | |
| 16:48:13 | gibi | per | |
| 16:48:18 | bauzas | yeah, devil is in the details of the productization | |
| 16:48:21 | dansmith | gibi: or even server group | |
| 16:48:42 | sean-k-mooney | anyway lets see what they actully bring up | |
| 16:48:42 | bauzas | gibi: honestly, the granularity sounds per project to me | |
| 16:48:53 | sean-k-mooney | and see if this type of solution would work for them or not | |
| 16:48:57 | dansmith | well, nfv people are all one project in some cases, so that probably won't work for them | |
| 16:49:01 | bauzas | gibi: are you done with this topic now that we drafted a solution for you ? :D | |
| 16:49:11 | gibi | I'm done | |
| 16:49:15 | bauzas | cool | |
| 16:49:16 | gibi | thanks for the discussion\ | |
| 16:49:22 | gibi | I will link this to the etherpad | |
| 16:49:27 | gibi | and I will report back from the summit | |
| 16:49:27 | bauzas | cool | |
| 16:49:37 | bauzas | I'll be back watching you at the Summit anyway | |
| 16:49:51 | bauzas | so if you promise too many things, I could yell :p | |
| 16:49:58 | gibi | bauzas: please do so | |
| 16:49:59 | gibi | :D | |
| 16:50:12 | gibi | I don't need another 3 years of "notification" work | |
| 16:50:22 | dansmith | heh | |
| 16:50:28 | dansmith | still got scars eh/ | |
| 16:50:30 | bauzas | ok, I was balancing the idea to paperwork the scaphandre and manila series but I'm exhausted of today | |
| 16:50:54 | bauzas | so, let's skip it and pretend it will be discussed in two weeks from now | |
| 16:51:01 | gibi | dansmith: time makes all these memory nicer and nicer actaully | |
| 16:51:06 | dansmith | heh | |
| 16:51:18 | gibi | so bauzas' has a good point watching me :) | |
| 16:51:32 | dansmith | gibi: https://www.youtube.com/watch?v=dLjNzwEULG8 | |
| 16:51:48 | bauzas | gibi: you're fortunate, canadians don't open carry | |
| 16:52:05 | bauzas | sorry, was a terrible joke :) | |
| 16:52:16 | gibi | dansmith: I need to check this out after the meeting :) | |
| 16:52:20 | bauzas | anyway, I think we're done for today | |
| 16:52:29 | gibi | indeed | |
| 16:52:40 | bauzas | thanks all | |
| 16:52:45 | bauzas | and thanks gibi for the chair | |
| 16:52:50 | bauzas | #endmeeting | |
| 16:52:50 | opendevmeet | Meeting ended Tue Jun 6 16:52:50 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 16:52:50 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2023/nova.2023-06-06-16.00.html | |
| 16:52:50 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2023/nova.2023-06-06-16.00.txt | |
| 16:52:50 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2023/nova.2023-06-06-16.00.log.html | |
| 16:53:02 | bauzas | gibi: there you go, you have your logs :p | |
| 16:54:11 | gibi | thank | |
| 16:54:25 | gibi | and it was the moment when my connection dropped first | |
| 16:54:34 | gibi | so it was not that flaky after all | |
| 16:55:13 | dansmith | are we canceling the nova meeting next week? | |
| 16:55:27 | bauzas | dansmith: damn shit, forgot to tell it | |
| 16:55:43 | bauzas | unless people wanna run it, which I'm cool wiht | |
| 16:56:29 | geguileo | dansmith: sean-k-mooney last week I mentioned the os-brick idempotency and we talked about reconstructing the disk XML on some operations. I've just opened a bug (#2023078) where Nova is not rebuilding the disk XML after block migration. Don't know nova enough to know if this would be fixed with the changes to the os-brick idempotency thingy. | |
| 16:57:14 | sean-k-mooney | the xml for live migration is built on the souce node based on info passed back form the destination host | |
| 16:57:22 | sean-k-mooney | for cold migration it build on the dest host | |
| 16:57:52 | sean-k-mooney | i assume you are refering to live-migration local block devices? | |
| 16:57:58 | geguileo | sean-k-mooney: the XML after live migration is wrong | |
| 16:58:18 | sean-k-mooney | that xml must be generated on the source host | |
| 16:58:20 | geguileo | sean-k-mooney: it doesn't have the right discard=unmap value even when the destination is saying that it is supported | |
| 16:58:38 | sean-k-mooney | is it enabeld on the source | |
| 16:59:11 | geguileo | sean-k-mooney: but if the source didn't support it and now it does? Then it cannot be supported until reboot? | |
| 16:59:28 | geguileo | Because then I think Cinder should start reporting that everything supports discard... | |
| 16:59:57 | sean-k-mooney | geguileo: correct | |
| 17:00:12 | sean-k-mooney | so this is not somethign that should be chnaging durign a live migrate | |
| 17:00:31 | geguileo | ok, so then why do we even report the support for this thing? | |
| 17:00:45 | geguileo | We should always set discard=unmap in the XML | |
| 17:01:01 | sean-k-mooney | well we cant because our min libvirt/qemu did not support it | |
| 17:01:05 | geguileo | if it works, nice, if it doesn't it will not prevent it from working after a live migration | |
| 17:01:22 | sean-k-mooney | that may have changed recently but it was a limitation in the past | |
| 17:01:37 | opendevreview | sean mooney proposed openstack/nova master: Allow discard with virtio-blk https://review.opendev.org/c/openstack/nova/+/878795 | |
| 17:01:42 | geguileo | sean-k-mooney: but if it does now then maybe we have to give it another go at this whole thing | |
| 17:01:52 | sean-k-mooney | geguileo: without ^ we also dont support discard for all disk buses | |
| 17:02:18 | sean-k-mooney | geguileo: sure but we have to be careful to ensure that the move ops work properly | |
| 17:02:22 | geguileo | sean-k-mooney: that patch should be abandoned... I added a comment to the LP bug | |
| 17:02:33 | sean-k-mooney | that is not your patch | |
| 17:02:38 | sean-k-mooney | its my patch to fix the bug | |
| 17:02:45 | sean-k-mooney | that for some reason was not in launchpad | |
| 17:02:49 | geguileo | sean-k-mooney: how is it different to mine? | |
| 17:03:02 | geguileo | it's literaly the same | |
| 17:03:06 | geguileo | and it doesn't work | |
| 17:03:07 | sean-k-mooney | i make discard work for disk buses | |
| 17:03:20 | geguileo | it just removes a debug log message | |
| 17:03:28 | geguileo | and the log is correct | |
| 17:03:32 | geguileo | discard doesn't work with virtio | |
| 17:03:37 | sean-k-mooney | yes it does | |
| 17:03:37 | geguileo | don't know why | |
| 17:03:49 | sean-k-mooney | it requires a min version fo qemu and libvirt | |
| 17:04:02 | geguileo | I could only make it work with IDE, SCSI, and SATA | |
| 17:04:29 | sean-k-mooney | i can try and find the downstream bz for virtio-blk again one sec | |
| 17:04:38 | geguileo | sean-k-mooney: I have the downstream BZ | |
| 17:04:45 | geguileo | sean-k-mooney: I'm just telling you I can't make it work | |
| 17:04:47 | sean-k-mooney | for qemu supprot of virtio-blk | |
| 17:04:51 | geguileo | (maybe I'm dumb) | |
| 17:04:54 | sean-k-mooney | *trim with virtio--blk | |
| 17:05:06 | geguileo | sean-k-mooney: sure, it says it supports it... I can't make it work without using IDE, SCSI or SATA | |
| 17:05:08 | sean-k-mooney | well it worked in our ci | |
| 17:05:16 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/879077/1 | |
| 17:05:20 | geguileo | sean-k-mooney: did you actually check that the size was reduced? | |
| 17:05:44 | sean-k-mooney | no but if its not that a qemu bug | |
| 17:05:55 | geguileo | sean-k-mooney: but then the log should remain | |
| 17:06:13 | sean-k-mooney | why it would not be correct | |
| 17:06:41 | geguileo | just because fstrim says it has freed space within the guest OS it doesn't mean that it has actually hapened | |
| 17:07:11 | sean-k-mooney | sure there are several layeres at play here | |
| 17:07:24 | geguileo | sean-k-mooney: oh, I'm sure there are many, but they are beyond my expertise | |
| 17:07:39 | geguileo | I'm just reporting as a storage guy saying, don't know why I can't make it work | |