| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-03-20 | |||
| 14:22:29 | sean-k-mooney | the same thing happens if you have the power managemnt feature enabeld and the core is offline | |
| 14:23:00 | sean-k-mooney | qemu check for that and reports an error to libvirt which we then get and we rollback in the same way as if the cpu were not compatiable | |
| 15:43:28 | opendevreview | Merged openstack/nova stable/2024.1: [stable-only] Update .gitreview for stable/2024.1 https://review.opendev.org/c/openstack/nova/+/913631 | |
| 15:43:36 | opendevreview | Merged openstack/nova stable/2024.1: [stable-only] Update TOX_CONSTRAINTS_FILE for stable/2024.1 https://review.opendev.org/c/openstack/nova/+/913632 | |
| 16:19:55 | zigo | Who's trying to use systemctl in unit tests?!? | |
| 16:19:55 | zigo | https://buildd.debian.org/status/fetch.php?pkg=nova&arch=all&ver=2%3A29.0.0~rc1-1&stamp=1710857860&file=log | |
| 16:20:30 | zigo | Not a good idea, IMO. Or at least, this has to be explicited somewhere. | |
| 16:20:58 | clarkb | the upstream unittest jobs should remove sudo powers before running the tests. Unprivileged systemctl isn't the worst thing? | |
| 16:21:27 | zigo | It's not that... | |
| 16:21:28 | zigo | FileNotFoundError: [Errno 2] No such file or directory: 'systemctl' | |
| 16:21:37 | zigo | There's no systemd in the Debian buildd chroots ... | |
| 16:21:48 | zigo | Plus WTF is it even trying to do? | |
| 16:23:33 | clarkb | its checking if the system is running systemd and assumes if systemd is present then so is systemdctl | |
| 16:23:44 | zigo | sysdout, sysderr = processutils.execute("systemctl", | |
| 16:23:44 | zigo | "is-system-running", | |
| 16:23:44 | zigo | check_exit_code=[0, 1]) | |
| 16:23:49 | zigo | Well, no it's not, thanks ... :) | |
| 16:24:09 | clarkb | does seem like the sort of check that could catch exceptions and then mark no to systemd | |
| 16:24:31 | clarkb | zigo: there must be some systemd due to the checks just above that statement. | |
| 16:24:41 | clarkb | so I don't think it is accurate that the build chroots have no systemd | |
| 16:24:56 | clarkb | if psutil.Process(1).name() == "systemd" or os.path.exists(SYSTEMCTL_CHECK_PATH): | |
| 16:25:05 | zigo | I could add a depends on systemd... | |
| 16:25:11 | zigo | Not sure if that will help though. | |
| 16:25:25 | zigo | It does build fine on my laptop, just not on buildd. | |
| 16:25:32 | zigo | That's kind of the worse situation ! :) | |
| 16:25:52 | zigo | I probably will end up blacklisting yet another bunch of tests ... | |
| 16:27:53 | clarkb | I think changing that or to an and may fix it for you | |
| 16:52:48 | opendevreview | Merged openstack/nova master: Fix tipo https://review.opendev.org/c/openstack/nova/+/907208 | |
| 16:54:44 | opendevreview | ribaudr proposed openstack/nova master: Fix: Live migrating to a host with cpu_shared_set configured will now update the VM's configuration accordingly. https://review.opendev.org/c/openstack/nova/+/903706 | |
| 16:54:45 | opendevreview | ribaudr proposed openstack/nova master: Test live migration between hosts with differnet cpu_shared_sets https://review.opendev.org/c/openstack/nova/+/913744 | |
| 19:34:33 | opendevreview | sean mooney proposed openstack/nova master: enable numa live migration in the ceph job https://review.opendev.org/c/openstack/nova/+/913842 | |
| 19:37:06 | opendevreview | sean mooney proposed openstack/nova master: enable numa live migration in the ceph job https://review.opendev.org/c/openstack/nova/+/913842 | |
| 21:14:38 | opendevreview | sean mooney proposed openstack/nova master: imporve nova object logging https://review.opendev.org/c/openstack/nova/+/898057 | |
| 21:18:42 | opendevreview | sean mooney proposed openstack/nova master: refactor numa claims https://review.opendev.org/c/openstack/nova/+/898056 | |
| 21:24:23 | sean-k-mooney | artom: ^ those are my patches i sstarted when debuging the numa aware vsiwtch issue as an fyi | |
| 21:24:42 | sean-k-mooney | i just inverted the order so the logging patch is first and the refactor patch is second | |
| 21:25:16 | sean-k-mooney | melwitt: ^ i would prefer to use my logging patch instead of https://review.opendev.org/c/openstack/nova/+/907976 by the way just because its more systemtic and general | |
| 21:25:46 | sean-k-mooney | it would be nice to have your review feedback on that when you have time | |
| 21:30:08 | opendevreview | sean mooney proposed openstack/nova master: enable numa live migration in the ceph job https://review.opendev.org/c/openstack/nova/+/913842 | |
| 21:35:56 | artom | sean-k-mooney, ack. I've got a couple of appointment on Friday, but between them, if you're up for it, I'll load context from you | |
| 21:50:56 | sean-k-mooney | sure | |
| 21:51:01 | melwitt | sean-k-mooney: ack, ok | |
| #openstack-nova - 2024-03-21 | |||
| 10:01:34 | bauzas | elodilles: I did a couple of single-core approvals for the reno unmaintenained patches but please ping me if you find that I forgot some patch for other nova repos | |
| 10:06:38 | frickler | bauzas: osc-placement is also nova team, isn't it? https://review.opendev.org/c/openstack/osc-placement/+/911294 etc. | |
| 10:07:00 | bauzas | that's correct, I'll do my duty then | |
| 10:07:56 | frickler | ty | |
| 10:09:11 | bauzas | I wish we could have some flag in gerrit query to say 'flag:nova-related' instead of 'project:<something> OR project:somethingelse> :) | |
| 10:09:35 | bauzas | I just don't know how other services that have like 50 repos do this :) | |
| 10:12:03 | opendevreview | Merged openstack/os-vif master: reno: Update master for unmaintained/xena https://review.opendev.org/c/openstack/os-vif/+/911292 | |
| 10:12:04 | opendevreview | Merged openstack/os-vif master: reno: Update master for unmaintained/wallaby https://review.opendev.org/c/openstack/os-vif/+/911282 | |
| 10:12:05 | opendevreview | Merged openstack/os-vif master: reno: Update master for unmaintained/victoria https://review.opendev.org/c/openstack/os-vif/+/911272 | |
| 10:12:36 | elodilles | bauzas: ACK, thanks (i added review-priority+1 for them) | |
| 10:12:44 | opendevreview | Merged openstack/python-novaclient master: reno: Update master for unmaintained/wallaby https://review.opendev.org/c/openstack/python-novaclient/+/911288 | |
| 10:12:46 | opendevreview | Merged openstack/python-novaclient master: reno: Update master for unmaintained/victoria https://review.opendev.org/c/openstack/python-novaclient/+/911278 | |
| 10:12:47 | opendevreview | Merged openstack/python-novaclient master: reno: Update master for unmaintained/xena https://review.opendev.org/c/openstack/python-novaclient/+/911298 | |
| 10:13:05 | bauzas | we gotta get a pile of them ^ :) | |
| 10:13:52 | elodilles | (well, sorry, i was talking about this ones: 'Update master for stable/2024.1') | |
| 10:14:28 | opendevreview | Merged openstack/osc-placement master: reno: Update master for unmaintained/victoria https://review.opendev.org/c/openstack/osc-placement/+/911274 | |
| 10:16:54 | opendevreview | Merged openstack/osc-placement master: reno: Update master for unmaintained/xena https://review.opendev.org/c/openstack/osc-placement/+/911294 | |
| 10:16:55 | opendevreview | Merged openstack/osc-placement master: reno: Update master for unmaintained/wallaby https://review.opendev.org/c/openstack/osc-placement/+/911284 | |
| 10:18:57 | opendevreview | Merged openstack/nova master: reno: Update master for unmaintained/xena https://review.opendev.org/c/openstack/nova/+/911290 | |
| 10:19:04 | opendevreview | Sylvain Bauza proposed openstack/nova master: Update min support for Dalmatian https://review.opendev.org/c/openstack/nova/+/913890 | |
| 10:22:27 | opendevreview | Merged openstack/nova master: reno: Update master for unmaintained/victoria https://review.opendev.org/c/openstack/nova/+/911270 | |
| 10:24:27 | opendevreview | Merged openstack/nova master: reno: Update master for unmaintained/wallaby https://review.opendev.org/c/openstack/nova/+/911280 | |
| 12:23:42 | sean-k-mooney[m] | bauzas: you can create a dashboard in gerrit | |
| 12:24:15 | bauzas | sure, I already use two of them | |
| 12:24:47 | bauzas | but OK, maybe I should create some dashboard for all the nova repos | |
| 12:25:12 | bauzas | (at least for the release bot ones :) | |
| 13:40:15 | bauzas | dansmith: gmann: I don't know what's the plan with grenade now we branched 2024.1 but I proposed https://review.opendev.org/c/openstack/grenade/+/913892 | |
| 13:40:29 | bauzas | oh, frickler gave me a procedural -2, cool | |
| 13:43:53 | opendevreview | Merged openstack/nova stable/2023.2: pwr mgmt: make API into a per-driver object https://review.opendev.org/c/openstack/nova/+/913196 | |
| 13:45:19 | sean-k-mooney | bauzas: that greneade patch looks correct to me | |
| 13:45:42 | sean-k-mooney | so ya once grenade is branched we shoudl be good | |
| 13:46:38 | bauzas | cool | |
| 13:47:20 | bauzas | dansmith: sean-k-mooney: while you're here, something procedural again about rolling-upgrades now we've branched 2024.1 : https://review.opendev.org/c/openstack/nova/+/913890 | |
| 13:48:14 | dansmith | needs a recheck? | |
| 13:48:53 | bauzas | because of multicell yeah | |
| 13:49:13 | bauzas | probably a guest kernel panic again, but haven't looked yet | |
| 13:49:30 | dansmith | ssh timeout | |
| 13:49:33 | bauzas | either way, grenade skip-level is happy so do I | |
| 13:49:36 | dansmith | no crash | |
| 13:49:59 | bauzas | (but skiplevels are actually setting the hack option, so meh) | |
| 13:50:12 | bauzas | dansmith: ack | |
| 14:38:59 | opendevreview | mike_mp@zzzcomputing.com proposed openstack/nova master: do not use str(url) to stringify a URL for subsequent use https://review.opendev.org/c/openstack/nova/+/913910 | |
| 16:06:29 | opendevreview | Merged openstack/nova stable/2023.2: Reproducer test for live migration with power management https://review.opendev.org/c/openstack/nova/+/913197 | |
| 17:07:54 | melwitt | sean-k-mooney: in case you missed it, I replied to your comment re: live migration with encryption a few days ago after I did some testing with cinder volume encryption https://review.opendev.org/c/openstack/nova/+/905512/21#message-23db60f810b5472343e21666c09292885b832a17 | |
| 17:30:26 | sean-k-mooney | i did ill read it now | |
| 17:31:12 | sean-k-mooney | ah so we do pull the key form barbican for cinder | |
| 17:32:35 | sean-k-mooney | melwitt: ill comment on the bug but ya we could copy the libvirt secret with either in the migration_data object or via ssh | |
| 17:32:48 | sean-k-mooney | im actully wonderign if we can tell libvirt to transfer the libivrt secret | |
| 17:33:51 | sean-k-mooney | i dont think it a teribly uncommon usecasue to have a domain and a secret related to that domain so i wonder if they have a native metond to copy such info | |
| 17:34:09 | sean-k-mooney | espically when in this case its a secret passed to qemu to decrypt the disks | |
| 17:34:46 | melwitt | I tend to think sending it over RPC wouldn't be good from a security perspective | |
| 17:35:27 | sean-k-mooney | i have been thinking about a solution for that for a diffent usecase but i agree unecypted it would be not good | |
| 17:35:54 | sean-k-mooney | i assumed that is why we dont do it today by the way | |
| 17:37:19 | melwitt | and I guess I also don't really see why we should not be able to require that barbican be available when live migrating. we expect all other services to be up (like keystone) | |
| 17:38:19 | sean-k-mooney | thats fair. the reans we need the other is we need to update there state to refect the move | |
| 17:38:32 | sean-k-mooney | we dont need to do that for barbaican swiift or glance | |
| 17:39:03 | sean-k-mooney | which keyston neutron cinder and placement yes | |
| 17:39:42 | sean-k-mooney | that a reasonable reason to pull form barbican i just was not expecting it | |