| opendevreview | Abhishek Kekane proposed openstack/nova master: [DNM] Revert^2 "Support glance's new location API" https://review.opendev.org/c/openstack/nova/+/950623 | 06:40 |
|---|---|---|
| opendevreview | Lajos Katona proposed openstack/os-vif master: OVS Trunk: Add bridge_name to external_ids https://review.opendev.org/c/openstack/os-vif/+/949736 | 08:53 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Translate scatter-gather to futurist https://review.opendev.org/c/openstack/nova/+/947966 | 12:46 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Use futurist for _get_default_green_pool() https://review.opendev.org/c/openstack/nova/+/948072 | 12:46 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Replace utils.spawn_n with spawn https://review.opendev.org/c/openstack/nova/+/948076 | 12:46 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Add spawn_on https://review.opendev.org/c/openstack/nova/+/948079 | 12:46 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Move ComputeManager to use spawn_on https://review.opendev.org/c/openstack/nova/+/948186 | 12:46 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Move ConductorManager to use spawn_on https://review.opendev.org/c/openstack/nova/+/948187 | 12:46 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Make nova.utils.pass_context private https://review.opendev.org/c/openstack/nova/+/948188 | 12:46 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Rename DEFAULT_GREEN_POOL to DEFAULT_EXECUTOR https://review.opendev.org/c/openstack/nova/+/948086 | 12:46 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Make the default executor configurable https://review.opendev.org/c/openstack/nova/+/948087 | 12:46 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Print ThreadPool statistics https://review.opendev.org/c/openstack/nova/+/948340 | 12:46 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Document threading mode and tuneables https://review.opendev.org/c/openstack/nova/+/949364 | 12:46 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Allow services to start with threading https://review.opendev.org/c/openstack/nova/+/948311 | 12:46 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-next with n-sch in threading mode https://review.opendev.org/c/openstack/nova/+/948450 | 12:46 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Translate scatter-gather to futurist https://review.opendev.org/c/openstack/nova/+/947966 | 12:46 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Use futurist for _get_default_green_pool() https://review.opendev.org/c/openstack/nova/+/948072 | 12:46 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Replace utils.spawn_n with spawn https://review.opendev.org/c/openstack/nova/+/948076 | 12:46 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Add spawn_on https://review.opendev.org/c/openstack/nova/+/948079 | 12:46 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Move ComputeManager to use spawn_on https://review.opendev.org/c/openstack/nova/+/948186 | 12:46 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Move ConductorManager to use spawn_on https://review.opendev.org/c/openstack/nova/+/948187 | 12:46 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Make nova.utils.pass_context private https://review.opendev.org/c/openstack/nova/+/948188 | 12:46 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Rename DEFAULT_GREEN_POOL to DEFAULT_EXECUTOR https://review.opendev.org/c/openstack/nova/+/948086 | 12:47 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Make the default executor configurable https://review.opendev.org/c/openstack/nova/+/948087 | 12:47 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Print ThreadPool statistics https://review.opendev.org/c/openstack/nova/+/948340 | 12:47 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Document threading mode and tuneables https://review.opendev.org/c/openstack/nova/+/949364 | 12:47 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Allow services to start with threading https://review.opendev.org/c/openstack/nova/+/948311 | 12:47 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-next with n-sch in threading mode https://review.opendev.org/c/openstack/nova/+/948450 | 12:47 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Centralize cooperative yield https://review.opendev.org/c/openstack/nova/+/950991 | 13:22 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: [hacking] N374 do not use time.sleep(0) to yield https://review.opendev.org/c/openstack/nova/+/950992 | 13:22 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Do not yield in threading mode https://review.opendev.org/c/openstack/nova/+/950994 | 13:32 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Add functional reproducer for bug 2102038 https://review.opendev.org/c/openstack/nova/+/944061 | 14:06 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Multiple spec per PCI alias limitation https://review.opendev.org/c/openstack/nova/+/944062 | 14:06 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Add functional reproducer for bug 2102038 https://review.opendev.org/c/openstack/nova/+/944061 | 14:52 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Multiple spec per PCI alias limitation https://review.opendev.org/c/openstack/nova/+/944062 | 14:52 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Return HTTP400 for multi spec pci alias if PCI in Placement https://review.opendev.org/c/openstack/nova/+/951007 | 14:52 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/nova master: Fix live migration error logging https://review.opendev.org/c/openstack/nova/+/951009 | 14:58 |
| Uggla | Nova meeting in ~1h | 15:00 |
| noonedeadpunk | sean-k-mooney: this is smth I've accidentally catched while investigating my live-migration issue ^ | 15:00 |
| noonedeadpunk | interestingly, failure happens sometimes on the source, but it still results in shutdown domain and ERROR state, with `shutting down, reason=crashed` in qemu | 15:02 |
| sean-k-mooney | oh i see | 15:07 |
| sean-k-mooney | its exepcting the pritn behavior | 15:07 |
| sean-k-mooney | or rahter the person that wrote it was | 15:08 |
| sean-k-mooney | print is variadic | 15:08 |
| noonedeadpunk | yup | 15:15 |
| noonedeadpunk | I will check on the unit test... | 15:21 |
| bauzas | Uggla: I have an appointment at 1630UTC so I'll miss the half of the meeting fwiw | 15:34 |
| Uggla | bauzas, no pb | 15:36 |
| Uggla | Meeting in ~10mn | 15:48 |
| Uggla | #startmeeting nova | 16:00 |
| opendevmeet | Meeting started Tue May 27 16:00:40 2025 UTC and is due to finish in 60 minutes. The chair is Uggla. Information about MeetBot at http://wiki.debian.org/MeetBot. | 16:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 16:00 |
| opendevmeet | The meeting name has been set to 'nova' | 16:00 |
| Uggla | Hello everyone | 16:00 |
| fwiesel | o/ | 16:01 |
| Callum027 | o/ | 16:01 |
| gibi | o/ | 16:01 |
| Uggla | awaiting a moment for people to join. | 16:01 |
| dpascual | o/ | 16:01 |
| Uggla | #topic Bugs (stuck/critical) | 16:02 |
| Uggla | #info No Critical bug | 16:03 |
| elodilles | o/ | 16:03 |
| Uggla | bauzas, still working on https://review.opendev.org/c/openstack/nova/+/922140 bauzas working on a patch to enable nova-next. | 16:03 |
| bauzas | o/ | 16:03 |
| Uggla | bauzas, something you want to say ? | 16:04 |
| Uggla | about ^ | 16:05 |
| gmaan | o/ | 16:05 |
| Uggla | #topic Gate status | 16:06 |
| Uggla | #link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs | 16:06 |
| Uggla | #link https://etherpad.opendev.org/p/nova-ci-failures-minimal | 16:06 |
| Uggla | #link https://zuul.openstack.org/builds?project=openstack%2Fnova&project=openstack%2Fplacement&branch=stable%2F*&branch=master&pipeline=periodic-weekly&skip=0 Nova&Placement periodic jobs status | 16:06 |
| Uggla | #info Please look at the gate failures and file a bug report with the gate-failure tag. | 16:06 |
| Uggla | #info Please look at the gate failures and file a bug report with the gate-failure tag. | 16:06 |
| Uggla | Seems that we are hitting a lot https://bugs.launchpad.net/glance/+bug/2109428 | 16:07 |
| Uggla | gibi, do you want to add something about it ? | 16:07 |
| gibi | no | 16:07 |
| gibi | it is not nova | 16:08 |
| gibi | and this week I did not checked if we hit it again | 16:08 |
| Uggla | yep I hit it too last week. | 16:08 |
| Uggla | maybe dansmith knows more about this issue ? | 16:09 |
| bauzas | I've seen a revert proposal | 16:09 |
| bauzas | from Glance IIRC | 16:09 |
| dansmith | is this pre-revert? | 16:09 |
| dansmith | https://review.opendev.org/c/openstack/nova/+/950336 | 16:10 |
| dansmith | merged may 20 | 16:10 |
| gibi | I last checked on the 19th | 16:10 |
| gibi | I can check for hits since then... | 16:10 |
| dansmith | tl;dr the new glance location API is fundamentally broken for ceph and cinder backends, we reverted nova's use of it while working on a solution for those backends in glance | 16:10 |
| Uggla | thx for the info dan | 16:12 |
| gibi | I see two hits after 20th | 16:12 |
| gibi | | 455491157c0f44cebf2cfcfe9f1c7b5c | 2025-05-22T07:20:41 | check | https://review.opendev.org/950623 | master | | 16:12 |
| gibi | | f61be3ef2b9a442d8843f65152343018 | 2025-05-20T12:55:29 | check | https://review.opendev.org/948450 | master | | 16:12 |
| dansmith | one is the revert of revert, so expected | 16:12 |
| dansmith | the other is too close to merge, probably used old code | 16:13 |
| gibi | OK then maybe we are moving to the good direction | 16:13 |
| gibi | dansmith: thanks for the info | 16:13 |
| dansmith | np | 16:13 |
| Uggla | ok mobing on | 16:13 |
| Uggla | #topic tempest-with-latest-microversion job status | 16:13 |
| Uggla | #link https://zuul.opendev.org/t/openstack/builds?job_name=tempest-with-latest-microversion&skip=0 | 16:13 |
| gmaan | no new update this week | 16:13 |
| Uggla | ok so let's continue | 16:14 |
| Uggla | #topic Release Planning | 16:14 |
| Uggla | #link https://releases.openstack.org/flamingo/schedule.html | 16:14 |
| Uggla | #info Nova deadlines are set in the above schedule | 16:14 |
| Uggla | #info Only 1 weeks before Nova Spec Soft Freeze, do not forget to submit your specs. | 16:15 |
| Uggla | #topic Review priorities | 16:15 |
| Uggla | #link https://etherpad.opendev.org/p/nova-2025.2-status | 16:15 |
| Uggla | fyi I have just asked for the Cloud hypervisor, to have the first patches. As we discussed last week. | 16:17 |
| Uggla | but tbh I have not looked closely yet to know if we progressed. | 16:18 |
| Uggla | #topic OpenAPI | 16:18 |
| Uggla | #link: https://review.opendev.org/q/topic:%22openapi%22+(project:openstack/nova+OR+project:openstack/placement)+-status:merged+-status:abandoned | 16:19 |
| Uggla | 22 patches remaining. (+1) | 16:19 |
| Uggla | #topic Stable Branches | 16:19 |
| Uggla | elodilles, your turn. | 16:19 |
| elodilles | thanks, i'll be short, as there's nothing to report | 16:20 |
| elodilles | #info stable branches (stable/2025.1 and stable/2024.*) seem to be in OK state | 16:20 |
| elodilles | Uggla: back to you | 16:20 |
| Uggla | cool thx elodilles | 16:20 |
| Uggla | #topic vmwareapi 3rd-party CI efforts Highlights | 16:20 |
| Uggla | fwiesel, something to share ? | 16:20 |
| fwiesel | So, from my side I want to introduce dpascual. He will be my stand-in, in case I should be inable to attend. | 16:21 |
| Uggla | dpascual, welcome | 16:21 |
| dpascual | hi everyone | 16:21 |
| fwiesel | He is operating the vmwareapi CI, and hopefully should keep you in the loop in case I can't | 16:22 |
| fwiesel | Otherwise, since last Tuesday, we have a problem with some change in neutron. I am debugging this, so the CI failures come from there. | 16:22 |
| fwiesel | That's from my side. | 16:22 |
| fwiesel | Uggla: back to you | 16:23 |
| Uggla | fwiesel, thx good to know. | 16:23 |
| Uggla | #topic Gibi's news about eventlet removal. | 16:24 |
| Uggla | #link Series: https://gibizer.github.io/categories/eventlet/ | 16:24 |
| Uggla | #link nova-scheduler series is ready for core review, starting at https://review.opendev.org/c/openstack/nova/+/947966 | 16:24 |
| gibi | no new blogpost as my prios mostly elsewhere these weeks | 16:24 |
| gibi | sean-k-mooney gave some review and I respin the series to fix them | 16:25 |
| gibi | oslo.service threading backend released and the global req bumped so we are now using the official oslo.service version in the series | 16:25 |
| gibi | that is all | 16:25 |
| Uggla | \o/ | 16:25 |
| Uggla | thx gibi | 16:26 |
| Uggla | #topic Open discussion | 16:26 |
| Uggla | #link Callum027: Spec-less blueprint for adding new attributes to libvirt domain XML metadata: https://blueprints.launchpad.net/nova/+spec/xml-image-meta | 16:26 |
| Callum027 | Hi guys, I am proposing a spec-less blueprint to add new metadata to Nova's libvirt domain metadata for Ceilometer and other telemetry-related OpenStack services. | 16:26 |
| Uggla | Hi Callum027 | 16:26 |
| Callum027 | To give some context, a long time ago Ceilometer used to perform API queries for polling instances. Because this was not scalable it was changed to reading the XML metadata Nova adds to the libvirt domain metadata on hypervisors. My understanding is that this was not the original intention for how this metadata is used, but Ceilometer now relies on | 16:27 |
| Callum027 | it and has done for a long time, and changing this at this point would be a big project. | 16:27 |
| Callum027 | The problem that Ceilometer faces is that this metadata does not provide enough information for Ceilometer to do what it needs, so Ceilometer still needs to perform Nova API queries for some things, which scales badly with the number of running instances in a cloud and increases API load. What I'm proposing are additions to Nova's libvirt domain | 16:27 |
| Callum027 | metadata to not only make it unnecessary for Ceilometer to make API queries for compute polling in most cases, but also add additional metadata that make it easier to use services like CloudKitty to implement OpenStack billing. | 16:27 |
| Callum027 | I'd appreciate feedback if anyone has any (sean-k-mooney has +1'd the spec and changes as an initial pass), and if everything looks good, approval to proceed with the patches created for this. | 16:27 |
| gibi | btw this metadata in the libvirt xml helps with troubleshooting as it tells the details of the VM from the xml instead of looking it up form the logs of from a DB dump | 16:27 |
| sean-k-mooney | yep | 16:28 |
| gibi | so I'm supporting | 16:28 |
| sean-k-mooney | this is why i want to addm ore to it | 16:28 |
| gibi | this | 16:28 |
| sean-k-mooney | i mean it help with billing and other ceilomenter usecases too | 16:28 |
| sean-k-mooney | but if im being selfish having the info in the xml/logs means i do not have to ask supprot to ask a custoemr for it | 16:29 |
| sean-k-mooney | so shortening that round trip time is a massive win IMO | 16:29 |
| Callum027 | In our experience, having the flavour metadata was useful for helping determine how an instance ended up on a hypervisor | 16:29 |
| Uggla | I understand we agree with this BP. Everybody ok with a specless one ? | 16:30 |
| gibi | yeah I'm OK with it being a specless one | 16:31 |
| sean-k-mooney | im supportive fo that and just proceedign witht he review and ensuring we have a good release note | 16:31 |
| sean-k-mooney | there are some askpsec that woudl be nice to have beyond what Callum027 would liek to do. but i dont neesisarly want to expand the scope of there request to include them | 16:32 |
| sean-k-mooney | so if we decied to work on those we can file a follow up bluepritn or revisit the discussion | 16:32 |
| gibi | +1 on follow up and keep the scope small and land it quickly | 16:33 |
| Callum027 | Sounds great, thanks guys, we've actually been using this in our production envs for a little while and it's been working great so I'm confident it's ready to go | 16:33 |
| gibi | Callum027 thanks for working on it | 16:33 |
| Uggla | Callum027, cool | 16:34 |
| Callum027 | That's it from me Uggla, thanks | 16:34 |
| Callum027 | Actually, do I need to add this to the Etherpad? | 16:35 |
| Uggla | Callum027, it is ok, I have just updated it | 16:36 |
| Callum027 | Awesome, thank you! | 16:36 |
| Uggla | btw, thanks for joining this call. | 16:36 |
| Uggla | I guess sean-k-mooney wanted to discuss this https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/NSW3OG5ME5RDPQYLHF4T4RCWPQYG57PK/ too | 16:37 |
| gibi | it seems they have an RFE but they don't have a person volunteering to impelemnt | 16:40 |
| gibi | I guess we can wait until we have somebody who at leat can propose a spec | 16:40 |
| sean-k-mooney | /me clicks | 16:40 |
| gibi | sean-k-mooney: this is the memballon RFE | 16:40 |
| sean-k-mooney | oh that | 16:40 |
| sean-k-mooney | ya i kind of felt like may i shoudl jsut spend an afternoon and write it | 16:41 |
| sean-k-mooney | basicaly the short version is | 16:41 |
| sean-k-mooney | nova orgianlly did not enabel the mem balloan at all unless you enabel memory stat reporting | 16:41 |
| sean-k-mooney | at some point libvirt started adding it by default | 16:41 |
| sean-k-mooney | but we have never actullly used it to release/reclaim unused memory form the guest | 16:41 |
| sean-k-mooney | because that used to require a special agent on the host | 16:42 |
| sean-k-mooney | proxmox and ovirt provided those | 16:42 |
| sean-k-mooney | that now built into libvirt | 16:42 |
| sean-k-mooney | so we coudl just hardcode 2 xml atributes | 16:42 |
| sean-k-mooney | which woudl allwo memory to be release in 2 cases | 16:42 |
| sean-k-mooney | oen when the OOM reaper is about to kill the vm if we have auto defalte on the vm gets a chane to free memory | 16:43 |
| sean-k-mooney | the other one "freePageReporting" | 16:43 |
| sean-k-mooney | allow the memory to be retruend to the host any time the guest free memory | 16:43 |
| sean-k-mooney | i think it would nice ot supprot both in nova | 16:43 |
| sean-k-mooney | but ya someone just needt to write it | 16:44 |
| sean-k-mooney | i kind of wodnered how other felt | 16:44 |
| gibi | if you have time to write it I'm not against it, but it is just a nice to have | 16:44 |
| sean-k-mooney | track this as a wishlist RFE bug? | 16:45 |
| sean-k-mooney | spec? | 16:45 |
| sean-k-mooney | i feel like this is one of those things where is quicker to write then disucss :) | 16:45 |
| Uggla | Yep that sounds quite strait forward so specless BP, I think. | 16:46 |
| gibi | if this is hardcode as enalbed then specless | 16:46 |
| sean-k-mooney | all of our qemu and linvert version miniums are above wehre this was added so there is no migration or upgrade aspect either | 16:46 |
| gibi | does it effect realtime / nfv guest performance? | 16:46 |
| sean-k-mooney | :) | 16:46 |
| sean-k-mooney | well yes but they normlaly turn off the mem ballon entirly | 16:47 |
| sean-k-mooney | because it addds overhead that they dont tolllerate | 16:47 |
| sean-k-mooney | whicih is why we shoudl fix the bug that stops you form being able to do that | 16:47 |
| gibi | ahh | 16:47 |
| sean-k-mooney | https://bugs.launchpad.net/nova/+bug/2069192 | 16:47 |
| gibi | so we should not make that situation worse :) Lets fix the bug first then | 16:47 |
| sean-k-mooney | so this would be hard code on if you have the mem balloan enabled | 16:48 |
| sean-k-mooney | gibi: rajesh ahs a patch up to fix that bug https://review.opendev.org/c/openstack/nova/+/945621 | 16:48 |
| gibi | cool | 16:48 |
| gibi | so I'm OK with a specless | 16:49 |
| gibi | and I will look at that bugfix | 16:49 |
| sean-k-mooney | Uggla: gibi ok lets do this. if i find time to work on this before m2 ill file a specless blueprint and we can aprove it then | 16:49 |
| sean-k-mooney | if not i can file a RFE bug or similar tracker and if someone has time we can work on it later | 16:49 |
| gibi | sounds good to me | 16:50 |
| Uggla | +1 | 16:50 |
| Uggla | I have tracked that in the etherpad no to forget. | 16:51 |
| Uggla | Something else to discuss ? | 16:52 |
| gibi | - | 16:52 |
| Uggla | Do you have still energy left to triage 1 or 2 bugs ? | 16:53 |
| gibi | - | 16:53 |
| sean-k-mooney | we can proably do one | 16:53 |
| sean-k-mooney | although if you have not noticed im a littel disrtrated today. but we can try to do one | 16:54 |
| Uggla | #topic Bug scrubbing | 16:54 |
| Uggla | #link: https://etherpad.opendev.org/p/nova-bug-selection-for-triaging#L4 | 16:54 |
| Uggla | Just one | 16:55 |
| Uggla | https://bugs.launchpad.net/nova/+bug/2103500 | 16:55 |
| Uggla | To my mind that sounds a valid one. At least regarding the comment from Brian. | 16:56 |
| gibi | db error in the neutron code, this is a neutron bug | 16:58 |
| gibi | at least neutron should not return 500 | 16:58 |
| sean-k-mooney | so we had a vm create request with --network | 16:59 |
| sean-k-mooney | it existsed at the api but by the time we got to the compute it as deleteed | 16:59 |
| sean-k-mooney | i guesss it coudl happen when we do the quota check as well | 16:59 |
| gmaan | and nova do capture NetworkNotFound exception if it is not found and requested https://github.com/openstack/nova/blob/221a3e89e8988bc664298106ee691a4e41ca71f9/nova/api/openstack/compute/servers.py#L842 | 17:00 |
| sean-k-mooney | in anycase ya we shoudl get back a 400 not a 500 | 17:00 |
| gmaan | or somewhere neutron exception is lost and not raised to controller ? | 17:00 |
| sean-k-mooney | the expected behivor here shold be that nova puts the vm in error | 17:00 |
| sean-k-mooney | and when the vm is deleted we would clean up any prot that were created then | 17:01 |
| sean-k-mooney | based on teh delete on termeinate behaivor | 17:01 |
| gmaan | yeah, bug say ignore the deleted network port and continue VM creation which does not seems right | 17:01 |
| sean-k-mooney | its not form a nova point of view because we could not honner the orginal request | 17:01 |
| sean-k-mooney | so depending on where this failed it soudl be burried in cell 0, rejected at the quoata check with a 400 or in error in the cell db if it got all the way to the compute | 17:02 |
| sean-k-mooney | reading it more careflyy im tempeted to mark it inviald in nova | 17:04 |
| * gibi bows to Uggla and disappers | 17:04 | |
| sean-k-mooney | in there case its the quota check that is failing whne we list all prots for a tenatnt | 17:04 |
| Uggla | sean-k-mooney, if you are unsure we can keep it open for next week. | 17:05 |
| sean-k-mooney | althogh if we are missing the try accpate on teh network not found then thats a valid bug i guess | 17:05 |
| Uggla | sale for valid ? | 17:07 |
| gmaan | I have not tracked the exact call of validate_networks() but nova handle NetworkNotFound in _validate_requested_network_ids | 17:07 |
| gmaan | https://github.com/openstack/nova/blob/221a3e89e8988bc664298106ee691a4e41ca71f9/nova/network/neutron.py#L988 | 17:07 |
| gmaan | does bug mention 500 is returned? or 400 ? | 17:09 |
| sean-k-mooney | gmaan: i think neturon is returnign a 500 | 17:09 |
| sean-k-mooney | https://paste.openstack.org/show/bt3BWnyrBbQspiFx8KPZ/ | 17:09 |
| gmaan | ohk | 17:09 |
| sean-k-mooney | i think we need more info to be sure | 17:09 |
| sean-k-mooney | the do have a NetworkNotFound excption https://paste.openstack.org/show/bJL9b5aY99XpZlLZdzgy/ | 17:10 |
| gmaan | I think nova hanlding the error correctly if neuton return NetworkNotFound | 17:10 |
| gmaan | and controller also handle NetworkNotFound so 400 will be return | 17:10 |
| sean-k-mooney | can you comment to that effect on the bug | 17:11 |
| gmaan | sure, will do after TC meeting | 17:11 |
| sean-k-mooney | the first step to fixing this if we were to fix it in nova woudl be a repoducer test anyway | 17:11 |
| sean-k-mooney | iesally a functional one | 17:12 |
| Uggla | sean-k-mooney, does that mean we can set it as valid ? | 17:13 |
| Uggla | I need to close the meeting, so I propose to look at that bug offline later or next week. | 17:15 |
| Uggla | Thanks all and thanks for the extended time for bug triage. | 17:15 |
| gmaan | Uggla: I will check exact trace of validate_network and see if nova missing NetworkNotFound handling anywhere | 17:15 |
| Uggla | #endmeeting | 17:15 |
| opendevmeet | Meeting ended Tue May 27 17:15:54 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 17:15 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2025/nova.2025-05-27-16.00.html | 17:15 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2025/nova.2025-05-27-16.00.txt | 17:15 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2025/nova.2025-05-27-16.00.log.html | 17:15 |
| gmaan | and comment in bug | 17:15 |
| gmaan | aftr TC meeting | 17:16 |
| Uggla | thanks gmaan | 17:16 |
| Uggla | Sorry I close the meeting a bit too quickly | 17:16 |
| Uggla | Anyway time for me to afk, see you tomorrow. | 17:17 |
| gmaan | its fine, actually 17 UTC, TC meeting start so difficult to respond here on time :) | 17:17 |
| Uggla | no worries, I think I would close the meeting without bug triage because we were almost on top of the hour. | 17:19 |
| sean-k-mooney | i set it to incomplete | 17:21 |
| sean-k-mooney | we can come back next week | 17:21 |
| Uggla | ok sean-k-mooney | 17:22 |
| sean-k-mooney | but i suspect if the neutorn issue is fixed nova will do the right thing | 17:22 |
| opendevreview | Callum Dickinson proposed openstack/nova master: Fix image ID in libvirt metadata when unshelving https://review.opendev.org/c/openstack/nova/+/942973 | 21:04 |
| opendevreview | Callum Dickinson proposed openstack/nova master: Add image meta to libvirt XML metadata https://review.opendev.org/c/openstack/nova/+/942766 | 21:04 |
| opendevreview | Callum Dickinson proposed openstack/nova master: Add more flavor metadata to libvirt guest XML https://review.opendev.org/c/openstack/nova/+/942974 | 21:09 |
| Callum027 | sean-k-mooney: I know it's not strictly necessary but I've rebased the branches to make sure they're fine to merge with the newer master branch | 21:09 |
| Callum027 | (the image meta blueprint ones, and the bugfix for image ID in libvirt metadata) | 21:10 |
| Callum027 | Also added some more detail to the release notes as discussed in the meeting | 21:10 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!