| dvo-plv | sean-k-mooney, Hello | 06:44 |
|---|---|---|
| dvo-plv | regarding our yesterday discussion. I see two options. we can get back to the separate mech driver, like was suggested year ago | 06:45 |
| dvo-plv | or we can pass socket name + pci endpoint in some way | 06:46 |
| Uggla | Hello Nova. | 07:25 |
| gibi | Uggla: o/ | 07:34 |
| opendevreview | melanie witt proposed openstack/nova master: nova-manage: Add 'limits migrate_to_unified_limits' https://review.opendev.org/c/openstack/nova/+/891646 | 08:24 |
| opendevreview | Maksim Malchuk proposed openstack/nova stable/xena: Fix to implement 'pack' or 'spread' VM's NUMA cells https://review.opendev.org/c/openstack/nova/+/829804 | 12:56 |
| *** artom_ is now known as artom | 12:57 | |
| *** tbachman_ is now known as tbachman | 12:57 | |
| *** ozzzo1 is now known as ozzzo | 12:57 | |
| *** NeilHanlon_ is now known as NeilHanlon | 13:11 | |
| *** ykarel_ is now known as ykarel | 14:01 | |
| opendevreview | Dan Smith proposed openstack/nova master: WIP: Detach disks using alias when possible https://review.opendev.org/c/openstack/nova/+/893068 | 14:47 |
| dansmith | kashyap: was just going to point you at this, first PoC of using the stable alias to detach disks: https://review.opendev.org/c/openstack/nova/+/893068 | 14:49 |
| dansmith | seems to work, without having to muck with all the actual detach plumbing like I was originally thinking | 14:49 |
| dansmith | I think something similar should work for interfaces too | 14:49 |
| dansmith | there's a lot more looking-up-of-devices elsewhere that would need to change, but I'm hoping those places will be able to use that helper too | 14:49 |
| opendevreview | Elod Illes proposed openstack/nova stable/yoga: Don't provide MTU value in metadata service if DHCP is enabled https://review.opendev.org/c/openstack/nova/+/879701 | 14:50 |
| kashyap | dansmith: Nice that you preserve the fallback to using 'disk_dev' | 14:50 |
| kashyap | dansmith: I guess you're (me for sure) pleasnatly surprised that it turned out to be much less of code-footprint than anticipated | 14:52 |
| kashyap | (Unless there are no nasty surprises lurking somewhere) | 14:52 |
| dansmith | yeah I originally had rewritten all the detach stuff to use that per-alias method, but realized it was really not necessary and that I could just change how we looked it up instead | 14:52 |
| dansmith | which is much smaller *and* should apply to the other places where we just need to look it up and not detach | 14:53 |
| kashyap | Smart. (On changing how we looked it up. I would've racked my brains longer) | 14:53 |
| opendevreview | Pavlo Shchelokovskyy proposed openstack/nova master: Replace -- as 3-4 char in hostname with - https://review.opendev.org/c/openstack/nova/+/893072 | 14:53 |
| dansmith | so I'm hoping that helper on guest that looks up all devices and then finds by alias will work for interfaces in a similar workflow | 14:54 |
| kashyap | You already ran nova.tests.unit.virt.libvirt.* unit tests locallY? | 14:55 |
| dansmith | no, no tests, it just works in a devstack | 14:57 |
| opendevreview | Elod Illes proposed openstack/nova stable/xena: Don't provide MTU value in metadata service if DHCP is enabled https://review.opendev.org/c/openstack/nova/+/879703 | 14:57 |
| opendevreview | Pavlo Shchelokovskyy proposed openstack/nova master: Replace -- as 3-4 char in hostname with - https://review.opendev.org/c/openstack/nova/+/893072 | 14:57 |
| dansmith | kashyap: i.e.: Aug 29 14:40:22 jammy nova-compute[306476]: DEBUG nova.virt.libvirt.driver [None req-323b5960-d0c5-4729-b269-5743cd4fdf90 demo demo] Found disk vdb by alias ua-ada5af06-300e-4d07-931d-3cc2bff8a8a9 {{(pid=306476) _get_guest_disk_device /opt/stack/nova/nova/virt/libvirt/driver.py:2722}} | 14:57 |
| kashyap | Ah, nice. | 14:58 |
| opendevreview | Elod Illes proposed openstack/nova stable/wallaby: Don't provide MTU value in metadata service if DHCP is enabled https://review.opendev.org/c/openstack/nova/+/879704 | 15:01 |
| kashyap | dansmith: I'm just quietly wondering if there are scenarios where we have to care about transient disk configs at all | 15:02 |
| bauzas | auniyal: left a comment on https://review.opendev.org/c/openstack/nova/+/889947 but also on the dependent patch, please review it as soon as you can | 15:02 |
| opendevreview | Elod Illes proposed openstack/nova stable/victoria: Don't provide MTU value in metadata service if DHCP is enabled https://review.opendev.org/c/openstack/nova/+/879705 | 15:03 |
| dansmith | kashyap: meaning all the detaching from live+persistent you mean? | 15:04 |
| dansmith | AFAIK we pretty much always keep them in sync, so I'm not sure why we're going to all that trouble, yeah | 15:04 |
| kashyap | (Yeah, detach from both live and persistent) | 15:05 |
| kashyap | I didn't realize we're keeping them in sync. Then one less trouble. | 15:05 |
| opendevreview | Elod Illes proposed openstack/nova stable/ussuri: Don't provide MTU value in metadata service if DHCP is enabled https://review.opendev.org/c/openstack/nova/+/879702 | 15:06 |
| bauzas | reminder : nova meeting in 55 mins (heya, back) | 15:06 |
| opendevreview | Pavlo Shchelokovskyy proposed openstack/nova master: Replace -- as 3-4 char in hostname with - https://review.opendev.org/c/openstack/nova/+/893072 | 15:32 |
| bauzas | auniyal: also gave -1 for a few nits on https://review.opendev.org/c/openstack/nova/+/882284 | 15:48 |
| auniyal | ack bauzas, thanks for reviews | 16:00 |
| bauzas | #startmeeting nova | 16:00 |
| opendevmeet | Meeting started Tue Aug 29 16:00:07 2023 UTC and is due to finish in 60 minutes. The chair is bauzas. 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 |
| bauzas | hey folks, welcome | 16:00 |
| auniyal | o/ | 16:00 |
| bauzas | I'm eventually released from Canada | 16:00 |
| bauzas | #link https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting | 16:00 |
| bauzas | who's around ? | 16:00 |
| Uggla | o/ | 16:01 |
| gmann | o/ | 16:01 |
| elodilles | o/ | 16:01 |
| * gibi is partially around (on a call in parallel) | 16:01 | |
| bauzas | we can softly start | 16:01 |
| bauzas | #topic Bugs (stuck/critical) | 16:01 |
| bauzas | #info No Critical bug | 16:01 |
| bauzas | #link https://bugs.launchpad.net/nova/+bugs?search=Search&field.status=New 49 new untriaged bugs (+9 since the last meeting) | 16:01 |
| dansmith | o/ | 16:01 |
| bauzas | #info Add yourself in the team bug roster if you want to help https://etherpad.opendev.org/p/nova-bug-triage-roster | 16:02 |
| bauzas | so, we would like to do some bug triage | 16:02 |
| bauzas | auniyal: you're the next in the roster list, fancy with this ? | 16:02 |
| auniyal | ack bauzas | 16:02 |
| auniyal | yes | 16:03 |
| bauzas | very much appreciated | 16:03 |
| bauzas | #info bug baton is auniyal | 16:03 |
| bauzas | thanks | 16:03 |
| bauzas | moving on, I don't see any critical bug to discuss | 16:03 |
| bauzas | #topic Gate status | 16:03 |
| bauzas | #link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs | 16:03 |
| bauzas | #link https://etherpad.opendev.org/p/nova-ci-failures | 16:04 |
| bauzas | #link https://zuul.openstack.org/builds?project=openstack%2Fnova&project=openstack%2Fplacement&pipeline=periodic-weekly Nova&Placement periodic jobs status | 16:04 |
| bauzas | #info nova-emulation seems flakey | 16:04 |
| bauzas | #info https://zuul.openstack.org/build/209dc9676c4149b281420d9ae18f889c | 16:04 |
| dansmith | things are better lately, thanks to a lot of hard work by a number of people, most recently probably melwitt with an LVM fix that might help with some stability, and a functional fix from gibi | 16:05 |
| bauzas | dansmith: ++, I was about to ask for news :) | 16:05 |
| dansmith | still need to work on some random timeouts we get which I think is related to stress on the api workers (at least in part) and melwitt has a green/native threads refactor up which I hope will eventually help with that | 16:05 |
| bauzas | I have to admit the issues I've seen are out of my knowledge area :( | 16:06 |
| dansmith | also still occasional OOMs which are hard to address when we're legitimately running out of memory if we get unlucky with what gets started when | 16:06 |
| bauzas | dansmith: ack, any change in particular for greenthreading ? | 16:06 |
| dansmith | bauzas: we need to move to native threads in nova-api when under wsgi so that we can handle more than two things at once | 16:06 |
| dansmith | *and* so we can enable it in devstack so that other services which can already do so can leverage that :) | 16:07 |
| bauzas | I see | 16:07 |
| dansmith | anyway, that's pretty much the status.. I encourage everyone to look critically at any failure they see and try to resolve it or nail it down while waiting for your recheck :) | 16:08 |
| bauzas | well, I'd say, I'm a bad doctor, I'm only able to tell what's wrong without any cure | 16:08 |
| bauzas | and my diagnostics are always biased | 16:09 |
| bauzas | but yeah, teamwork, agreed :) | 16:09 |
| dansmith | well, doctors require continuing education to get better ;) | 16:09 |
| bauzas | not only doctors :) | 16:09 |
| bauzas | anyway, thanks for the update | 16:09 |
| bauzas | I'm still wondering if the GPT errors I'm seeing in the console are somehow related to the timeouts | 16:10 |
| dansmith | I don't think so, | 16:11 |
| dansmith | we get those every time we attach a blank disk to an instance | 16:11 |
| bauzas | ack, ok | 16:11 |
| dansmith | the correlation is lots of timeouts come from volume tests, | 16:11 |
| dansmith | which always dump console logs, and you see those things that look error-like | 16:11 |
| dansmith | correlated, but not caused | 16:12 |
| bauzas | yeah, correlation is not cause, I get it :) | 16:12 |
| bauzas | I know sean-k-mooney was also investing any alping image usage in some job | 16:13 |
| bauzas | have we moved forward on it ? | 16:13 |
| bauzas | alpine* | 16:13 |
| bauzas | anyway, I think gate stability deserves some discussions at the PTG, if we haven't closed the gaps yet | 16:14 |
| dansmith | not that I know of | 16:15 |
| bauzas | moving on then | 16:16 |
| sean-k-mooney | i havent looked at that in a while | 16:16 |
| bauzas | #info Please look at the gate failures and file a bug report with the gate-failure tag. | 16:16 |
| sean-k-mooney | but it was failing becuase it did not have growfs i.e. the disk root partition was not being expanded | 16:16 |
| sean-k-mooney | easy fix jsut have not had time to work on it | 16:16 |
| bauzas | sean-k-mooney: ack, this week I'm busy but next week, I could help | 16:17 |
| sean-k-mooney | we can move on but https://review.opendev.org/q/topic:alpine is the topic for context | 16:17 |
| bauzas | moving on indeed | 16:17 |
| bauzas | #topic Release Planning | 16:18 |
| bauzas | #link https://releases.openstack.org/bobcat/schedule.html | 16:18 |
| bauzas | #link https://etherpad.opendev.org/p/nova-bobcat-blueprint-status Etherpad for tracking blueprints status | 16:18 |
| bauzas | #info 2 days before FeatureFreeze | 16:18 |
| bauzas | so, I tried to update the etherpad based on my recent findings | 16:18 |
| bauzas | we have about 5 series that are reviewable | 16:18 |
| bauzas | and we only have 2 days to get them into some state | 16:19 |
| bauzas | of those 5 series, 2 of them are quite large | 16:19 |
| bauzas | so I don't expect a full merge, but I'd appreciate if some reviews could be made, particularly in order to let them move forward quickly once we open Caracal feature dev | 16:20 |
| bauzas | and we can discuss whether we can half-merge some series or hold the whole | 16:20 |
| bauzas | so, please, take your pens | 16:20 |
| bauzas | I'll continue to give a few review rounds till the last day, but I also need traction from others :) | 16:21 |
| bauzas | and if you're an owner of one of those series, feel free to ping us on IRC if you think you'd like feedback | 16:22 |
| bauzas | I think we're done on this topic | 16:24 |
| bauzas | I'll probably ping a couple of folks after this meeting to get a proper status | 16:24 |
| bauzas | #topic Review priorities | 16:25 |
| bauzas | #link https://review.opendev.org/q/status:open+(project:openstack/nova+OR+project:openstack/placement+OR+project:openstack/os-traits+OR+project:openstack/os-resource-classes+OR+project:openstack/os-vif+OR+project:openstack/python-novaclient+OR+project:openstack/osc-placement)+(label:Review-Priority%252B1+OR+label:Review-Priority%252B2) | 16:25 |
| bauzas | #topic Stable Branches | 16:26 |
| bauzas | elodilles: floor is yours | 16:26 |
| elodilles | well, i was on PTO last week, | 16:26 |
| elodilles | and i didn't see much activity on stable branches | 16:27 |
| elodilles | i guess people are more focused on Bobcat nowadays ;) | 16:27 |
| bauzas | this week, for sure | 16:27 |
| elodilles | anyway, i'm not aware of any stable gate failures | 16:27 |
| elodilles | #info stable branch status / gate failures tracking etherpad: https://etherpad.opendev.org/p/nova-stable-branch-ci | 16:27 |
| elodilles | feel free to add anything there if you find any issue ^^^ | 16:28 |
| elodilles | that's all from me | 16:28 |
| auniyal | #info Please{10} *review* these backport patches of stable 2023.1, zed and yoga for next minor release | 16:28 |
| auniyal | #info most of these already have one +2. | 16:28 |
| auniyal | #link https://etherpad.opendev.org/p/release-liaison-PatchesToReview | 16:29 |
| bauzas | yeah, it would be nice to deliver some stable releases while we reach the rc period | 16:30 |
| elodilles | ++ | 16:30 |
| bauzas | I'll personnally doublecheck if some are missing my +2 | 16:30 |
| auniyal | I added 2 more patches for stable/2023.1 | 16:31 |
| auniyal | earlier bauzas had merged all | 16:31 |
| bauzas | a 2023.1 stable release before we cut rc1 is important | 16:31 |
| bauzas | operators can then upgrade to the latest .z release on Antelope before upgrading smoothly to Bobcat | 16:32 |
| bauzas | but ok, I'll try to chase them down | 16:32 |
| bauzas | on a note, my usual stable branches item : | 16:32 |
| bauzas | #info train-eol patch proposed https://review.opendev.org/c/openstack/releases/+/885365 | 16:33 |
| bauzas | I guess it's waiting a second releases core | 16:33 |
| elodilles | oh, it's still not merged... :S | 16:33 |
| elodilles | yes, it is waiting for that | 16:33 |
| bauzas | I know US trains are pretty long | 16:33 |
| elodilles | :) | 16:33 |
| bauzas | anyway, moving on, last topic | 16:33 |
| bauzas | #topic Open discussion | 16:34 |
| bauzas | nothing on my agenda, ever after a wiki refresh | 16:34 |
| bauzas | so, anything anyone ? | 16:34 |
| bauzas | looks not | 16:37 |
| bauzas | thanks all, and tbh I'm happy to see all of you back :) | 16:37 |
| bauzas | #endmeetinh | 16:37 |
| bauzas | doh | 16:37 |
| bauzas | #endmeeting | 16:37 |
| opendevmeet | Meeting ended Tue Aug 29 16:37:43 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:37 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2023/nova.2023-08-29-16.00.html | 16:37 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2023/nova.2023-08-29-16.00.txt | 16:37 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2023/nova.2023-08-29-16.00.log.html | 16:37 |
| elodilles | thanks o/ | 16:37 |
| gibi | o/ | 16:38 |
| opendevreview | Merged openstack/os-vif master: Don't break traffic if port already exists https://review.opendev.org/c/openstack/os-vif/+/885127 | 17:38 |
| opendevreview | Raul Moldovan proposed openstack/nova master: Update resources post live migration https://review.opendev.org/c/openstack/nova/+/892614 | 18:11 |
| *** melwitt_ is now known as melwitt | 18:34 | |
| opendevreview | Raul Moldovan proposed openstack/nova master: Update resources post live migration https://review.opendev.org/c/openstack/nova/+/892614 | 20:28 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!