| *** mhen_ is now known as mhen | 01:10 | |
| 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:17 |
|---|---|---|
| bauzas | gibi: sure I can take a look | 07:44 |
| gibi | bauzas: thanks! | 07:53 |
| 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:05 |
| 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:12 |
| bauzas | gibi: I understand, let's just give a bit of time, that's what I'm just saying | 08:13 |
| gibi | do you have a limit how many successful run you need before you accept it? | 08:13 |
| gibi | (some of our voting jobs has a lot worse than 50:0 success:pass ratio) | 08:14 |
| gibi | s/success:pass/success:fail/ | 08:14 |
| bauzas | sorry was otp | 08:31 |
| 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:32 |
| gibi | I don't think there will be much run of it up until tomorrow. But I can do some rechecks | 08:53 |
| 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:06 |
| 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:10 |
| gibi | tkajinam: approved the job change, thanks | 09:13 |
| tkajinam | gibi, thank you ! | 09:14 |
| sean-k-mooney[m] | we also have a job on jammy on os-vif which i guess we should drop | 09:14 |
| sean-k-mooney[m] | ill review the follow up shortly after im at my desk | 09:15 |
| tkajinam | sean-k-mooney[m], thanks | 09:37 |
| tkajinam | agree with you about os-vif. let me propose a change (so that you can +2) | 09:37 |
| 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 |
| tkajinam | sean-k-mooney[m], done ^^^ | 09:40 |
| sean-k-mooney | tkajinam: :) i was not asking you to create it but i wont say no to approving it | 09:43 |
| sean-k-mooney | tkajinam: they all look good. thanks for creating them | 09:57 |
| opendevreview | Merged openstack/nova master: Run unit test with threading mode https://review.opendev.org/c/openstack/nova/+/953475 | 10:11 |
| opendevreview | Merged openstack/nova master: [test]RPC using threading or eventlet selectively https://review.opendev.org/c/openstack/nova/+/953815 | 10:11 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add runtime check for general additionalProperties https://review.opendev.org/c/openstack/nova/+/958858 | 10:30 |
| opendevreview | Merged openstack/nova stable/2024.2: [doc]Clarify where to set pci_in_placement https://review.opendev.org/c/openstack/nova/+/958623 | 11:00 |
| sean-k-mooney | gibi: fixign the futurist bug was fast | 11:06 |
| sean-k-mooney | but at this point the conductor changes are targeting next cycle yes? | 11:07 |
| gibi | sean-k-mooney: yes, conductor change is for G | 11:26 |
| gibi | I did not imagine we will bump the global reqs so late, but it seems we did | 11:27 |
| gibi | so the only thing left from eventlet in F is making the threading unit test job voting | 11:28 |
| gibi | I will write up a list of topics regarding eventlet for the PTG to do the planning for G | 11:28 |
| gibi | but we don't have https://etherpad.opendev.org/p/nova-2026.1-ptg yet | 11:29 |
| gibi | cc Uggla ^^ :) | 11:29 |
| opendevreview | Merged openstack/nova master: Replace remaining usage of Ubuntu Jammy https://review.opendev.org/c/openstack/nova/+/958989 | 11:30 |
| opendevreview | Merged openstack/nova master: Drop CentOS 8 Stream https://review.opendev.org/c/openstack/nova/+/930445 | 11:30 |
| sean-k-mooney | gibi: i added the boiler plate to that if you want ot start adding things | 11:43 |
| 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:52 |
| gibi | cool thanks folks | 11:54 |
| opendevreview | Merged openstack/nova master: Follow-up of AMD SEV-ES support https://review.opendev.org/c/openstack/nova/+/958822 | 11:59 |
| sean-k-mooney | Uggla: with ^ you can mark the sev blueprint as implmented just an fyi | 12:00 |
| tkajinam | \o/ | 12:00 |
| sean-k-mooney | this might be a nice feature for the cycle highlisths | 12:03 |
| sean-k-mooney | i need to start on those for watcher | 12:03 |
| sean-k-mooney | it proably does nto need to be in release note prelude but its a nice marketable feature for the highlights | 12:04 |
| 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:05 |
| 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 |
| sean-k-mooney | Uggla: the cycle highlight are ment to aid the marketing team | 12:08 |
| sean-k-mooney | so that why i was suggesting not | 12:08 |
| 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:13 |
| bauzas | sean-k-mooney: I was about to discuss with Uggla on how to create cycle highlights and prelude indeed | 12:41 |
| bauzas | the foundation asks for highlights but we also need the prelude note | 12:41 |
| 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 |
| sean-k-mooney | but im not to pushsed either way | 13:30 |
| 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:07 |
| sean-k-mooney | stephenfin: all the request schemas are there right and most of the respocne schemas | 14:32 |
| sean-k-mooney | so maybe | 14:33 |
| * sean-k-mooney hopes we will complete merging that serise before the actual 2025.2 offical release | 14:33 | |
| opendevreview | Merged openstack/os-vif master: Remove remaining job with Ubuntu Jammy (22.04) https://review.opendev.org/c/openstack/os-vif/+/959019 | 14:35 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add runtime check for general additionalProperties https://review.opendev.org/c/openstack/nova/+/958858 | 15:00 |
| *** bauzas4 is now known as bauzas | 19:46 | |
| *** thelounge5514 is now known as thelounge551 | 21:47 | |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!