| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-11-07 | |||
| 16:08:33 | bauzas | #link https://zuul.openstack.org/builds?project=openstack%2Fnova&project=openstack%2Fplacement&pipeline=periodic-weekly Nova&Placement periodic jobs status | |
| 16:08:43 | Uggla_ | https://bugs.launchpad.net/nova/+bug/2039381 | |
| 16:08:47 | bauzas | #info Please look at the gate failures and file a bug report with the gate-failure tag. | |
| 16:09:13 | bauzas | Uggla_: oh sorry we moved to the gate section, can we discuss your bug in the open discussion then ? | |
| 16:09:46 | dansmith | melwitt has a fix up for the vnc ting.. I've asked her a question, but assume we'll get that on the way soon once she's around | |
| 16:09:58 | bauzas | fwiw, all periodics are in green | |
| 16:10:04 | Uggla_ | yep sure | |
| 16:10:08 | bauzas | dansmith: oh, nice to hear, any change I could dent ? | |
| 16:10:41 | dansmith | sec | |
| 16:11:17 | dansmith | https://review.opendev.org/c/openstack/grenade/+/900257 | |
| 16:11:50 | bauzas | dansmith: cool, CCing it | |
| 16:12:18 | bauzas | oh that's why it's failing intermittently | |
| 16:12:25 | bauzas | depending on the host you land | |
| 16:12:53 | bauzas | I nacked the idea when gibi suggested it because I was too lazy to look at the job and I just trusted the fact we got green runs | |
| 16:12:56 | bauzas | what a shame | |
| 16:13:04 | dansmith | bauzas: that's what I'm wondering | |
| 16:13:19 | dansmith | seems like we should be failing a lot more though, | |
| 16:13:34 | dansmith | and why this has become a thing all of a sudden also seems weird if it's been broken like this for a while | |
| 16:13:43 | dansmith | but otherwise yeah, hopefully an easy fix | |
| 16:13:56 | bauzas | because yesterday we merged my patch that added the servers actions checks in grenade | |
| 16:14:18 | bauzas | this test was flakey but never run, and I just opened the can of worms | |
| 16:14:33 | dansmith | but I think it has failed on other patches, not just yours | |
| 16:15:03 | bauzas | well, then the vnc check itself could be present on other tempest tests | |
| 16:15:20 | bauzas | or we could run the server actions list in other jobs, rather | |
| 16:15:22 | bauzas | anyway | |
| 16:16:01 | bauzas | moving on | |
| 16:16:49 | bauzas | #topic Release Planning | |
| 16:16:54 | bauzas | #link https://releases.openstack.org/caracal/schedule.html | |
| 16:16:58 | bauzas | #info Nova deadlines will be proposed in the schedule above | |
| 16:17:12 | bauzas | I just need to file a release patch with the correct dates :) | |
| 16:17:21 | bauzas | #info Caracal-1 milestone in 1 week | |
| 16:17:30 | bauzas | #info Spec review day today | |
| 16:17:56 | bauzas | I think I made a correct round of reviews but there are still some specs I haven't looked yet | |
| 16:18:09 | bauzas | I'll continue my duty until EOB | |
| 16:18:47 | dansmith | any list of specs just waiting for a +W? | |
| 16:19:02 | bauzas | sure | |
| 16:19:19 | opendevreview | Steven Relf proposed openstack/nova master: Adding basic auth to dynamic vendordata api calls https://review.opendev.org/c/openstack/nova/+/900252 | |
| 16:19:21 | gibi | there is a set of reproposals that are easy wins | |
| 16:19:30 | bauzas | dansmith: you mean a second +2 or just a plain +W ? :) | |
| 16:19:46 | dansmith | bauzas: second review I mean? | |
| 16:19:48 | bauzas | https://review.opendev.org/q/(project:openstack/nova-specs)+status:open+NOT+owner:self+NOT+label:Workflow%253C%253D-1+label:Verified%253E%253D1%252Czuul+NOT+reviewedby:self+is:mergeable+NOT+label:Code-Review%253C%253D-1%252Cnova-core+label:Code-Review%253E%253D2 | |
| 16:19:59 | bauzas | not sure the link will render correctly | |
| 16:20:11 | bauzas | but tl;dr: dansmith you have the sole spec that needs a second +2 :) | |
| 16:20:37 | bauzas | and yeah, there are reproposals on the way | |
| 16:20:44 | bauzas | btw. I wonder why they don't show up | |
| 16:21:09 | dansmith | bauzas: ah bummer.. maybe gibi can circle back on that .. that's the only one I can't finish :) | |
| 16:21:14 | bauzas | https://review.opendev.org/q/project:openstack/nova-specs+status:open+label:Code-Review%253E%253D2 | |
| 16:21:31 | dansmith | but yeah I'll look at the re-proposals | |
| 16:21:38 | bauzas | sorry, my dash link seems to be wrong | |
| 16:21:59 | bauzas | fwiw, had no time yet to correctly write a mdev live-migration spec | |
| 16:22:01 | auniyal6 | on CI bugs, nova-emulation job is failing too (I wanted to check if its failing for more patches, but builds page is not opening for me right now ) | |
| 16:22:04 | bauzas | I'll bug folks directly :) | |
| 16:22:04 | sean-k-mooney | i can also try and loop back | |
| 16:22:09 | sean-k-mooney | ill be doing more review later today | |
| 16:22:21 | sean-k-mooney | dansmith: on your spec | |
| 16:22:41 | johnthetubaguy | (I keep trying to do the ironic re-proposal, but I keep getting distracted, sorry) | |
| 16:22:43 | opendevreview | Dmitriy Rabotyagov proposed openstack/nova stable/2023.1: Fix rebuild compute RPC API exception for rolling-upgrades https://review.opendev.org/c/openstack/nova/+/900336 | |
| 16:22:44 | opendevreview | Dmitriy Rabotyagov proposed openstack/nova stable/2023.1: Adding server actions tests to grenade-multinode https://review.opendev.org/c/openstack/nova/+/900337 | |
| 16:22:58 | bauzas | gibi: you left a +1 because of Uggla_'s concern but AFAICR, it was resolved | |
| 16:23:11 | bauzas | so I just went +2 | |
| 16:23:18 | gibi | bauzas: OK, thanks | |
| 16:23:31 | opendevreview | Dmitriy Rabotyagov proposed openstack/nova stable/2023.2: add a regression test for all compute RPCAPI 6.x pinnings for rebuild https://review.opendev.org/c/openstack/nova/+/900309 | |
| 16:23:40 | bauzas | okay, moving on | |
| 16:24:21 | bauzas | dvo-plv wanted us to do a group discussion on https://review.opendev.org/c/openstack/nova-specs/+/895924/ since I asked to *not* use a trait but let's not discuss this now | |
| 16:24:32 | opendevreview | Dmitriy Rabotyagov proposed openstack/nova stable/2023.2: Fix rebuild compute RPC API exception for rolling-upgrades https://review.opendev.org/c/openstack/nova/+/900338 | |
| 16:24:33 | opendevreview | Dmitriy Rabotyagov proposed openstack/nova stable/2023.2: Adding server actions tests to grenade-multinode https://review.opendev.org/c/openstack/nova/+/900339 | |
| 16:24:41 | bauzas | if people want, we could discuss this during open discussion | |
| 16:25:17 | bauzas | (or just read my gerrit comments, you'll get my point, which is please avoid adding traits for just a libvirt version check that's already supported for all computes except Antelope and olders) | |
| 16:25:53 | bauzas | anyway, moving on | |
| 16:26:16 | dansmith | yeah | |
| 16:26:28 | dansmith | needs to be capability-based, IMHO | |
| 16:26:38 | dansmith | "supports X" not "is version X" | |
| 16:27:09 | sean-k-mooney | yep | |
| 16:27:17 | sean-k-mooney | so supprot virtio-packed format | |
| 16:27:20 | sean-k-mooney | is a capablity | |
| 16:27:22 | bauzas | I said we could resolve that with a service version check | |
| 16:27:22 | sean-k-mooney | not a version check | |
| 16:27:27 | sean-k-mooney | so it shoudl be a trait | |
| 16:27:35 | sean-k-mooney | and we shoudl not need a comptue service check | |
| 16:27:43 | bauzas | sean-k-mooney: that's not exactly what's written in both code and spec | |
| 16:27:48 | sean-k-mooney | as this is not a feature that is impemnted at the comptue mager level | |
| 16:27:56 | bauzas | the requirement is "is libvirt >6.7" | |
| 16:28:12 | sean-k-mooney | right that is the requireemtn for the feature too work | |
| 16:28:33 | bauzas | since bobcat supports 7.0, we're good | |
| 16:28:35 | sean-k-mooney | but it is modeled by reportign a triat for the host where the capablitys is supproted | |
| 16:28:38 | sean-k-mooney | yep | |
| 16:28:51 | sean-k-mooney | but we now need to supprot n-2 and diffent oss | |
| 16:29:00 | sean-k-mooney | *operating systems | |
| 16:29:02 | bauzas | so it only leaves the rolling upgrade case with a caracal env + an antelope node | |
| 16:29:21 | bauzas | which will no longer be a problem with D | |
| 16:29:29 | sean-k-mooney | well this featur like many other can be docusmeed as only supproted after a full upgrade is complete | |
| 16:29:49 | bauzas | so again, I'm very against adding a trait for modeling a libvirt version that's gonna be minimum for all our computes next cycle anyway | |
| 16:29:59 | sean-k-mooney | bauzas: that not what its modeling | |
| 16:30:27 | sean-k-mooney | and its required in my view for as long as we have any virt driver (other then ironic and libvirt) that supprot vms | |
| 16:30:36 | bauzas | https://review.opendev.org/c/openstack/nova/+/876075/25/nova/virt/libvirt/driver.py | |
| 16:30:57 | bauzas | that exactly models the libvirt version | |
| 16:31:14 | dansmith | not really | |
| 16:31:16 | sean-k-mooney | its the only way to detect it because libvirt does not have a way to detect it without a version check | |
| 16:31:19 | sean-k-mooney | but | |
| 16:31:26 | sean-k-mooney | since we now have the min versio nrequiremnet | |