| *** bauzas8 is now known as bauzas | 00:49 | |
| opendevreview | Ghanshyam Maan proposed openstack/nova master: Add manager graceful shutdown, timeout, and wait https://review.opendev.org/c/openstack/nova/+/975586 | 01:10 |
|---|---|---|
| opendevreview | Ghanshyam Maan proposed openstack/nova master: WIP: Use 2nd RPC server in compute operations https://review.opendev.org/c/openstack/nova/+/975588 | 01:11 |
| opendevreview | Ilia Baikov proposed openstack/nova master: compute: Pre-load flavor and system_metadata in init_host https://review.opendev.org/c/openstack/nova/+/977037 | 02:17 |
| opendevreview | Ghanshyam Maan proposed openstack/nova master: WIP: Use 2nd RPC server in compute operations https://review.opendev.org/c/openstack/nova/+/975588 | 02:40 |
| opendevreview | Ghanshyam Maan proposed openstack/nova master: WIP: Use 2nd RPC server in compute operations https://review.opendev.org/c/openstack/nova/+/975588 | 04:04 |
| opendevreview | Ghanshyam Maan proposed openstack/nova master: WIP: Use 2nd RPC server in compute operations https://review.opendev.org/c/openstack/nova/+/975588 | 04:21 |
| *** ykarel_ is now known as ykarel | 05:22 | |
| opendevreview | Kamil Sambor proposed openstack/nova master: Destroy scatter_gather in conductor https://review.opendev.org/c/openstack/nova/+/977055 | 09:55 |
| frickler | Uggla: gibi: (or other stable cores) given the current situation, I have lent my global-stable-core +2 on https://review.opendev.org/c/openstack/nova/+/976991 and https://review.opendev.org/c/openstack/nova/+/976373. it would be really good if these could be merged today to unblock stable branches | 10:58 |
| sean-k-mooney | frickler: done | 11:33 |
| frickler | ty | 11:42 |
| *** aloke_dev_ is now known as aloke_dev | 13:01 | |
| tobias-urdin | sean-k-mooney: when you've some time, can you check neutron core responses in https://bugs.launchpad.net/neutron/+bug/2141745 - i'm not sure what the correct path forward is | 13:06 |
| aloke_dev | Hi #openstack-nova! We are planning an InfoScale driver integration for 2026.2. | 13:15 |
| aloke_dev | It involves a new Cinder driver + os-brick connector. On the Nova side, the proposal is to add a new Libvirt volume driver that | 13:15 |
| aloke_dev | uses the new os-brick connector. It won't change existing volume APIs or XML generation. | 13:15 |
| aloke_dev | Would this qualify as a specless blueprint for Nova? Any early feedback welcome! | 13:15 |
| aloke_dev | We have posted blueprint: https://blueprints.launchpad.net/nova/+spec/infoscale-nova-driver | 13:15 |
| frickler | sean-k-mooney: elodilles: yay more brokenness :( | 14:13 |
| elodilles | yes. "finally" virtualenv bundled the latest setuptools release, so now <py312 based jobs will start failing :'( | 14:18 |
| frickler | perfect timing | 14:19 |
| elodilles | we need now virtualenv pinning in tox.ini (on stable branches) with 'requires'. (it's fun that >=py312 jobs needs direct pinning in 'deps', and <py312 jobs needs pinning in '[tox]requires') | 14:20 |
| elodilles | frickler: well, we could have counted with this. and at least we are not at release time (yeah, i know, FF deadline is quite close o:)) | 14:21 |
| sean-k-mooney | is that why the older tox jobs on jammy were passign without the pin they were using an old virutalenv with older setuptools | 14:23 |
| sean-k-mooney | that makes sense but its sucky timing indeed | 14:23 |
| elodilles | yepp. that's the reason. | 14:24 |
| opendevreview | Dan Smith proposed openstack/nova master: Make disk.extend() pass format to qemu-img https://review.opendev.org/c/openstack/nova/+/977100 | 14:35 |
| opendevreview | Dan Smith proposed openstack/nova stable/2025.2: Make disk.extend() pass format to qemu-img https://review.opendev.org/c/openstack/nova/+/977101 | 14:37 |
| frickler | regarding timing ^^ is what I was referring to, but we'll have to live with that now | 14:37 |
| opendevreview | Dan Smith proposed openstack/nova stable/2025.1: Make disk.extend() pass format to qemu-img https://review.opendev.org/c/openstack/nova/+/977103 | 14:38 |
| opendevreview | Dan Smith proposed openstack/nova stable/2024.2: Make disk.extend() pass format to qemu-img https://review.opendev.org/c/openstack/nova/+/977104 | 14:39 |
| priteau | Will you be able to bypass Zuul to force merging the security patches? | 14:39 |
| dansmith | priteau: we typically don't do that and I wouldn't think these are severe enough to warrant anything like that | 14:41 |
| dansmith | frickler: elodilles: what's the fix for the gate, pin setuptools? | 14:48 |
| dansmith | or pin virtualenv I guess? | 14:49 |
| frickler | good question, I'm not sure yet | 14:50 |
| bauzas | dansmith: I think the master CI works | 14:54 |
| bauzas | (hopefully) | 14:55 |
| bauzas | not unfortunately for stable | 14:55 |
| bauzas | elodilles: right? | 14:55 |
| dansmith | oh? I figured this was more likely a master break, but cool if not | 14:55 |
| priteau | If it is still related to the same pkg_resources issue, I think it mostly affects 2025.1 and below (because pbr is too old) | 14:57 |
| priteau | Related: https://review.opendev.org/c/openstack/requirements/+/976903 | 14:58 |
| dansmith | ah okay cool | 15:00 |
| opendevreview | Stephen Finucane proposed openstack/nova master: Add ruff-check https://review.opendev.org/c/openstack/nova/+/974441 | 15:01 |
| opendevreview | Stephen Finucane proposed openstack/nova master: tests: Filter out (more) eventlet deprecation warnings https://review.opendev.org/c/openstack/nova/+/974686 | 15:01 |
| opendevreview | Stephen Finucane proposed openstack/nova master: console: Fix type error https://review.opendev.org/c/openstack/nova/+/974687 | 15:01 |
| opendevreview | Stephen Finucane proposed openstack/nova master: typing: Add hints to nova.cmd https://review.opendev.org/c/openstack/nova/+/705657 | 15:01 |
| opendevreview | Stephen Finucane proposed openstack/nova master: typing: Add hints to nova.conf https://review.opendev.org/c/openstack/nova/+/974688 | 15:01 |
| opendevreview | Stephen Finucane proposed openstack/nova master: typing: Add hints to nova.console https://review.opendev.org/c/openstack/nova/+/974689 | 15:01 |
| opendevreview | Stephen Finucane proposed openstack/nova master: typing: Add hints to remaining top-level modules https://review.opendev.org/c/openstack/nova/+/705658 | 15:01 |
| opendevreview | Stephen Finucane proposed openstack/nova master: typing: Add hints to nova.virt, nova.virt.libvirt https://review.opendev.org/c/openstack/nova/+/974220 | 15:01 |
| opendevreview | Stephen Finucane proposed openstack/nova master: typing: Correct import issues https://review.opendev.org/c/openstack/nova/+/974221 | 15:01 |
| opendevreview | Stephen Finucane proposed openstack/nova master: mypy: Enable incremental checks https://review.opendev.org/c/openstack/nova/+/974222 | 15:01 |
| opendevreview | Stephen Finucane proposed openstack/nova master: Disable flake8 rules https://review.opendev.org/c/openstack/nova/+/974442 | 15:01 |
| opendevreview | Stephen Finucane proposed openstack/nova master: mypy: Disallow incomplete defs https://review.opendev.org/c/openstack/nova/+/974690 | 15:01 |
| opendevreview | Stephen Finucane proposed openstack/nova master: mypy: Disallow untyped defs (where possible) https://review.opendev.org/c/openstack/nova/+/974725 | 15:01 |
| opendevreview | Stephen Finucane proposed openstack/nova master: privsep: Remove dead code https://review.opendev.org/c/openstack/nova/+/977111 | 15:01 |
| opendevreview | Stephen Finucane proposed openstack/nova master: WIP: typing: Add hints to nova.privsep https://review.opendev.org/c/openstack/nova/+/977112 | 15:01 |
| elodilles | dansmith bauzas : i think this is an issue for stable branches where virtualenv is not pinned (yet) in tox.ini | 15:07 |
| elodilles | i need to check | 15:07 |
| dansmith | ack | 15:07 |
| opendevreview | Elod Illes proposed openstack/nova stable/2025.1: [CI][stable-only] Fix docs job and set skip level grenade non-voting https://review.opendev.org/c/openstack/nova/+/976991 | 15:10 |
| elodilles | if i'm not mistaken, this new issue is only present in stable/2024.2 | 15:13 |
| elodilles | (in nova) | 15:16 |
| priteau | Yes I think you're right elodilles. Although 2025.1 is broken in another way: pkg_resources docs job | 15:19 |
| priteau | https://zuul.opendev.org/t/openstack/build/ff6ec5e7a0a049bba32410d8f9b329ab[<65;124;24M[<65;124;24M | 15:19 |
| priteau | Sorry for the bad link, not sure what happened | 15:19 |
| priteau | https://zuul.opendev.org/t/openstack/build/ff6ec5e7a0a049bba32410d8f9b329ab | 15:19 |
| elodilles | priteau: yes, but that is not a new gate blocker (it's not related to virtualenv, rather the last week's setuptools release itself, which) i have a fix on the gate already o:) https://review.opendev.org/c/openstack/nova/+/976991 | 15:32 |
| priteau | Nice :) | 15:33 |
| opendevreview | Elod Illes proposed openstack/nova stable/2024.2: [CI][stable-only] Workaround for missing pkg_resources https://review.opendev.org/c/openstack/nova/+/976373 | 15:45 |
| sean-k-mooney | elodilles: why are you only fixing this for 3.12? https://review.opendev.org/c/openstack/nova/+/976373/2/tox.ini | 17:10 |
| sean-k-mooney | on 2024.2 that not a required testing reuntime and fi we are to fix it we shoudl do ti properly for all teh tagets no? | 17:10 |
| sean-k-mooney | i just realise that is why 3.9-3.11 were broken in ci | 17:11 |
| melwitt | looks like the nova-live-migration-ceph job has been broken since sometime on 2026-02-12 ... https://zuul.opendev.org/t/openstack/builds?job_name=nova-live-migration-ceph&project=openstack%2Fnova&skip=50 | 17:31 |
| melwitt | "Unit devstack@q-agt.service could not be found." and "Could not find the requested service virtlockd.service: host" and "Could not find the requested service virtlockd-admin.socket: host" https://zuul.opendev.org/t/openstack/build/5e3f246e8f11458099d1fc9b3f8d9a5a/log/job-output.txt#37346 | 17:34 |
| sean-k-mooney | ya... | 17:37 |
| sean-k-mooney | i noticed that but hoped it was trasiant | 17:37 |
| sean-k-mooney | melwitt: i have not had time to did into what is happening there | 17:37 |
| melwitt | ok I guess the q-agt thing is ignored error so it must be the virtlockd stuff actually causing the fail | 17:38 |
| sean-k-mooney | i dont think its 100% broken | 17:38 |
| melwitt | it looks 100% as far as I can tell, on the master branch anyway | 17:38 |
| sean-k-mooney | maybe but i qoudl guest q-agt mightr be form world dump? | 17:38 |
| melwitt | last passing run on master was from 2/12 https://zuul.opendev.org/t/openstack/builds?job_name=nova-live-migration-ceph&project=openstack%2Fnova&branch=master&skip=0 | 17:39 |
| sean-k-mooney | its a psot failure right so its the post job hook that failing | 17:39 |
| melwitt | right | 17:39 |
| sean-k-mooney | we may want to make this non voting while we debug it | 17:40 |
| sean-k-mooney | ok | 17:40 |
| melwitt | on this step [run-evacuate-hook : Stop libvirtd on "controller"] | 17:40 |
| sean-k-mooney | so the evacuate hook is looking for q-agt | 17:40 |
| melwitt | https://zuul.opendev.org/t/openstack/build/5e3f246e8f11458099d1fc9b3f8d9a5a/log/job-output.txt#37372 | 17:40 |
| sean-k-mooney | i think neutron might have renamed that to neutron-ovn-agnet or simialr | 17:41 |
| melwitt | I think the q-agt error is being ignored so I don't think it's causing the failure. it is wrong and should be fixed I think but I don't think it's causing this | 17:41 |
| sean-k-mooney | ah i see | 17:41 |
| sean-k-mooney | ya so its failing on virtlockd but that is an optional service | 17:42 |
| sean-k-mooney | and it looks like we are using monolitic libvirt i think | 17:42 |
| sean-k-mooney | not modular | 17:42 |
| sean-k-mooney | that may have change recently | 17:42 |
| melwitt | hm | 17:42 |
| sean-k-mooney | we moved the ceph job to debian 13 form 12 a few days ago | 17:43 |
| sean-k-mooney | that not nessiarly the cause but it one of the possiblities | 17:44 |
| sean-k-mooney | https://termbin.com/7519 | 17:45 |
| sean-k-mooney | that what my debina 13 host looks liek | 17:45 |
| sean-k-mooney | so virtlockd.service is there | 17:45 |
| elodilles | sean-k-mooney: i've answered your questions on the patch: https://review.opendev.org/c/openstack/nova/+/976373 | 17:45 |
| sean-k-mooney | we can proably make the post hook condtional | 17:45 |
| sean-k-mooney | elodilles: on so ya virtualenv or settup toolsl is not installed by default in later versions of tox | 17:46 |
| elodilles | sean-k-mooney: yes, at least that is how i found it worked while i was testing these issues | 17:47 |
| gmaan | ah I also thought ceph job is not 100% failing, thanks melwitt for bringing it here | 17:48 |
| gmaan | bauzas: gibi btw this is ready for review (ceph job failing is not related to that change, rest all jobs are green), https://review.opendev.org/c/openstack/nova/+/975586 | 17:48 |
| sean-k-mooney | gmaan: i was sure we merged some thing in the last few days but ya it was failing on my change bug i didnt have a chance to dig in | 17:49 |
| melwitt | the timing seems to line up for the change to debian trixie https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/976279 | 17:49 |
| sean-k-mooney | ya so i think the best path forward woudl eb to fix the hook | 17:49 |
| sean-k-mooney | i woudl prefer not to revert that if we can avoid it | 17:50 |
| melwitt | merged at 10:04 on Feb 12 and first failure of nova-live-migration-ceph was at 10:38 on Feb 12 (both my time zone) | 17:50 |
| sean-k-mooney | ya so i asked for a dnm on the nova gate but it disable the nova jobs | 17:50 |
| sean-k-mooney | or at least this one | 17:50 |
| sean-k-mooney | but i expect we woudl catuch it with nova-next | 17:50 |
| sean-k-mooney | i guess nova-next is lvm? | 17:51 |
| sean-k-mooney | not ceph? | 17:51 |
| sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/976551 was the nova dnm | 17:51 |
| gmaan | for future, i think that ceph devstack plugin need more integration jobs from nova/other-proejct as it break those many times | 17:51 |
| sean-k-mooney | ya perhaps | 17:51 |
| sean-k-mooney | it did run nova-ceph-multistore as teh smoke test | 17:52 |
| sean-k-mooney | but normally we woudl urn all the nova jobs | 17:52 |
| gmaan | yeah | 17:52 |
| melwitt | lol the dnm patch disabled a ceph job | 17:52 |
| sean-k-mooney | well it disabeld one of hte ceph jobs | 17:53 |
| sean-k-mooney | but ya | 17:53 |
| sean-k-mooney | lest quickly se if we can update the hook | 17:53 |
| sean-k-mooney | we can overeid it inour gate to go back to bookworm as well until we fix it if needed | 17:53 |
| gmaan | git diff | 17:54 |
| gmaan | oops | 17:54 |
| melwitt | I don't understand what the "host" part of this message means "Could not find the requested service virtlockd.service: host" | 17:57 |
| sean-k-mooney | im wondering if maybe there is a chagne in the invetory or soemthing in ansible | 17:58 |
| sean-k-mooney | but ya im trying to figure that out currenlty | 17:58 |
| melwitt | hm yeah might be an ansible thing https://stackoverflow.com/questions/75850522/failed-changed-false-msg-could-not-find-the-requested-service-mysql this unrelated random error also says ": host" | 17:59 |
| sean-k-mooney | its actully failing in the evacuate hook role | 18:02 |
| opendevreview | Merged openstack/nova stable/2025.1: [CI][stable-only] Fix docs job and set skip level grenade non-voting https://review.opendev.org/c/openstack/nova/+/976991 | 18:02 |
| sean-k-mooney | so here https://github.com/openstack/nova/blob/master/roles/run-evacuate-hook/tasks/main.yaml#L51 | 18:02 |
| melwitt | right | 18:03 |
| sean-k-mooney | we have hardcoded hosts https://github.com/openstack/nova/blob/master/roles/run-evacuate-hook/tasks/main.yaml#L20-L44 | 18:03 |
| melwitt | I know where it's failing, just trying to figure out why the service is missing | 18:03 |
| sean-k-mooney | im wondering if the subnode is called soemthign sele in the trixe node set? | 18:04 |
| melwitt | on one of the last passing runs, it looks like those services are part of libvirt-daemon-system https://zuul.opendev.org/t/openstack/build/3a45604d38ae4af683a5e7583bf91708/log/job-output.txt#32097 | 18:04 |
| sean-k-mooney | hum no its compute1 | 18:04 |
| melwitt | on the failing run there is libvirt-daemon-system but it's not showing the same sort of output about creating symlinks https://zuul.opendev.org/t/openstack/build/5e3f246e8f11458099d1fc9b3f8d9a5a/log/job-output.txt#11250 | 18:05 |
| sean-k-mooney | yes that is the package that is provideing them | 18:06 |
| sean-k-mooney | https://zuul.opendev.org/t/openstack/build/5e3f246e8f11458099d1fc9b3f8d9a5a/log/job-output.txt#11060 | 18:06 |
| melwitt | on the passing run it actually says "virtlockd.service is a disabled or a static unit, not starting it." after creating symlinks ?? | 18:06 |
| sean-k-mooney | virtlockd is an optional part of libvirt that nova does not use | 18:07 |
| sean-k-mooney | looking at teh installation on 13 | 18:07 |
| sean-k-mooney | its not being isntalled by default | 18:07 |
| melwitt | oh huh | 18:07 |
| sean-k-mooney | so the fix is to only stop the service if its active | 18:08 |
| melwitt | if nova doesn't use it why is it in this evacuate post hook (?) | 18:08 |
| melwitt | yeah. ok | 18:08 |
| sean-k-mooney | because it was installed by default before | 18:08 |
| sean-k-mooney | and we just stop all fo them | 18:08 |
| sean-k-mooney | we dont install it down stream either by the ay | 18:08 |
| melwitt | I see. ok, so check for active before trying to stop | 18:09 |
| sean-k-mooney | virtlockd is for livbirt api to act as a distribute lock manger for liek pci devices and that sort of thing | 18:09 |
| melwitt | I don't know much about ansible so I'm looking what is a good way to have a conditional run upon each item in the "with_items" | 18:11 |
| sean-k-mooney | we can use an ansible block | 18:13 |
| sean-k-mooney | but im also verifyitn iwth claude as we speak | 18:13 |
| sean-k-mooney | im thinking maybe a resuabel task file i shoudl prably look at how i implemtned it in edpm-ansible | 18:14 |
| sean-k-mooney | i coudl also jsut use failed_when or ignore errors | 18:14 |
| sean-k-mooney | but i woudl like a cleaner solution | 18:14 |
| melwitt | yeah I'm wary of a blanket ignore errors | 18:15 |
| melwitt | I see a thing called "when:" but not sure if a service active check could fit into that | 18:16 |
| melwitt | where it could run what's in "when" per item in "with_items" | 18:17 |
| sean-k-mooney | so the "simple" approch is factor out the test if its ensbeld and skip into a task file then include that in a loop for ech serivce | 18:18 |
| sean-k-mooney | that what i do in the molecule test in edpm-ansibel im gong to push an exmaple of that shortly | 18:19 |
| melwitt | that makes sense. I'm browsing in codesearch bc it seems like this would be something we already have a role (or something) for in general in openstack infra somewhere? maybe not | 18:20 |
| sean-k-mooney | amy i dont think so but maybe in zuul-jobs | 18:21 |
| opendevreview | sean mooney proposed openstack/nova master: Handle missing libvirt services in evacuate hook https://review.opendev.org/c/openstack/nova/+/977150 | 18:22 |
| sean-k-mooney | its not the cleanest way to do that but that should work | 18:23 |
| melwitt | yeah I'm not finding anything | 18:23 |
| sean-k-mooney | so i wrote these https://github.com/openstack-k8s-operators/edpm-ansible/blob/main/molecule/test-helpers/verify_systemd_unit.yaml to asert the behvior of edpm-ansible roles | 18:24 |
| sean-k-mooney | so this is mostly folling the same patern of have a reusable task file that we inovke in a loop to emulate funcitons | 18:24 |
| melwitt | only thing I find is an example from openstack-ansible checking a service status https://opendev.org/openstack/openstack-ansible-os_nova/src/branch/master/tasks/nova_install.yml#L109 | 18:25 |
| sean-k-mooney | ya that doing something smilar | 18:26 |
| sean-k-mooney | its using is-active to test if its running | 18:26 |
| sean-k-mooney | in the patch i just pusth im using list-unit | 18:26 |
| sean-k-mooney | to test if its defined | 18:26 |
| gmaan | it could not done by checking service status via systemctl status virtlovkd.service and store it in variable via 'register' | 18:26 |
| sean-k-mooney | it could that also similar | 18:27 |
| gmaan | and we can separate the virtlockd stop things as a seprate task with that vartiable | 18:27 |
| sean-k-mooney | gmaan: check the review i pushed above | 18:27 |
| sean-k-mooney | its puthing the check if it exisnt then apply state change into a task file and including that task file in a loop | 18:28 |
| gmaan | I checked, it was little more complicated that just store it in register, same way we do for q-agt. no? | 18:29 |
| melwitt | yeah, I'm not a fan of the complicated things AI seems to like | 18:29 |
| gmaan | sean-k-mooney: but your solution is more usable for other things also | 18:29 |
| sean-k-mooney | it is but i didn twant to jsut do this for virtlog d as a one off | 18:29 |
| sean-k-mooney | i guess i could | 18:29 |
| gmaan | but do not know if we will be required it for other things | 18:30 |
| sean-k-mooney | well we could refactor the q-agt code ot use this | 18:30 |
| sean-k-mooney | if we wanted too | 18:30 |
| gmaan | yeah, that is easy to read also | 18:30 |
| sean-k-mooney | the complciation with that is teh delegate_to line | 18:31 |
| sean-k-mooney | i can resue it just not in the same block | 18:32 |
| opendevreview | sean mooney proposed openstack/nova master: Generalize service check to handle q-agt https://review.opendev.org/c/openstack/nova/+/977151 | 18:35 |
| sean-k-mooney | done as a sperate commit to not kick it out of hte gate | 18:35 |
| sean-k-mooney | i want to confirm that actully fixes it | 18:35 |
| sean-k-mooney | melwitt: gmaan if ye prefer ot do it another way that aslo ok by me | 18:36 |
| sean-k-mooney | that just following the way i woudl normally do this in my own ansibel coding | 18:36 |
| gmaan | sean-k-mooney: ohk, I thought you said do it in q-agt way but i re-read that and realized it said 'yeah' for changing q-agt also this way :) | 18:37 |
| gmaan | but I am fine if it works fine | 18:38 |
| sean-k-mooney | oh you ment do a one off check with registr for the virtlockd part | 18:38 |
| sean-k-mooney | so the reason im not doing that right now | 18:38 |
| gmaan | yeah | 18:38 |
| sean-k-mooney | is i just realsise we are using monolitch libvirt in this ci job | 18:38 |
| sean-k-mooney | we are not using modular libvirt | 18:38 |
| gmaan | i am in between of something else also so I am not much clear on communication here, sorry | 18:39 |
| sean-k-mooney | and if we want to suprpot that or both in the future we will ned to manage virtqemud ectra and all the other modular services in this role eventually | 18:39 |
| sean-k-mooney | rhel/centos9 moved to modular libvirt by default it looks like debian never did | 18:40 |
| gmaan | i see. anyways let's wait for gate and if green, I am ok with that solution | 18:40 |
| sean-k-mooney | im basicly future profing because with modular libvirt these are the service we mange instead https://github.com/openstack-k8s-operators/edpm-ansible/blob/main/roles/edpm_libvirt/defaults/main.yml#L29-L34 | 18:41 |
| opendevreview | Ghanshyam Maan proposed openstack/nova master: WIP: Use 2nd RPC server in compute operations https://review.opendev.org/c/openstack/nova/+/975588 | 18:44 |
| sean-k-mooney | so ya https://zuul.opendev.org/t/openstack/build/98dc2d060560442b9bf51a8452d048d4 is green now | 19:19 |
| sean-k-mooney | the other jobs are still pending but that works | 19:20 |
| opendevreview | Stephen Finucane proposed openstack/nova master: typing: Add hints to nova.cmd https://review.opendev.org/c/openstack/nova/+/705657 | 19:22 |
| opendevreview | Stephen Finucane proposed openstack/nova master: typing: Add hints to nova.conf https://review.opendev.org/c/openstack/nova/+/974688 | 19:22 |
| opendevreview | Stephen Finucane proposed openstack/nova master: typing: Add hints to nova.console https://review.opendev.org/c/openstack/nova/+/974689 | 19:22 |
| opendevreview | Stephen Finucane proposed openstack/nova master: typing: Add hints to remaining top-level modules https://review.opendev.org/c/openstack/nova/+/705658 | 19:22 |
| opendevreview | Stephen Finucane proposed openstack/nova master: typing: Add hints to nova.virt, nova.virt.libvirt https://review.opendev.org/c/openstack/nova/+/974220 | 19:22 |
| opendevreview | Stephen Finucane proposed openstack/nova master: typing: Correct import issues https://review.opendev.org/c/openstack/nova/+/974221 | 19:22 |
| opendevreview | Stephen Finucane proposed openstack/nova master: mypy: Enable incremental checks https://review.opendev.org/c/openstack/nova/+/974222 | 19:22 |
| opendevreview | Stephen Finucane proposed openstack/nova master: Disable flake8 rules https://review.opendev.org/c/openstack/nova/+/974442 | 19:22 |
| opendevreview | Stephen Finucane proposed openstack/nova master: mypy: Disallow incomplete defs https://review.opendev.org/c/openstack/nova/+/974690 | 19:22 |
| opendevreview | Stephen Finucane proposed openstack/nova master: mypy: Disallow untyped defs (where possible) https://review.opendev.org/c/openstack/nova/+/974725 | 19:22 |
| opendevreview | Stephen Finucane proposed openstack/nova master: typing: Add hints to nova.privsep https://review.opendev.org/c/openstack/nova/+/977112 | 19:22 |
| opendevreview | Stephen Finucane proposed openstack/nova master: typing: Add hints to nova.policy https://review.opendev.org/c/openstack/nova/+/977153 | 19:22 |
| opendevreview | Merged openstack/nova stable/2024.2: [CI][stable-only] Workaround for missing pkg_resources https://review.opendev.org/c/openstack/nova/+/976373 | 19:24 |
| gmaan | looks good, it is skipping 'run-evacuate-hook: Manage virtlockd units' as expected | 19:29 |
| dansmith | so I should depends-on that for the master CVE fix looks like? | 19:42 |
| dansmith | or maybe just get it in the gate ahead of that so I don't have to fix the cherry-pick hashes | 19:43 |
| sean-k-mooney | ya the latteer sound like less work | 19:43 |
| dansmith | it's failing the multi-cell job though | 19:44 |
| sean-k-mooney | for a related reason or cinder volume failres | 19:44 |
| dansmith | guest crash it looks like, but unrelated yeah.. I just mean I can't kick it right into gate | 19:44 |
| sean-k-mooney | no but we can ask fungi or one of the ohter zuul admins to requeue it once it reported back | 19:45 |
| fungi | yeah, or i can dequeue and reenqueue it now if you'd prefer | 19:46 |
| dansmith | queue that one to gate seems safe to me, and then I'll recheck the CVE fix when it finishes | 19:46 |
| sean-k-mooney | nova-multi-cell is in gate as well anyway so it will run on it before it merges again anyway | 19:47 |
| fungi | which change number needs to go into the gate? | 19:48 |
| dansmith | 977150 | 19:48 |
| fungi | thanks, working on it | 19:48 |
| sean-k-mooney | dansmith: i have not been following the securtiy fix but are we going driect to stable with the fixes or are they all gated behind master | 19:49 |
| dansmith | I see no reason not to way for master like usual | 19:49 |
| dansmith | *wait | 19:49 |
| sean-k-mooney | ack | 19:49 |
| dansmith | fungi: if the CVE fix passes everything else other than the hook thing fixed by the above, do you want to just enqueue that one behind it as well?.. otherwise we just have to wait for another check round | 19:50 |
| fungi | 977150,1 is at the top of the integrated gate queue now | 19:51 |
| dansmith | fungi: I see it, thanks | 19:51 |
| fungi | dansmith: yeah, i can do that | 19:51 |
| dansmith | I don't think we need it to go to the head of the line, I assume you can just put it at the end right? | 19:51 |
| fungi | i can, though we can certainly expedite the security fixes | 19:52 |
| dansmith | up to you.. doesn't seem that urgent to me, just bypassing another check round would be a nice speedup on its own | 19:53 |
| fungi | sure, happy to, just let me know which change(s) once you get the test results you're looking for and set workflow +1 on it | 19:54 |
| dansmith | ack thanks, just waiting for the last slowpoke job to finish on it | 19:54 |
| dansmith | fungi: okay 977100 should be good to go please, thanks! | 20:02 |
| fungi | and it's at the end of the gate now (behind changes for neutron and devstack) | 20:03 |
| dansmith | thanks! | 20:05 |
| fungi | my pleasure, as always | 20:06 |
| opendevreview | Ghanshyam Maan proposed openstack/nova master: Use 2nd RPC server in compute operations https://review.opendev.org/c/openstack/nova/+/975588 | 21:35 |
| opendevreview | Merged openstack/nova master: Handle missing libvirt services in evacuate hook https://review.opendev.org/c/openstack/nova/+/977150 | 22:01 |
| *** erlon5 is now known as erlon | 22:30 | |
| opendevreview | Ghanshyam Maan proposed openstack/nova master: Use 2nd RPC server in compute operations https://review.opendev.org/c/openstack/nova/+/975588 | 22:45 |
| fungi | looks like the gate fix merged, but the ossa change bounced out | 22:46 |
| fungi | died on nova-multi-cell | 22:47 |
| fungi | that job seems cursed | 22:47 |
| fungi | i've directly enqueued 977100,1 to the gate again | 22:47 |
| fungi | maybe it'll have more luck this time, but i won't be around to monitor it further until tomorrow | 22:48 |
| gmaan | I think so, it is failing test_resize_volume_backed_server_confirm test which is know failure | 22:49 |
| gmaan | I will monitor in case it fail again | 22:49 |
| fungi | thanks! | 22:49 |
| opendevreview | melanie witt proposed openstack/nova master: TPM: fixups for live migration of `host` secret security https://review.opendev.org/c/openstack/nova/+/976316 | 23:43 |
| opendevreview | melanie witt proposed openstack/nova master: TPM: support live migration of `deployment` secret security https://review.opendev.org/c/openstack/nova/+/925771 | 23:43 |
| opendevreview | melanie witt proposed openstack/nova master: TPM: bump service version to enable live migration https://review.opendev.org/c/openstack/nova/+/975724 | 23:43 |
| opendevreview | melanie witt proposed openstack/nova master: TPM: test live migration between hosts with different security https://review.opendev.org/c/openstack/nova/+/952629 | 23:43 |
| opendevreview | melanie witt proposed openstack/nova master: TPM: add late check for supported TPM secret security https://review.opendev.org/c/openstack/nova/+/956975 | 23:43 |
| opendevreview | melanie witt proposed openstack/nova master: TPM: enable conversion of secret security modes via resize https://review.opendev.org/c/openstack/nova/+/962052 | 23:43 |
| opendevreview | melanie witt proposed openstack/nova master: DNM vtpm tempest https://review.opendev.org/c/openstack/nova/+/957477 | 23:43 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!