| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-08-29 | |||
| 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. | |
| 12:08:48 | sean-k-mooney | Uggla: the cycle highlight are ment to aid the marketing team | |
| 12:08:54 | sean-k-mooney | so that why i was suggesting not | |
| 12:13:52 | Uggla | sean-k-mooney what do you suggest instead og highlights ? (note that maybe it is something we could discuss at the meeting tomorrow). | |
| 12:41:07 | bauzas | sean-k-mooney: I was about to discuss with Uggla on how to create cycle highlights and prelude indeed | |
| 12:41:33 | bauzas | the foundation asks for highlights but we also need the prelude note | |
| 13:30:44 | sean-k-mooney | so we have releasenotes for the eventlet stuff in general with a call to arms for operator to try it an provide feedback. i dont think that makes sense in the cycle highlight since they are emnt to be high level and marketing focused rather then technial. | |
| 13:30:55 | sean-k-mooney | but im not to pushsed either way | |
| 14:07:22 | stephenfin | idk if it's worth calling out the OpenAPI stuff. It's still not complete, but its damn close now and it's likely very relevant from a marketing perspective (particularly with an AI slant: machine readability etc. etc.) | |
| 14:32:53 | sean-k-mooney | stephenfin: all the request schemas are there right and most of the respocne schemas | |
| 14:33:12 | sean-k-mooney | so maybe | |
| 14:35:56 | opendevreview | Merged openstack/os-vif master: Remove remaining job with Ubuntu Jammy (22.04) https://review.opendev.org/c/openstack/os-vif/+/959019 | |
| 15:00:02 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add runtime check for general additionalProperties https://review.opendev.org/c/openstack/nova/+/958858 | |
| #openstack-nova - 2025-09-02 | |||
| 06:53:16 | opendevreview | Rajesh Tailor proposed openstack/nova master: Fix 'nova-manage image_property set' command https://review.opendev.org/c/openstack/nova/+/946733 | |
| 08:39:14 | gibi | sean-k-mooney: Uggla: I've added the eventlet stuff to the PTG etherpad https://etherpad.opendev.org/p/nova-2026.1-ptg | |
| 09:34:45 | Uggla | gibi 👍 | |
| 11:02:43 | sean-k-mooney | elodilles: thank you for approving https://review.opendev.org/c/openstack/requirements/+/959051 in my head i was willing to grant the watcher-dashbaord review until our next watcher irc meeting to merge but that last of the 3 needed that release | |