| opendevreview | Jorge San Emeterio proposed openstack/nova master: Have host look for CPU controller of cgroupsv2 location. https://review.opendev.org/c/openstack/nova/+/873127 | 08:47 |
|---|---|---|
| opendevreview | Sylvain Bauza proposed openstack/nova master: Add service version for Antelope https://review.opendev.org/c/openstack/nova/+/874932 | 09:12 |
| opendevreview | Sylvain Bauza proposed openstack/nova master: DNM (yet) Update min support for Bobcat https://review.opendev.org/c/openstack/nova/+/875621 | 09:12 |
| bauzas | gibi: I'd appreciate your review on both https://review.opendev.org/c/openstack/nova/+/874932/5 and https://review.opendev.org/c/openstack/nova/+/875380 | 09:14 |
| kgube | Hi! So, I ran out of time with the implementation of my specs that got accepted for antelope and I want to resubmit them for bobcat. | 09:33 |
| kgube | Should I create a new change for this, or is it possible to reuse the old change, even though it was already merged, to keep previous discussions? | 09:33 |
| gibi | bauzas: on it | 09:35 |
| Uggla | Hi gibi, I need help regarding this comment https://review.opendev.org/c/openstack/nova/+/851029/19/doc/notification_samples/common_payloads/InstancePayload.json#42. As InstancePayload is the "root" of all notifications, I can not change it adding a share as it will impact all notifications tests. Is there a way to do that without copying this file and calling it only on specific cases ? | 09:37 |
| gibi | bauzas: Im OK with the prelude, I left comments in the service version patch | 09:51 |
| gibi | Uggla: looking | 09:52 |
| gibi | Uggla: duplicate the file and use it for the share related notification samples | 09:52 |
| gibi | Uggla: alternativel you could manipulate the sample in the test code but that would be missleading | 09:53 |
| gibi | you are right we probably don't want to add shares for each notification test | 09:54 |
| gibi | still we want to have samples with shares as that is basically our API definition | 09:54 |
| Uggla | @gibi, ok if I duplicate, then it will be documented automatically or should I do something special ? | 09:56 |
| gibi | Uggla: the doc generation happens via the @base.notification_sample decorators | 10:05 |
| gibi | doc generation code is here https://github.com/openstack/nova/blob/master/doc/ext/versioned_notifications.py | 10:06 |
| gibi | so if you create a sample that has shares | 10:07 |
| gibi | then you need to add that to the related class via the @base.notification_sample | 10:07 |
| gibi | decorator | 10:07 |
| gibi | kgube: do you mean resubmitting the spec? | 10:08 |
| gibi | kgube: if so then you need a new change, but you can copy the already approved spec there. And please note that this is a re-propose in the commit message so that people can look at the old discussion | 10:09 |
| bauzas | sorry folks, was in meeting | 10:14 |
| bauzas | gibi: ack, will be looking | 10:15 |
| bauzas | kgube: the bobcat specs directory is now present, you can indeed resubmit | 10:15 |
| bauzas | like gibi said, just copy the rst file and mention it was previously-approved | 10:15 |
| bauzas | so you would benefit from a straight fast-approval if no modification is made in the spec file besides the mention of the previous approval | 10:16 |
| bauzas | see other specs in Antelope, that pattern is often used :) | 10:16 |
| Uggla | gibi ok, I ll check | 10:18 |
| kgube | bauzas, gibi: alright, thanks! | 10:20 |
| opendevreview | Merged openstack/nova stable/train: Refactor volume connection cleanup out of _post_live_migration https://review.opendev.org/c/openstack/nova/+/864670 | 10:47 |
| opendevreview | Merged openstack/nova stable/train: Adds a repoducer for post live migration fail https://review.opendev.org/c/openstack/nova/+/863806 | 10:47 |
| opendevreview | Sylvain Bauza proposed openstack/nova master: Add service version for Antelope https://review.opendev.org/c/openstack/nova/+/874932 | 11:18 |
| opendevreview | Sylvain Bauza proposed openstack/nova master: DNM (yet) Update min support for Bobcat https://review.opendev.org/c/openstack/nova/+/875621 | 11:18 |
| opendevreview | Danylo Vodopianov proposed openstack/os-traits master: Add 'COMPUTE_NET_VIRTIO_PACKED' https://review.opendev.org/c/openstack/os-traits/+/876069 | 11:34 |
| opendevreview | Danylo Vodopianov proposed openstack/nova master: Packed virtqueue support was added. https://review.opendev.org/c/openstack/nova/+/876075 | 11:47 |
| opendevreview | Jorge San Emeterio proposed openstack/nova master: WIP: Have schema for 'lock' action be applied to all microversions. https://review.opendev.org/c/openstack/nova/+/875653 | 11:55 |
| opendevreview | Jorge San Emeterio proposed openstack/nova master: WIP: Have schema for 'lock' action be applied to all microversions. https://review.opendev.org/c/openstack/nova/+/875653 | 13:37 |
| bauzas | man, the gate is sloooooow https://zuul.openstack.org/status#873584 | 13:47 |
| elodilles | bauzas: it's rc1 day :D | 13:47 |
| bauzas | elodilles: we may need to defer RC1 branching tomorrow given that | 13:48 |
| elodilles | :S | 13:48 |
| bauzas | I'm fighting against treadmills, y'know | 13:48 |
| elodilles | actually, talked a bit about this with gibi at today's lunch :) | 13:49 |
| elodilles | until -1 is on the patch, release team will wait | 13:49 |
| opendevreview | Jorge San Emeterio proposed openstack/nova master: WIP: Have schema for 'lock' action be applied to all microversions. https://review.opendev.org/c/openstack/nova/+/875653 | 13:49 |
| bauzas | elodilles: gibi: damn I should take my car and visit you | 13:49 |
| elodilles | bauzas: sure, anytime, we are here :) | 13:50 |
| bauzas | we're only separated by the Alps | 13:50 |
| elodilles | bauzas: just switch on FSD and the car will bring you here :D | 13:50 |
| bauzas | anyway, time for review beggins | 13:50 |
| bauzas | gibi: https://review.opendev.org/c/openstack/nova/+/874932/6 could you please ? | 13:50 |
| bauzas | elodilles: nah, no way I'd expense an incredibly insane amount of money just for having my car changing lanes automatically | 13:51 |
| elodilles | :) | 13:51 |
| bauzas | (and VW makes it waaaay better than Tesla, if you want MHO) | 13:51 |
| * bauzas is absolutely underwhelmed by those Autopilot crazypants, he had a Skoda that performed better | 13:52 | |
| elodilles | :-o | 13:52 |
| bauzas | true experience, don't be freaked by the buzz | 13:53 |
| elodilles | good to know, i couldn't imagine that it is better :-o | 13:53 |
| bauzas | surely my car can better stick within lanes, but the overall experience is way less autonomous than my previous TravelAssist | 13:53 |
| elodilles | but at least it can be improved in the future :) | 13:54 |
| bauzas | elodilles: you're getting buzzed by the FSD Beta, which is only delivered to US customers on a private-beta basis | 13:54 |
| bauzas | but even the FSD Beta somehow is less efficient than the recent upgrades of VAG TravelAssists, trust me | 13:55 |
| elodilles | (yeah, i know it is only in US, i was just joking o:)) | 13:55 |
| bauzas | anyway, this isn't #openstack-auto | 13:56 |
| elodilles | :D | 13:57 |
| bauzas | cores, I urgently need eyes on https://review.opendev.org/c/openstack/nova/+/875380 and https://review.opendev.org/c/openstack/nova/+/874932/ | 13:57 |
| elodilles | bauzas: I'm waiting for Kitt from Knight Rider (yes, I am that old :D) | 13:58 |
| bauzas | elodilles: I'm old enough to share the reference | 13:58 |
| elodilles | :] | 13:58 |
| gibi | bauzas: yeah we just happened to have a good lunch with elodilles lajoskatona rubasov here :) | 14:10 |
| bauzas | I'm jealous | 14:11 |
| * bauzas is not specifically a travel person but he likes to see his colleagues and coworkers | 14:11 | |
| opendevreview | Jorge San Emeterio proposed openstack/nova master: WIP: Have schema for 'lock' action be applied to all microversions. https://review.opendev.org/c/openstack/nova/+/875653 | 14:17 |
| gibi | bauzas: I'm +2 on both pending patches | 14:19 |
| bauzas | ta | 14:19 |
| bauzas | I'll proxy dansmith's vote if you agree | 14:19 |
| opendevreview | Justas Poderys proposed openstack/nova-specs master: VirtIO PackedRing Configuration support https://review.opendev.org/c/openstack/nova-specs/+/868377 | 14:20 |
| gibi | bauzas: sure | 14:21 |
| gibi | bauzas: I added a topic to the ptg to discuss cleaning up our upgrade documentation in the light of SLURP | 14:22 |
| bauzas | cool | 14:22 |
| kashyap | SLURP? | 14:22 |
| gibi | kashyap: yepp, what a nice naming https://governance.openstack.org/tc/resolutions/20220210-release-cadence-adjustment.html | 14:23 |
| gibi | (Skip Level Upgrade Release Process | 14:23 |
| gibi | ) | 14:23 |
| kashyap | Aah, /me clicks | 14:23 |
| kashyap | Hehe, yeah, catchy acronyms are a good psychological trick | 14:24 |
| bauzas | formerly unofficially named tick-tock releases | 14:24 |
| bauzas | well, SLURP release is a former 'tick' release | 14:24 |
| opendevreview | Justas Poderys proposed openstack/nova-specs master: VirtIO PackedRing Configuration support https://review.opendev.org/c/openstack/nova-specs/+/868377 | 14:24 |
| kashyap | gibi: The name is dangerously close to ... "SLIRP", QEMU's user-mode networking: https://wiki.qemu.org/Documentation/Networking#User_Networking_(SLIRP) | 14:24 |
| bauzas | nooooo https://zuul.openstack.org/status#873584 says it will fail | 14:32 |
| bauzas | pressure on the gate I guess, as today is RC1 | 14:32 |
| bauzas | we got problems before, and now we fail again | 14:32 |
| bauzas | while last weeks were quite ok | 14:32 |
| gibi | bauzas: then we can conclude that the load on the CI causes that some of our edge cases are more visible. | 14:33 |
| bauzas | yup | 14:34 |
| * bauzas running again logsearch | 14:34 | |
| bauzas | https://b37c807ec9e6a66916f3-1265f2e9e3a0fcf75a00d1718ae30bb8.ssl.cf1.rackcdn.com/873584/1/check/tempest-integrated-compute-enforce-scope-new-defaults/f4dafd1/testr_results.html seems coming from an unresponsive API service | 14:34 |
| gibi | I think I opened a bug for that class of bugs | 14:35 |
| gibi | https://bugs.launchpad.net/tempest/+bug/1999893 | 14:35 |
| bauzas | and https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_19b/873584/1/check/nova-live-migration/19ba9dd/testr_results.html shows some weird neutron behaviour | 14:35 |
| gibi | here it is ^^ | 14:35 |
| opendevreview | Jorge San Emeterio proposed openstack/nova master: Adding a default schema for requests to the 'lock', 'migrate' and 'unshelve' actions. https://review.opendev.org/c/openstack/nova/+/875653 | 14:35 |
| gibi | hm, "Bad router request: Router already has a port on subnet 352d6d1a-3324-4786-8f7c-4c2fb20b73d4" is new to me | 14:36 |
| bauzas | I'll add the new exerpts to the bug | 14:36 |
| gibi | thanks | 14:36 |
| bauzas | this time it looks to me that it was neutron API which was unresponsive | 14:37 |
| bauzas | oh wait no | 14:37 |
| opendevreview | Justas Poderys proposed openstack/nova-specs master: VirtIO PackedRing Configuration support https://review.opendev.org/c/openstack/nova-specs/+/868377 | 14:37 |
| bauzas | neutron, glance, nova | 14:37 |
| bauzas | all close in time | 14:38 |
| bauzas | that can't be a service API problem | 14:38 |
| opendevreview | Danylo Vodopianov proposed openstack/os-traits master: Add 'COMPUTE_NET_VIRTIO_PACKED' https://review.opendev.org/c/openstack/os-traits/+/876069 | 14:40 |
| opendevreview | Danylo Vodopianov proposed openstack/nova master: Packed virtqueue support was added. https://review.opendev.org/c/openstack/nova/+/876075 | 14:41 |
| opendevreview | Justas Poderys proposed openstack/nova-specs master: VirtIO PackedRing Configuration support https://review.opendev.org/c/openstack/nova-specs/+/868377 | 14:52 |
| bauzas | dansmith: can you give https://review.opendev.org/c/openstack/nova/+/875380 a swing ? | 15:08 |
| dansmith | ack | 15:08 |
| * bauzas is atm grabbing his Columbo coat and a cigar to identify the new CI failures he saw | 15:09 | |
| dansmith | bauzas: do you want me to +W or let it soak for others to review? | 15:15 |
| bauzas | well, we can wait for gmann and sean-k-mooney to review that prelude | 15:16 |
| bauzas | the CI is fucked today | 15:16 |
| bauzas | anyway | 15:16 |
| dansmith | is it? | 15:16 |
| bauzas | and we don't have yet merged the logging revert and the service min version patch | 15:16 |
| bauzas | dansmith: yah, between your patch and https://review.opendev.org/c/openstack/nova/+/873584 today is a nice day | 15:17 |
| dansmith | my patch? did I break something? | 15:17 |
| bauzas | no no | 15:17 |
| bauzas | saying that I did some rechecks on your own patch | 15:18 |
| dansmith | oh the grenade job one? | 15:18 |
| bauzas | speaking of https://review.opendev.org/c/openstack/nova/+/875773 | 15:18 |
| bauzas | oh heh, it failed again | 15:18 |
| dansmith | so, not that it helps, | 15:19 |
| bauzas | the fun is that every single failure is different | 15:19 |
| dansmith | but I realized a bit ago that we never merged the test for the rebuild-bfv feature last cycle: https://review.opendev.org/c/openstack/tempest/+/831018 | 15:20 |
| dansmith | I don't know why that needs its own job, so I hope we can roll that in, | 15:20 |
| dansmith | but we have no coverage for that currently, unfortunately :/ | 15:20 |
| bauzas | today i haven't seen (yet) any volume-related issue | 15:20 |
| bauzas | so far this was timeouts and a few unresponsive calls | 15:21 |
| bauzas | none of them being cinder-related | 15:21 |
| dansmith | well, that's progress IMHO :) | 15:21 |
| dansmith | but yeah, timeouts | 15:21 |
| bauzas | I have a rule of thumb | 15:21 |
| bauzas | I had problems during the week of milestone-3 | 15:22 |
| bauzas | milestone-3 went and the weekend after that, all patches in queue merged without problems | 15:22 |
| bauzas | now we're on RC1 period | 15:22 |
| bauzas | the gate is mostly busted and the check pipeline is horribly on delay | 15:22 |
| bauzas | I just think we should be able to branch RC1 this weekend, yay | 15:23 |
| bauzas | anyway, me needs to disappear for a dozen of minutes for parenting responsibilities | 15:23 |
| bauzas | bbias | 15:24 |
| dosaboy | hi folks, if anyone has time to land these yoga backports it would be greatly appreciated - https://review.opendev.org/q/topic:bug%252F1890244 | 15:32 |
| dosaboy | they've got a +2 already | 15:33 |
| sean-k-mooney1 | i fixed that... hum does that mean i need to drink more or less coffee if i dont remember it | 15:34 |
| dosaboy | more coffee always, that's my M.O. at least | 15:35 |
| bauzas | dosaboy: I'll give them a swing | 15:57 |
| bauzas | dosaboy: aaaand done | 16:00 |
| dosaboy | bauzas: many thanks! | 16:35 |
| bauzas | np | 16:35 |
| bauzas | I'm saying it here too, but we're busted to get the RC1 patch being merged today | 16:40 |
| *** bhagyashris|ruck is now known as bhagyashris | 16:51 | |
| bauzas | folks, I end my day on a sad note : since we can't deliver RC1 today, we'll try again tomorrow | 17:38 |
| bauzas | that's a bummer, but looks like nobody really cares about it | 17:39 |
| * bauzas bails out with a bit of bitter in his mouth | 17:40 | |
| dansmith | bauzas: is this because you can't merge the log revert? | 17:40 |
| bauzas | that + the prelude + the service object bump | 17:40 |
| bauzas | 3 patches on fly | 17:40 |
| dansmith | oh right, those, gotcha | 17:40 |
| bauzas | we should end up releasing RC1 on the weekend | 17:40 |
| dansmith | we can certainly merge the prelude right? | 17:40 |
| bauzas | dansmith: the prelude is still in flight | 17:41 |
| bauzas | oh my bad, not | 17:41 |
| dansmith | bauzas: in flight meaning not reviewed by enough people right? not because you can't land due to tests | 17:41 |
| bauzas | dansmith: send it to the gate, folks that care should have voted earlier | 17:41 |
| dansmith | done | 17:41 |
| bauzas | yeah, my bad I was paying attention to the two others | 17:42 |
| bauzas | the revert logging may be merged later and backported | 17:42 |
| dansmith | yeah the revert is not a huge deal | 17:42 |
| bauzas | this is just a silly logging | 17:42 |
| dansmith | would be nice, but not critical | 17:42 |
| dansmith | I'll watch the service version one | 17:42 |
| bauzas | but the prelude has to land, ditto the service check | 17:42 |
| dansmith | yeah | 17:42 |
| bauzas | thanks for caring this | 17:43 |
| bauzas | I'll look at those later today | 17:43 |
| dansmith | maybe if I can grind these two you can post the release tomorrow morning | 17:43 |
| bauzas | maybe | 17:44 |
| bauzas | gerrit will be restarted later today | 17:44 |
| bauzas | that was an ask from the infra team | 17:44 |
| bauzas | zuul will only be recycled this weekend | 17:44 |
| dansmith | oye | 17:44 |
| clarkb | note zuul is zero downtime updates | 17:46 |
| bauzas | cool | 17:46 |
| clarkb | maybe that wasn't clear in my comments in the release channel. Its fully rolling and starts at 00:00 UTC saturday and usually takes about a day | 17:46 |
| dansmith | ah I assumed that meant some queue resets | 17:53 |
| opendevreview | melanie witt proposed openstack/nova master: testing: Reset affinity support global variables https://review.opendev.org/c/openstack/nova/+/875991 | 18:18 |
| gmann | bauzas: +1 on prelude | 18:19 |
| clarkb | dansmith: we've been doing fully automatic weekly rolling zuul upgrades since like july? | 18:35 |
| dansmith | clarkb: cool, didn't know (which I guess means it's working) | 18:35 |
| clarkb | and before that we were doing them manually. Its been about a year since we had to intentionally take a zuul downtime for normal updates (they do still occasionally happen to apply bug fixes or security updates more quickly) | 18:35 |
| opendevreview | Merged openstack/nova stable/yoga: add repoducer test for bug 1890244 https://review.opendev.org/c/openstack/nova/+/872663 | 20:41 |
| sean-k-mooney1 | bauzas: https://bugs.launchpad.net/nova/+bug/2008883 | 20:42 |
| sean-k-mooney1 | bauzas: did you ever test 2 mdev types with MIG on the same card concurrently | 20:42 |
| sean-k-mooney1 | bauzas: i think this would need an a100 to test and we normally only used t4's so im not sure if we have ever tested that | 20:43 |
| *** sean-k-mooney1 is now known as sean-k-mooney | 21:06 | |
| opendevreview | Merged openstack/nova master: Add the 2023.1 Antelope prelude section https://review.opendev.org/c/openstack/nova/+/875380 | 22:14 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!