| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-06-20 | |||
| 16:15:10 | bauzas | #link https://lists.openstack.org/pipermail/openstack-discuss/2023-June/034152.html | |
| 16:15:20 | bauzas | in case you haven't read it yet :) | |
| 16:15:48 | bauzas | we won't do a retrospective, we don't have a quorum here :) | |
| 16:15:58 | bauzas | and actually, all the folks around were on the Summit too :p | |
| 16:16:11 | bauzas | so, next topic | |
| 16:17:27 | gibi | :) | |
| 16:18:16 | bauzas | #topic Review priorities | |
| 16:18:24 | bauzas | #link https://review.opendev.org/q/status:open+(project:openstack/nova+OR+project:openstack/placement+OR+project:openstack/os-traits+OR+project:openstack/os-resource-classes+OR+project:openstack/os-vif+OR+project:openstack/python-novaclient+OR+project:openstack/osc-placement)+(label:Review-Priority%252B1+OR+label:Review-Priority%252B2) | |
| 16:18:29 | bauzas | #info As a reminder, cores eager to review changes can +1 to indicate their interest, +2 for committing to the review | |
| 16:18:34 | bauzas | #topic Stable Branches | |
| 16:18:50 | bauzas | I haven't seen elodilles_pto since a while and his nick tells me something | |
| 16:19:51 | bauzas | maybe "pto" means something like "people to obligate" :) | |
| 16:19:59 | bauzas | but meh | |
| 16:20:08 | bauzas | let's say he's not here | |
| 16:20:34 | bauzas | so I won't tell anything about the stable branches | |
| 16:20:38 | bauzas | maybe just | |
| 16:20:44 | bauzas | #info stable branch status / gate failures tracking etherpad: https://etherpad.opendev.org/p/nova-stable-branch-ci | |
| 16:20:51 | bauzas | but I also have | |
| 16:20:56 | bauzas | #info train-eol patch proposed https://review.opendev.org/c/openstack/releases/+/885365 | |
| 16:21:04 | bauzas | so, there was a question in ^ | |
| 16:21:30 | bauzas | should we just abandon the existing changes in this branch ? | |
| 16:21:49 | bauzas | gibi: would be nice if you could vote about it | |
| 16:23:03 | bauzas | ok, last topic | |
| 16:23:17 | bauzas | #topic Open discussion | |
| 16:23:24 | bauzas | (bauzas) Specless blueprint approval request for https://blueprints.launchpad.net/nova/+spec/num-instances-weigher | |
| 16:23:40 | bauzas | given we don't have quorum today, I'll ask him again next week | |
| 16:24:00 | bauzas | I'm done for me, anything else? | |
| 16:25:20 | gibi | I have nothing extra for today | |
| 16:26:09 | bauzas | ok, so, | |
| 16:26:13 | bauzas | thanks folks | |
| 16:26:15 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2023/nova.2023-06-20-16.01.log.html | |
| 16:26:15 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2023/nova.2023-06-20-16.01.txt | |
| 16:26:15 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2023/nova.2023-06-20-16.01.html | |
| 16:26:15 | opendevmeet | Meeting ended Tue Jun 20 16:26:15 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 16:26:15 | bauzas | #endmeeting | |
| 16:27:18 | bauzas | was quick and silent :) | |
| 16:28:59 | Uggla_ | yep but nothing special this week I guess | |
| 16:34:21 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (objects) https://review.opendev.org/c/openstack/nova/+/839401 | |
| 16:34:21 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (db) https://review.opendev.org/c/openstack/nova/+/831193 | |
| 16:34:24 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (drivers and compute manager part) https://review.opendev.org/c/openstack/nova/+/833090 | |
| 16:34:24 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (manila abstraction) https://review.opendev.org/c/openstack/nova/+/831194 | |
| 16:34:25 | opendevreview | ribaudr proposed openstack/nova master: Deletion of associated share mappings on instance deletion https://review.opendev.org/c/openstack/nova/+/881472 | |
| 16:34:25 | opendevreview | ribaudr proposed openstack/nova master: Mounting the shares as part of the initialization process https://review.opendev.org/c/openstack/nova/+/880075 | |
| 16:34:26 | opendevreview | ribaudr proposed openstack/nova master: Add share_info parameter to reboot method for each driver (driver part) https://review.opendev.org/c/openstack/nova/+/854823 | |
| 16:34:26 | opendevreview | ribaudr proposed openstack/nova master: Add metadata for shares https://review.opendev.org/c/openstack/nova/+/850500 | |
| 16:34:27 | opendevreview | ribaudr proposed openstack/nova master: Support rebooting an instance with shares (compute manager part) https://review.opendev.org/c/openstack/nova/+/854824 | |
| 16:34:28 | opendevreview | ribaudr proposed openstack/nova master: Support resuming an instance with shares (compute manager part) https://review.opendev.org/c/openstack/nova/+/860285 | |
| 16:34:28 | opendevreview | ribaudr proposed openstack/nova master: Add share_info parameter to resume method for each driver (driver part) https://review.opendev.org/c/openstack/nova/+/860284 | |
| 16:34:30 | opendevreview | ribaudr proposed openstack/nova master: Support rescuing an instance with shares (driver part) https://review.opendev.org/c/openstack/nova/+/860287 | |
| 16:34:30 | opendevreview | ribaudr proposed openstack/nova master: Add helper methods to rescue/unrescue shares https://review.opendev.org/c/openstack/nova/+/860286 | |
| 16:34:32 | opendevreview | ribaudr proposed openstack/nova master: Allow to mount manila share using Cephfs protocol https://review.opendev.org/c/openstack/nova/+/883862 | |
| 16:34:32 | opendevreview | ribaudr proposed openstack/nova master: Support rescuing an instance with shares (compute manager part) https://review.opendev.org/c/openstack/nova/+/860288 | |
| 16:34:34 | opendevreview | ribaudr proposed openstack/nova master: Check shares support (only API exception) https://review.opendev.org/c/openstack/nova/+/885752 | |
| 16:34:34 | opendevreview | ribaudr proposed openstack/nova master: Check shares support (compute manager) https://review.opendev.org/c/openstack/nova/+/885751 | |
| 16:34:36 | opendevreview | ribaudr proposed openstack/nova master: Check shares support (API) https://review.opendev.org/c/openstack/nova/+/850499 | |
| 16:34:36 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (API) https://review.opendev.org/c/openstack/nova/+/836830 | |
| 16:34:38 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach notification https://review.opendev.org/c/openstack/nova/+/850501 | |
| 16:34:38 | opendevreview | ribaudr proposed openstack/nova master: Add helper methods to attach/detach shares https://review.opendev.org/c/openstack/nova/+/885753 | |
| 16:34:40 | opendevreview | ribaudr proposed openstack/nova master: Add shares to InstancePayload https://review.opendev.org/c/openstack/nova/+/851029 | |
| 16:34:40 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach notification https://review.opendev.org/c/openstack/nova/+/851028 | |
| 16:34:42 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach_error notification https://review.opendev.org/c/openstack/nova/+/860283 | |
| 16:34:42 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach_error notification https://review.opendev.org/c/openstack/nova/+/860282 | |
| 16:34:44 | opendevreview | ribaudr proposed openstack/nova master: Add virt/libvirt error test cases https://review.opendev.org/c/openstack/nova/+/852087 | |
| 16:34:44 | opendevreview | ribaudr proposed openstack/nova master: Add libvirt test to ensure metadata are working. https://review.opendev.org/c/openstack/nova/+/852086 | |
| 16:34:46 | opendevreview | ribaudr proposed openstack/nova master: Allow config to support virtiofs (driver) https://review.opendev.org/c/openstack/nova/+/886522 | |
| 16:34:46 | opendevreview | ribaudr proposed openstack/nova master: Docs about Manila shares API usage https://review.opendev.org/c/openstack/nova/+/871642 | |
| 16:35:25 | gibi | it was almost like the PTG :) | |
| 16:36:15 | sean-k-mooney | ouch | |
| 16:36:30 | sean-k-mooney | i can feel the heat form that burn form here | |
| 16:43:28 | gibi | :) | |
| 16:54:51 | dansmith | heh | |
| 19:34:30 | opendevreview | melanie witt proposed openstack/nova master: database: Archive parent and child rows "trees" one at a time https://review.opendev.org/c/openstack/nova/+/877056 | |
| 20:04:56 | opendevreview | melanie witt proposed openstack/nova master: database: Archive parent and child rows "trees" one at a time https://review.opendev.org/c/openstack/nova/+/877056 | |
| #openstack-nova - 2023-06-21 | |||
| 10:11:46 | opendevreview | Jakub Skunda proposed openstack/nova master: Drop Fedora support https://review.opendev.org/c/openstack/nova/+/886596 | |
| 11:37:11 | opendevreview | Amit Uniyal proposed openstack/nova-specs master: Adds cleanup to remove dangling volumes https://review.opendev.org/c/openstack/nova-specs/+/878757 | |
| 11:47:50 | noonedeadpunk | hey folks, I've found this wiki page lately, and it looks like this should be supported by nova, as the `migrate_set_capability` is a command that's passed to the qemu monitor https://wiki.qemu.org/Features/Migration-Multiple-fds | |
| 11:48:16 | noonedeadpunk | So I was wondering, if that's smth that should be supported by nova, or I can set it in some other way as well? | |
| 12:02:39 | noonedeadpunk | And also according to https://bugzilla.redhat.com/show_bug.cgi?id=1968540 it seems there's another way around in progress now. But maybe it's worth mentioning in docs as a note about performance hit by using TLS for migrations? | |
| 13:43:58 | bauzas | noonedeadpunk: sorry was barely paying attention to the chan | |
| 13:46:09 | bauzas | noonedeadpunk: so, basically you have concerns about migration performance ? | |
| 13:48:03 | noonedeadpunk | well, yes, as they can hardly finish with 1.2gbitp/s | |
| 13:48:36 | noonedeadpunk | It's slightly different on different hardware, but basically it's always single thread now that's handling migration/encryption | |
| 13:50:01 | noonedeadpunk | I've tried to check if gnutls 3.7.3 (default for ubuntu 22.04) helps (as it has this merged) https://gitlab.com/gnutls/gnutls/-/commit/6462916d2f6810409d5da1e13c4a0720f412c166 | |
| 13:50:23 | noonedeadpunk | But seems that it's not enough to have that only in gnutls, apparently qemu should have smth as well | |
| 13:51:06 | noonedeadpunk | But was also wondering if anybody heard anything about x-multiple-fds in qemu capabilities | |
| 13:54:52 | bauzas | I'm not really a SME of this matter | |
| 13:54:59 | bauzas | maybe kashyap may help | |
| 13:56:14 | bauzas | kashyap: thanks, tl;dr noonedeadpunk has some concerns by the migration TLS performance due to a QEMU single threading | |
| 13:56:21 | kashyap | noonedeadpunk: Before I say anything further, please note that the "x-" prefix in QMP commands (like "x-multiple-fds") means it's experimental | |
| 13:56:34 | kashyap | That _said_: the multiple-fd work did come out of experimental, IIRC. Lemme check | |
| 13:56:52 | kashyap | bauzas: I see; I missed the earlier context; so would be good to know what kind of performance concerns are these | |
| 13:57:41 | noonedeadpunk | so using TLS for libvirt gives migration speed like 1.2gbitps - 3 gbitps depending on CPU, TCP live migrations are 20gbitps | |
| 13:57:54 | noonedeadpunk | (basically full link) | |
| 14:01:54 | kashyap | noonedeadpunk: Hmm, I mean I'd definitely expect some loss w/ built-in TLS -- as it involves the additional encryption over head -- but I don't have numbers off-hand as to what's "expected" | |
| 14:02:17 | kashyap | noonedeadpunk: Can you come to #virt (on OFTC) and ask there too? More libvirt migration experts hang out there | |
| 14:02:21 | noonedeadpunk | so diffference is quite dramatical | |
| 14:02:40 | noonedeadpunk | yes, sure | |
| 14:02:50 | kashyap | Yeah; it's almost 10 fold diff | |
| 14:03:02 | bauzas | noonedeadpunk: ohoh, good to know | |
| 14:07:50 | noonedeadpunk | enabling auto-convergance kinda solves issue, at least migrations do not fail | |