| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-10-10 | |||
| 19:11:53 | tobias-urdin | im just a little bit scared about the impact for large instances removing that entirely but i guess i dont have a choice, i felt like scaling the value for cgroupv2 in cputune.shares and introducing a new cputune.weight was more elegant, anyway | |
| #openstack-nova - 2023-10-11 | |||
| 10:49:08 | whoami-rajat | bauzas, hey, i was planning to keep a cross project meeting with nova regarding the topic https://etherpad.opendev.org/p/caracal-ptg-cinder-planning#L50 | |
| 10:50:01 | whoami-rajat | bauzas, let me know when would be a good time for the same | |
| 11:37:51 | sean-k-mooney | elodilles: can i add https://review.opendev.org/c/openstack/nova/+/865334 and the following 5 patches to your review radar. tthey have all merged on xena so the wallaby backports are all that is left to close out those backport series | |
| 11:54:28 | elodilles | sean-k-mooney: ACK, will review them shortly | |
| 11:55:26 | bauzas | whoami-rajat: hey, sorry quite busy today, sorry for the late reply | |
| 11:55:35 | opendevreview | sean mooney proposed openstack/nova stable/wallaby: Add compute restart capability for libvirt func tests https://review.opendev.org/c/openstack/nova/+/865571 | |
| 11:55:41 | bauzas | whoami-rajat: I'm open to any dates, you're the first to ask :) | |
| 11:56:27 | bauzas | I'd prefer if we could stick with a common timeslot for both nova and cinder, but we can book another room if needed | |
| 11:57:05 | whoami-rajat | bauzas, great, sure, how about wednesday 1600 utc? | |
| 11:57:19 | bauzas | cool with me | |
| 11:58:36 | whoami-rajat | thanks, let me know if there are any topics from nova side for cross project and i will add it to our etherpad | |
| 11:58:40 | opendevreview | sean mooney proposed openstack/nova stable/wallaby: Add compute restart capability for libvirt func tests https://review.opendev.org/c/openstack/nova/+/865571 | |
| 11:59:17 | bauzas | whoami-rajat: cool, I'll mark it also in our etherpad | |
| 11:59:28 | bauzas | whoami-rajat: I'll leave you host the session :) | |
| 11:59:39 | bauzas | your etherpad, your room :D | |
| 11:59:57 | whoami-rajat | bauzas, sounds good to me! | |
| 12:01:01 | opendevreview | sean mooney proposed openstack/nova stable/wallaby: enable blocked VDPA move operations https://review.opendev.org/c/openstack/nova/+/865572 | |
| 12:01:02 | opendevreview | sean mooney proposed openstack/nova stable/wallaby: Reproducer for bug 1951656 https://review.opendev.org/c/openstack/nova/+/866157 | |
| 12:01:03 | opendevreview | sean mooney proposed openstack/nova stable/wallaby: Handle mdev devices in libvirt 7.7+ https://review.opendev.org/c/openstack/nova/+/866158 | |
| 12:01:38 | sean-k-mooney | elodilles: there was a pep8 issue in one of the patches in the middle so i just fixed that and cherrypicked the follow on patches ^ | |
| 12:01:52 | sean-k-mooney | so we will need to wait for ci in anycase so no rush | |
| 12:02:38 | sean-k-mooney | the first 2 should be fine the 3rd patch and an extra new line | |
| 12:10:01 | elodilles | sean-k-mooney: OK, i'll add it then to my TODO :) it's a big nice pack of backports as i see, fixing multiple bugs if i'm not mistaken :) | |
| 12:12:50 | sean-k-mooney | yep they have been sitting for a while so the new ci runs probaly dont hurt | |
| 12:13:13 | sean-k-mooney | they are all related to device passthough so vdpa+sriov+vgpu | |
| 12:16:40 | elodilles | sean-k-mooney: ACK, i had a commit message nit for the 2nd patch until zuul is back with the results for the rests | |
| 12:31:55 | sean-k-mooney | elodilles: if you like i can adress that and just rebase all the rest on the tip of stable | |
| 12:32:13 | sean-k-mooney | i intentonlly did not chagne the orgianl base so there are a littel behind | |
| 12:32:30 | sean-k-mooney | they should not have merge conflict but it never hurts to actully have them rebased | |
| 12:35:38 | elodilles | sean-k-mooney: ACK, then please do o:) | |
| 13:03:16 | opendevreview | sean mooney proposed openstack/nova stable/wallaby: refactor: remove duplicated logic https://review.opendev.org/c/openstack/nova/+/865334 | |
| 13:03:17 | opendevreview | sean mooney proposed openstack/nova stable/wallaby: Record SRIOV PF MAC in the binding profile https://review.opendev.org/c/openstack/nova/+/865336 | |
| 13:03:18 | opendevreview | sean mooney proposed openstack/nova stable/wallaby: Add compute restart capability for libvirt func tests https://review.opendev.org/c/openstack/nova/+/865571 | |
| 13:03:19 | opendevreview | sean mooney proposed openstack/nova stable/wallaby: enable blocked VDPA move operations https://review.opendev.org/c/openstack/nova/+/865572 | |
| 13:03:20 | opendevreview | sean mooney proposed openstack/nova stable/wallaby: Reproducer for bug 1951656 https://review.opendev.org/c/openstack/nova/+/866157 | |
| 13:03:21 | opendevreview | sean mooney proposed openstack/nova stable/wallaby: Handle mdev devices in libvirt 7.7+ https://review.opendev.org/c/openstack/nova/+/866158 | |
| 13:03:39 | sean-k-mooney | ok commit message updateand and trivial rebase to the tip of stable | |
| 13:04:02 | sean-k-mooney | as i said no rush they have just been sitting ther a while | |
| 13:08:56 | elodilles | sean-k-mooney: thanks! added them to my TODO! | |
| 13:21:26 | sean-k-mooney | dansmith: im +2w on https://review.opendev.org/q/topic:extra-rename-check | |
| 13:21:57 | sean-k-mooney | all nice enhancements | |
| 13:31:17 | bauzas | shit, forgot to do my round of reviews | |
| 13:38:45 | dansmith | sean-k-mooney: thanks | |
| 14:04:50 | opendevreview | Pavlo Shchelokovskyy proposed openstack/nova master: Update disk.info in finish_migration https://review.opendev.org/c/openstack/nova/+/897842 | |
| 14:48:05 | opendevreview | Dan Smith proposed openstack/nova master: Compile mdev samples for nova-next https://review.opendev.org/c/openstack/nova/+/897708 | |
| 15:41:33 | opendevreview | Dan Smith proposed openstack/nova master: Compile mdev samples for nova-next https://review.opendev.org/c/openstack/nova/+/897708 | |
| 15:48:03 | opendevreview | Merged openstack/nova master: Sanity check that new hosts have no instances https://review.opendev.org/c/openstack/nova/+/897076 | |
| 17:12:58 | opendevreview | Dan Smith proposed openstack/nova master: Compile mdev samples for nova-next https://review.opendev.org/c/openstack/nova/+/897708 | |
| #openstack-nova - 2023-10-12 | |||
| 03:35:28 | opendevreview | sean mooney proposed openstack/nova master: refactor numa claims https://review.opendev.org/c/openstack/nova/+/898056 | |
| 03:37:40 | opendevreview | sean mooney proposed openstack/nova master: refactor numa claims https://review.opendev.org/c/openstack/nova/+/898056 | |
| 04:29:02 | opendevreview | sean mooney proposed openstack/nova master: imporve nova object logging https://review.opendev.org/c/openstack/nova/+/898057 | |
| 09:08:20 | tobias-urdin | sean-k-mooney: when you have some seconds, based on https://review.opendev.org/c/openstack/nova/+/824048 and the issue from libvirt down expressed in https://gitlab.com/libvirt/libvirt/-/issues/161 I assume it would be thrown upon implementing the workaround in the nova layer by checking for cgroupsv1/cgroupsv2 and reintroducing the default | |
| 09:08:20 | tobias-urdin | cputune.shares value based on that (lets ignore the live migration part of it now and just focus on "having the correct value" based on the crazy decision made in libvirt).. any thoughts on that? | |
| 09:40:08 | opendevreview | Sylvain Bauza proposed openstack/nova master: WIP: add mtty support for vgpus https://review.opendev.org/c/openstack/nova/+/898100 | |
| 12:46:31 | bauzas | dansmith: I know that's too early for you but my devstack fails on your mtty patch https://paste.opendev.org/show/bwpHZPMuePifFk0JWi7v/ (just reply when you want) | |
| 12:46:38 | bauzas | sean-k-mooney: ^ | |
| 12:47:53 | bauzas | actually the whole stack is here https://paste.opendev.org/show/bLirDAMuGmxhLLod0S9p/ | |
| 12:49:19 | bauzas | stack@sbauza-dev2:~/devstack$ cat /etc/lsb-release | |
| 12:49:19 | bauzas | DISTRIB_ID=Ubuntu | |
| 12:49:19 | bauzas | DISTRIB_RELEASE=22.04 | |
| 12:49:19 | bauzas | DISTRIB_CODENAME=jammy | |
| 12:49:20 | bauzas | DISTRIB_DESCRIPTION="Ubuntu 22.04.1 LTS" | |
| 12:49:35 | bauzas | stack@sbauza-dev2:~/devstack$ uname -r | |
| 12:49:35 | bauzas | 5.15.0-1037-kvm | |
| 13:19:21 | frickler | bauzas: is there some linux-modules-extra pkg for that kernel? | |
| 13:22:22 | dansmith | bauzas: looks like maybe it has module symbols from a different kernel | |
| 13:22:37 | dansmith | since it failed to create /opt/stack/kernel .. did you run it once, upgrade kernel, run again? | |
| 13:36:29 | bauzas | dansmith: it didn't failed to create kernel | |
| 13:36:37 | bauzas | (sorry was on a meeting) | |
| 13:36:39 | dansmith | didn't fail? | |
| 13:36:52 | dansmith | ++/opt/stack/nova/devstack/lib/mdev_samples:compile_mdev_samples:19 mkdir /opt/stack/kernel | |
| 13:36:52 | dansmith | mkdir: cannot create directory ‘/opt/stack/kernel’: File exists | |
| 13:36:53 | bauzas | the kernel modules, yeah | |
| 13:36:57 | bauzas | ah | |
| 13:37:14 | bauzas | probably because devstack failed before on OVS | |
| 13:37:25 | bauzas | but I hadn't upgraded my kernel | |
| 13:37:38 | dansmith | that's what I just said yeah :) | |
| 13:37:51 | dansmith | blow that away and try again, I'll add cleaning of that to clean.sh | |
| 13:37:55 | bauzas | want me to unstack, delete the kernel dir and stack ? | |
| 13:38:02 | bauzas | ok | |
| 13:38:04 | bauzas | I can try | |
| 13:38:24 | bauzas | it's a pure compute node, no controllers on it so should be small and quick | |
| 13:39:33 | dansmith | well, you could just run the plugin itself again (after cleaning the kernel dir) but re-stacking would be better | |
| 13:39:59 | bauzas | yeah | |
| 13:40:15 | bauzas | if that fails again, I'll run the plugin itself | |
| 13:40:35 | bauzas | dansmith: fwiw, I started to provide the mtty change on nova | |
| 13:41:21 | bauzas | I could have missed something (and I know I did, since I haven't modified the method that checks the mdevs when we restart, but meh not a problem for testing) but I think it's a start | |
| 13:42:14 | dansmith | cool | |
| 13:42:50 | bauzas | dansmith: failing again https://paste.opendev.org/show/bFTPQR1VBSJNQj2jrknG/ | |
| 13:43:10 | bauzas | this time, this is linking the modules | |
| 13:43:18 | bauzas | but still failing on the definitions | |
| 13:43:42 | dansmith | bauzas: show me "dpkg -l | grep linux" and "ls /usr/src" | |
| 13:43:49 | bauzas | cool | |
| 13:44:20 | dansmith | also uname -r and "ls -l /lib/modules/*/build" | |
| 13:44:45 | dansmith | worked fine for me and sean-k-mooney, but it gets weird if you have a bunch of mismatched kernels and things installed sometimes | |
| 13:45:54 | bauzas | dansmith: there you go https://paste.opendev.org/show/bSXwvpbc6Z96awnWuIRS/ | |
| 13:46:27 | bauzas | I can reclone the machine and reinstall devstack again | |
| 13:46:41 | bauzas | I mean, if my env is dirty, meh | |
| 13:46:50 | bauzas | probably better to start with fresh things | |