| opendevreview | Rodolfo Alonso proposed openstack/os-vif master: ovs: Pass ``multi_queue`` inside ``ifr`` dict for tuntap devices https://review.opendev.org/c/openstack/os-vif/+/995933 | 06:58 |
|---|---|---|
| opendevreview | Masahito Muroi proposed openstack/nova master: Add bug 2106085 reproducer test case https://review.opendev.org/c/openstack/nova/+/971153 | 07:08 |
| opendevreview | Masahito Muroi proposed openstack/nova master: Add check_instance_state validation to the four action API https://review.opendev.org/c/openstack/nova/+/946223 | 07:08 |
| artem_vasilyev | Hi, please make a review for this bugfix: https://review.opendev.org/c/openstack/nova/+/984048 It fixes race between resize and update_available_resource for dedicated CPUs, which can lead to CPUUnpinningInvalid errors | 07:50 |
| opendevreview | Merged openstack/os-vif master: typing: Adapt to oslo.versionedobjects 3.11 https://review.opendev.org/c/openstack/os-vif/+/997289 | 08:20 |
| opendevreview | Merged openstack/os-vif master: typing: Define supported OVS VIF types https://review.opendev.org/c/openstack/os-vif/+/997296 | 08:20 |
| gibi | folks nova-alt-configuration is broken and blocking the gate https://zuul.opendev.org/t/openstack/builds?job_name=nova-alt-configurations&project=openstack/nova | 08:37 |
| gibi | do we have a bug report and somebody already actively working on it? | 08:37 |
| bauzas | gibi: sorry, no idea | 08:44 |
| bauzas | but this is clear that it failed at some point on Monday | 08:45 |
| gibi | yep | 08:45 |
| frickler | ah, that's a known issue in OSC I think "public endpoint for block-storage service in RegionOne region not found" | 08:47 |
| frickler | see https://review.opendev.org/c/openstack/openstacksdk/+/997117 | 08:47 |
| gibi | frickler: thanks | 08:49 |
| frickler | actually it looks like osc 10.2.1 fixes this, so just need to wait for the u-c bump to merge https://review.opendev.org/c/openstack/releases/+/997204 | 08:50 |
| gibi | cool then I will wait for https://review.opendev.org/c/openstack/releases/+/997204 | 08:51 |
| bauzas | thanks frickler | 08:51 |
| gibi | yeah nova-alt-configuration disables cinder so the bug description matches | 08:52 |
| *** ykarel_ is now known as ykarel | 09:13 | |
| opendevreview | Merged openstack/nova stable/2025.2: Make QEMU_IMG_LIMITS process limits configurable https://review.opendev.org/c/openstack/nova/+/997028 | 10:16 |
| opendevreview | Balazs Gibizer proposed openstack/placement master: Reproduce bug XXX https://review.opendev.org/c/openstack/placement/+/997362 | 10:22 |
| gibi | sean-k-mooney: ^^ this is the repro for the next placement a_c query slowness we saw downstream | 10:23 |
| sean-k-mooney | gibi: oh cool you had time to start triaging it | 10:23 |
| gibi | yepp | 10:23 |
| gibi | at least with the existing functional test it is easy to reproduce | 10:23 |
| sean-k-mooney | i have a intuition of what we shoudl be doing at the sql level | 10:23 |
| gibi | I will see if I have time this sprint to look for an optimization | 10:24 |
| sean-k-mooney | but im not sure how to express that in placement | 10:24 |
| sean-k-mooney | do you have time to chat about it on meet breifly | 10:24 |
| gibi | not right now | 10:25 |
| gibi | I will ping you later | 10:25 |
| sean-k-mooney | ok cool my idea is pretty simple and i tought something placement already idd but apprently not | 10:25 |
| sean-k-mooney | effectivly we shoudl be flattenting both the provider tree and aagreating the counts of all free resouce classe in the tree and flatteing the request and droping any root provider where the recquest exceed any RC in the flatttened tree | 10:26 |
| sean-k-mooney | that tells use that regardless of the toplogy that root provider cant supprot the request | 10:27 |
| sean-k-mooney | if that flanned check passes then and only then do we process it in python | 10:27 |
| sean-k-mooney | i think that can be expressed as a select root_uuid, rc, capcity group by root_uuid or similar to get the flattend trees | 10:28 |
| sean-k-mooney | we can trivially flaten the request in python to constuct a where clause | 10:29 |
| sean-k-mooney | and basiclly seleect out the root_uuids form that where the flattend free capsity is greater then then the requested ammount | 10:30 |
| sean-k-mooney | obviously its more complicated then that as we need to compute the free capsty not just the total and genericly we need to account for the allocation ratio as well | 10:31 |
| sean-k-mooney | so conceptually simple but expressign that in placment could be tricky | 10:31 |
| sean-k-mooney | if we can implemnt that optimistation we will never do the expnesive combinitorial decent if it cant fit | 10:32 |
| sean-k-mooney | when i type it out it feels more complicated the it is in my head :) | 10:38 |
| opendevreview | Kamil Sambor proposed openstack/nova master: libvirt: retry get_guest() on InstanceNotFound during post-copy VIF reattach https://review.opendev.org/c/openstack/nova/+/997380 | 12:15 |
| sean-k-mooney | gibi: this shoudl not require a spec but if i was to have ai formualt my idea into spec format it woudl looks somethign like https://paste.opendev.org/show/bVKIYH6Om0svZbMwlCbg/ :) | 12:15 |
| gibi | sean-k-mooney: I feel that this flatten and do a pre-flight check make sense. It solve the current problem. There is probably more problems. I.e. when this pre-flight check passes but the normal expansion still finds, in the long way, that the host is not having enough resources. | 14:10 |
| gibi | this is a wack-a-mole as we knew since the first perf bug was triaged :) | 14:10 |
| sean-k-mooney | https://github.com/SeanMooney/tachyon/blob/master/reports/almost-fitting-root-bug-report.md | 14:11 |
| sean-k-mooney | im not saying my ai generate placment replacemnt is a valid solution | 14:11 |
| sean-k-mooney | but i quickly tested the prefilterign there and it seam to resolve the currnt issue | 14:11 |
| gibi | sean-k-mooney: if you would like to take the bug (once I file it upstream) I'm happy to pass it on :) | 14:13 |
| sean-k-mooney | gibi: i proably have to many other things on my plate | 14:13 |
| sean-k-mooney | if you have time to do it properly in placment then that might be more efficent | 14:13 |
| sean-k-mooney | i just left ai run on the idea to validate the concept | 14:14 |
| sean-k-mooney | im interested in this because cyborg makes this problem worse | 14:14 |
| sean-k-mooney | becaus it report every VF as its onw rp today | 14:14 |
| sean-k-mooney | that on my todo list to fix next cycle but i am generally conferedn/interested in the placment performance | 14:15 |
| gibi | yeah. right now I have no time to read the 1000 lines proposal. If this bug gets into our next sprint then I will load context on the shape of the placement code and then it would make sense to me to read you proposal to see if that fits in to the code | 14:16 |
| gibi | if/how | 14:16 |
| opendevreview | Kamil Sambor proposed openstack/nova master: Replace StaticallyDelayingWrapper with futurist DelayingExecutor https://review.opendev.org/c/openstack/nova/+/997410 | 14:38 |
| gibi | fyi, nova-alt-configuration is green again, so the gate is unblocked | 14:47 |
| gibi | nova upstream bug triage call https://meet.google.com/zjr-rxus-hzj | 15:28 |
| sean-k-mooney | https://meetings.opendev.org/#Nova_Team_Meeting is there but we shoudl proably add that meetign sepreatly | 15:28 |
| gibi | (I expect it to be short as Uggla is on PTO) | 15:28 |
| opendevreview | Balazs Gibizer proposed openstack/placement master: Reproduce bug 2160721 https://review.opendev.org/c/openstack/placement/+/997362 | 17:07 |
| opendevreview | Merged openstack/nova master: Remove unused method for image v1 API https://review.opendev.org/c/openstack/nova/+/995532 | 18:30 |
| opendevreview | Merged openstack/nova master: Remove unused UpdateGlanceImage https://review.opendev.org/c/openstack/nova/+/995726 | 18:33 |
| opendevreview | Merged openstack/nova master: Accept hw_firmware_stateless=True for BIOS firmware type https://review.opendev.org/c/openstack/nova/+/995835 | 19:31 |
| opendevreview | Merged openstack/nova stable/2025.1: Make QEMU_IMG_LIMITS process limits configurable https://review.opendev.org/c/openstack/nova/+/997029 | 20:42 |
| opendevreview | Merged openstack/os-vif master: ovs: Preserve versioned field compatibility https://review.opendev.org/c/openstack/os-vif/+/997297 | 21:08 |
| opendevreview | Merged openstack/os-vif master: ovs: Pass ``multi_queue`` inside ``ifr`` dict for tuntap devices https://review.opendev.org/c/openstack/os-vif/+/995933 | 21:08 |
| opendevreview | Merged openstack/nova master: Add vTPM startup validation reproducer https://review.opendev.org/c/openstack/nova/+/990551 | 21:34 |
| opendevreview | Merged openstack/nova master: Delegate startup instance processing to drivers https://review.opendev.org/c/openstack/nova/+/990552 | 22:22 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!