| opendevreview | melanie witt proposed openstack/nova master: TPM: support instances with `deployment` secret security https://review.opendev.org/c/openstack/nova/+/942021 | 00:18 |
|---|---|---|
| opendevreview | melanie witt proposed openstack/nova master: Add vtpm_secret_(uuid|value) to LibvirtLiveMigrateData https://review.opendev.org/c/openstack/nova/+/952628 | 00:18 |
| opendevreview | melanie witt proposed openstack/nova master: TPM: prepare to bump service version for live migration https://review.opendev.org/c/openstack/nova/+/962051 | 00:18 |
| opendevreview | melanie witt proposed openstack/nova master: TPM: support live migration of `host` secret security https://review.opendev.org/c/openstack/nova/+/941483 | 00:18 |
| opendevreview | melanie witt proposed openstack/nova master: TPM: support live migration of `deployment` secret security https://review.opendev.org/c/openstack/nova/+/925771 | 00:18 |
| opendevreview | melanie witt proposed openstack/nova master: TPM: test live migration between hosts with different security https://review.opendev.org/c/openstack/nova/+/952629 | 00:18 |
| opendevreview | melanie witt proposed openstack/nova master: TPM: add late check for supported TPM secret security https://review.opendev.org/c/openstack/nova/+/956975 | 00:18 |
| opendevreview | melanie witt proposed openstack/nova master: TPM: opt-in to new TPM secret security via resize https://review.opendev.org/c/openstack/nova/+/962052 | 00:18 |
| opendevreview | melanie witt proposed openstack/nova master: DNM vtpm tempest https://review.opendev.org/c/openstack/nova/+/957477 | 00:18 |
| *** mhen_ is now known as mhen | 02:46 | |
| opendevreview | Merged openstack/nova master: Replace obsolete PCRE packages https://review.opendev.org/c/openstack/nova/+/970063 | 05:00 |
| opendevreview | Merged openstack/nova master: Faults from cell DB missing in GET /servers/detail https://review.opendev.org/c/openstack/nova/+/699176 | 05:00 |
| opendevreview | Merged openstack/nova master: Use *_OR_ADMIN policy defaults for server shares https://review.opendev.org/c/openstack/nova/+/967709 | 05:00 |
| opendevreview | Merged openstack/nova stable/2025.1: libvirt: Skip unsupported firmware types https://review.opendev.org/c/openstack/nova/+/973821 | 05:01 |
| opendevreview | Merged openstack/nova stable/2025.2: Replace removed os-vif-ovs job https://review.opendev.org/c/openstack/nova/+/972663 | 05:01 |
| opendevreview | Merged openstack/nova stable/2025.1: Replace removed os-vif-ovs job https://review.opendev.org/c/openstack/nova/+/972664 | 05:06 |
| opendevreview | Merged openstack/nova stable/2025.2: Add a regression test for ImagePropsWeigher https://review.opendev.org/c/openstack/nova/+/964287 | 05:07 |
| opendevreview | Esra Ozkan proposed openstack/nova master: Fix Concurrent VM Live Migrate - Volume Backup Error https://review.opendev.org/c/openstack/nova/+/973750 | 06:27 |
| opendevreview | Merged openstack/nova stable/2025.2: Fix fill_metadata usage for the ImagePropertiesWeigher https://review.opendev.org/c/openstack/nova/+/964288 | 06:34 |
| opendevreview | Merged openstack/nova stable/2025.2: Fix string format specifier https://review.opendev.org/c/openstack/nova/+/961336 | 06:36 |
| zigo | Hi there! | 08:52 |
| zigo | Has any of you already encounter this ? | 08:52 |
| zigo | https://bugs.launchpad.net/nova/+bug/2138923 | 08:52 |
| gibi | zigo: does not ring a bell to me | 08:59 |
| obre | Hi! Are there any chance to get this fix in the G-release? https://review.opendev.org/c/openstack/nova/+/916409 | 09:14 |
| obre | It seems to be 90% there | 09:15 |
| obre | ? | 09:15 |
| gibi | obre: I see dansmith has some unanswered comment in the patch. I think the next step there is to handle that | 09:16 |
| sean-k-mooney | zigo: i have only seen that if there is a qemu crash | 10:37 |
| sean-k-mooney | zigo: i.e. the isntance was not found because the domain terminated following the crash | 10:38 |
| sean-k-mooney | it might happen other ways btu that the only time i have seen that | 10:38 |
| opendevreview | Takashi Kajinami proposed openstack/nova stable/2024.2: libvirt: Skip unsupported firmware types https://review.opendev.org/c/openstack/nova/+/974409 | 10:53 |
| *** ChanServ changes topic to "This channel is for Nova development | development-planning: https://etherpad.opendev.org/p/nova-2026.1-status | This channel is logged at https://meetings.opendev.org/irclogs/%23openstack-nova/" | 10:55 | |
| zigo | tkajinam: What's your patch about ? UEFI variables ? | 11:20 |
| sean-k-mooney | zigo: are you askign about https://review.opendev.org/c/openstack/nova/+/974409 | 11:23 |
| zigo | yeah | 11:23 |
| sean-k-mooney | its a temory fix to make sure the firmware that is selected by nova is compatibel with the vm devition if your using the stateless firmware option | 11:24 |
| sean-k-mooney | so yes it for uefi guest | 11:24 |
| sean-k-mooney | we are planning to delegate this logic back to libvirt now that it supprots it | 11:24 |
| sean-k-mooney | i.e. allowign libvirt to selct the correct firmware based on the funcitonaltiy you have requested | 11:24 |
| sean-k-mooney | for example if you request secure boot we have to select the secure boot enabeld ovmf uefi firmware or it wont work i belvie the opiste it ture as well if you have it disabled then we need to use the non secure boot version | 11:26 |
| sean-k-mooney | when you enabel stateless vms for confidental compute you similarly need to select the correct firmware file. this is fixing that in nova before we adtop the libvirt feature so it can potitrally be backported | 11:27 |
| sean-k-mooney | well its already being backported but you get the point we exended nova logic before riping it out beacuse we are not backporting the use of the new libvirt feature | 11:28 |
| zigo | Ok, thanks for explaining. | 11:29 |
| opendevreview | Merged openstack/nova stable/2024.2: Fix bug 2000069 https://review.opendev.org/c/openstack/nova/+/970634 | 11:56 |
| DominikDanelski[m] | sean-k-mooney: Sorry, could you go through my replies to your comments in https://review.opendev.org/c/openstack/nova/+/969251?tab=comments ? I think it's very close to being merged after I addressed your questions there, I hope. | 12:01 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Remove networks key from quota schemas https://review.opendev.org/c/openstack/nova/+/974417 | 13:46 |
| stephenfin | sean-k-mooney: gmaan: Could you re-add your +2s to https://review.opendev.org/c/openstack/nova/+/956140/? | 13:48 |
| stephenfin | sean-k-mooney: sorry https://review.opendev.org/c/openstack/nova/+/956141/ for you (need gmaan's +2 on the other one) | 13:48 |
| sean-k-mooney | sure once im off this call ill take a look | 13:49 |
| stephenfin | ta | 13:49 |
| opendevreview | Merged openstack/nova stable/2024.2: Replace removed os-vif-ovs job https://review.opendev.org/c/openstack/nova/+/972665 | 13:52 |
| opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron subnets https://review.opendev.org/c/openstack/nova/+/962190 | 14:04 |
| opendevreview | Stephen Finucane proposed openstack/nova master: typing: Add hints to nova.cmd, nova.conf, nova.console https://review.opendev.org/c/openstack/nova/+/705657 | 14:30 |
| opendevreview | Stephen Finucane proposed openstack/nova master: typing: Add hints to remaining top-level modules https://review.opendev.org/c/openstack/nova/+/705658 | 14:30 |
| opendevreview | Stephen Finucane proposed openstack/nova master: typing: Add hints to nova.virt, nova.virt.libvirt https://review.opendev.org/c/openstack/nova/+/974220 | 14:30 |
| opendevreview | Stephen Finucane proposed openstack/nova master: typing: Correct import issues https://review.opendev.org/c/openstack/nova/+/974221 | 14:30 |
| opendevreview | Stephen Finucane proposed openstack/nova master: mypy: Enable incremental checks https://review.opendev.org/c/openstack/nova/+/974222 | 14:30 |
| opendevreview | Dominik proposed openstack/nova master: NUMA Topology with Resource Providers: Scheduler changes https://review.opendev.org/c/openstack/nova/+/971176 | 15:21 |
| opendevreview | Dominik proposed openstack/nova master: NUMA Topology with Resource Providers: Libvirt NUMA Migrate https://review.opendev.org/c/openstack/nova/+/971177 | 15:21 |
| sean-k-mooney | DominikDanelski[m]: so melwitt and i chated about this breifly a few weeks ago and we were not configed the repoduce actully repoduce things properly. or that the proposed fix didnt create ohter problems | 15:36 |
| sean-k-mooney | but to be honestl i have lost all context on it since then | 15:37 |
| opendevreview | Stephen Finucane proposed openstack/nova master: typing: Add hints to nova.cmd, nova.conf, nova.console https://review.opendev.org/c/openstack/nova/+/705657 | 15:45 |
| opendevreview | Stephen Finucane proposed openstack/nova master: typing: Add hints to remaining top-level modules https://review.opendev.org/c/openstack/nova/+/705658 | 15:45 |
| opendevreview | Stephen Finucane proposed openstack/nova master: typing: Add hints to nova.virt, nova.virt.libvirt https://review.opendev.org/c/openstack/nova/+/974220 | 15:45 |
| opendevreview | Stephen Finucane proposed openstack/nova master: typing: Correct import issues https://review.opendev.org/c/openstack/nova/+/974221 | 15:45 |
| opendevreview | Stephen Finucane proposed openstack/nova master: mypy: Enable incremental checks https://review.opendev.org/c/openstack/nova/+/974222 | 15:45 |
| opendevreview | Stephen Finucane proposed openstack/nova master: Add ruff-check https://review.opendev.org/c/openstack/nova/+/974441 | 15:45 |
| opendevreview | Stephen Finucane proposed openstack/nova master: Disable flake8 rules https://review.opendev.org/c/openstack/nova/+/974442 | 15:45 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-compute in native threading mode https://review.opendev.org/c/openstack/nova/+/965467 | 15:55 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: DNM:Test with oslo.vmware + compute eventlet removal patches https://review.opendev.org/c/openstack/nova/+/973468 | 15:55 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Use an executor to delay STOPPED events https://review.opendev.org/c/openstack/nova/+/974445 | 15:55 |
| gmaan | stephenfin: thanks for followup for 'network' field. I +w on quota ones now and re-approvd other one which were rebased | 17:20 |
| stephenfin | ty! | 17:20 |
| -opendevstatus- NOTICE: The Gerrit service on review.opendev.org will be offline momentarily while we restart it onto new images after an update to the build toolchain | 18:01 | |
| opendevreview | sean mooney proposed openstack/nova master: [WIP] Add PCI device groups for atomic allocation https://review.opendev.org/c/openstack/nova/+/973604 | 18:52 |
| melwitt | dansmith, gmaan: thank you both for reviews :) but I regret to inform you there is a prereq refactor patch underneath it for the service_auth.py commonizing https://review.opendev.org/c/openstack/nova/+/974267 | 19:26 |
| rm_work[m] | anyone offhand have any ideas why placement might take a long time to show a node within openstack allocation candidate list after it goes available in ironic? it's usually about 5 minutes in normal operation and sometimes we see it take up to 20 minutes :( | 19:32 |
| rm_work[m] | is there some setting we can tweak? is this a periodic task that is running too slowly? | 19:33 |
| sean-k-mooney | melwitt: +1 for now ill try and take a look properly on monday but it looks ok at first glance | 19:34 |
| melwitt | thanks sean-k-mooney :) | 19:34 |
| sean-k-mooney | rm_work[m]: is the RP not there or is it there but disabled? | 19:35 |
| sean-k-mooney | rm_work[m]: my guess of the top of my head is that its related to the update_aviable_resouces periodic task tiem of 5 mins and the delay is just in the virt driver runnign that and noticing the new node | 19:35 |
| rm_work[m] | ah ok so we could maybe tune that task down to 1m or something? | 19:38 |
| gmaan | melwitt: yeah, I was reviewing that also, lgtm. thanks for refactoring. | 19:38 |
| melwitt | thanks gmaan | 19:38 |
| sean-k-mooney | rm_work[m]: perhaps this is the optioni mentioned https://docs.openstack.org/nova/latest/configuration/config.html#DEFAULT.update_resources_interval | 19:39 |
| rm_work[m] | i think the BAD case (the 20m I talked about) might be when we had other issues going on slowing things down, so maybe it's as simple as setting the update_aviable_resouces periodic to run faster | 19:39 |
| sean-k-mooney | rm_work[m]: what exactly do you mean by aviable by the way | 19:39 |
| sean-k-mooney | do you meain aviable in the ironic node list | 19:39 |
| *** gmaan is now known as gmaan_afk | 19:39 | |
| rm_work[m] | the ironic state yeah | 19:40 |
| sean-k-mooney | ack i dont recall if there is a specifric addtioanl dely in the ironic case but maybe JayF woudl know | 19:40 |
| rm_work[m] | he said he didn't :D | 19:40 |
| JayF | I sent rm_work[m] to you :) | 19:41 |
| sean-k-mooney | ack i was onderign if it could be relate the the lease stuff | 19:41 |
| rm_work[m] | we'll try tweaking that periodic | 19:41 |
| rm_work[m] | or, that config value | 19:41 |
| JayF | I honestly have no idea where to even start from the Ironic side; we just fetch nodes and pass them in | 19:41 |
| JayF | *Ironic driver side | 19:41 |
| JayF | and I don't know much about the mechanisms on the other side of that API interface | 19:41 |
| rm_work[m] | the default being "0" to "run at the default interval" does not tell me what the default interval is, lol | 19:42 |
| rm_work[m] | why would the default not just BE the default interval 😅 | 19:42 |
| sean-k-mooney | the default interval for perodics in oslo and as a result nova is 500 seconds | 19:42 |
| sean-k-mooney | sory 300 | 19:42 |
| rm_work[m] | ah ok | 19:42 |
| sean-k-mooney | so 5 mins | 19:42 |
| rm_work[m] | yeah | 19:42 |
| rm_work[m] | alright, thanks we will poke at it | 19:42 |
| JayF | rm_work[m]: please make sure if you figure out a set of good configs for your scale, please document it -- even if just in a mailing list half-blogpost | 19:43 |
| sean-k-mooney | so oen thing to look at would be the logs in the relevent nova-comptue between the previous start of that perodic and then first one after the node appeard | 19:44 |
| sean-k-mooney | nova-comnpute when its managing a lot fo ironic nodes can take a long time for that perodic to complete | 19:44 |
| rm_work[m] | we have 240ish nodes so i don't think it's considered "a lot" | 19:45 |
| sean-k-mooney | we shoudl eventually improve that but ironic shardign might help | 19:45 |
| sean-k-mooney | 240 nodes manged by a singel nova compute agent? | 19:45 |
| rm_work[m] | our clouds are relatively small ATM | 19:45 |
| sean-k-mooney | or are you still using the deprecated peers parmaters with the hashring that loadblances them between multiple agents? | 19:46 |
| rm_work[m] | just one agent | 19:46 |
| rm_work[m] | also FWIW we're on 2024.2 | 19:46 |
| sean-k-mooney | ya so 250 ironics on 1 nova compute is both a lot and not a lot. for perodics that loop over all the nodes they manage and update placement that can be slow since itd getign the infor one ironic node at a time | 19:47 |
| sean-k-mooney | for actully triging the provisioning ectra its fine | 19:47 |
| rm_work[m] | hmm | 19:47 |
| rm_work[m] | but it sounds like you're saying we CAN'T have multiple? because whatever way they spread out is deprecated? or is that just one of multiple methods | 19:48 |
| rm_work[m] | surprised it's sequential for the node data gathering | 19:48 |
| sean-k-mooney | there is a repalcement know as ironic sharding | 19:48 |
| sean-k-mooney | that allow you to map subset of ironic nodes to diffent nova compute agents | 19:48 |
| sean-k-mooney | im not sure at what scale using that is recommended | 19:49 |
| sean-k-mooney | rm_work[m]: i would try seeign how long the perodic takes to run | 19:49 |
| sean-k-mooney | if its like 90 seconds your fine | 19:49 |
| sean-k-mooney | if its 15 mins then that the issue | 19:49 |
| rm_work[m] | hmm k | 19:50 |
| rm_work[m] | will do some timing and log diving | 19:50 |
| sean-k-mooney | https://specs.openstack.org/openstack/nova-specs/specs/2024.1/implemented/ironic-shards.html is the new way of scaling nova-compute with ironic | 19:51 |
| sean-k-mooney | the peer_list is deprecated but still in master because we never got aroudn to deleting it | 19:51 |
| sean-k-mooney | but we really should at some point | 19:51 |
| sean-k-mooney | rm_work[m]: for what its worht it woudl be posible for us to optimize this in nova in the future by say delegting the node update to a thread pool on each perodic but we have not been inveting time in that space with the eventlet removal ectra that has been happening | 19:53 |
| rm_work[m] | right yeah, no worries. thanks for taking the time to explain! | 19:58 |
| rm_work[m] | yeah my thought would just be to parallelize the node checks in the job | 19:58 |
| rm_work[m] | I wonder if that would actually be that complicated... | 19:58 |
| sean-k-mooney | in the past yes with all the work we are doing to make the compute agent work with thread pool executors instead of eventlet i suspect its simpler then before | 19:59 |
| sean-k-mooney | i would focus on confiming this is the issue first before going much further however | 20:00 |
| sean-k-mooney | if we can confirm this as a problem it would be a good topic for a ptg session with the ironic folks | 20:00 |
| opendevreview | Merged openstack/nova master: Refactor service user authentication https://review.opendev.org/c/openstack/nova/+/974267 | 22:30 |
| opendevreview | Merged openstack/nova master: TPM: support instances with `deployment` secret security https://review.opendev.org/c/openstack/nova/+/942021 | 22:30 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!