| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-08-29 | |||
| 14:08:58 | gibi | dansmith: yepp yepp, failure is on my, success is on the team :) | |
| 14:09:06 | gibi | s/on my/on me/ | |
| 14:09:12 | amorin | jkulik: sounds like my exact issue | |
| 14:09:40 | dansmith | jkulik: yep | |
| 14:09:58 | jkulik | maybe we can get that mentioned spec back on track? :D | |
| 14:11:07 | jkulik | since eventlet will go away, we also need a solution for this problem. currently, we're still on the eventlet-based webserver for only this reason :/ | |
| 14:11:19 | dansmith | we could just make this an async operation in a later microversion | |
| 14:11:41 | amorin | +1 | |
| 14:11:42 | dansmith | easiest would be to delegate to conductor and keep compute the same | |
| 14:11:53 | dansmith | that's the least upgrade-impact too | |
| 14:12:02 | jkulik | > simply an approximation by most virt drivers as their underlying hypervisors cannot guarantee how the device will eventually be presented. | |
| 14:12:17 | jkulik | my understanding was, that we cannot guarantee the device name anyways | |
| 14:12:19 | dansmith | but it might require blocking earlier microversions to prevent people from triggering it | |
| 14:12:25 | jkulik | so the whole call is ... senseless | |
| 14:12:35 | dansmith | jkulik: on libvirt it's fairly pointless, but it was not on other hypervisors | |
| 14:12:52 | jkulik | I can say it's also pointless on vmware | |
| 14:15:03 | dansmith | gibi: so maybe we should make this under the eventlet removal umbrella.. moving this to an async task on conductor | |
| 14:15:19 | jkulik | https://github.com/sapcc/nova/commit/e4e8e6276ab0324704f31d6a3cc4e47f061d25ca this is how we currently try to get relieve on the wait time for the lock, FYI | |
| 14:15:27 | dansmith | the spec (as skimmed) looks far more complex than it needs to be if we just kick this to conductor in a newer microversion | |
| 14:15:39 | gibi | dansmith: maybe. I'm a bit affraid of increasing the scope. But I understand that it is related | |
| 14:16:24 | dansmith | well, if jkulik is sticking on eventlet because of this that tells me that (a) people might be less likely to try threaded until they're forced and (b) a lot more people will hit this once they move | |
| 14:16:54 | dansmith | and tbh, it's probably worth a security hardening classification since this is sort of DoSable | |
| 14:17:22 | jkulik | I agree to ^ | |
| 14:17:32 | jkulik | DoS waiting to happen | |
| 14:19:21 | dansmith | since it's friday maybe I'll at least write this up in a spec rough draft to capture it while it is in my head | |
| 14:19:56 | jkulik | please CC me on that. I'm quite interested in the topic :) | |
| 14:20:22 | amorin | CC me as well :) | |
| 14:20:58 | dansmith | are either of you interested in working on the implementation? | |
| 14:21:55 | jkulik | yes, I can probably do that | |
| 14:22:21 | amorin | not sure I would have a lot of time very soon, but we I probably help (or someone from our team) | |
| 14:23:23 | dansmith | cool | |
| 14:23:32 | amorin | but I try to motivate our management, as this is affecting us quite badly | |
| 14:23:37 | gibi | I can be pulled in as reviewer for sure. If we decide that this should be fixed before the n-cpu gets threading support then I can be pulled in as author as well | |
| 14:24:23 | gibi | dansmith: thank for writing up a spec skeleton. My friday brain is already pretty useless anyhow | |
| 14:25:01 | dansmith | I don't think this is going to be very big and I don't think it needs to be serialized against n-cpu work | |
| 14:26:00 | gibi | I just try to limit the number of things open in parallel waiting for me | |
| 14:26:55 | dansmith | ack | |
| 15:10:23 | opendevreview | Dan Smith proposed openstack/nova-specs master: WIP: Spec for async volume attach https://review.opendev.org/c/openstack/nova-specs/+/958900 | |
| 15:10:45 | dansmith | jkulik: amorin ^ .. just rough brain dump of what we talked about | |
| 15:11:05 | amorin | yay! | |
| 15:11:50 | ratailor | sean-k-mooney, gibi could you please review this https://review.opendev.org/q/Ifc20894801f723627726e3c9bed7076144542660 and provide your suggestion on approach here https://review.opendev.org/c/openstack/nova/+/954460 | |
| 15:12:35 | dansmith | I'm sure I saw the bug years ago when it was filed, but this is the sort of latent thing I don't want to lose track of as we push for eventlet deprecation, | |
| 15:13:03 | dansmith | where we think "yeah everything works in devstack" but we have lots of smaller reasons like this that prevent people from actually being able to move | |
| 15:13:43 | dansmith | this isn't even the class of things that are highest concern (i.e. general threading problems, deadlocks, etc) but they are just as much of a problem for getting people transitioned | |
| 15:19:29 | ratailor | stephenfin, submitted fix for microversion v2.100 as well https://review.opendev.org/c/openstack/python-openstackclient/+/958829 and backported v2.96 to relevant stable/releases https://review.opendev.org/q/I1e398bb3379fa6443b0a44db76baaf6241a945e7 | |
| 15:43:53 | gmaan | stephenfin: perfect, that is really helpful. thanks. ack about targeting for next cycle | |
| 19:23:37 | gmaan | sean-k-mooney: glancer service role fix is ready for review now https://review.opendev.org/q/topic:%22glance-service-api%22 | |
| 21:12:31 | opendevreview | Merged openstack/nova master: api: Correct expected errors https://review.opendev.org/c/openstack/nova/+/951640 | |
| #openstack-nova - 2025-08-31 | |||
| 07:36:58 | opendevreview | Takashi Kajinami proposed openstack/nova master: Replace remaining usage of Ubuntu Jammy https://review.opendev.org/c/openstack/nova/+/958989 | |
| 18:12:47 | opendevreview | Merged openstack/nova master: api: Fix validators for hw:cpu_max_* extra specs https://review.opendev.org/c/openstack/nova/+/957732 | |
| 18:13:08 | opendevreview | Merged openstack/nova master: Update min support for Flamingo https://review.opendev.org/c/openstack/nova/+/944018 | |
| #openstack-nova - 2025-09-01 | |||
| 07:17:19 | gibi | bauzas: when you are up, we have 3 test only (unit test enabling) commits left from eventlet removal series. Each has +2 from Sean. Neither of them is complicated. Could you please take a look. This is the first https://review.opendev.org/c/openstack/nova/+/953475 and the next two are in the same bucket. | |
| 07:44:43 | bauzas | gibi: sure I can take a look | |
| 07:53:54 | gibi | bauzas: thanks! | |
| 08:05:50 | bauzas | gibi: sean-k-mooney: +Wd two of them but I have an open question about when we could have the vote for the job https://review.opendev.org/c/openstack/nova/+/955791 | |
| 08:12:29 | gibi | bauzas: I think we can make it voting now. See my answer with data in https://review.opendev.org/c/openstack/nova/+/955791/12#message-22f01d1fcd5abdf549b2dd3e206d1292eebcae8b | |
| 08:13:29 | bauzas | gibi: I understand, let's just give a bit of time, that's what I'm just saying | |
| 08:13:50 | gibi | do you have a limit how many successful run you need before you accept it? | |
| 08:14:38 | gibi | (some of our voting jobs has a lot worse than 50:0 success:pass ratio) | |
| 08:14:54 | gibi | s/success:pass/success:fail/ | |
| 08:31:07 | bauzas | sorry was otp | |
| 08:32:05 | bauzas | honestly, I don't have any number, if the job works up to tomorrow, then if so, let's leave it to be voting :) | |
| 08:53:16 | gibi | I don't think there will be much run of it up until tomorrow. But I can do some rechecks | |
| 09:06:56 | tkajinam | I noticed one job still running on Jammy and proposed a change to migrate it to Noble. Can we merge this for 2025.2 ? (because jammy is no longer supported) https://review.opendev.org/c/openstack/nova/+/958989 | |
| 09:10:15 | tkajinam | sean-k-mooney, could you review https://review.opendev.org/c/openstack/nova/+/958822 when you have time ? This is a follow-up of SEV-ES support which I'm hoping to include in 2025.2 | |
| 09:13:10 | gibi | tkajinam: approved the job change, thanks | |
| 09:14:17 | tkajinam | gibi, thank you ! | |
| 09:14:46 | sean-k-mooney[m] | we also have a job on jammy on os-vif which i guess we should drop | |
| 09:15:07 | sean-k-mooney[m] | ill review the follow up shortly after im at my desk | |
| 09:37:05 | tkajinam | sean-k-mooney[m], thanks | |
| 09:37:26 | tkajinam | agree with you about os-vif. let me propose a change (so that you can +2) | |
| 09:40:14 | opendevreview | Takashi Kajinami proposed openstack/os-vif master: Remove remaining job with Ubuntu Jammy (22.04) https://review.opendev.org/c/openstack/os-vif/+/959019 | |
| 09:40:38 | tkajinam | sean-k-mooney[m], done ^^^ | |
| 09:43:47 | sean-k-mooney | tkajinam: :) i was not asking you to create it but i wont say no to approving it | |
| 09:57:00 | sean-k-mooney | tkajinam: they all look good. thanks for creating them | |
| 10:11:13 | opendevreview | Merged openstack/nova master: Run unit test with threading mode https://review.opendev.org/c/openstack/nova/+/953475 | |
| 10:11:40 | opendevreview | Merged openstack/nova master: [test]RPC using threading or eventlet selectively https://review.opendev.org/c/openstack/nova/+/953815 | |
| 10:30:50 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add runtime check for general additionalProperties https://review.opendev.org/c/openstack/nova/+/958858 | |
| 11:00:29 | opendevreview | Merged openstack/nova stable/2024.2: [doc]Clarify where to set pci_in_placement https://review.opendev.org/c/openstack/nova/+/958623 | |
| 11:06:50 | sean-k-mooney | gibi: fixign the futurist bug was fast | |
| 11:07:19 | sean-k-mooney | but at this point the conductor changes are targeting next cycle yes? | |
| 11:26:58 | gibi | sean-k-mooney: yes, conductor change is for G | |
| 11:27:37 | gibi | I did not imagine we will bump the global reqs so late, but it seems we did | |
| 11:28:17 | gibi | so the only thing left from eventlet in F is making the threading unit test job voting | |
| 11:28:42 | gibi | I will write up a list of topics regarding eventlet for the PTG to do the planning for G | |
| 11:29:38 | gibi | but we don't have https://etherpad.opendev.org/p/nova-2026.1-ptg yet | |
| 11:29:46 | gibi | cc Uggla ^^ :) | |
| 11:30:29 | opendevreview | Merged openstack/nova master: Replace remaining usage of Ubuntu Jammy https://review.opendev.org/c/openstack/nova/+/958989 | |
| 11:30:42 | opendevreview | Merged openstack/nova master: Drop CentOS 8 Stream https://review.opendev.org/c/openstack/nova/+/930445 | |
| 11:43:06 | sean-k-mooney | gibi: i added the boiler plate to that if you want ot start adding things | |
| 11:52:22 | Uggla | gibi, sean-k-mooney, I was going to create it, but sean-k-mooney did it already (thx). Please go ahead and add what you need. I will organize it later with the timeslot, owners etc... | |
| 11:54:59 | gibi | cool thanks folks | |
| 11:59:28 | opendevreview | Merged openstack/nova master: Follow-up of AMD SEV-ES support https://review.opendev.org/c/openstack/nova/+/958822 | |
| 12:00:11 | sean-k-mooney | Uggla: with ^ you can mark the sev blueprint as implmented just an fyi | |
| 12:00:23 | tkajinam | \o/ | |
| 12:03:17 | sean-k-mooney | this might be a nice feature for the cycle highlisths | |
| 12:03:24 | sean-k-mooney | i need to start on those for watcher | |
| 12:04:00 | sean-k-mooney | it proably does nto need to be in release note prelude but its a nice marketable feature for the highlights | |
| 12:05:01 | sean-k-mooney | the eventlet changes is kind of the reverse it would be good to note in the prelude but not really needed for the cycle highlights as it should be transparent ot end users | |
| 12:08:26 | Uggla | sean-k-mooney, I think that even though Eventlet is transparent for end users, it’s important to include it in the highlights and mention it, especially if we want users to test it and provide feedback. | |