| *** mhen_ is now known as mhen | 01:48 | |
| gibi | bauzas: Hi! if you have a bit of time this backport will look familiar to you https://review.opendev.org/q/topic:%22bug/2115905%22 :)h | 06:57 |
|---|---|---|
| bauzas | lolilol | 07:00 |
| jlejeune | gibi: hello, same request for mine https://review.opendev.org/c/openstack/nova/+/961601 :) | 07:01 |
| gibi | jlejeune: on it | 07:07 |
| jlejeune | thanks :) | 07:08 |
| gibi | jlejeune: maybe its me but I only see the regression test being backported there. do you have the actual fix backported on top of that as well? | 07:11 |
| jlejeune | yes but I haven't yet create the changes, doing | 07:12 |
| gibi | OK. I'm +2 on the regression test backports. Feel free to ping me when the fix backports are up. | 07:14 |
| jlejeune | perfect | 07:14 |
| opendevreview | Julien LE JEUNE proposed openstack/nova stable/2025.1: Adds regression test for bug LP#2044235 https://review.opendev.org/c/openstack/nova/+/961601 | 07:36 |
| opendevreview | Julien LE JEUNE proposed openstack/nova stable/2025.1: nova-conductor puts instance in error state https://review.opendev.org/c/openstack/nova/+/961774 | 07:36 |
| opendevreview | Julien LE JEUNE proposed openstack/nova stable/2024.2: nova-conductor puts instance in error state https://review.opendev.org/c/openstack/nova/+/961775 | 07:40 |
| opendevreview | Julien LE JEUNE proposed openstack/nova stable/2024.1: nova-conductor puts instance in error state https://review.opendev.org/c/openstack/nova/+/961778 | 07:41 |
| jlejeune | gibi: here there are ^ | 07:42 |
| opendevreview | Merged openstack/nova stable/2025.1: Reproduce bug/2115905 https://review.opendev.org/c/openstack/nova/+/960241 | 07:47 |
| gibi | jlejeune: thanks | 08:02 |
| gibi | jlejeune: one small suggestion, you can put all these to the same gerrit topic that was used for the original bugfix so it is easy to see the list of backports in a single query | 08:06 |
| jlejeune | gibi: indeed, can I add a topic after having created the change ? | 08:30 |
| gibi | jlejeune: yes on the web UI you can add it by hand | 08:33 |
| jlejeune | nice, done :) | 08:40 |
| gibi | jlejeune: thanks | 08:46 |
| gibi | elodilles: there is a simple clean backport series from jlejeune in https://review.opendev.org/q/topic:%22bug-2044235%22 if you have time :) | 08:47 |
| elodilles | gibi: ACK, will add it to my TODO :] | 08:48 |
| gibi | elodilles: thanks! | 08:48 |
| gibi | Uggla: (and others) I kept monitoring the nova-tox-py312-threading job to see if we have unexplained failures. I found three cases where this job failed but the eventlet based unit test job does not. I filed a bug and I will patch the ignore list to not run these two test cases until I have time to go back to burning down that ignore list https://bugs.launchpad.net/nova/+bug/2125185 | 08:50 |
| gibi | If anybody sees a nova-tox-py312-threading run that fails while the normal unit test job passes then please let me know | 08:51 |
| Uggla | gibi, ok I think we could share this in our next meeting and follow up on it if you wish. | 08:53 |
| gibi | sure I will try to remember of it in the next meeting j) | 08:57 |
| gibi | :) | 08:57 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: [nova-tox-py312-threading]Ignore failing tests https://review.opendev.org/c/openstack/nova/+/961781 | 09:00 |
| gibi | Uggla: ^^ this is the immediate fix | 09:00 |
| Uggla | 👍 | 09:00 |
| gibi | bauzas: sean-k-mooney: ^^ if you are around this fix a small gate instability | 09:00 |
| bauzas | gibi: looking (sorry was on meeting) | 09:31 |
| bauzas | gibi: thanks for continuing looking at our CI <3 | 09:32 |
| gibi | yeah I feel like I introduced a voting job so I responsible keeping that job in good health :) | 09:33 |
| sean-k-mooney | gibi: done | 11:22 |
| sean-k-mooney | gibi: sorry was not around in my morning today | 11:23 |
| gibi | sean-k-mooney: thanks, and no worries :) | 12:03 |
| gibi | heads up I saw multiple multinode job failing due to the fact that the subnode nova-compute cannot connet to rabbit. | 12:15 |
| gibi | maybe it is intermitten infra issue but if you also see it happening on your patches then it might worth to report to infra | 12:16 |
| frickler | gibi: infra is watching you already ;) do you have an example I could look at? | 12:17 |
| sean-k-mooney | gibi: the last tim ethat happend it was a build where the nodes came form 2 diffent providers | 12:19 |
| sean-k-mooney | so it woudl be good to check if all nodes came form the same provider in that failed job run | 12:20 |
| gibi | frickler: https://zuul.opendev.org/t/openstack/build/488552424c654e2e8a4d8c5cb82f02a0/logs | 12:20 |
| gibi | frickler: and thanks for keeping an eye on it (on me?!) | 12:21 |
| frickler | sean-k-mooney: yes, that was one of my ideas, but both from sjc3 in the above case | 12:22 |
| sean-k-mooney | https://zuul.opendev.org/t/openstack/build/488552424c654e2e8a4d8c5cb82f02a0/log/zuul-info/inventory.yaml | 12:22 |
| sean-k-mooney | they are both form raxflex | 12:22 |
| sean-k-mooney | https://zuul.opendev.org/t/openstack/build/488552424c654e2e8a4d8c5cb82f02a0/log/zuul-info/inventory.yaml#320 | 12:22 |
| sean-k-mooney | so it not that htis time | 12:22 |
| gibi | https://zuul.opendev.org/t/openstack/build/7055bb2bfcb84b6388f67ad3181f8043 this was the another case I hit today | 12:22 |
| sean-k-mooney | frickler: the two node might be in diffent subnets | 12:24 |
| sean-k-mooney | im not sure what the cider is but once is sueing 10.0.16.132 and the other is 10.0.17.239 for the private ip | 12:25 |
| sean-k-mooney | 255.255.240.0 | 12:26 |
| sean-k-mooney | ok so they are in a /20 so in the same subnet | 12:26 |
| sean-k-mooney | so there shoudl nto eb any firewalling between the nodes then | 12:27 |
| sean-k-mooney | well assumign they are in the same secuiryt group and we allow access from member of the same secuirty group but i dont think secuirty groups are likely to be the issue here | 12:28 |
| sean-k-mooney | if it was it woudl affect all jobs on that provider | 12:28 |
| frickler | there should not be two ethernet interfaces. we same some changes recently that could cause this. let's move to #opendev | 12:28 |
| sean-k-mooney | ack, ya i notcied there are 2 interface and 2 diffent private ips | 12:29 |
| sean-k-mooney | ack on swaping | 12:29 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Centralize cooperative yield https://review.opendev.org/c/openstack/nova/+/950991 | 12:35 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: [hacking] N374 do not use time.sleep(0) to yield https://review.opendev.org/c/openstack/nova/+/950992 | 12:35 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Make RBD Tpool usage conditional https://review.opendev.org/c/openstack/nova/+/956089 | 12:40 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Make libvirt Tpool proxying conditional https://review.opendev.org/c/openstack/nova/+/956090 | 12:40 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Fix ProviderTree copying with threading Lock https://review.opendev.org/c/openstack/nova/+/956091 | 12:40 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: [test]Further categorization of disabled unit tests https://review.opendev.org/c/openstack/nova/+/956092 | 12:40 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: [vncproxy]Handle ssl.wrap_socket removal in py312 https://review.opendev.org/c/openstack/nova/+/955915 | 12:40 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Warn on long task wait time for executor https://review.opendev.org/c/openstack/nova/+/952666 | 12:40 |
| bauzas | gibi: frickler: I probably have another related failure https://zuul.opendev.org/t/openstack/build/2ba0b6a8665e4c66883119ca7c602f6a | 12:59 |
| gibi | FYI, next week I want to get back to the normal eventlet sync meeting schedule. I updated the doc with the current status and things that can be reviewed already https://docs.google.com/document/d/1vCOmD7kxmRFk9IG8WbVhyqKlCpOPDZFJgkUSwYdQHbM/edit?tab=t.0 | 13:01 |
| *** haleyb|out is now known as haleyb | 13:04 | |
| bauzas | gibi: cool, I guess that procedurally, we just need another blueprint for the Gazpacho release to be accepted | 13:07 |
| gibi | uhh ohh yeah | 13:14 |
| gibi | I promised to do that | 13:14 |
| gibi | let me remember if I did it already :) | 13:14 |
| gibi | ohh hey I did it https://blueprints.launchpad.net/nova/+spec/eventlet-removal-gazpacho | 13:15 |
| gibi | as acceptance I'm not affraid of asking for it on the next meeting | 13:15 |
| * gibi goes updating the agenda for the next nova meeting | 13:16 | |
| bauzas | cool :) | 13:18 |
| * bauzas plays the lawyer role, not the PTL one :) | 13:18 | |
| bauzas | lawyer czar, lemme be it :) | 13:19 |
| bauzas | I'm French, I'm naturally leaning towards this role | 13:19 |
| gibi | meeting agend has been amended, French lawsuite suit successfully avoided :)j | 13:22 |
| bauzas | heh | 13:26 |
| opendevreview | Merged openstack/nova master: Add admin context when filling metadata on ImagePropsWeigher https://review.opendev.org/c/openstack/nova/+/961673 | 14:17 |
| opendevreview | Sylvain Bauza proposed openstack/nova stable/2025.2: Regression test for ImagePropsWeigher due to missing context https://review.opendev.org/c/openstack/nova/+/961803 | 14:21 |
| opendevreview | Sylvain Bauza proposed openstack/nova stable/2025.2: Add admin context when filling metadata on ImagePropsWeigher https://review.opendev.org/c/openstack/nova/+/961804 | 14:21 |
| opendevreview | Sylvain Bauza proposed openstack/nova stable/2025.1: Regression test for ImagePropsWeigher due to missing context https://review.opendev.org/c/openstack/nova/+/961805 | 14:26 |
| opendevreview | Sylvain Bauza proposed openstack/nova stable/2025.1: Add admin context when filling metadata on ImagePropsWeigher https://review.opendev.org/c/openstack/nova/+/961806 | 14:26 |
| elodilles | gibi jlejeune : the bug fix has landed AFTER stable/2025.2 was cut, so it needs to be cherry picked to stable/2025.2 first :S https://review.opendev.org/q/topic:bug-2044235 | 16:07 |
| elodilles | and note that as long as we are before the Final RC deadline, it should be merged to stable/2025.2 only if this bug is release critical :/ | 16:09 |
| opendevreview | Merged openstack/nova stable/2024.1: libvirt: Disable VMCoreInfo device for SEV-encrypted instances https://review.opendev.org/c/openstack/nova/+/959914 | 17:49 |
| -opendevstatus- NOTICE: All hosted mailing lists are undergoing maintenance for the next hour: https://lists.opendev.org/archives/list/service-announce@lists.opendev.org/message/UTMXRWWTE5WA3IF6WS3BIEJAORI2D62V/ | 20:01 | |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!