| opendevreview | Konrad Gube proposed openstack/nova-specs master: Propose assisted-volume-extend API https://review.opendev.org/c/openstack/nova-specs/+/949504 | 08:16 |
|---|---|---|
| gibi | dansmith: thanks for the review on https://review.opendev.org/c/openstack/nova/+/966446/1#message-d56cbd036e036b8a6962e6023c1c4213fbadf7ab I replied inline | 08:45 |
| *** elodilles is now known as elodilles_pto | 09:18 | |
| opendevreview | Henry Richter proposed openstack/nova master: Preserve vTPM state between power off and power on https://review.opendev.org/c/openstack/nova/+/955657 | 10:33 |
| opendevreview | Henry Richter proposed openstack/nova master: Preserve vTPM state between power off and power on https://review.opendev.org/c/openstack/nova/+/955657 | 10:34 |
| opendevreview | sean mooney proposed openstack/nova master: ensure correct cleanup of multi-attach volumes https://review.opendev.org/c/openstack/nova/+/916322 | 10:54 |
| LarsErikP | tobias-urdin: do you know anything about the status of merging this? https://review.opendev.org/c/openstack/nova/+/916409 | 11:29 |
| opendevreview | sean mooney proposed openstack/nova master: ensure correct cleanup of multi-attach volumes https://review.opendev.org/c/openstack/nova/+/916322 | 13:27 |
| sean-k-mooney | gibi: good catch i missed that i reordered it ^ shoudl adress your comments | 13:28 |
| opendevreview | Henry Richter proposed openstack/nova master: Preserve vTPM state between power off and power on https://review.opendev.org/c/openstack/nova/+/955657 | 13:35 |
| opendevreview | Henry Richter proposed openstack/nova master: Preserve vTPM state between power off and power on https://review.opendev.org/c/openstack/nova/+/955657 | 13:37 |
| gibi | sean-k-mooney: thanks. +2 | 13:38 |
| sean-k-mooney | long term ther is a more generic version fo the FairLockGuard concept that i may eventually upstream to oslo. if i do write that ill eventurally refactort nova to use it but i want thsi simpler version in nova so we can backport the fix. gibi the more generic version would take to sets of lock names to aquire as readers and writers, this would allow me to create a more | 13:47 |
| sean-k-mooney | generic form of the @serial decorator that we use for tempest so that we can declare annotate test that need exclusive access to a subset of resouces with a request for the wright lock for that resouce, the use case is watcher senario tests have some tests that exclsilve operatate on voluem and ohter that exlucsitlvy operato on instnaces and some that work on both | 13:47 |
| opendevreview | Rajesh Tailor proposed openstack/nova master: Clear image_ref for boot-from-volume instances https://review.opendev.org/c/openstack/nova/+/967369 | 13:47 |
| sean-k-mooney | today we run the warcher tempest test with concurrancy 1 so that they dont interfer | 13:47 |
| sean-k-mooney | evecutnally i think we can do better with an enhanced version fo this but that is over kill for what nova needs today | 13:48 |
| gibi | sean-k-mooney: could you check this small hacking rule https://review.opendev.org/c/openstack/nova/+/958408 ? | 14:27 |
| *** haleyb|out is now known as haleyb | 14:32 | |
| sean-k-mooney | sure | 14:33 |
| sean-k-mooney | ok so we are trying to block import or direct use via fully qulaifed name of eventlets semaphore timeout or event | 14:36 |
| sean-k-mooney | :) and it found one incorrect usage in a test | 14:37 |
| sean-k-mooney | i need to check but we have already converted LibvirtDriver._handle_conn_event to actully use ThreadingEventWithResult() right? | 14:37 |
| gibi | https://github.com/openstack/nova/blob/8a993d583ffedb608a67acac9fce0c3d065e2dd2/nova/compute/manager.py#L268 | 14:39 |
| gibi | yepp | 14:39 |
| sean-k-mooney | _InstanceEvents = ty.Dict[ty.Tuple[str, str], ThreadingEventWithResult] ah ok then yes i think this all looks fine to me | 14:41 |
| sean-k-mooney | gibi: i didnt have a chance to look last week but you mentioend the first copule of patches in the eventlet serise are ready to review | 14:42 |
| sean-k-mooney | ah found the base https://review.opendev.org/c/openstack/nova/+/966443/1 looks like stephen has alrady taken a look at that one at least | 14:43 |
| sean-k-mooney | i have maybe 30-40 mins before my next meeting ill see if i can take a look at one or two of them | 14:44 |
| gibi | thanks | 15:08 |
| dansmith | gibi: I replied... I'm concerned we have such differing views of what init_host() is for | 15:27 |
| dansmith | but, I +W'd as I don't think I'm going to convince you and a fixed bug is better of course | 15:28 |
| dansmith | I would offer to tighten up the definition of init_host() but I'd do so in a way incompatible with your definition, so probably better for me to not | 15:28 |
| gibi | dansmith: thanks. "Initialize anything that is necessary for the driver to function" for me means do not use the driver until this is done. I understand that technically we could use parts of the driver already after host.initialize() like querying the uuids. But what you suggest would actually require the better part of the current libvirt/driver.init_host() to be move to __init__ not just | 15:34 |
| gibi | host.initialize() like the libvirt version check. I really don't want to query anything about libvirt before we are sure we are talking about a libvirt we support | 15:34 |
| gibi | and really changing the meaning of init_host during a bugfix seems dangerous | 15:36 |
| dansmith | except we call host.initialize() before we check the libvirt version :) | 15:36 |
| gibi | we do but we don't return from init_host until we check the libvirt version | 15:36 |
| dansmith | well, as I noted, I don't think the current meaning agrees with what you think it is | 15:37 |
| gibi | the current libvirt implementation agrees with the meaning I hold here and that is our biggest and most used impl | 15:37 |
| dansmith | hence me asking you to change the actual definitions to match, for the future | 15:37 |
| gibi | I thought you asked me to change the implementation of the libvirt driver | 15:38 |
| gibi | btw we probably initalize the host before the libvirt version check as we ask libvirt to tell us its version to check and for that we need the host to be initalized | 15:40 |
| dansmith | "Might be good to nail down the expectations for those two interfaces at some point." and then " I would offer to tighten up the definition of init_host() but I'd do so in a way incompatible with your definition, so probably better for me to no" | 15:40 |
| gibi | sorry I'm still at the point where you suggested to move host.initalize to __init__. So you say we should just update the init_host interface comment to say, "do not catch up with instances" | 15:41 |
| dansmith | ^ this is me saying that I don't think the docstrings match your interpretation of the interfaces, so if we're not going to change the code we should update the docstrings to encode the current state you want to codify | 15:41 |
| gibi | ? | 15:41 |
| gibi | I can change the docstring | 15:42 |
| gibi | So I will propose a follow up with some wording in virt.driver.init_host(). Which are the other interface you see needing adjustment in the docs? | 15:44 |
| dansmith | I just mean the split between __init__ and init_host, which have some delineation of what should be done in each, but which I clearly don't understand | 15:48 |
| Uggla | Reminder : nova meeting in ~10mn | 15:50 |
| gibi | dansmith: OK. Currently virt.driver.__init__ has no docstring. I will add some there too | 15:55 |
| Uggla | #startmeeting nova | 16:04 |
| opendevmeet | Meeting started Mon Nov 17 16:04:05 2025 UTC and is due to finish in 60 minutes. The chair is Uggla. Information about MeetBot at http://wiki.debian.org/MeetBot. | 16:04 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 16:04 |
| opendevmeet | The meeting name has been set to 'nova' | 16:04 |
| Uggla | Hello everyone | 16:04 |
| gibi | o/ | 16:04 |
| fwiesel | o/ | 16:05 |
| bauzas | o. | 16:05 |
| bauzas | I was importing the ics because I was afraid to miss the nova meeting :) | 16:05 |
| Uggla | Let's start | 16:06 |
| Uggla | I expect a short one | 16:06 |
| Uggla | #topic Bugs (stuck/critical) | 16:06 |
| Uggla | #info No Critical bug | 16:06 |
| Uggla | #topic Gate status | 16:07 |
| Uggla | #link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs | 16:07 |
| tkajinam | o/ | 16:07 |
| Uggla | #link https://etherpad.opendev.org/p/nova-ci-failures-minimal | 16:07 |
| 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:07 |
| Uggla | #info Please look at the gate failures and file a bug report with the gate-failure tag. | 16:07 |
| Uggla | #info Please try to provide a meaningful comment when you recheck | 16:07 |
| Uggla | Nothing special on the gate unless I missed it | 16:08 |
| tobias-urdin | LarsErikP: haven't heard anything about https://review.opendev.org/c/openstack/nova/+/916409 but i would like it merged as well | 16:08 |
| Uggla | gibi, bauzas from what I read it would be good to have your point of view about ^ | 16:10 |
| sean-k-mooney | tobias-urdin: im now aware of anythign blockign that specific other then lack fo review | 16:10 |
| sean-k-mooney | dan did have one query about fucntional tests but it was not dreictly a blocker | 16:11 |
| Uggla | sean-k-mooney it seems Dan wants a fn reproducer | 16:11 |
| sean-k-mooney | i dont have full context loaded on this to say if that is hard or easy to do | 16:11 |
| sean-k-mooney | Uggla: they +1 and said they were ok to proceed if other were but yes that was tehre prefernce | 16:12 |
| Uggla | yep, I think we can review it later, but I keep this in mind. | 16:12 |
| tobias-urdin | sorry everyone, i was responding to a ping didn't see it was in the middle of a meeting, my bad! | 16:13 |
| Uggla | tobias-urdin, no worries, I though it was linked more or less to gate. :) | 16:13 |
| Uggla | skipping next point because gmaan seems not available. | 16:15 |
| Uggla | #topic Release Planning | 16:15 |
| Uggla | #link https://releases.openstack.org/gazpacho/schedule.html | 16:15 |
| Uggla | #info Soft spec freeze November 20th ! | 16:15 |
| Uggla | yes this is this Thursday already. | 16:16 |
| bauzas | as I said, I'm a bit sad about this deadline but OK | 16:16 |
| bauzas | Uggla: can you at least send an email about this deadline ? | 16:17 |
| bauzas | I mean, a reminder | 16:17 |
| Uggla | yep I did it last week, but I will send an update. | 16:17 |
| Uggla | update/reminder | 16:18 |
| opendevreview | Merged openstack/nova master: [hacking] Improve N373 to catch also other primitives https://review.opendev.org/c/openstack/nova/+/958408 | 16:18 |
| Uggla | #topic Review priorities | 16:19 |
| Uggla | #link https://etherpad.opendev.org/p/nova-2026.1-status | 16:19 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: [virt]Clarify the meaning of init_host https://review.opendev.org/c/openstack/nova/+/967398 | 16:19 |
| Uggla | ^ not fully updated, | 16:19 |
| Uggla | #topic OpenAPI | 16:20 |
| Uggla | #link: https://review.opendev.org/q/topic:%22openapi%22+(project:openstack/nova+OR+project:openstack/placement)+-status:merged+-status:abandoned | 16:20 |
| Uggla | #info still 28 remaining atm. | 16:20 |
| sean-k-mooney | the first 9/10 are approved just pending ci | 16:20 |
| sean-k-mooney | im hoping to be able to review the rest in hte next 2 weeks | 16:20 |
| Uggla | cool 👍 | 16:21 |
| sean-k-mooney | my personal goal woudl be to try and land the rest by the end of hte month | 16:21 |
| Uggla | sean-k-mooney would be super ! | 16:21 |
| Uggla | I'll try to review as well, I wanted last week, but did not manage to because of a shorter week. | 16:22 |
| Uggla | skipping stable branch point as elodilles is on pto | 16:22 |
| Uggla | #topic vmwareapi 3rd-party CI efforts Highlights | 16:22 |
| Uggla | fwiesel anything on your side ? | 16:22 |
| fwiesel | Hi, no update from my side. | 16:22 |
| Uggla | thx, fwiesel | 16:23 |
| Uggla | #topic Gibi's news about eventlet removal | 16:23 |
| * Uggla passing the mic to gibi | 16:23 | |
| gibi | o/ | 16:23 |
| gibi | (gibi): The sync_power_state_pool_size and max_concurrent_live_migrations options are reworked now based on last week’s discussion. https://review.opendev.org/c/openstack/nova/+/966016 | 16:24 |
| gibi | (gibi):Waiting for an oslo.service 4.4.x release to have fixes available for nova-compute | 16:24 |
| gibi | but oslo.service 4.4.0 with our fix broke Heat so we need to wait for 4.4.1 now | 16:25 |
| gibi | that is it from me | 16:25 |
| tkajinam | We can hopefully merge that heat patch soon once we merge another change required to fix the broken ec2 token API call caused by recent CVE fix ... | 16:25 |
| tkajinam | I feel like that problem appearing heat is a broken compatibility in oslo.service but fixing heat may be an quicker path | 16:26 |
| tkajinam | just fyi, mostly for gibi | 16:26 |
| gibi | tkajinam: damani fixed the oslo.service incompatibility and the suggestion is to go with that instead of the heat fisx | 16:26 |
| tkajinam | ah, ok | 16:26 |
| gibi | https://review.opendev.org/c/openstack/oslo.service/+/966458 no_fork interface fix. (mandatory) | 16:26 |
| gibi | sorry not this | 16:26 |
| gibi | https://review.opendev.org/c/openstack/oslo.service/+/967218/4 | 16:26 |
| gibi | this | 16:27 |
| gibi | tkajinam: so your +2 would be appreciated ^^ there | 16:27 |
| * tkajinam is approving it :-) | 16:27 | |
| gibi | tkajinam: thanks | 16:27 |
| gibi | Uggla: that is all from me | 16:27 |
| Uggla | Thanks gibi | 16:28 |
| Uggla | #topic Open discussion | 16:29 |
| Uggla | There is nothing on the agenda | 16:29 |
| Uggla | Does someone want to talk about something ? | 16:29 |
| tkajinam | I just want to quickly mention that I proposed a spec for firmware detection we discussed during ptg | 16:30 |
| tkajinam | the link can be found in review priority etherpad | 16:30 |
| tkajinam | https://review.opendev.org/c/openstack/nova-specs/+/966584 | 16:30 |
| tkajinam | and appreciate any feedback :-) | 16:30 |
| Uggla | tkajinam 👍 | 16:31 |
| Uggla | thanks for submitting it | 16:31 |
| tkajinam | ;-) | 16:31 |
| Uggla | #topic Bug scrubbing | 16:33 |
| Uggla | #info up to 189 | 16:33 |
| Uggla | #link: https://etherpad.opendev.org/p/nova-bug-triage-roster | 16:33 |
| Uggla | please add your nick to the roster if you'd like to participate the bug triage. | 16:33 |
| Uggla | I will send an update / reminder to the list, and I'll try to assign the first bugs this week. | 16:34 |
| Uggla | ATM we are 5 volunteers on the list. | 16:35 |
| Uggla | I thinks that' all for today. | 16:35 |
| Uggla | Last chance to share something. | 16:35 |
| Uggla | 3 | 16:36 |
| Uggla | 2 | 16:36 |
| Uggla | 1 | 16:36 |
| Uggla | Thanks for joining this meeting. Have a nice day/evening. | 16:37 |
| Uggla | #endmeeting | 16:37 |
| opendevmeet | Meeting ended Mon Nov 17 16:37:17 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:37 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2025/nova.2025-11-17-16.04.html | 16:37 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2025/nova.2025-11-17-16.04.txt | 16:37 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2025/nova.2025-11-17-16.04.log.html | 16:37 |
| gibi | Uggla: thanks | 16:38 |
| bauzas | thanks Uggla | 16:39 |
| Uggla | gibi, bauzas you are welcome | 16:40 |
| opendevreview | Merged openstack/nova master: Reproduce bug/2130881 https://review.opendev.org/c/openstack/nova/+/966443 | 16:44 |
| opendevreview | Ghanshyam proposed openstack/nova master: PoC: Graceful shutodwn of nova services https://review.opendev.org/c/openstack/nova/+/967261 | 17:32 |
| gibi | fwiesesl | 18:47 |
| gibi | fwiesel: I have a series for oslo.vmware to remove eventlet: https://review.opendev.org/q/project:openstack/oslo.vmware+topic:eventlet-removal but I have no way to test it. The last patch changing the timeout handling of image_transfer can be problematic so that needs extra care | 18:48 |
| sean-k-mooney | gibi: doesnt the ci trigger automaticly on new patchs | 19:01 |
| sean-k-mooney | gibi: given you have now enabeld threadign by default it hsoudl trigger on it no | 19:02 |
| sean-k-mooney | ? | 19:02 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!