| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-03-23 | |||
| 10:44:07 | sean-k-mooney | i dont rememebr that but i am interested in that | |
| 10:44:15 | sean-k-mooney | maybe dansmith | |
| 10:44:54 | sean-k-mooney | or artom i certenlly ask that question or one like it in our intenal meetings a few week ago | |
| 10:45:41 | sean-k-mooney | my understandign is the glance folks were going to also work on the nova patches | |
| 10:45:43 | bauzas | I'm just mentioning glance will discuss this with cinder | |
| 10:45:47 | sean-k-mooney | and we just needed to reveiew | |
| 10:45:55 | bauzas | and people will join | |
| 10:48:24 | sean-k-mooney | ok are you suggesting we also join that at teh same time | |
| 10:52:46 | bauzas | yeah, if interested | |
| 10:52:53 | bauzas | I wrote that in our etherpad | |
| 10:53:13 | bauzas | but we can ask for a specific glance/nova session if after that some questions remain | |
| 10:53:25 | bauzas | I just arranged this with pdeore | |
| 11:15:26 | artom | sean-k-mooney, bauzas, no, not me | |
| 11:15:47 | bauzas | k | |
| 12:28:22 | stephenfin | Could I get another reviewer on that series to remove sqlalchemy-migrate, please? https://review.opendev.org/q/topic:sqlalchemy-20+project:openstack/nova+is:open | |
| 12:54:12 | sean-k-mooney | so you have two diffent patch changes there | |
| 12:54:41 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/860829 andhttps://review.opendev.org/c/openstack/nova/+/872428 | |
| 12:54:55 | sean-k-mooney | i reviewd the later before and was ok to proceed with it | |
| 12:55:00 | sean-k-mooney | i have not looked at teh former | |
| 13:27:05 | opendevreview | Elod Illes proposed openstack/nova stable/victoria: DNM: gate test https://review.opendev.org/c/openstack/nova/+/878386 | |
| 13:34:16 | opendevreview | Merged openstack/nova stable/victoria: [stable-only][cve] Check VMDK create-type against an allowed list https://review.opendev.org/c/openstack/nova/+/871699 | |
| 13:34:22 | opendevreview | Merged openstack/nova stable/yoga: Reproducer for bug 1951656 https://review.opendev.org/c/openstack/nova/+/866153 | |
| 13:38:13 | dansmith | bauzas: sean-k-mooney wasn't me | |
| 13:38:42 | bauzas | anyway, we'll see this next Thursday then | |
| 13:38:59 | bauzas | elodilles: awesome \o/ https://review.opendev.org/c/openstack/nova/+/871699 | |
| 13:39:31 | bauzas | dansmith: gibi or sean-k-mooney: we need to merge this one now https://review.opendev.org/c/openstack/nova/+/875621 given grenade was modified | |
| 13:41:52 | dansmith | bauzas: yep, got it | |
| 13:43:28 | bauzas | ++ | |
| 13:43:52 | elodilles | bauzas: wow! finally... \o/ | |
| 14:12:54 | dansmith | gmann: around yet? | |
| 14:13:38 | dansmith | gmann: I'm not sure I understand the skip-level-always comment.. you made it gate/voting in the template and thus it needs to be in the gate pipeline in nova's zuul as well is that right? | |
| 14:13:47 | dansmith | just confused by the irrelevant-files part I guess | |
| 14:14:33 | sean-k-mooney | you can override it in nova .zuul.yaml | |
| 14:14:43 | sean-k-mooney | if we wanted too | |
| 14:14:51 | sean-k-mooney | the in repo options take precidnece | |
| 14:15:57 | sean-k-mooney | you would just need to add the job by name to check/gate and set voting false. but i think your really wondiering if ti shoudl be voting? | |
| 14:16:12 | sean-k-mooney | or was this a zuul mechanics question | |
| 14:16:57 | opendevreview | Dan Smith proposed openstack/nova master: Add grenade-skip-level-always to nova https://review.opendev.org/c/openstack/nova/+/875773 | |
| 14:17:12 | dansmith | sean-k-mooney: I'm talking about a specific comment *on* nova's zuul.yaml from gmann | |
| 14:18:57 | sean-k-mooney | oh hum well you have it in both check and gate and we have irrelevnet-files in both | |
| 14:19:38 | sean-k-mooney | i think they were suggesting editing the project template | |
| 14:19:43 | sean-k-mooney | maybe | |
| 14:20:00 | dansmith | maybe we just wait and see what he meant :) | |
| 14:21:24 | sean-k-mooney | sure but im not sure i really like the idea of having to have teh irrelevnt file we use in the tempest repo | |
| 14:21:48 | sean-k-mooney | since that depnes on the nova repo strcuture. granted that changes very in frequesntly but still | |
| 14:21:55 | sean-k-mooney | it does not feel liek this shoudl be there | |
| 14:22:42 | sean-k-mooney | granted i have also said in the past that i woudl preer if the integrated-gate-compute template was in the nova repo but i knwo why the qa team want to keep those all in one repo | |
| 14:23:14 | dansmith | are you talking about the policies-irrelevant list? | |
| 14:24:36 | dansmith | I dunno why that's named that way, but AFAIK it's defined in this file, not in tempest | |
| 14:29:35 | opendevreview | Dmitry Tantsur proposed openstack/nova master: ironic: clean up references to memory_mb/cpus/local_gb https://review.opendev.org/c/openstack/nova/+/878418 | |
| 14:57:36 | opendevreview | Merged openstack/nova stable/yoga: Handle mdev devices in libvirt 7.7+ https://review.opendev.org/c/openstack/nova/+/866154 | |
| 14:59:53 | bauzas | whoami-rajat: it's quite late for your time, but it looks like we need a cinder-nova cross-project session, IIRC my emails :) | |
| 15:00:03 | bauzas | for the vPTG | |
| 15:00:54 | whoami-rajat | bauzas, yeah i was just reading through it, I haven't finalized timing for cinder topics yet, do you have any time in mind? | |
| 15:01:02 | whoami-rajat | we're going to have a cross project with glance on thursday | |
| 15:02:39 | bauzas | yup, I discussed that this morning with pdeore | |
| 15:04:41 | bauzas | whoami-rajat: Sofia was requesting the last Thursday slot or the first Wed slot | |
| 15:05:10 | bauzas | whoami-rajat: I can somehow set it for Wed 1300UTC, would that work for you ? | |
| 15:05:56 | whoami-rajat | bauzas, hmm, the problem with first slots is we don't have full gathering, would thursday last slot work for you? 1600-1700 UTC | |
| 15:06:07 | bauzas | whoami-rajat: sure | |
| 15:06:11 | bauzas | let's take it | |
| 15:06:17 | whoami-rajat | great! | |
| 15:06:19 | whoami-rajat | thanks | |
| 15:06:54 | bauzas | whoami-rajat: would it work if that would be in our room ? | |
| 15:07:25 | whoami-rajat | bauzas, sure, we can move there, is it on Zoom or any other platform? | |
| 15:07:55 | bauzas | whoami-rajat: we use the diablo room (zoom) | |
| 15:08:06 | whoami-rajat | cool, we will be there | |
| 15:08:26 | bauzas | I just added it in our etherpad https://etherpad.opendev.org/p/nova-bobcat-ptg#L91 | |
| 15:08:43 | bauzas | whoami-rajat: and if you see other topics to discuss, please add them there | |
| 15:09:30 | whoami-rajat | bauzas, sure, sounds good! | |
| 15:10:15 | bauzas | ++ | |
| 15:10:18 | bauzas | g'night | |
| 15:44:34 | dtantsur | Hey folks! Seeing this in the ironic grenade job: https://zuul.opendev.org/t/openstack/build/d90374e9b6554704a2f84b7fe8a9d411/log/controller/logs/screen-n-api.txt#4182 | |
| 15:44:39 | dtantsur | rings any bells? | |
| 15:45:26 | dtantsur | It's quite possible that the ironic virt driver does not indeed support 'openstack console log show', but why is it called? | |
| 15:47:21 | dtantsur | hmm, maybe a red herring. judging by https://opendev.org/openstack/grenade/commit/adcb563b185416451da419186a8d7773ffb6b913 it happens if ping fails. | |
| 15:47:33 | dtantsur | (would be cool to check the virt driver before doing it) | |
| 15:48:23 | clarkb | dtantsur: I think one of the responses to a failed tempest test is to dump the instance console log. This is often useful if there are networking issues because with a VM the console is accessed via libvirt and not the network and the logs often show you if dhcp failed etc | |
| 15:48:49 | dtantsur | Right. Probably needs to exclude VIRT_DRIVER=ironic | |
| 17:08:03 | gmann | dansmith: hi | |
| 17:08:42 | gmann | dansmith: I mean to add irrelevnet-files in gate pipeline in same way you did in check pipeline to avoid running grenade on doc only changes etc | |
| 17:50:17 | sean-k-mooney | gmann: that in the curernt patch but i dont knwo if you left your comment on a previous version | |
| 17:50:56 | sean-k-mooney | gmann: https://review.opendev.org/c/openstack/nova/+/875773/6/.zuul.yaml#798 | |
| 17:52:15 | sean-k-mooney | ah i see so in v5 it was being added to the gate pipeline via the template | |
| 17:52:21 | sean-k-mooney | not explictly | |
| 17:52:40 | sean-k-mooney | so ya v6 adressed your comment | |
| 17:53:05 | sean-k-mooney | however it failed in v6 for somereason when it ran | |
| 17:53:46 | sean-k-mooney | test_security_group_rules_create | |
| 17:53:57 | sean-k-mooney | weired that looks unrealted | |
| 17:54:01 | sean-k-mooney | i wonder wy that failed | |
| 17:55:09 | gmann | sean-k-mooney: yes, already +2 on that | |
| 17:55:44 | sean-k-mooney | im jsut quickly checking the logs before rechecking and +2ing | |
| 17:56:30 | gmann | ohk | |
| 17:56:44 | gmann | I checked it from test failing log but did not go into deep | |
| 17:56:58 | sean-k-mooney | nova.api.openstack.wsgi.Fault: Instance 08070e41-68b0-4dd3-9eb6-1926c3082060 could not be found. | |
| 17:57:13 | sean-k-mooney | there are a bunch of faults like that in the nova api | |
| 17:57:53 | sean-k-mooney | although im not sure its the same test | |
| 17:59:14 | gmann | api log might be confusing on NotFound due to negative tests | |
| 17:59:26 | sean-k-mooney | ya i was assumign that too | |
| 17:59:38 | sean-k-mooney | but i was just checkign to see if there are any erference to that est | |