| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-10-13 | |||
| 17:14:10 | 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:17:19 | 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:21:01 | gibi | they have two backend, we need their eventlet backend to exists | |
| 17:21:41 | gibi | but in threading mode we does not depend on the eventlet backend of oslo.service | |
| 17:21:54 | gibi | so we should be able to install oslo.service without the eventlet dependency | |
| 17:22:25 | gibi | I think that needs the tweeking tkajinam mentioned on the etherpad making eventlet and extra dep of oslo.service | |
| 17:22:42 | gmaan | I mean if no eventlet is install and service is running in threading mode should not fail. | |
| 17:23:47 | 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:24:38 | 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:51 | gibi | instead of just removing eventlet but keeping oslo.service | |
| 17:26:14 | gibi | https://github.com/openstack/nova/blob/8729d67412feb60d5c167df63aa6394a1329a44f/requirements.txt#L46 | |
| 17:27:00 | 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:31:56 | gmaan | but some other project might be installing it right, in single job we do install lot of other service, libs | |
| 17:32:38 | gmaan | or maybe we need to try the uninstall path and see what all broken when service is runing in threading mode | |
| 17:32:57 | gibi | yeah that was discussed on the call we have a shared venv in devstack that is a potential problem | |
| 17:33:04 | tkajinam | gmaan, we can first start with unit tests and functiona tests without eventlet and then move on to devstack thing | |
| 17:33:30 | 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:32 | 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:34:08 | tkajinam | while we do want to test integration bits within nova without eventlet | |
| 17:34:11 | gmaan | yeah, same i did for RBAC and json policy yaml to show what all fail | |
| 17:34:12 | tkajinam | (cinder is just an example | |
| 17:34:49 | tkajinam | we can look for it, but I won't try to expand that scope too much at this stage | |
| 17:34:50 | gmaan | yeah, and this is cycle all service needs to be ready | |
| 17:35:17 | 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:25 | tkajinam | that's what I suggested in the session basically | |
| 17:35:46 | gmaan | yeah, doing unit/functional tests first is good idea | |
| 17:35:53 | gibi | +1 | |
| 17:41:08 | opendevreview | Ahmed Syed proposed openstack/nova master: Tolerate transient InstanceActionNotFound on instance actions https://review.opendev.org/c/openstack/nova/+/1010180 | |
| 17:43:21 | tkajinam | I'm likely be packed this week but can start looking at that slowly from next week | |
| 17:43:30 | tkajinam | I think stephen prepared something for oslo.service already | |
| 17:46:13 | tkajinam | I'm going to leave soon to survive through rest of this busy week ;-P | |
| 18:00:02 | gibi | tkajinam: :) | |
| 18:18:25 | 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:19:55 | gmaan | its non-voting in check pipeline but voting in gate pipeline | |
| 19:08:33 | opendevreview | Leo Jenkins proposed openstack/nova-specs master: Support for Hostname-based Ceph Mon References https://review.opendev.org/c/openstack/nova-specs/+/1000098 | |
| 19:20:53 | opendevreview | Merged openstack/nova master: Remove gabbi from test-requirements.txt https://review.opendev.org/c/openstack/nova/+/953578 | |
| 20:23:48 | opendevreview | Merged openstack/nova master: Handle GPT image detection https://review.opendev.org/c/openstack/nova/+/933928 | |
| 20:24:02 | opendevreview | Merged openstack/nova master: Bump requirement to PrettyTable>=2.4.0 https://review.opendev.org/c/openstack/nova/+/939157 | |
| 20:24:15 | opendevreview | Merged openstack/nova master: Fix exception raising in _set_passwd() https://review.opendev.org/c/openstack/nova/+/935640 | |
| 20:24:28 | opendevreview | Merged openstack/nova stable/2026.2: Fix KeyError: 'body' in os-volumes_boot API proxy https://review.opendev.org/c/openstack/nova/+/1010193 | |
| 21:58:19 | opendevreview | Merged openstack/nova master: Prevent concurrent attach_volume with task_state guard https://review.opendev.org/c/openstack/nova/+/1006786 | |
| #openstack-nova - 2026-10-14 | |||
| 03:35:30 | opendevreview | Ghanshyam Maan proposed openstack/nova-specs master: Spec for graceful shutdown part3: safe termination https://review.opendev.org/c/openstack/nova-specs/+/1010320 | |
| 03:50:46 | opendevreview | Ghanshyam Maan proposed openstack/nova-specs master: Spec for graceful shutdown part3: safe termination https://review.opendev.org/c/openstack/nova-specs/+/1010320 | |
| 09:21:02 | opendevreview | Max proposed openstack/nova master: revert: remove populate_missing_availability_zones https://review.opendev.org/c/openstack/nova/+/971894 | |
| 11:23:44 | opendevreview | Merged openstack/nova master: Fix wrong Neutron install guide link for RDO controller https://review.opendev.org/c/openstack/nova/+/992035 | |
| 11:23:57 | opendevreview | Merged openstack/nova master: Delete the suse reference in the documentation. https://review.opendev.org/c/openstack/nova/+/927705 | |
| 11:24:32 | 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:26:23 | opendevreview | Alexandre Engloner proposed openstack/nova master: Replace ServerUpdateResponse server_groups array property https://review.opendev.org/c/openstack/nova/+/995470 | |
| 11:28:12 | opendevreview | Alexandre Engloner proposed openstack/nova master: Replace ServerUpdateResponse server_groups array property https://review.opendev.org/c/openstack/nova/+/995470 | |
| 13:02:12 | opendevreview | Merged openstack/nova master: Re-enable "test_tagged_attachment" test. https://review.opendev.org/c/openstack/nova/+/876699 | |
| 13:02:51 | opendevreview | Merged openstack/nova master: Replace outdated usage of egrep by grep https://review.opendev.org/c/openstack/nova/+/1002754 | |
| 13:03:03 | opendevreview | Merged openstack/nova master: Fix the libvirt.images_type value in KVM docs https://review.opendev.org/c/openstack/nova/+/896651 | |
| 13:33:02 | tkajinam | I think we can abandon https://review.opendev.org/c/openstack/nova/+/853786 now, because the related changes for cinder and os-brick have been abandoned due to inactivity. https://review.opendev.org/q/topic:%22bp/add-lustre-driver%22 | |
| 13:33:28 | tkajinam | could anyone from core spend a few minutes to abandon it (to reduce our review queue length) | |
| 13:33:59 | gibi | tkajinam: ohh that is the exact mentally non challenging task I need as a disconnect before the PTG :) On it... | |
| 13:34:54 | tkajinam | gibi, thx ! | |
| 13:34:59 | gibi | is this a single patch? | |
| 13:35:18 | gibi | (I prepared for a long list of patches to mechanically close and gain lot of free dopamine) | |
| 13:35:49 | tkajinam | that's a single one, sorry | |
| 13:35:54 | gibi | :) | |
| 13:36:04 | tkajinam | I'm just going through patches appearing in my patches as conflicts | |
| 13:36:21 | tkajinam | that's a not big win but a easy small step | |
| 13:36:52 | gibi | I abandoned the snp series from your colleague the other day | |
| 13:36:57 | gibi | as we landed yours | |
| 13:37:02 | tkajinam | gibi, yeah I saw that. thanks | |
| 13:37:10 | gibi | that was at least 5 patches to get rid off | |
| 13:37:28 | tkajinam | by the way I noticed we didn't discuss if nova-reviewers (or whatever it is named) should have right to abandon changes or -2. -2 might be too powerful but I personally feel like having more people to sweep the queue may be helpful | |
| 13:38:25 | tkajinam | gibi, let's go into lfx insight to find how we get "number of reviews closed by the person" | |
| 13:38:44 | tkajinam | and abandon 1000 reviews before you bring that stat number to your mgmt :-P | |
| 13:39:01 | gmaan | Uggla: I might be away from 15 - 15.15 UTC ish to drop kids to school. I think cross cell migration and graceful shutdown is schedules after that 15 UTC? | |
| 13:39:46 | gibi | tkajinam: I'm totally OK for nova-reviewers to got abadon rights. | |
| 13:39:50 | gibi | to get | |
| 13:40:19 | tkajinam | :-) | |
| 13:40:21 | gibi | -2 is something that needs a dicussion for sure | |
| 13:40:43 | gibi | I think -2 is sticky across patchsets so yeah that might be too much | |
| 13:41:05 | Uggla | gmaan true for graceful. But x cell LM is starting at 15UTC | |
| 13:41:10 | tkajinam | gmaan, 1500-1600UTC : Cross-cell live migration | |
| 13:41:20 | tkajinam | gmaan, 1700-1800UTC : Nova graceful shutdown part-3: operations self-termination point | |
| 13:41:36 | gmaan | yeah, i will be away for about 15 min for the cross-cell one | |
| 13:41:39 | tkajinam | (just copy-pasting agenda lines) | |
| 13:42:33 | gmaan | gibi: as you are here, or maybe dansmith or elodilles knows, do you know why nova-tox-validate-backport is non voting in check pipeline and voting in gate pipeline? catching invalid backport in check pipeline is worth right? | |
| 13:43:20 | gibi | I do remember it was intentional but I cannot come up with a why | |
| 13:43:30 | gibi | I think we need elodilles' memories | |
| 13:43:31 | dansmith | gmaan: I think it might be because we wanted to be able to line up backports before master merged without having to recheck them again after master merged, but still prevent them landing out of order | |
| 13:43:47 | gibi | ohh that sounds reasonable | |
| 13:43:50 | gibi | one less recheck | |
| 13:44:02 | gmaan | ohk, yeah, prevent recheck is good reason | |
| 13:44:02 | dansmith | right, else we'd sort of serialize the whole thing | |
| 13:45:34 | gmaan | let me add it reason in zuul file. i was wondering why we made it like this. thanks | |
| 13:53:29 | gibi | thanks gmaan for documenting it | |
| 14:01:18 | sean-k-mooney[m] | gmaan: its very intentional that its non-voting n check | |
| 14:01:52 | sean-k-mooney[m] | and yes we anted it to not make the job red when we were proposing the backports | |
| 14:02:06 | sean-k-mooney[m] | to avoid having to do a recheck alwasy | |
| 14:02:18 | gmaan | yeah | |
| 14:02:39 | Leo[m] | Hello, my PTG topic "Support for Hostname-based Ceph Mon References" is scheduled for now. Is this the right chat for Nova PTG discussions? Thank you | |
| 14:03:00 | dansmith | Leo[m]: https://meetpad.opendev.org/nova-2027.1-ptg | |
| 14:03:57 | dansmith | Leo[m]: we reordered you after another session since you weren't on, so just hold up for a bit and you'll be next | |
| 14:05:00 | Leo[m] | Ok, thank you | |
| 14:19:32 | dansmith | abhishekk: we have a question for you about image trusted certificate verification, maybe we can circle back to it when you're in the nova room later | |
| 14:35:23 | abhishekk | dansmith: ack, you can post the question now as well | |