Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2025-08-29
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
11:04:30 sean-k-mooney its one of those things where its nice to complete the feature by including the horizon supprot this cycle but i also am not willing ot keep merging thing too much past FF even if we are just wait on release ectra
11:08:43 opendevreview Merged openstack/nova master: Reproduce that only half of the PCI devs are removed https://review.opendev.org/c/openstack/nova/+/953971
11:55:57 elodilles sean-k-mooney: no problem. the python-watcherclient release happened yesterday, and new-release patches (upper constraint version bumps) should land as soon as possible anyway
11:57:38 elodilles sean-k-mooney: in the future: you could propose releases anytime if that is needed in other repository, no need to wait for the generated milestone release patches :)
11:58:13 sean-k-mooney gibi: i finished reviweeing https://review.opendev.org/q/topic:%22bug/2115905%22 the first 2 im +2 on although there are some nits related to the commit mesaage that would be nice to fix. https://review.opendev.org/c/openstack/nova/+/954149 i have a question/suggeetion on how to clsoe the schduer race for you to consider but im +1 over all on that
11:58:52 sean-k-mooney elodilles: actully that not what happend in this case, we only merged the api change on FF
11:59:17 elodilles (on the other hand watcher-dashboard is cycle-with-rc release typed project, so it still has time until the release, and FF is mostly discussed within the given project teams -> if the team decides that it's not risky for the release, then things can be merged after FF)
11:59:22 sean-k-mooney so we had to wait for that to merge before merging the clint change and doing the release
12:00:15 sean-k-mooney elodilles: the main issue with watcher-dashboard is there is almost no testing or test infra in that and we need to figure out how to resolve that going forward.
12:00:31 elodilles :S
12:00:34 sean-k-mooney elodilles: that means we need to carfully manually tests all changes so that takes time
12:00:44 elodilles i see
12:01:45 sean-k-mooney i want to eventully explore using playwrite (instead of creating selenium tests) to see if we can get some end to end test coverage but we also coudl use django unit test framework more then we do
12:02:07 sean-k-mooney horizon has historically never relaly facilated testing plugins well
12:02:23 sean-k-mooney so as a resutl most plugins were nto proeprly tested
12:02:53 sean-k-mooney fixing that is on our todo list just not as high as it proably shoudl be
12:06:13 elodilles maybe next cycle ;)
12:11:27 grami[m] Has anyone run into the problem with Nvidia GPU and requiring pci-expander-bus and vswitch-pcis?
12:19:56 sean-k-mooney grami[m]: it should not require them however they recommend them
12:20:01 sean-k-mooney for performance reasons
12:20:23 sean-k-mooney vswitch-pcis is not something im famialr with
12:21:20 sean-k-mooney nvidia recomemtn in one of there kvm docs that you attache thre gpus and nics to a virutal pcie switch in the vm
12:21:48 sean-k-mooney nova cant do that but its too hlep with device to device comuication within the vm
12:26:49 sean-k-mooney instead of conencting the gpu to a pcie-root-port you add a pcie-switch-upstream-port contoler instead and you can then group multipel gpus or other device as downstream ports on that switch.

Earlier   Later