| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-02-17 | |||
| 12:50:13 | bauzas | https://f89b63837ed61d9739ac-76da2058f7382f685ecdff725a6049b3.ssl.cf1.rackcdn.com/821228/7/check/nova-ceph-multistore/6617ac9/job-output.txt | |
| 12:50:21 | bauzas | despite the image being 0.5.2 | |
| 12:56:23 | kashyap | bauzas: So you're saying the image is 0.5.2, but it logs it as 0.6.1? | |
| 13:22:33 | bauzas | kashyap: the guest console, yes | |
| 13:22:59 | bauzas | and it uses the dhcpcd client for getting the lease | |
| 13:24:51 | bauzas | "use Ubuntu 22.04 (5.15) kernels for aarch64, arm, ppc64le, x86_64" | |
| 13:24:55 | opendevreview | Samuel Kunkel proposed openstack/nova master: fix: handle missing img properties https://review.opendev.org/c/openstack/nova/+/874248 | |
| 13:25:15 | bauzas | compared to 0.5.0 "Linux kernel 5.3.0-26.28~18.04.1 for aarch64, arm, i386, ppc64le, x86_64 (Ubuntu 18.04 HWE)" | |
| 13:25:52 | samuelkunkel[m] | please have a look and provide me some feedback if the approach is correct. With that "fix" everything works properly for deploying AMD SEV | |
| 13:25:59 | bauzas | so, yeah the guest that was booted was using 0.6.1 | |
| 13:26:02 | bauzas | 2023-02-17 09:45:15.450272 | controller | [ 0.000000] Linux version 5.15.0-48-generic (buildd@lcy02-amd64-080) (gcc (Ubuntu 11.2.0-19ubuntu1) 11.2.0, GNU ld (GNU Binutils for Ubuntu) 2.38) #54-Ubuntu SMP Fri Aug 26 13:26:29 UTC 2022 (Ubuntu 5.15.0-48.54-generic 5.15.53) | |
| 13:26:36 | bauzas | samuelkunkel[m]: I'm not really knowing about AMD SEV but I'll try to review your change | |
| 13:29:15 | kashyap | bauzas: Hmm, thx for the kernel comparision. | |
| 13:30:37 | samuelkunkel[m] | thanks bauzas I am also not that familiar but I tried to describe the implications | |
| 13:31:54 | kashyap | bauzas: I see the code from samuelkunkel[m] is not in core SEV functionality itself, but in sched/utils | |
| 13:32:27 | kashyap | samuelkunkel[m]: What triggerred this change? Would be useful if you could write-up a small bug report / tracker | |
| 13:34:05 | samuelkunkel[m] | I can write up a bug report | |
| 13:34:46 | samuelkunkel[m] | yes, its with in the scheduler utils but its triggert by the AMD SEV implementation | |
| 13:35:15 | samuelkunkel[m] | This fix "could" (in a somewhat similar way) also be moved to the memory encryption section | |
| 13:35:29 | samuelkunkel[m] | e.g. directly into _check_mem_encryption_uses_uefi_image where its triggered | |
| 13:37:53 | kashyap | samuelkunkel[m]: I see, then what made you move it scheduler/utils? If it could be dealt with in nova/virt/hardware.py) | |
| 13:38:18 | kashyap | Thanks for writing it! (And link it in the commit message on your next respin, plese) | |
| 13:41:18 | opendevreview | Kashyap Chamarthy proposed openstack/nova stable/wallaby: Add a workaround to skip hypervisor version check on LM https://review.opendev.org/c/openstack/nova/+/851206 | |
| 13:41:19 | opendevreview | Kashyap Chamarthy proposed openstack/nova stable/wallaby: libvirt: At start-up rework compareCPU() usage with a workaround https://review.opendev.org/c/openstack/nova/+/872402 | |
| 13:59:07 | opendevreview | Andre Aranha proposed openstack/nova stable/yoga: [stable-only] Test setting the nova job to centos-9-stream https://review.opendev.org/c/openstack/nova/+/860087 | |
| 14:00:15 | opendevreview | Andre Aranha proposed openstack/nova stable/yoga: [stable-only] Test setting the nova job to centos-9-stream https://review.opendev.org/c/openstack/nova/+/860087 | |
| 14:00:58 | opendevreview | Andre Aranha proposed openstack/nova stable/yoga: [stable-only] Test setting the nova job to centos-9-stream https://review.opendev.org/c/openstack/nova/+/860087 | |
| 14:27:06 | samuelkunkel[m] | https://bugs.launchpad.net/nova/+bug/2007697 | |
| 14:27:34 | samuelkunkel[m] | I hope I got everything. I will try to move the fix into the virt/hardware.py. It should not a big issue to just exclude the uefi check if no image informations are present at all | |
| 15:22:00 | bauzas | was fucking close to get merged https://review.opendev.org/c/openstack/nova/+/872413 :cries: | |
| 15:23:37 | opendevreview | Samuel Kunkel proposed openstack/nova master: fix: handle missing img properties https://review.opendev.org/c/openstack/nova/+/874248 | |
| 15:24:25 | gibi | bauzas: I will look at the etherpad during the weekend and try to recheck-in thigns we already approved | |
| 15:24:27 | opendevreview | Samuel Kunkel proposed openstack/nova master: fix: handle missing img properties https://review.opendev.org/c/openstack/nova/+/874248 | |
| 15:24:53 | bauzas | gibi: meh, don't do this, you'll waste your weekend time | |
| 15:25:19 | bauzas | I'm trying to merge changes since one week, and about only one change is done | |
| 15:25:36 | gibi | bauzas: I will do it as part of any slack time, I won't reorg my weekend around it :) | |
| 15:26:29 | bauzas | meh | |
| 15:27:28 | opendevreview | Samuel Kunkel proposed openstack/nova master: fix: amd-sev handle missing img properties https://review.opendev.org/c/openstack/nova/+/874264 | |
| 15:28:12 | samuelkunkel[m] | hope I got it now everything. Reviews appreciated, this approach to the fix also works properly in our setup | |
| 15:31:02 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/train: Add functional tests to reproduce bug #1960412 https://review.opendev.org/c/openstack/nova/+/873579 | |
| 15:40:44 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/train: Test aborting queued live migration https://review.opendev.org/c/openstack/nova/+/873578 | |
| 16:03:38 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/train: Test aborting queued live migration https://review.opendev.org/c/openstack/nova/+/873578 | |
| 16:47:34 | sean-k-mooney | samuelkunkel[m] i dont have time to review that today but i dont think we can just skip it like that | |
| 16:48:51 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/train: Test aborting queued live migration https://review.opendev.org/c/openstack/nova/+/873578 | |
| 16:56:05 | samuelkunkel[m] | I thought so. No worries - for now it runs just patched in our environment | |
| 16:56:23 | samuelkunkel[m] | just let me know how I can make it better. Next week is enough :) | |
| 19:09:54 | opendevreview | Artom Lifshitz proposed openstack/nova master: DNM: Try out new flake8 https://review.opendev.org/c/openstack/nova/+/874296 | |
| 19:57:55 | opendevreview | melanie witt proposed openstack/nova master: doc: Add details about the behavior of server delete https://review.opendev.org/c/openstack/nova/+/874188 | |
| 20:43:30 | fnordahl | fungi: apologies for the tardy response, you would need to add jammy-proposed to your sources.list to get the package before it is promoted to -updates, yes. | |
| 20:44:12 | fnordahl | fungi: once a package is in -proposed it will normally be promoted to -updates within a week or two, but due to the interdependencies between OVS and OVN this specific package will most likely take a bit longer. | |
| 20:46:11 | fnordahl | fungi: once https://launchpadlibrarian.net/646504743/openvswitch_2.17.5-0ubuntu0.22.04.1_source.changes is accepted into -proposed we should have both packages in -updates within a couple of weeks. | |
| 20:46:46 | fnordahl | fungi: the bug to track for that one is https://bugs.launchpad.net/ubuntu/+source/openvswitch/+bug/2003060 | |
| 20:50:57 | fungi | fnordahl: noted, thanks for the details! | |
| 20:51:47 | fungi | so anyone looking for a sooner resolution to the ovn-related job failures may want to have devstack add jammy-proposed to the sources.list temporarily | |
| #openstack-nova - 2023-02-18 | |||
| 00:20:39 | opendevreview | David Hill proposed openstack/nova master: Change maximum value of pcie-ports to 32 https://review.opendev.org/c/openstack/nova/+/871305 | |
| 00:43:21 | opendevreview | David Hill proposed openstack/nova master: Change maximum value of pcie-ports to 32 https://review.opendev.org/c/openstack/nova/+/871305 | |
| 02:21:43 | gmann | elodilles: anything you want to be updated here? https://review.opendev.org/c/openstack/osc-placement/+/869753 | |
| 08:28:33 | gibi | bauzas: I rerechecked the power mgmt series I think everything else we approved is merged | |
| 12:34:10 | opendevreview | Merged openstack/nova master: libvirt: let CPUs be power managed https://review.opendev.org/c/openstack/nova/+/821228 | |
| 12:34:19 | opendevreview | Merged openstack/nova master: Enable cpus when an instance is spawning https://review.opendev.org/c/openstack/nova/+/868237 | |
| 15:56:12 | prometheanfire | looks like oslo.log 5.1.0 is causing fails for nova https://review.opendev.org/873390 | |
| 23:10:55 | opendevreview | Merged openstack/nova master: fup: support evacuate target state https://review.opendev.org/c/openstack/nova/+/872413 | |
| #openstack-nova - 2023-02-19 | |||
| 01:56:27 | opendevreview | Takashi Natsume proposed openstack/nova master: doc: mark the max microversion for 2023.1 Antelope https://review.opendev.org/c/openstack/nova/+/874103 | |
| 09:11:14 | gibi | bauzas: power mgmt series is merged. \o/ | |
| 20:26:17 | opendevreview | Tobias Urdin proposed openstack/nova stable/yoga: Reproducer for bug 1951656 https://review.opendev.org/c/openstack/nova/+/866153 | |
| 20:26:27 | opendevreview | Tobias Urdin proposed openstack/nova stable/yoga: Handle mdev devices in libvirt 7.7+ https://review.opendev.org/c/openstack/nova/+/866154 | |
| 20:26:43 | opendevreview | Tobias Urdin proposed openstack/nova stable/xena: Reproducer for bug 1951656 https://review.opendev.org/c/openstack/nova/+/866155 | |
| 20:26:47 | opendevreview | Tobias Urdin proposed openstack/nova stable/xena: Handle mdev devices in libvirt 7.7+ https://review.opendev.org/c/openstack/nova/+/866156 | |
| #openstack-nova - 2023-02-20 | |||
| 06:13:46 | opendevreview | Amit Uniyal proposed openstack/nova master: WIP: Allow swap resize from non-zero to zero https://review.opendev.org/c/openstack/nova/+/857339 | |
| 07:23:26 | opendevreview | Amit Uniyal proposed openstack/nova master: WIP: Allow swap resize from non-zero to zero https://review.opendev.org/c/openstack/nova/+/857339 | |
| 08:27:30 | bauzas | gibi: thanks a lot | |
| 08:30:38 | bauzas | gibi: I wonder | |
| 08:31:21 | bauzas | gibi: maybe the gate was having way less changes to verify, so we got a good enviromnent without race conditions | |
| 08:32:01 | gibi | probably less load on the unerdlying cloud infra made it a smoother run | |
| 08:37:53 | bauzas | yup | |
| 08:38:03 | bauzas | gibi: anyway, you made my day :) | |
| 08:38:25 | bauzas | I'm quite crying :p | |
| 08:39:13 | bauzas | (of joy, heh) | |
| 08:43:18 | Uggla | o/ | |
| 08:59:35 | bauzas | Uggla: welcome back \o | |
| 09:00:19 | Uggla | bauzas, hi, I have just seen you and Gibi merged power management ! cool ! | |
| 09:03:02 | gibi | it was an easy win then. I'm happy that you are happy :) | |
| 09:05:07 | bauzas | gibi: I wouldn't say "it was an easy win" :D | |
| 09:05:11 | bauzas | for you, surely | |
| 09:05:20 | bauzas | for me... well, Gerrit knows my comments anyway | |
| 10:04:52 | samuelkunkel[m] | thanks bauzas for linking the MR to the bug. Somehow I abandoned one MR and created basically the same instead? | |
| 10:04:52 | samuelkunkel[m] | https://review.opendev.org/c/openstack/nova/+/874248 vs https://review.opendev.org/c/openstack/nova/+/874264 | |
| 10:05:10 | samuelkunkel[m] | I will work on a recreational test in the next few days | |
| 10:05:22 | bauzas | samuelkunkel[m]: doing things now, but I'll add some other comments | |
| 10:07:12 | samuelkunkel[m] | Just wanted to mentioned, because I made a mess of my pull requests you may linked the abandoned one... sorry | |
| 10:07:22 | samuelkunkel[m] | s/mentioned/mention/ | |
| 11:09:54 | opendevreview | Rajesh Tailor proposed openstack/nova master: Fix case-sensitivity for metadata keys https://review.opendev.org/c/openstack/nova/+/873901 | |
| 11:16:28 | opendevreview | Amit Uniyal proposed openstack/nova master: WIP: Allow swap resize from non-zero to zero https://review.opendev.org/c/openstack/nova/+/857339 | |
| 13:14:38 | opendevreview | Jorge San Emeterio proposed openstack/nova master: WIP: Look for cpu controller on cgroups v2 https://review.opendev.org/c/openstack/nova/+/873127 | |
| 13:18:59 | tobias-urdin | backports for the mdev naming change in newer libvirts looks ok and just needs reviews, zed backport https://review.opendev.org/c/openstack/nova/+/866152 see parent and all cherry-picks for yoga and xena | |
| 13:19:01 | tobias-urdin | bauzas: ^ | |
| 13:19:46 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/train: Test aborting queued live migration https://review.opendev.org/c/openstack/nova/+/873578 | |
| 13:50:42 | bauzas | tobias-urdin: ack, I'll try to help | |
| 14:53:32 | tobias-urdin | bauzas: thanks, and it's your backports so thanks for fixing that :) | |
| 15:16:54 | bauzas | tobias-urdin: I +2d the zed patches, we'll go thru the backports one after the others | |