| opendevreview | Takashi Kajinami proposed openstack/nova master: Catch ValueError from from_image_ref in rescue https://review.opendev.org/c/openstack/nova/+/1000969 | 02:25 |
|---|---|---|
| opendevreview | Rocky proposed openstack/nova master: Drop placement connections inherited across fork https://review.opendev.org/c/openstack/nova/+/1010106 | 03:39 |
| opendevreview | taha hosseini proposed openstack/nova master: libvirt: assign virtio storage devices to guest IOThread https://review.opendev.org/c/openstack/nova/+/1008546 | 04:37 |
| opendevreview | taha hosseini proposed openstack/nova master: libvirt: assign virtio storage devices to guest IOThread https://review.opendev.org/c/openstack/nova/+/1008546 | 06:02 |
| opendevreview | Merged openstack/nova master: Convert [libvirt] nfs_mount_options to list https://review.opendev.org/c/openstack/nova/+/986258 | 06:38 |
| *** s0okju[m] is now known as sookju[m] | 07:52 | |
| *** ykarel_ is now known as ykarel | 08:04 | |
| opendevreview | Maximilian Stinsky proposed openstack/nova stable/2026.1: Remove experimental warning from UnifiedLimitsDriver https://review.opendev.org/c/openstack/nova/+/1010135 | 09:14 |
| opendevreview | Maximilian Stinsky proposed openstack/nova stable/2025.2: Remove experimental warning from UnifiedLimitsDriver https://review.opendev.org/c/openstack/nova/+/1010136 | 09:14 |
| opendevreview | Maximilian Stinsky proposed openstack/nova stable/2025.1: Remove experimental warning from UnifiedLimitsDriver https://review.opendev.org/c/openstack/nova/+/1010137 | 09:14 |
| *** sambork__ is now known as sambork | 09:30 | |
| stephenfin | bauzas: gibi: Just another poke for https://review.opendev.org/c/openstack/nova/+/1009877/ It's tiny, and unblocks the SQLA bump in upper-constraints | 11:17 |
| stephenfin | (ping ye specifically because EU tz) | 11:17 |
| opendevreview | taha hosseini proposed openstack/nova master: libvirt: assign virtio storage devices to guest IOThread https://review.opendev.org/c/openstack/nova/+/1008546 | 11:22 |
| opendevreview | Maximilian Stinsky proposed openstack/nova stable/2026.1: Fix KeyError: 'body' in os-volumes_boot API proxy https://review.opendev.org/c/openstack/nova/+/1010162 | 11:40 |
| opendevreview | Moritz Tanner proposed openstack/nova master: Implement extend_volume for local devices https://review.opendev.org/c/openstack/nova/+/878763 | 12:04 |
| opendevreview | Merged openstack/nova master: Fix KeyError: 'body' in os-volumes_boot API proxy https://review.opendev.org/c/openstack/nova/+/1009982 | 12:37 |
| gibi | stephenfin: does sqlite :memory: still works in the new SQLAlchemy that start quoting ":"? | 12:50 |
| gibi | I mean is this just a test assert issue or is it affect sqlite URL usage? | 12:52 |
| stephenfin | it must since oslo.db tests it and has a tips job that runs against `main` of SQLA | 12:53 |
| gibi | cool then | 12:53 |
| stephenfin | we also don't use memory in production, and I'm not sure we use it in our tests (though if we do, that's more proof it does still work since the nova tests are passing in https://review.opendev.org/c/openstack/requirements/+/1008776 with that change) | 12:54 |
| stephenfin | thanks for taking a look at that 🙏 | 12:55 |
| *** haleyb|out is now known as haleyb | 12:55 | |
| dansmith | mhen: I think we should scrub my confusion from your ptg topic this morning.. I (obviously) pattern-matched that to be about encrypted images given our recent discussions and it would probably be good to avoid confusing others. okay with that? | 13:02 |
| mhen | sure | 13:03 |
| gibi | stephenfin: I think the default is :memory: if no url is provided. In nova at least in the funtional test we switched to file based sqlite due to threading complications | 13:04 |
| opendevreview | Nathan proposed openstack/nova master: libvirt: Attach disk.config.rescue in stable device rescue https://review.opendev.org/c/openstack/nova/+/1009444 | 13:21 |
| opendevreview | Ahmed Syed proposed openstack/nova master: Tolerate transient InstanceActionNotFound on instance actions https://review.opendev.org/c/openstack/nova/+/1010180 | 13:42 |
| opendevreview | Ahmed Syed proposed openstack/nova master: Tolerate transient InstanceActionNotFound on instance actions https://review.opendev.org/c/openstack/nova/+/1010180 | 13:49 |
| opendevreview | Ahmed Syed proposed openstack/nova master: Tolerate transient InstanceActionNotFound on instance actions https://review.opendev.org/c/openstack/nova/+/1010180 | 13:54 |
| opendevreview | Ahmed Syed proposed openstack/nova master: Tolerate transient InstanceActionNotFound on instance actions https://review.opendev.org/c/openstack/nova/+/1010180 | 13:55 |
| opendevreview | Maximilian Stinsky proposed openstack/nova stable/2026.2: Fix KeyError: 'body' in os-volumes_boot API proxy https://review.opendev.org/c/openstack/nova/+/1010193 | 14:34 |
| opendevreview | Maximilian Stinsky proposed openstack/nova stable/2026.2: Remove experimental warning from UnifiedLimitsDriver https://review.opendev.org/c/openstack/nova/+/1010194 | 14:35 |
| opendevreview | Merged openstack/nova master: tests: Remove DB URL render test https://review.opendev.org/c/openstack/nova/+/1009877 | 14:36 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Reproduce cross cell move losing trusted certs https://review.opendev.org/c/openstack/nova/+/1010021 | 14:41 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Reject shelve and cross-cell moves with trusted certs https://review.opendev.org/c/openstack/nova/+/1010197 | 14:41 |
| opendevreview | Merged openstack/nova master: Replace use of datetime.timezone.UTC https://review.opendev.org/c/openstack/nova/+/989515 | 14:57 |
| opendevreview | Gabin Lanore proposed openstack/nova master: libvirt: stop the event dispatch loops on cleanup https://review.opendev.org/c/openstack/nova/+/1005022 | 15:14 |
| opendevreview | Gabin Lanore proposed openstack/nova master: Reproduce bug #2170719 https://review.opendev.org/c/openstack/nova/+/1010210 | 15:14 |
| sean-k-mooney | gibi to answer a question you asked, there is a usecase for grouping the cybrog devices in a single swich even if the vm does not have a numa toplogy but if we were to make that change anyway we would detail it in the spec. on the nova side i think we would want to default to 1 pci switch per virutal numa node in the guest and then decied if cybrog devices go into there own or | 15:18 |
| sean-k-mooney | get spread between them. again topic to be detiled in any future spec | 15:18 |
| gibi | sean-k-mooney: ack, thanks. I feel like here are a big bunch of incoming complexity that I'm a bit reluctant to blindly accept as something nova needs to take. At least the livemigration side of this complexity feels similar to how nova started deciding if two hosts' cpu are compatible for live migration and then libvirt implemented that eventually and we still not using the better libvirt | 15:23 |
| gibi | implementation properly ever since | 15:23 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Reproduce trusted cert bugs in cross-cell move and shelve https://review.opendev.org/c/openstack/nova/+/1010021 | 16:00 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Reject shelve and cross-cell moves with trusted certs https://review.opendev.org/c/openstack/nova/+/1010197 | 16:00 |
| gmaan | gibi: sambork: somehow i missed the eventlet sessions, but read the etherpad. I replied to the gibi question about 'nova testing without installing eventlet', its hard to "not install" it in devstack based job but we can uninatll it explicitly https://etherpad.opendev.org/p/oct2026-ptg-eventlet-removal#L117 | 16:32 |
| sambork | sounds reasonable, and easy to do | 16:33 |
| 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 | 16:36 |
| gibi | gmaan: uninstalling it today will probably uninstall oslo.service as oslo.service hard depends on eventlet today.(or is pip dumber than that?) So we need to make the oslo.service -> eventlet dependency soft | 17:14 |
| gmaan | gibi: ohk, i thought oslo.services are ready with no-eventlet code path but if not then yeah that is first thing to do | 17:17 |
| gibi | they have two backend, we need their eventlet backend to exists | 17:21 |
| gibi | but in threading mode we does not depend on the eventlet backend of oslo.service | 17:21 |
| gibi | so we should be able to install oslo.service without the eventlet dependency | 17:21 |
| gibi | I think that needs the tweeking tkajinam mentioned on the etherpad making eventlet and extra dep of oslo.service | 17:22 |
| gmaan | I mean if no eventlet is install and service is running in threading mode should not fail. | 17:22 |
| gmaan | but if we do that via extra dep the testing env has the eventlet and i think you want to test things with no eventlet at all in testing env? | 17:23 |
| gibi | I haven't tried I just assume that today I cannot remove eventlet from devstack as that would remove oslo.service alltogether | 17:24 |
| gibi | instead of just removing eventlet but keeping oslo.service | 17:24 |
| gibi | https://github.com/openstack/nova/blob/8729d67412feb60d5c167df63aa6394a1329a44f/requirements.txt#L46 | 17:26 |
| gibi | this suggests that we can do oslo.service[eventlet,threading] to request both backend (after eventlet made optional) and then change that to oslo.service[threading] to signal not to install eventlet due to oslo.service is installed | 17:27 |
| gmaan | but some other project might be installing it right, in single job we do install lot of other service, libs | 17:31 |
| gmaan | or maybe we need to try the uninstall path and see what all broken when service is runing in threading mode | 17:32 |
| gibi | yeah that was discussed on the call we have a shared venv in devstack that is a potential problem | 17:32 |
| tkajinam | gmaan, we can first start with unit tests and functiona tests without eventlet and then move on to devstack thing | 17:33 |
| gibi | but in general having a job that fails and showing which services is not ready for 2027.1 is a good signal :) | 17:33 |
| tkajinam | one problem with devstack job is that it shares global venv so you can't remove eventlet from nova unless you disable cinder or you kill cinder | 17:33 |
| tkajinam | while we do want to test integration bits within nova without eventlet | 17:34 |
| gmaan | yeah, same i did for RBAC and json policy yaml to show what all fail | 17:34 |
| tkajinam | (cinder is just an example | 17:34 |
| tkajinam | we can look for it, but I won't try to expand that scope too much at this stage | 17:34 |
| gmaan | yeah, and this is cycle all service needs to be ready | 17:34 |
| tkajinam | removing hard dependencies for sure helps us achieving that path. so first focus on local unit tests and functionat tests and complete that step then look at devstack | 17:35 |
| tkajinam | that's what I suggested in the session basically | 17:35 |
| gmaan | yeah, doing unit/functional tests first is good idea | 17:35 |
| gibi | +1 | 17:35 |
| opendevreview | Ahmed Syed proposed openstack/nova master: Tolerate transient InstanceActionNotFound on instance actions https://review.opendev.org/c/openstack/nova/+/1010180 | 17:41 |
| tkajinam | I'm likely be packed this week but can start looking at that slowly from next week | 17:43 |
| tkajinam | I think stephen prepared something for oslo.service already | 17:43 |
| tkajinam | I'm going to leave soon to survive through rest of this busy week ;-P | 17:46 |
| gibi | tkajinam: :) | 18:00 |
| gmaan | is there ay reason nova-tox-validate-backport job on stable branches are non voting? I was expecting it to catch the invalid backport and it did but zuul +1 because it is non voting | 18:18 |
| gmaan | its non-voting in check pipeline but voting in gate pipeline | 18:19 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!