| *** eandersson15 is now known as eandersson | 02:08 | |
| opendevreview | melanie witt proposed openstack/nova master: testing: Use inspect.isfunction() to check signatures https://review.opendev.org/c/openstack/nova/+/883217 | 02:21 |
|---|---|---|
| opendevreview | melanie witt proposed openstack/nova master: testing: Use inspect.isfunction() to check signatures https://review.opendev.org/c/openstack/nova/+/883217 | 03:08 |
| elodilles | hi nova team! Happy *** Nova Stable Branches Review Day *** to everyone! \o/ o:) | 07:29 |
| bauzas | elodilles: thanks for the reminder :) | 07:56 |
| jamespage | dansmith, fungi: thanks - looking now | 08:01 |
| bauzas | gibi: can you give a swing on that stable review day for https://review.opendev.org/c/openstack/nova/+/882786 ? :) | 08:45 |
| gibi | bauzas: done | 09:01 |
| gibi | on a similar note I need stable review on https://review.opendev.org/q/topic:bug%252F1994526 | 09:02 |
| bauzas | gibi: I did it for 2023.1, will look at zed | 09:02 |
| elodilles | gibi: 2023.1 version is on the gate now | 09:16 |
| elodilles | and if you have time for these simple gate fix backports to review: https://review.opendev.org/q/topic:validate-backport-release-id | 09:17 |
| elodilles | o:) | 09:17 |
| gibi | looking | 09:19 |
| gibi | done :) | 09:21 |
| elodilles | thx \o/ | 09:24 |
| opendevreview | Amit Uniyal proposed openstack/nova-specs master: Adds cleanup to remove dangling volumes https://review.opendev.org/c/openstack/nova-specs/+/878757 | 09:27 |
| auniyal_ | gibi, elodilles in case you are done with stable reviews, only these 2 stable/2023.1 patches missed: https://review.opendev.org/c/openstack/nova/+/882293 , https://review.opendev.org/c/openstack/nova/+/882913 | 09:42 |
| auniyal_ | bauzas, already had +2 in one, and other one is from bauzas | 09:43 |
| bauzas | np | 09:44 |
| zigo | melwitt: I was able to get Nova / Victoria to build correctly with the patch. thanks for your usefull hint ! | 09:50 |
| elodilles | zigo: if you could upload the victoria patch then i can review it as well ;) | 09:59 |
| zigo | elodilles: It would need the os-brick patch to be merged first, no? | 10:34 |
| elodilles | zigo: probably yes, but i guess then you have that too | 11:05 |
| opendevreview | Merged openstack/nova stable/zed: Revert "Debug Nova APIs call failures" https://review.opendev.org/c/openstack/nova/+/882786 | 11:09 |
| opendevreview | Merged openstack/nova stable/2023.1: Reproduce asym NUMA mixed CPU policy bug https://review.opendev.org/c/openstack/nova/+/882508 | 11:09 |
| opendevreview | Elod Illes proposed openstack/nova stable/victoria: Cleanup old resize instances dir before resize https://review.opendev.org/c/openstack/nova/+/864730 | 11:21 |
| opendevreview | Merged openstack/nova stable/2023.1: Handle zero pinned CPU in a cell with mixed policy https://review.opendev.org/c/openstack/nova/+/882509 | 11:21 |
| opendevreview | Merged openstack/nova stable/wallaby: CI: fix backport validator for new branch naming https://review.opendev.org/c/openstack/nova/+/882968 | 11:21 |
| zigo | elodilles: I'm currently finishing the debian/bullseye package updates, I'll see if I can do that later on. | 11:30 |
| elodilles | zigo: thanks in advance! please ping me for those backports so that i can review them (both nova and os-brick) | 11:32 |
| opendevreview | Merged openstack/nova stable/2023.1: Have host look for CPU controller of cgroupsv2 location. https://review.opendev.org/c/openstack/nova/+/882913 | 12:05 |
| auniyal_ | nice all stable/2023.1 patches merged | 12:12 |
| auniyal_ | oh 1 pending | 12:13 |
| elodilles | hmmm, it seems stable/wallaby gate is broken as nova-ceph-multistore is failing: https://zuul.opendev.org/t/openstack/builds?job_name=nova-ceph-multistore&project=openstack%2Fnova&branch=stable%2Fwallaby&skip=0 | 13:36 |
| elodilles | it seems cinder-tempest-plugin is too new and needs latest tempest which is not allowed (?) on wallaby | 13:38 |
| elodilles | (yepp, upper constraints pins tempest===27.0.0) | 13:39 |
| dansmith | elodilles: yeah I mentioned that yesterday | 14:00 |
| dansmith | elodilles: I think the only reasonable solution is to drop that job from wallaby | 14:00 |
| dansmith | I mean, we can certainly fix it, I'm just not sure it's worth it for something so old | 14:00 |
| elodilles | dansmith: isn't that enough if we just ensure that the job should use older cinder-tempest-plugin? | 14:02 |
| dansmith | elodilles: probably, but the fixes to c-t-p were for stability and it just seems like it better fits the "best effort" approach to drop jobs that need that level of work | 14:03 |
| dansmith | I guess devstack-plugin-ceph is branched so that will be okay (there was a tangled web of changes amongst all three) | 14:04 |
| elodilles | dansmith: i can accept to drop the job, just thought to better fix it if possible | 14:05 |
| dansmith | elodilles: your call | 14:06 |
| elodilles | :) | 14:06 |
| elodilles | sure, i'm looking it now and will propose a job dropping patch if i don't find an easy solution :D | 14:06 |
| opendevreview | Danylo Vodopianov proposed openstack/nova master: Packed virtqueue support was added. https://review.opendev.org/c/openstack/nova/+/876075 | 14:11 |
| bauzas | elodilles: dansmith: I'll create a release for 2023.1 and Zed tonight then | 14:12 |
| elodilles | bauzas: ack, thanks in advance! | 14:13 |
| elodilles | bauzas: can you propose a yoga release patch as well? o:) I think that is also good to go | 14:14 |
| bauzas | elodilles: sure | 14:14 |
| elodilles | just ping me and i'll review them ASAP :) | 14:15 |
| bauzas | reminder: nova meeting in 1 hour-ish | 15:03 |
| bauzas | here | 15:03 |
| elodilles | bauzas: i'll quickly update stable status for the meeting if you are done with your editing | 15:47 |
| bauzas | #startmeeting nova | 16:02 |
| opendevmeet | Meeting started Tue May 16 16:02:14 2023 UTC and is due to finish in 60 minutes. The chair is bauzas. Information about MeetBot at http://wiki.debian.org/MeetBot. | 16:02 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 16:02 |
| opendevmeet | The meeting name has been set to 'nova' | 16:02 |
| bauzas | sorry folks I was in the wrong chan :facepalm | 16:02 |
| gibi | o/ | 16:02 |
| elodilles | o/ | 16:02 |
| bauzas | elodilles: I didn't had time to modify the agenda yet | 16:02 |
| dansmith | o/ | 16:02 |
| elodilles | bauzas: no problem, i haven't updated it either o:) | 16:02 |
| auniyal | o/ | 16:03 |
| bauzas | ok, done | 16:04 |
| bauzas | #link https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting | 16:04 |
| bauzas | let's start, we're already late | 16:04 |
| bauzas | #topic Bugs (stuck/critical) | 16:04 |
| bauzas | #info No Critical bug | 16:05 |
| bauzas | #link https://bugs.launchpad.net/nova/+bugs?search=Search&field.status=New 15 new untriaged bugs (+0 since the last meeting) | 16:05 |
| bauzas | #info Add yourself in the team bug roster if you want to help https://etherpad.opendev.org/p/nova-bug-triage-roster | 16:05 |
| bauzas | #info bug baton is being passed to elodilles | 16:05 |
| bauzas | elodilles: are you okay with this ? | 16:05 |
| elodilles | note that i'll be OoO on Thursday and Friday o:) | 16:05 |
| bauzas | ditto here | 16:05 |
| bauzas | + Wed for me | 16:06 |
| elodilles | otherwise +1 | 16:06 |
| Uggla | o/ | 16:06 |
| bauzas | ok, cool thanks | 16:06 |
| bauzas | any bug people want to discuss ? | 16:06 |
| bauzas | looks not | 16:07 |
| bauzas | moving on ? | 16:07 |
| bauzas | looks so | 16:09 |
| bauzas | #topic Gate status | 16:09 |
| bauzas | #link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs | 16:09 |
| bauzas | #link https://etherpad.opendev.org/p/nova-ci-failures | 16:09 |
| bauzas | honestly, it looked to me a quiet week | 16:09 |
| bauzas | we were able to merge a few changes without rechecking a lot | 16:09 |
| gibi | that sounds good | 16:10 |
| dansmith | anyone seen any volume detach fails? | 16:10 |
| gibi | - | 16:10 |
| dansmith | (other than those caused by guest kernel crashes or other things) | 16:10 |
| bauzas | nope, not yet | 16:10 |
| dansmith | cool | 16:10 |
| bauzas | ok, let's move on then | 16:11 |
| auniyal | in stable/2023.1 this too can merge - https://review.opendev.org/c/openstack/nova/+/882293, need 1 +2 | 16:12 |
| bauzas | #link https://zuul.openstack.org/builds?project=openstack%2Fnova&project=openstack%2Fplacement&pipeline=periodic-weekly Nova&Placement periodic jobs status | 16:12 |
| bauzas | auniyal: sorry, wrong topic atm, we'll discuss the stable branches later | 16:12 |
| auniyal | ack bauzas | 16:12 |
| bauzas | all greens for the periodics, cool. | 16:13 |
| elodilles | \o/ | 16:13 |
| bauzas | #info Please look at the gate failures and file a bug report with the gate-failure tag. | 16:13 |
| bauzas | #info STOP DOING BLIND RECHECKS aka. 'recheck' https://docs.openstack.org/project-team-guide/testing.html#how-to-handle-test-failures | 16:13 |
| bauzas | ok, next topic | 16:14 |
| bauzas | #topic Release Planning | 16:14 |
| bauzas | #link https://releases.openstack.org/bobcat/schedule.html | 16:14 |
| bauzas | #info Nova deadlines are set in the above schedule | 16:14 |
| bauzas | #info Today is stable branches review day | 16:15 |
| bauzas | so, right time for reviewing some changes like what said auniyal | 16:15 |
| bauzas | I also prepared some releases for our stable releases, due to the CVE fix | 16:15 |
| bauzas | sec | 16:15 |
| bauzas | #link https://review.opendev.org/c/openstack/releases/+/883277 | 16:15 |
| bauzas | #link https://review.opendev.org/c/openstack/releases/+/883280 | 16:16 |
| bauzas | #link https://review.opendev.org/c/openstack/releases/+/883283 | 16:16 |
| bauzas | we could wait for other bugfixes to be merged, but honestly I'd prefer to create first a release and then release again | 16:16 |
| elodilles | +1 | 16:16 |
| elodilles | thanks! i will review them after the meeting with my release mgmt hat on o:) | 16:17 |
| bauzas | elodilles: thanks | 16:17 |
| bauzas | so, please review our stable branches today :) | 16:18 |
| bauzas | anyway, moving on | 16:18 |
| bauzas | #topic Review priorities | 16:18 |
| 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:18 |
| bauzas | #info As a reminder, cores eager to review changes can +1 to indicate their interest, +2 for committing to the review | 16:18 |
| bauzas | #topic Stable Branches | 16:18 |
| bauzas | elodilles: now, please take your stable branch hat :) | 16:18 |
| elodilles | :] | 16:19 |
| elodilles | #info stable gates should be OK except on stable/xena | 16:19 |
| elodilles | nova-ceph-multistore job is broken on xena, | 16:19 |
| elodilles | and we discussed with dansmith today | 16:19 |
| elodilles | i'll try to find a simple fix, | 16:19 |
| elodilles | but if i won't succeed, then we can drop that job on stable/xena | 16:20 |
| * bauzas nods | 16:20 | |
| elodilles | as it is alread in EM branch | 16:20 |
| elodilles | #info stable branch status / gate failures tracking etherpad: https://etherpad.opendev.org/p/nova-stable-branch-ci | 16:20 |
| elodilles | i think that's it from me | 16:20 |
| dansmith | yup | 16:20 |
| elodilles | and thanks for all the stable reviews :) | 16:21 |
| elodilles | quite some stable patches have been merged today :) | 16:21 |
| bauzas | indeed | 16:22 |
| gibi | \o/ | 16:22 |
| bauzas | but yeah we have other bugfixes that could be looked | 16:22 |
| bauzas | so, please continue :) | 16:23 |
| elodilles | ++ | 16:23 |
| auniyal | bauzas, quick update on dansmith above question and as we are discussing stable branches, | 16:23 |
| auniyal | dansmith, similar to volume attach/detach fails - testtest_live_block_migration_with_attached_volume failed for stable/wallaby | 16:23 |
| auniyal | I feel like I should give more info, so here https://14cb8680ad7e2d5893c2-a0a2161f988b6356e48326da15450ffb.ssl.cf1.rackcdn.com/877334/2/check/nova-grenade-multinode/abcf382/testr_results.html | 16:25 |
| dansmith | the fixes for those failures definitely won't be seen on wallaby right now | 16:26 |
| dansmith | but also that one looks like a guest panic to me | 16:26 |
| dansmith | so not the normal volume detach problems | 16:26 |
| auniyal | gate is frequently failing in these 2 patches on diff jobs: https://review.opendev.org/c/openstack/nova/+/877334, https://review.opendev.org/c/openstack/nova/+/882939 | 16:27 |
| auniyal | ack | 16:27 |
| elodilles | auniyal: unfortunately wallaby and victoria gates are the ones with the most intermittent failures :/ | 16:28 |
| bauzas | I've seen this pattern a lot | 16:28 |
| bauzas | sec, will find you the upstream bug report for rechecks | 16:28 |
| auniyal | ack elodilles | 16:29 |
| elodilles | correction for my early statement: *stable/wallaby* gate is broken, NOT stable/xena (sorry :S) | 16:29 |
| elodilles | (with nova-ceph-multistore) | 16:30 |
| auniyal | nova-next also fails, but yeah mostly nova-ceph-multistore and node | 16:30 |
| bauzas | auniyal: see https://bugs.launchpad.net/nova/+bug/1950310 | 16:31 |
| bauzas | are we done with this ? | 16:33 |
| auniyal | yes bauzas | 16:33 |
| bauzas | ok, last topic then | 16:33 |
| bauzas | #topic Open discussion | 16:33 |
| bauzas | the agenda is empty | 16:33 |
| bauzas | so, anything anyone ? | 16:34 |
| bauzas | looks not | 16:35 |
| bauzas | thanks all, | 16:35 |
| bauzas | #endmeeting | 16:35 |
| opendevmeet | Meeting ended Tue May 16 16:35:24 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:35 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2023/nova.2023-05-16-16.02.html | 16:35 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2023/nova.2023-05-16-16.02.txt | 16:35 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2023/nova.2023-05-16-16.02.log.html | 16:35 |
| gibi | thanks bauzas | 16:35 |
| bauzas | np | 16:35 |
| bauzas | as I said, I'll be off starting tonight until end of the week | 16:35 |
| elodilles | thanks bauzas, have a nice time off o/ | 16:36 |
| dansmith | bauzas: I hope you get some well-needed rest.. it's been like at least a week since you had one | 16:36 |
| bauzas | (taking the opportunity of the bank holiday to visit some friends on the West coast of France, 7 hours by drive) | 16:36 |
| dvo-plv | sean-k-monney, gibi, bauzas: Hello, i have updated bp. Could you please review it again | 16:36 |
| dvo-plv | https://review.opendev.org/c/openstack/nova/+/876075/6..7 | 16:36 |
| bauzas | dansmith: that's why I refused our gorgeous all-company PTO :) | 16:37 |
| dansmith | smh :) | 16:37 |
| bauzas | but yeah, May is our French thanskgiving-like month :) | 16:37 |
| dvo-plv | bauzas, have a nice holiday) | 16:37 |
| bauzas | thanks | 16:38 |
| opendevreview | Merged openstack/nova stable/2023.1: Fix get_segments_id with subnets without segment_id https://review.opendev.org/c/openstack/nova/+/882293 | 18:07 |
| opendevreview | Merged openstack/nova stable/xena: Use force=True for os-brick disconnect during delete https://review.opendev.org/c/openstack/nova/+/882867 | 19:34 |
| melwitt | elodilles: I added a link on the etherpad too but fyi gmann is working on fixing ceph on stable/wallaby here https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/871920 | 19:44 |
| elodilles | melwitt: oh, cool, thanks for the information \o/ | 19:58 |
| opendevreview | Carl Morris proposed openstack/nova master: Improving logging at '_numa_cells_support_network_metadata'. https://review.opendev.org/c/openstack/nova/+/860930 | 22:08 |
| opendevreview | Merged openstack/nova stable/xena: Enable use of service user token with admin context https://review.opendev.org/c/openstack/nova/+/882868 | 22:35 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!