| melwitt | did anyone else stop receiving gerrit email notifications after about 4 hours ago? | 01:43 |
|---|---|---|
| *** mhen_ is now known as mhen | 02:17 | |
| opendevreview | Merged openstack/nova master: libvirt: add configuration option for volume AIO mode https://review.opendev.org/c/openstack/nova/+/964848 | 05:20 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Fix missing guest.migrate(parallel=) mock arg https://review.opendev.org/c/openstack/nova/+/969503 | 09:10 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Collect result of _live_migration_operation https://review.opendev.org/c/openstack/nova/+/969501 | 09:10 |
| gibi | Uggla: I'm not sure anything is left tirageing in https://etherpad.opendev.org/p/nova-bug-triage-roster#L24 Does that duplicted bug shows up in your query as untriaged? It should not. As it is connected to another bug as duplicate and that bug is fixed | 09:45 |
| Uggla | gibi, if you set it as duplicate that's fine. It should not appear anymore. | 09:53 |
| noonedeadpunk | hey folks! do you know if passing of any of these flags are supproted by nova? https://www.qemu.org/docs/master/system/i386/hyperv.html | 10:10 |
| noonedeadpunk | I guess not, unless I'm missing a way to pass something completely arbitrary... | 10:11 |
| gibi | Uggla: it is set as duplicate since 2025-07-29 hence my question | 10:12 |
| Uggla | gibi, oh ok that's probably because I get the bugs from a previous doc I did, not fresh at all. Sorry about that. | 10:14 |
| gibi | no worries | 10:14 |
| gibi | thanks for organizing the triage | 10:14 |
| gibi | noonedeadpunk: appreantly yes, based on the fields here we have set of enlightments https://github.com/openstack/nova/blob/1712ae48e3111e0a2c178b9d601572d624a56374/nova/virt/libvirt/config.py#L2964 | 10:15 |
| gibi | noonedeadpunk: https://github.com/openstack/nova/blob/1712ae48e3111e0a2c178b9d601572d624a56374/nova/virt/libvirt/driver.py#L6678 you need set os_type | 10:17 |
| noonedeadpunk | huh | 10:17 |
| noonedeadpunk | Ok, so it's done on it's own, nice! | 10:17 |
| noonedeadpunk | thanks gibi! | 10:17 |
| gibi | https://docs.openstack.org/glance/latest/admin/useful-image-properties.html | 10:17 |
| noonedeadpunk | yeah, I knew about os_type, but I didn't know about hidden logic behind it:) | 10:18 |
| noonedeadpunk | will propose documentation update probably :) | 10:19 |
| gibi | noonedeadpunk: thanks | 10:19 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Upgrade note for concurrency mode default change https://review.opendev.org/c/openstack/nova/+/969888 | 10:32 |
| gibi | gmaan: sean-k-mooney: dansmith: ^^ this is what I come up with based on the discussion about the upgrade impact of eventlet removal and our default mode change in G | 10:33 |
| opendevreview | Lajos Katona proposed openstack/nova master: WIP: Use SDK for Neutron Ports https://review.opendev.org/c/openstack/nova/+/969298 | 10:54 |
| sean-k-mooney | gibi: im not sure i agree with the recommendation to decouple the cahnge in mode in gerneal but its a more reasonable recomendtion for slurp upgrades | 11:00 |
| sean-k-mooney | i dont really think we shoudl be recommendign that operators every use the env var out side of optin in to threadign to test early | 11:01 |
| sean-k-mooney | it does exist as an escape hatch if we messed up | 11:02 |
| sean-k-mooney | so i dont object to it existing just that for me makeing the swtich trasnparent to the end user is very imporant but i get the performance tuning concern | 11:03 |
| sean-k-mooney | gibi: do we know if any of the larger operators have started testign treading mode since the 2025.2 release? | 11:03 |
| opendevreview | Will Szumski proposed openstack/nova master: WIP: Adds regression test for bug LP#2132984 https://review.opendev.org/c/openstack/nova/+/969380 | 11:15 |
| opendevreview | Will Szumski proposed openstack/nova master: Fix race betweem instance claim and anti affinity check https://review.opendev.org/c/openstack/nova/+/969894 | 11:15 |
| gibi | sean-k-mooney: I got no indication if any large operator testing or tested threading mode in nova | 12:14 |
| gibi | sean-k-mooney: I think one of the main reason to have a way to switch between modes is to avoid big bang integration of the threading mode during upgrade. Especially due to the performance tuning needs. | 12:16 |
| gibi | sean-k-mooney: if it helps then I can try to give an oslo config flag instead of a env variable, but I don't see a good ROI of that at the moment | 12:16 |
| sean-k-mooney | so that was explcit not a goal for me because i saw it as a failure on our part if operatos needed to think about htis to that degree | 12:17 |
| sean-k-mooney | ie.. the uptrade shoudl be tansparent becuase we shoudl get the default right before chanigng them | 12:17 |
| sean-k-mooney | but ya i dont think we need to make it an oslo config | 12:18 |
| sean-k-mooney | im just really hoping that most operator will upgrade form 2026.1 to 2027.1 and not notice that we changed the threadning mode | 12:18 |
| gibi | the problem for me is that I cannot define good default that will work for differntly sized and shaped deployments | 12:19 |
| gibi | in the past we used large numbers as pool size so that covered a wide range of deployments | 12:19 |
| gibi | with threading we cannot use those large numbers any more without consequences | 12:19 |
| sean-k-mooney | yep i understand. given we only supprot say 10 boot operation on a server or a limite number of migraton ectra i think the small numbers for the comptue are actully likely to be reasonble | 12:26 |
| sean-k-mooney | for the api the scaling mode has alwasy been mainly via proceses, | 12:27 |
| sean-k-mooney | the schduler and conductor are probaly the ones i woudl be most concerned with | 12:27 |
| sean-k-mooney | both are horizontally scable and virtucally scalabel with workers but ya they may need different tuneing | 12:28 |
| sean-k-mooney | gibi: i am quitely hopeful that we will see a similar uplift in performance that ironic saw although we may have been using less problematic patterns | 12:28 |
| gibi | scatter gather across multi cell clouds will not scale by scaling the nova-api process | 12:58 |
| gibi | it will only scale if you scale the number of threads close to the number of cells in the threadpool | 12:58 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Enable mypy on nova/utils.py https://review.opendev.org/c/openstack/nova/+/969936 | 14:16 |
| dansmith | bauzas: can you +W this trivial one-liner mock fix? https://review.opendev.org/c/openstack/nova/+/969503 | 15:01 |
| dansmith | we didn't catch it on initial merge because of the bug fixed by the patch above it | 15:01 |
| bauzas | ack | 15:02 |
| dansmith | thanks | 15:02 |
| dansmith | the patch above would be nice too, but not as trivial | 15:03 |
| gmaan | gibi: ack, will take alook today | 16:33 |
| gmaan | bauzas: gibi sean-k-mooney I think +w was missed until something we are waiting for, otherwise, can either of you please hit +w https://review.opendev.org/c/openstack/nova-specs/+/969543 | 16:34 |
| sean-k-mooney | gmaan: well for the backlog spec i guess | 16:53 |
| gmaan | sean-k-mooney: did not get? this is in the backlog spec | 17:06 |
| JayF | Stable backport of an Ironic bugfix, cleanly applied, V+1 already, for your consideration: https://review.opendev.org/c/openstack/nova/+/969833 | 17:33 |
| sean-k-mooney | gmaan: we are now techinically past spec freeze but given that is a backlog spec change that was +2id but not +w to allow folks to review i think we can proceed with it | 17:38 |
| opendevreview | Merged openstack/nova master: Fix missing guest.migrate(parallel=) mock arg https://review.opendev.org/c/openstack/nova/+/969503 | 17:44 |
| opendevreview | Merged openstack/nova-specs master: Add spec3 in graceful shutdown backlog spec https://review.opendev.org/c/openstack/nova-specs/+/969543 | 17:47 |
| opendevreview | sean mooney proposed openstack/nova-specs master: add spec for resource tracker notifications https://review.opendev.org/c/openstack/nova-specs/+/967712 | 18:09 |
| opendevreview | Merged openstack/nova master: Collect result of _live_migration_operation https://review.opendev.org/c/openstack/nova/+/969501 | 18:10 |
| opendevreview | melanie witt proposed openstack/nova master: Make QEMU_IMG_LIMITS process limits configurable https://review.opendev.org/c/openstack/nova/+/969538 | 18:38 |
| melwitt | gibi: changed to Closes-Bug ^ and added a few more words to the commit message | 18:40 |
| gmaan | sean-k-mooney: yeah, that is my understanding too. | 19:06 |
| opendevreview | melanie witt proposed openstack/nova master: Make QEMU_IMG_LIMITS process limits configurable https://review.opendev.org/c/openstack/nova/+/969538 | 20:42 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!