| *** erlon4 is now known as erlon | 00:33 | |
| *** ykarel_ is now known as ykarel | 06:30 | |
| opendevreview | chandan kumar proposed openstack/nova master: Report OWNER_NOVA trait on resource providers https://review.opendev.org/c/openstack/nova/+/994299 | 11:26 |
|---|---|---|
| opendevreview | chandan kumar proposed openstack/nova master: Report OWNER_NOVA trait on resource providers https://review.opendev.org/c/openstack/nova/+/994299 | 11:34 |
| opendevreview | Joan Gilabert proposed openstack/nova master: Add libvirt driver MDEV ARQ support for Cyborg https://review.opendev.org/c/openstack/nova/+/995840 | 11:56 |
| opendevreview | Pierre Libeau proposed openstack/nova master: libvirt: Add reproducer for unstable vNIC PCI addresses on hard reboot https://review.opendev.org/c/openstack/nova/+/1000961 | 13:40 |
| opendevreview | Pierre Libeau proposed openstack/nova master: libvirt: Render guest interface PCI address in config https://review.opendev.org/c/openstack/nova/+/1000962 | 13:40 |
| opendevreview | Pierre Libeau proposed openstack/nova master: libvirt: Preserve vNIC guest PCI address on domain XML regeneration https://review.opendev.org/c/openstack/nova/+/1000963 | 13:40 |
| melwitt | I saw that somebody posted a new comment on this bug https://bugs.launchpad.net/nova/+bug/2076614 saying they hit the problem while upgrading 2025.1 => 2026.1 ... there was a fix proposed but it's in merge conflict now and sean-k-mooney had a comment on it with some follow up steps, I wondered are those steps still valid now (> 1 year later) https://review.opendev.org/c/openstack/nova/+/948328 or is it that we just need to rebase the | 14:51 |
| melwitt | patch? | 14:51 |
| sean-k-mooney | melwitt: triging to remeber context | 14:54 |
| sean-k-mooney | but reading my old comment i was just noting we could start droping older migrations in a followup for things older then caracal | 14:55 |
| sean-k-mooney | i think as wriggen the archive/pruge woudl still be required before doign the online migrations | 14:55 |
| sean-k-mooney | and it might still be worth addign an upgrade release note | 14:56 |
| sean-k-mooney | but i think just rebaseing that commit would be fine | 14:56 |
| sean-k-mooney | melwitt: i.e. if you wanted to progress that i would be ok wiht just resolving the merge conflict and proceeding with it | 14:57 |
| sean-k-mooney | with that said this only affect upgrades form unsupproted releases to my knowldage | 14:57 |
| sean-k-mooney | so it has not been a high priorty | 14:57 |
| sean-k-mooney | oh ok the latest comment confirm its an issue for current release | 14:58 |
| melwitt | sean-k-mooney: ok cool thank you. yeah ... it caught my eye bc they are upgrading _to_ a supported release though so it set off my alarm bells | 14:58 |
| sean-k-mooney | when upgrading after caracal so ya | 14:58 |
| sean-k-mooney | right it was the other way around its only an issue after caracal but also not sure why we dont see this grenade | 14:59 |
| melwitt | it happens due to the "fake" marker instance for that fill_virtual_interface_list online migration right ... so the grenade test wouldn't have that old of a migration artifact in its database | 15:01 |
| melwitt | it would have to be a long lived deployment that has been upgraded over the years, IIUC | 15:02 |
| sean-k-mooney | i guess i tought that marker was adde by the older migration | 15:11 |
| sean-k-mooney | but ok | 15:11 |
| sean-k-mooney | its still a valid bug so it makes sesne to fix it on master in anycase | 15:14 |
| melwitt | yes I think that is correct that the marker was added by the older migration (rocky => stein). I was trying to say that CI wouldn't show it bc CI would not ever have that old of a migration remnant in it | 15:15 |
| melwitt | maybe i'm misunderstanding it, I guess you are saying the old migration still running today. I have to think about it more | 15:16 |
| sean-k-mooney | i dont recall if alembic/sql aclahme in devstack is instcating the db directly from the model or if it apples the base schema and then runs the migrations | 15:17 |
| sean-k-mooney | i thinink sqlalchemy-migrate may have done the latter | 15:18 |
| sean-k-mooney | as i recall seeign devstack run migraitons locally in teh past | 15:18 |
| sean-k-mooney | but i dont knwo if we skip that step in a normal install with alembic | 15:18 |
| sean-k-mooney | and just create the db directly with the final state | 15:18 |
| melwitt | ah ok I get what you mean | 15:19 |
| sean-k-mooney | so we call migrtion.db_sync and pass the version wihc default to None | 15:20 |
| sean-k-mooney | and that does an alembic upgrade https://opendev.org/openstack/nova/src/branch/master/nova/db/migration.py#L54-L94 | 15:21 |
| sean-k-mooney | but we are not going to run the online data migration until the db is already at the latest revision | 15:22 |
| sean-k-mooney | i guess if we have no cells at that point | 15:25 |
| sean-k-mooney | https://opendev.org/openstack/nova/src/branch/master/nova/objects/virtual_interface.py#L186-L187 | 15:25 |
| sean-k-mooney | we never enter that for loop | 15:25 |
| sean-k-mooney | and if we never enter that loop we never set the fake marker instnace | 15:25 |
| sean-k-mooney | https://opendev.org/openstack/nova/src/branch/master/nova/objects/virtual_interface.py#L284-L287 | 15:25 |
| sean-k-mooney | so in a normal install you wont hit this problem, still not sure why greade wont | 15:26 |
| sean-k-mooney | but proably not worth diging into | 15:27 |
| melwitt | yeah :/ needs more brain power | 15:28 |
| melwitt | unrelated to that, I just saw a new TestComputeGracefulShutdown func test failure in the openstack-tox-cover job https://16fdd6c48f9c26f8b19a-1184c5a69e25bd403bd8a2f61f3cbab7.ssl.cf2.rackcdn.com/openstack/424ac0a1e84d486eacd5fda46d788ab6/testr_results.html not sure if this is already represented in an open bug cc gmaan | 15:31 |
| *** __ministry is now known as Guest15841 | 15:37 | |
| Uggla | Reminder: upstream meeting in ~20mn | 15:38 |
| opendevreview | ribaudr proposed openstack/nova master: Add regression test for bug 2062523 https://review.opendev.org/c/openstack/nova/+/1001185 | 15:52 |
| opendevreview | ribaudr proposed openstack/nova master: fix: preserve task_state during evacuation to prevent race https://review.opendev.org/c/openstack/nova/+/1001186 | 15:52 |
| gibi | o/ | 16:00 |
| Uggla | #startmeeting nova | 16:00 |
| opendevmeet | Meeting started Mon Aug 17 16:00:18 2026 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 |
| sean-k-mooney | o/ | 16:00 |
| antia | o/ | 16:00 |
| bryanfraschetti | o/ | 16:01 |
| lajoskatona | o/ | 16:01 |
| tkajinam | o/ | 16:02 |
| Uggla | Let's start | 16:02 |
| dansmith | o/ | 16:02 |
| Uggla | #topic Bugs (stuck/critical) | 16:02 |
| Uggla | #info No Critical bug | 16:03 |
| Uggla | #topic Gate status | 16:03 |
| Uggla | #link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs | 16:03 |
| Uggla | #link https://etherpad.opendev.org/p/nova-ci-failures-minimal | 16:03 |
| 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:03 |
| Uggla | #info Please look at the gate failures and file a bug report with the gate-failure tag. | 16:04 |
| Uggla | #info Please try to provide a meaningful comment when you recheck | 16:04 |
| Uggla | I have not seen something bad. Please tell me if I'm wrong. | 16:04 |
| Uggla | #topic Release Planning | 16:05 |
| Uggla | #link https://releases.openstack.org/hibiscus/schedule.html | 16:05 |
| Uggla | #info Nova deadlines are set in the above schedule | 16:05 |
| Uggla | #info PTG etherpad for 2026.2 is available: https://etherpad.opendev.org/p/nova-2026.2-ptg | 16:05 |
| Uggla | #info Feature freeze is next week. | 16:05 |
| Uggla | #info Thursday is 2nd Nova Implementation Review Day | 16:05 |
| Uggla | #topic Review priorities | 16:06 |
| sean-k-mooney | i think the non clinet lib freeze would be thrusday | 16:06 |
| Uggla | sean-k-mooney yes you are right. | 16:06 |
| sean-k-mooney | meaning we need anything for os-triats/os-vif ectra in this week | 16:06 |
| sean-k-mooney | not sure we have anything we need for those just calling it out | 16:06 |
| Uggla | sure no worries | 16:07 |
| gibi | we had a change landed in futurist | 16:07 |
| gibi | I guess we get a release of it this week | 16:07 |
| gibi | I assume futurist is also non client lib | 16:07 |
| sean-k-mooney | that should get done with the rest of the oslo lib yes | 16:07 |
| gibi | cool | 16:07 |
| dansmith | if I could, | 16:08 |
| dansmith | I've been going back and forth on the share migration stuff | 16:08 |
| dansmith | it's progressing but still lacking some robustness around the error handling | 16:08 |
| dansmith | I'm going to be out part of next week around the deadline and I'm the only one what has reviewed it AFAICT | 16:09 |
| dansmith | so just highlighting that it's a bit of a risk for FF | 16:09 |
| Uggla | dansmith ok, can I help on that side ? | 16:09 |
| dansmith | for sure, any reviews can help, of course | 16:10 |
| gibi | on the same note we will need a second set of eyes on the TDX series. I'm +2 but bauzas is on PTO | 16:11 |
| dansmith | I'm also on the hook for the graceful shutdown stuff, which has made progress for sure, but the next patch is pretty big | 16:11 |
| dansmith | I don't want to jeopardize the share stuff, but the shutdown stuff is progressed a bit farther | 16:11 |
| sean-k-mooney | gibi: i can maybe take a look at the tdx work | 16:12 |
| gibi | dansmith: I replied on your comment there but I'm not sure that helps:) | 16:12 |
| sean-k-mooney | but im alos trying to prgoress some cybrog work so we will see | 16:12 |
| gibi | sean-k-mooney: that would be awesom | 16:12 |
| gibi | e | 16:12 |
| Uggla | ok so I'll have a look tomorrow. I'll try to look the share cold migration and if possible tdx. | 16:12 |
| sean-k-mooney | i have most of the context still loaded form the sev work | 16:12 |
| gibi | I'm also off Thur / Fri (public holiay) | 16:12 |
| Uggla | sean-k-mooney 👍 | 16:13 |
| Uggla | anything else for review prio ? | 16:13 |
| dansmith | gibi: yeah I saw, will circle back | 16:13 |
| sean-k-mooney | im hoping https://review.opendev.org/c/openstack/nova/+/994579 can land before FF | 16:13 |
| sean-k-mooney | but we will see | 16:14 |
| Uggla | skipping couple of next topics because of Elod and fwiesel ptos. | 16:15 |
| Uggla | #topic Kamil's news about eventlet removal | 16:16 |
| Uggla | Kamil is on bank holiday, gibi something to report on that topic ? | 16:16 |
| gibi | o/ Kamil is off today | 16:16 |
| gibi | I substitute | 16:16 |
| Uggla | gibi 👍 | 16:16 |
| gibi | when futurist is released we can progress on removing code form nova by https://review.opendev.org/c/openstack/nova/+/997410 | 16:17 |
| gibi | this will conflict (trivially) with gmann's refactor patch dansmith mentioned above | 16:17 |
| gibi | just to complicate things :) | 16:17 |
| dansmith | oh wonderful | 16:18 |
| gibi | I think the graceful shutdown series is more important that this futurist code move so I'm fine delaying https://review.opendev.org/c/openstack/nova/+/997410 | 16:18 |
| sean-k-mooney | right | 16:18 |
| sean-k-mooney | so that is just adopting the futureist port | 16:19 |
| gibi | yepp | 16:19 |
| sean-k-mooney | so we can merge that later | 16:19 |
| gibi | yepp | 16:19 |
| gibi | we have one more thing for eventlet, the functional test job | 16:19 |
| sean-k-mooney | provided the futurist release happens i think it woudl be fine to merge kamils change after FF as long as it happens before rc1 | 16:19 |
| gibi | https://review.opendev.org/c/openstack/nova/+/997216 this and its ancestors are ready to land | 16:19 |
| sean-k-mooney | those look small so i can take a look after the meeting | 16:20 |
| sean-k-mooney | looks lik gmaan is +2 on them alrady | 16:20 |
| gibi | yepp | 16:20 |
| gibi | thanks sean-k-mooney | 16:20 |
| gibi | this give use an extra safety net | 16:21 |
| gibi | for any later refactors | 16:21 |
| gibi | that is it from me about eventlet <EOM> | 16:21 |
| Uggla | thx gibi | 16:21 |
| Uggla | #topic Nova using openstack sdk for neutron | 16:21 |
| Uggla | lajoskatona, something to share ? | 16:22 |
| lajoskatona | not really the sdk patch is ready to merge and after to be released: https://review.opendev.org/c/openstack/openstacksdk/+/995724 | 16:22 |
| lajoskatona | so seems like the whole series is skipping again for the next cycle | 16:23 |
| lajoskatona | that's it for the Neutron and SDK work | 16:23 |
| Uggla | thx lajoskatona. | 16:24 |
| Uggla | I think this is bug fixes and not impacted by FF. | 16:24 |
| Uggla | we will see | 16:25 |
| Uggla | moving on | 16:25 |
| Uggla | #topic Confidential computing, status of SNP and TDX features | 16:25 |
| Uggla | antia, tkajinam, gibi, something you would like to share ? | 16:26 |
| gibi | as I mentioned above I'm +2 on TDX and it needs a second core | 16:27 |
| gibi | I'm not trackin anything for SNP any more for H | 16:27 |
| gibi | but tkajinam correct me if there are open things to land | 16:28 |
| gibi | for SNP | 16:28 |
| antia | I was trying to wrap my head around some Zuul logs earlier today, but didn't get too far | 16:28 |
| sean-k-mooney | i think the only SNP thing is the test cleanup | 16:29 |
| tkajinam | nothing is left for SNP, as far as I'm aware of | 16:29 |
| tkajinam | (nothing critical, I mean | 16:29 |
| sean-k-mooney | tkajinam: did the mock path fix merge yet | 16:29 |
| tkajinam | sean-k-mooney, not yet | 16:29 |
| sean-k-mooney | ack | 16:29 |
| * tkajinam is reloading the context after 1 week off | 16:30 | |
| gmaan | gibi: sean-k-mooney on threading jobs series, I was in between of this one on friday but will check today, rest all were good https://review.opendev.org/c/openstack/nova/+/999121/3 | 16:30 |
| tkajinam | there is one small simplification change related to SNP, waiting for 2nd +2 https://review.opendev.org/c/openstack/nova/+/998285 | 16:30 |
| tkajinam | so it would be nice if we can merge that too | 16:31 |
| tkajinam | https://review.opendev.org/c/openstack/nova/+/999780 is the mock path fix which sean-k-mooney mentioned, just for record | 16:31 |
| sean-k-mooney | tkajinam: ill review that shortly but ya only checkign x86 makes sense | 16:31 |
| tkajinam | yup | 16:32 |
| tkajinam | a similar change is proposed for tdx as well (found in the series) | 16:32 |
| sean-k-mooney | tkajinam: im ok with merging that as is and defering the rest till later | 16:32 |
| sean-k-mooney | but just want to make sure gibi and dansmith are ok with that before +2ing | 16:32 |
| sean-k-mooney | that for https://review.opendev.org/c/openstack/nova/+/999780 | 16:32 |
| tkajinam | sean-k-mooney, you are talking about 999780, right ? | 16:32 |
| tkajinam | ok | 16:32 |
| sean-k-mooney | we coudl do more but i would prefer to fix the test and followup rather then continue to refine that change but i dont really mind in this case as long as we fix it before the release | 16:33 |
| sean-k-mooney | * brefore rc1 | 16:33 |
| sean-k-mooney | i dont wnat those test to start randomly failing when distos stat doign buils for the RC | 16:34 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: DNM: Test unit tests on kvm https://review.opendev.org/c/openstack/nova/+/999729 | 16:34 |
| tkajinam | agree | 16:34 |
| Uggla | I guess we are done with the topic ? | 16:35 |
| sean-k-mooney | +1 | 16:35 |
| tkajinam | I think that's all from my end. We can continue discussion about 999780 in reviews (and that can be mereged after FF because that is basically test only change) | 16:35 |
| tkajinam | yup | 16:35 |
| Uggla | thx antia, tkajinam, gibi. | 16:36 |
| Uggla | #topic Bug scrubbing | 16:36 |
| Uggla | #info up to 73 (-9) | 16:36 |
| Uggla | #link https://etherpad.opendev.org/p/nova-bug-triage-roster | 16:36 |
| Uggla | #link https://truc.uggla.fr/ to follow the trend. | 16:36 |
| Uggla | Next meeting (this week): [public] Upstream bug triage. Wednesday, August 19th · 15:30 – 16:00 UTC. Video call link: meet.google.com/zjr-rxus-hzj | 16:36 |
| Uggla | #topic Open discussion | 16:36 |
| Uggla | Reminder: PTL Candidacy: I wanted to do another round for the 'I' cycle, but my manager suggested that it would be better to have someone else step in. I agree that, if possible, rotating the role is good for the project. | 16:37 |
| Uggla | So if someone interested to run as candidate please let me know. | 16:37 |
| Uggla | (Note: The deadline is August 19th.) | 16:37 |
| Uggla | all on my side, something you'd like to discuss. | 16:38 |
| lajoskatona | I have another 2 topics for migration mostly if you have some time for review that would be nice: | 16:38 |
| lajoskatona | https://review.opendev.org/c/openstack/nova/+/982085 (Suppress error in cleanup of pre live migration ) | 16:38 |
| lajoskatona | and the other one is for handling queue_size/mtu related values: https://review.opendev.org/c/openstack/nova/+/995841/3 | 16:39 |
| lajoskatona | thanks in advance | 16:39 |
| Uggla | lajoskatona 👍, I will add to the list if not already done. | 16:39 |
| gmaan | one thing on gate status, i observed nova-grenade-multinode job failing for test_shelve_unshelve_server test more frequently last week | 16:39 |
| gmaan | i reported bug for that | 16:40 |
| gmaan | #link https://bugs.launchpad.net/nova/+bug/2163552 | 16:40 |
| lajoskatona | Uggla: thanks, please let me know if I need for some extra meeting or similar | 16:40 |
| gmaan | i did not debug it yet but in case anyone see similar failure, this is known one | 16:40 |
| Uggla | gmann, I have seen it. I mean the bug. | 16:40 |
| gmaan | k, I will try to debug it after FF and happen more frequently | 16:41 |
| Uggla | gmaan any idea of the frequency ? | 16:41 |
| gmaan | at least 4-5 times I saw this in different changes | 16:41 |
| gmaan | #link https://zuul.opendev.org/t/openstack/builds?job_name=nova-grenade-multinode&skip=0 | 16:42 |
| gmaan | this ^^ is state currenyty whihc is not bad | 16:42 |
| sean-k-mooney | lajoskatona: updatign the mtu durring live migratrion is not a thing we supprot | 16:42 |
| sean-k-mooney | lajoskatona: and there is a good reason fro that which si we do not allow the mtu to be chagne on a port that is atched to a vm | 16:42 |
| sean-k-mooney | i kno nuetron can allow that but nova has never supproted that | 16:43 |
| lajoskatona | sean-k-mooney: it 's not updating the value, but not changing if on the dst there is a different thing in the cfg, and breaking things in the VM | 16:43 |
| sean-k-mooney | also adding or removign the mtu elemetn changes the virtio options | 16:43 |
| sean-k-mooney | which make the vm un live mgiratable | 16:43 |
| sean-k-mooney | so updating it woudl only work if the element already exitsed | 16:43 |
| sean-k-mooney | lajoskatona: so the value of the mtu in the xml shoudl not change if the dest port bining has a diffent value | 16:44 |
| sean-k-mooney | i.e. if you enable jumb frames on a network | 16:44 |
| sean-k-mooney | that shoudl only reuslt in teh xml value updatign after a hard reboot | 16:44 |
| sean-k-mooney | not on a live migration | 16:44 |
| lajoskatona | the patch handles only the case when the nova-comp.cfg is different, nothing from Neutron side today (life is difficult even without that) and the same can happen with min/max_queue_size as the bug you opened (https://bugs.launchpad.net/nova/+bug/1854844) | 16:46 |
| sean-k-mooney | we only use the mtu values form teh nova config as a fallback | 16:46 |
| sean-k-mooney | and that is really only there for nova-networks | 16:46 |
| sean-k-mooney | so the real fix is to stop usign those values entirly | 16:46 |
| sean-k-mooney | or even remove them at this point | 16:47 |
| lajoskatona | ok, that can be also a good point to avoid confusion with config | 16:47 |
| sean-k-mooney | for the quese leght and count that is difent | 16:47 |
| sean-k-mooney | as that is entrily config driven | 16:47 |
| lajoskatona | so than the MTU part can be forgotten now (and let's check if the cfg can be deprecated and removed) | 16:48 |
| sean-k-mooney | but for mtu you shoudl not config that in nova if your using neutron | 16:48 |
| sean-k-mooney | ya for the queue config those cant change | 16:48 |
| sean-k-mooney | and we need to use the souce value on live migriton | 16:48 |
| sean-k-mooney | as that changes the way the nic is virutalsied to the guest | 16:48 |
| sean-k-mooney | i think qemu will not alow the queue count to chagne whiel the guest is running | 16:49 |
| lajoskatona | actually I got it from field where they upgraded from old nova/libvirt and during migrating around the VMs the MTU value mismatch had broken their apps, but if that is not used any more let's remove it | 16:49 |
| lajoskatona | but I don't want to keep the meeting open, we can discuss this topic offline, or in gerrit | 16:50 |
| Uggla | lajoskatona 👍 | 16:50 |
| Uggla | anything else ? If not I think we can close and keep this meeting. | 16:50 |
| bryanfraschetti | Hi folks, I'm hoping to get some reviews on this series: https://review.opendev.org/q/topic:%22improve_attachment_resiliance_against_timeout%22 | 16:51 |
| sean-k-mooney | lajoskatona: it was common when changing form ml2/ovs to ml2/ovn | 16:51 |
| sean-k-mooney | but the fix was to cold migrate | 16:51 |
| sean-k-mooney | not to hack live meigtion to chnge the mtu | 16:51 |
| sean-k-mooney | lajoskatona: we can chat about this more later | 16:52 |
| lajoskatona | sean-k-mooney: ack, thanks | 16:52 |
| Uggla | bryanfraschetti, at least I will put your series in the list. | 16:52 |
| bryanfraschetti | Thank you! | 16:53 |
| Uggla | anything else ? | 16:54 |
| Uggla | No so Closing, thanks for joining this meeting. Have a nice day/evening. | 16:55 |
| Uggla | Thanks for joining this meeting. Have a nice day/evening. | 16:55 |
| Uggla | #endmeeting | 16:55 |
| opendevmeet | Meeting ended Mon Aug 17 16:55:17 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:55 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2026/nova.2026-08-17-16.00.html | 16:55 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2026/nova.2026-08-17-16.00.txt | 16:55 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2026/nova.2026-08-17-16.00.log.html | 16:55 |
| lajoskatona | o/ | 16:55 |
| dansmith | gmaan: I'm not sure I understand this: https://github.com/openstack/nova/blob/master/nova/compute/rpcapi.py#L981 | 17:30 |
| dansmith | this lets us start new live migrations after shutdown is already going.. I think your comment is about conductor having already checked with both source and dest, but is there any reason we need to actually proceed during shutdown there? | 17:31 |
| dansmith | like some state conductor has altered? | 17:32 |
| dansmith | point being, you can continue issuing new live migration requests during a shutdown which will eventually run out the timeout clock and cause a failure | 17:32 |
| gmaan | dansmith: no, new live migratin will be rejected by check_can_live_migrate when conductor check it on source and dest. check_can_live_migrate_ is listneing on rpcserver which is stopped durign shutdown | 17:33 |
| dansmith | ah okay, cool | 17:34 |
| gmaan | that is why check_can_live_migrate make sure both source and dest are not shutting down so proceed rest of the live migration on rpc server alt in cae shutdown happen in betwen | 17:34 |
| dansmith | gotcha | 17:34 |
| gouthamr | dansmith: hey there, ack on the virtiofs cold-migration notes from the nova meeting, and ty for explaining stuff on the patch. i think i agree with your hardening suggestion more than any pointed fix i can make to the recovery paths.. i'm testing an update | 17:56 |
| gouthamr | dansmith: ack also on the PTO :) i'm out on and for a couple days after feature freeze day too (27th Aug) .. but i'll react to any review feedback as soon as i can | 17:58 |
| gouthamr | and i'll bother melwitt, Uggla and anyone else that was looking at this :) ty! | 17:59 |
| opendevreview | Ghanshyam Maan proposed openstack/nova master: Use service shutdown related config options in nova-graceful-shutdown https://review.opendev.org/c/openstack/nova/+/1001017 | 18:26 |
| opendevreview | Ghanshyam Maan proposed openstack/nova master: Improve the graceful shutdown job ansible tasks name https://review.opendev.org/c/openstack/nova/+/1001212 | 18:38 |
| opendevreview | Alexandre Engloner proposed openstack/nova master: Typo fix in List Hypervisors details Policy path. https://review.opendev.org/c/openstack/nova/+/1001216 | 19:04 |
| *** erlon1 is now known as erlon | 19:24 | |
| opendevreview | Clif Houck proposed openstack/nova master: perf(ironic): eliminate O(N²) ProviderTree deepcopy at startup https://review.opendev.org/c/openstack/nova/+/980676 | 20:01 |
| opendevreview | Clif Houck proposed openstack/nova master: Parallelize per-node resource updates https://review.opendev.org/c/openstack/nova/+/980679 | 20:01 |
| opendevreview | Merged openstack/nova master: Stabilize func test waiting for VM ERROR state https://review.opendev.org/c/openstack/nova/+/998822 | 20:35 |
| opendevreview | Goutham Pacha Ravi proposed openstack/nova master: Support cold migration with virtiofs shares https://review.opendev.org/c/openstack/nova/+/989633 | 22:45 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!