| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [UT] Delete the OVN AgentCache instance in the cleanup https://review.opendev.org/c/openstack/neutron/+/946734 | 06:36 |
|---|---|---|
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2025.1: [UT] Delete the OVN AgentCache instance in the cleanup https://review.opendev.org/c/openstack/neutron/+/946735 | 06:36 |
| ralonsoh | ykarel, ^ same as in D/S | 08:14 |
| ykarel | thx ralonsoh , so you were able to reproduce this too locally? | 08:15 |
| ralonsoh | ykarel, no, but this is the only place where we doing this | 08:15 |
| ralonsoh | self.agent_cache = neutron_agent.AgentCache(driver=mock.ANY) | 08:15 |
| ralonsoh | so the mech driver is ANY, the error we are seeing | 08:16 |
| ykarel | ralonsoh, ack, and why we seeing that randomly? | 08:16 |
| ralonsoh | ykarel, I don't know how the tests are executed in a tox worker (I'm investigating this) | 08:17 |
| ralonsoh | if the test load is deterministic, then I don't know how this could be happening | 08:18 |
| ralonsoh | but if that depends on other workers execution, then it could be possible that we are executing a test using the agent cache after one of these tests | 08:18 |
| ykarel | ralonsoh, ack thx, was just trying to be sure if this will fix the issue we seeing randomly, patch in itself looks good though | 08:18 |
| opendevreview | yatin proposed openstack/neutron master: [DNM] Check fips job with python requires update https://review.opendev.org/c/openstack/neutron/+/946642 | 09:02 |
| opendevreview | Merged openstack/neutron master: Update jobs based on testing runtime for 2025.2 https://review.opendev.org/c/openstack/neutron/+/944809 | 09:37 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN][QoS] Update the Logical_Swith_Port options with the QoS values https://review.opendev.org/c/openstack/neutron/+/946542 | 10:36 |
| opendevreview | Merged openstack/neutron master: Fix LRP.external_ids router name prefix https://review.opendev.org/c/openstack/neutron/+/940445 | 10:42 |
| opendevreview | yatin proposed openstack/neutron master: [DNM] Check fips job with python requires update https://review.opendev.org/c/openstack/neutron/+/946642 | 11:21 |
| opendevreview | yatin proposed openstack/neutron master: [DNM] Check fips job with python requires update https://review.opendev.org/c/openstack/neutron/+/946642 | 12:57 |
| opendevreview | yatin proposed openstack/neutron master: [DNM] Check fips job with python requires update https://review.opendev.org/c/openstack/neutron/+/946642 | 14:01 |
| cardoe | ralonsoh: wrt https://opendev.org/openstack/neutron/src/commit/99a9470da551310fd24ded3476a1ad36eeab3f95/neutron/agent/metadata/agent.py#L194-L196 we're not actually getting a 404 back just an empty response. it seems like we need to encode that HTTPNotFound to bytes and write it out? | 14:08 |
| ralonsoh | cardoe, let me check | 14:10 |
| cardoe | Trying to help the OpenStack Helm PTL with Kubernetes health checks and it's coming back with a completely empty reply so we cannot get it to succeed. If we got a 404 then we'd know it was healthy. | 14:12 |
| ralonsoh | cardoe, right, in the previous implementation it used the same http exception method | 14:14 |
| ralonsoh | I'll add a patch asap | 14:14 |
| ralonsoh | cardoe, let me ping you asap I have something and you'll be able to test it | 14:15 |
| cardoe | ralonsoh: https://github.com/openstack/neutron/compare/stable/2025.1...kozhukalov:neutron:stable/2025.1 that's what he's hacked together thus far. | 14:25 |
| cardoe | Trying to get him on IRC. | 14:25 |
| ralonsoh | ^ I think this last message is not for here | 14:26 |
| ralonsoh | cardoe, I'm in the PTG right now but tomorrow mornign I'll push a patch for this | 14:26 |
| cardoe | ralonsoh: No. I was sharing the PTL's code that he's done to debug this and make it work so you could see. I'm in the PTG as well. I wish I was in the neutron one but I'm in the ironic one and we're debating networking things. :D | 14:33 |
| cardoe | I'll be on neutron tomorrow to talk the VXLAN stuff. | 14:36 |
| cardoe | I appreciate you're quick response. | 14:36 |
| opendevreview | Lajos Katona proposed openstack/tap-as-a-service master: Policies: use new policy definion for tap mirrors and flows https://review.opendev.org/c/openstack/tap-as-a-service/+/941109 | 15:52 |
| *** dansmith is now known as dansmith_pto | 16:52 | |
| Uggla | haleyb, just to confirm it is still ok for nova to host the session tomorrow. | 16:56 |
| haleyb | Uggla: ack, yes that's fine with me | 17:21 |
| haleyb | MengyangZhang[m]: just wanted to confirm the time for QoS discussion tomorrow - 13:00 UTC, in the Nova room, https://meetpad.opendev.org/nova-2025.2-ptg | 17:23 |
| haleyb | tobias-urdin: and just a pre-ping that your item is on the Neutron agenda tomorrow at 14:10 UTC | 17:24 |
| haleyb | cardoe: and also a pre-ping to you that your item is on the Neutron agenda tomorrow at 15:00 UTC, just be aware it could be earlier/later | 17:25 |
| cardoe | Thank you. I'll be there early. | 17:25 |
| haleyb | thanks | 17:26 |
| opendevreview | Merged openstack/neutron master: [UT] Delete the OVN AgentCache instance in the cleanup https://review.opendev.org/c/openstack/neutron/+/946734 | 18:31 |
| opendevreview | Brian Haley proposed openstack/neutron master: Fix use of pyroute2 Stats tuple import https://review.opendev.org/c/openstack/neutron/+/946817 | 18:46 |
| opendevreview | Merged openstack/neutron stable/2025.1: [UT] Delete the OVN AgentCache instance in the cleanup https://review.opendev.org/c/openstack/neutron/+/946735 | 19:14 |
| opendevreview | Terry Wilson proposed openstack/neutron master: Add persist_uuid upgrade scenario test https://review.opendev.org/c/openstack/neutron/+/946455 | 21:40 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!