| opendevreview | Goutham Pacha Ravi proposed openstack/nova-specs master: Add spec for virtiofs live migration https://review.opendev.org/c/openstack/nova-specs/+/983801 | 07:16 |
|---|---|---|
| opendevreview | Goutham Pacha Ravi proposed openstack/nova-specs master: Add spec for virtiofs live migration https://review.opendev.org/c/openstack/nova-specs/+/983801 | 07:18 |
| bauzas | gibi: sambork: gmaan: got hurt by a graceful shutdown job failure https://review.opendev.org/c/openstack/nova/+/1005727/5 | 08:18 |
| bauzas | looks potentially related to https://review.opendev.org/c/openstack/nova/+/1005022 | 08:18 |
| bauzas | Uggla: noticed there is a conflicting session today at 3pm UTC for myself with os-agen,tic | 08:29 |
| bauzas | I'll be on and off | 08:29 |
| Uggla | bauzas ok | 08:31 |
| bauzas | I think I'll probably join once I'm tired with the conflicted session :) | 08:32 |
| * bauzas looks at the agentic etherpad and honestly I don't have anything important | 08:32 | |
| Uggla | I pushed the start today at 15h to avoid conflict with tc session. The agentic one was not on my radar I seemed to appear late in the preparation. | 08:33 |
| bauzas | no worries, it was created very late AFAICS | 08:35 |
| opendevreview | YushoYamaguchi proposed openstack/nova-specs master: Add spec for configurable virtio-net multiqueue count https://review.opendev.org/c/openstack/nova-specs/+/1009977 | 08:39 |
| opendevreview | Maximilian Stinsky proposed openstack/nova master: Fix KeyError: 'body' in os-volumes_boot API proxy https://review.opendev.org/c/openstack/nova/+/1009982 | 09:25 |
| opendevreview | Stephen Finucane proposed openstack/nova master: tests: Fix DB migration URL test for SQLAlchemy 2.1+ https://review.opendev.org/c/openstack/nova/+/1009877 | 09:52 |
| opendevreview | Stephen Finucane proposed openstack/nova master: Replace use of datetime.timezone.UTC https://review.opendev.org/c/openstack/nova/+/989515 | 09:52 |
| opendevreview | Stephen Finucane proposed openstack/nova master: trivial: Fix outstanding flake8 issues https://review.opendev.org/c/openstack/nova/+/999714 | 09:52 |
| opendevreview | Stephen Finucane proposed openstack/nova master: Add ruff-check https://review.opendev.org/c/openstack/nova/+/974441 | 09:52 |
| opendevreview | Stephen Finucane proposed openstack/nova master: Enable G (flake8-logging-format) rules https://review.opendev.org/c/openstack/nova/+/998557 | 09:52 |
| opendevreview | YushoYamaguchi proposed openstack/nova-specs master: Configurable virtio-net multiqueue count via flavor and image https://review.opendev.org/c/openstack/nova-specs/+/1009977 | 10:08 |
| *** ykarel_ is now known as ykarel | 10:18 | |
| stephenfin | bauzas: sean-k-mooney: Before PTG sessions start, this is a tiny test-only patch that should address the outstanding SQLAlchemy 2.1 blocker. I'd appreciate a review 🙏 https://review.opendev.org/c/openstack/nova/+/1009877 | 11:32 |
| stephenfin | (actual bump is happening here https://review.opendev.org/c/openstack/requirements/+/1008776) | 11:32 |
| gibi | bauzas: do we know why the graceful shutdon slowness is not hitting the graceful shutdown job at every run? The patch suggest we never stop the event thread and therefore never exit before the full graceful shutdown timeout, that sonds like something we should hit all the time. Why it is then not blocking the gate? | 11:34 |
| sean-k-mooney | stephenfin: my only reall issue with that is it makes it pretty hard to tell at a clance if its correct | 11:36 |
| stephenfin | Yeah, I had the same concern, but I wanted to avoid hardcoding since there's a chance this will change again | 11:38 |
| sean-k-mooney | well that kind of my concern | 11:38 |
| stephenfin | The other approach I could do is just to drop the test. It's really testing functionlity of alembic/sqlalchemy | 11:38 |
| stephenfin | *functionality | 11:38 |
| sean-k-mooney | that the internal will change again and we will need to replace render_as_string with somethign else | 11:39 |
| stephenfin | Yeah, okay, let me rework this to just drop the test so. There's nothing nova-specific in here anymore https://github.com/openstack/nova/blob/2b5fedace12dba41600bec10ca73476dbcfc7d10/nova/db/migration.py#L39-L51 | 11:44 |
| opendevreview | Stephen Finucane proposed openstack/nova master: tests: Remove DB URL render test https://review.opendev.org/c/openstack/nova/+/1009877 | 11:48 |
| stephenfin | sean-k-mooney: ^ | 11:53 |
| gibi | bauzas: gmaan: I reviewed https://review.opendev.org/c/openstack/nova/+/1005022 and I think this is a valid bug. But I'm very curious why our testing did not caught it as compute never exiting before the full graceful timeout? | 11:57 |
| gibi | I mean caught it in the devstack based graceful testing as in functional test we almost never enable libvirt event handling | 11:58 |
| opendevreview | HyungJoon Kim proposed openstack/nova-specs master: Add spec for booting from Cinder backup-based image https://review.opendev.org/c/openstack/nova-specs/+/1009014 | 11:59 |
| opendevreview | Stephen Finucane proposed openstack/nova master: Replace use of datetime.timezone.UTC https://review.opendev.org/c/openstack/nova/+/989515 | 11:59 |
| opendevreview | Stephen Finucane proposed openstack/nova master: trivial: Fix outstanding flake8 issues https://review.opendev.org/c/openstack/nova/+/999714 | 12:00 |
| opendevreview | Thibaut Démaret proposed openstack/nova-specs master: Spec for default SSD rotation rate on instance disks https://review.opendev.org/c/openstack/nova-specs/+/979696 | 12:22 |
| opendevreview | Stephen Finucane proposed openstack/nova master: trivial: Fix outstanding flake8 issues https://review.opendev.org/c/openstack/nova/+/999714 | 12:49 |
| opendevreview | Stephen Finucane proposed openstack/nova master: Add ruff-check https://review.opendev.org/c/openstack/nova/+/974441 | 12:49 |
| opendevreview | Stephen Finucane proposed openstack/nova master: Enable G (flake8-logging-format) rules https://review.opendev.org/c/openstack/nova/+/998557 | 12:49 |
| bauzas | sorry folks, I was AWOL, gibi : gtk | 12:50 |
| bauzas | gibi: I don't know why we don't hit this libvirt event thread issue every time, this sounds like a performance issue only happening on slow nodes I'd say | 13:10 |
| bauzas | I've noticed some job failures on my change only occurring due to long times (like a volume resize), which leads me thinking we may have a CI nodepool performance problem on some nodes | 13:11 |
| gibi | the patch you pointed at showing a bug that should affect all nova-compute services with libvirt virt driver | 13:14 |
| gibi | in tests where we shut down such service | 13:15 |
| gibi | which I think we do in the graceful shutdown job | 13:15 |
| bauzas | honestly, I'm also lost in translation, it should have been a blocking issue | 13:15 |
| opendevreview | ribaudr proposed openstack/nova master: Add reproducer for bug 2037585: trunk port deletion fails https://review.opendev.org/c/openstack/nova/+/1010008 | 13:21 |
| opendevreview | ribaudr proposed openstack/nova master: Preserve trunk parent ports when deallocating instance networking https://review.opendev.org/c/openstack/nova/+/1010009 | 13:21 |
| opendevreview | Lucas HENRY proposed openstack/nova master: Start the manager graceful shutdown once the RPC server drained https://review.opendev.org/c/openstack/nova/+/1009164 | 13:33 |
| opendevreview | Lucas HENRY proposed openstack/nova master: libvirt: Stop the event threads in Host.cleanup() https://review.opendev.org/c/openstack/nova/+/1010011 | 13:33 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Reproduce cross cell move losing trusted certs https://review.opendev.org/c/openstack/nova/+/1010021 | 14:18 |
| gmaan | gibi: ack, will check it | 14:46 |
| gibi | gmaan: thanks | 14:50 |
| mhen | dansmith: assuming you are "danms" on the PTG etherpad, I added some responses to your questions in the volume encryption cross session section for tomorrow to hopefully clarify a bit more. | 15:12 |
| gouthamr | Uggla: hola, running late in the os-agentic room | 15:58 |
| Uggla | gouthamr, we have a 5mn to 10mn break. I hope you'll be able to join right after. | 16:01 |
| gouthamr | woah, that helps me :) | 16:01 |
| Uggla | gouthamr I hope :) | 16:07 |
| bauzas | gibi: see, I'm tempted to think more and more that we have some not performant nodepools https://review.opendev.org/c/openstack/nova/+/1005727/5 | 16:44 |
| *** M00SE8 is now known as M00SE | 16:49 | |
| opendevreview | YushoYamaguchi proposed openstack/nova-specs master: Disable guest CPU flags per flavor https://review.opendev.org/c/openstack/nova-specs/+/1010041 | 16:52 |
| tkajinam | could anyone review this backport https://review.opendev.org/c/openstack/nova/+/990096 (of regression test) to unblock the following actual fix ? | 18:54 |
| sean-k-mooney | tkajinam: done | 19:01 |
| tkajinam | sean-k-mooney, thx ! | 19:01 |
| opendevreview | Merged openstack/nova stable/2026.1: Reproduce bug/2152697 https://review.opendev.org/c/openstack/nova/+/990096 | 19:33 |
| opendevreview | Merged openstack/nova stable/2026.1: Fix live migration with pinned VM and iothreads https://review.opendev.org/c/openstack/nova/+/990097 | 21:06 |
| opendevreview | Merged openstack/nova master: Guard against deleting a concurrent request's BDM on timeout https://review.opendev.org/c/openstack/nova/+/1005727 | 21:06 |
| opendevreview | Arnaud Morin proposed openstack/nova master: Add info log when destination hypervisor is too old https://review.opendev.org/c/openstack/nova/+/907972 | 21:39 |
| opendevreview | Arnaud Morin proposed openstack/nova master: Only set device_id at port creation with a service token https://review.opendev.org/c/openstack/nova/+/861172 | 22:13 |
| opendevreview | Arnaud Morin proposed openstack/nova master: Only set device_id at port creation with a service token https://review.opendev.org/c/openstack/nova/+/861172 | 22:16 |
| opendevreview | Takashi Natsume proposed openstack/nova master: Update contributor guide for 2027.1 Indri https://review.opendev.org/c/openstack/nova/+/1008569 | 23:38 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!