| opendevreview | yatin proposed openstack/neutron stable/2024.1: [DNM] check fix https://review.opendev.org/c/openstack/neutron/+/937956 | 04:03 |
|---|---|---|
| ykarel | we have a real issue with jobs running on ubuntu jammy + ovs/linuxbridge or the ones which using iptables | 04:23 |
| ykarel | all branches are impacted | 04:23 |
| frickler | ykarel: seems to be a kernel bug that made it into jammy, saw the same in kolla https://lore.kernel.org/all/20241019-xtables-typos-v2-1-6b8b1735dc8e@0upti.me/ | 04:50 |
| ykarel | frickler, yeah just reported https://bugs.launchpad.net/neutron/+bug/2091990 with details | 04:50 |
| ykarel | frickler, any way to hold this kernel update? | 04:55 |
| ykarel | as i believe the fix will take some time to be available in ubuntu jammy | 04:55 |
| ykarel | i see next update in line https://bugs.launchpad.net/kernel-sru-workflow/+bug/2090163 but that also don't have the fix included | 05:02 |
| *** ykarel_ is now known as ykarel | 06:02 | |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Set always the GW LRP "gateway_mtu" option https://review.opendev.org/c/openstack/neutron/+/937026 | 06:50 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Allow network owner reader to get subnets https://review.opendev.org/c/openstack/neutron/+/900236 | 08:26 |
| ralonsoh | slaweq, ^^ please check this patch again | 08:26 |
| ralonsoh | if approved, I'll backport it up to 2023.2 | 08:26 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2024.2: Allow network owner reader to get subnets https://review.opendev.org/c/openstack/neutron/+/937966 | 08:28 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2024.1: Allow network owner reader to get subnets https://review.opendev.org/c/openstack/neutron/+/937967 | 08:32 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2023.2: Allow network owner reader to get subnets https://review.opendev.org/c/openstack/neutron/+/937968 | 08:32 |
| slaweq | ralonsoh looks good for me, I added one comment but it can be addressed in yet another follow up (it is only about unit tests) | 08:32 |
| ralonsoh | slaweq, right, this new policy deserves a new UTs | 08:33 |
| ralonsoh | not new policy, new target | 08:33 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Allow network owner reader to get subnets https://review.opendev.org/c/openstack/neutron/+/900236 | 08:47 |
| sahid | o/ | 09:18 |
| sahid | hello guys, quick question, regarding enventlet removal, you may already have discussed that, is it ok to use future.ThreadPoolExecutor? we don't have any specific lib to use? | 09:19 |
| ralonsoh | sahid, not from future library, but from futurist library | 09:24 |
| ralonsoh | the thread, green and process executors have the same API | 09:25 |
| sahid | ralonsoh: ack thanks | 09:27 |
| ralonsoh | sahid, what are you trying to implement? | 09:27 |
| ralonsoh | can you point to the code? | 09:27 |
| sahid | ralonsoh: git me few seconds i will publish the code | 09:39 |
| sahid | s/git/give | 09:39 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: ovs: remove the usage of eventlet in the OVS agent https://review.opendev.org/c/openstack/neutron/+/937765 | 09:40 |
| sahid | ralonsoh: https://review.opendev.org/c/openstack/neutron/+/937765/6/neutron/services/trunk/drivers/openvswitch/agent/ovsdb_handler.py | 09:41 |
| sahid | it's that part | 09:41 |
| sahid | you feel it's the right way to move on it? | 09:41 |
| sahid | I'm going to add some tests | 09:42 |
| sahid | well, if the impl looks reasonable | 09:42 |
| ralonsoh | sahid, I don't think the context would work here | 09:44 |
| ralonsoh | we need a thread working in parallel executing the trunk add command | 09:45 |
| ralonsoh | but we need to return the event AFTER_READ | 09:45 |
| ralonsoh | if you use the thread context, you'll wait until the thread finishes | 09:45 |
| ralonsoh | that was not the goal | 09:45 |
| sahid | ralonsoh: i see, that makes sense, so you prefer we just let the threads making their life? or perhaps we could send the event before to enter in the context? | 09:49 |
| ralonsoh | for now we should just translate, if possible, the current code | 09:50 |
| ralonsoh | with eventlet we were just spawning a thread and that's all | 09:50 |
| sahid | yer, ack, previously it was like that | 09:51 |
| sahid | yes | 09:51 |
| ralonsoh | maybe we can do something more elaborate, but as a follow-up | 09:51 |
| sahid | sure i understand, i will fix that point and look how this part is coverade with tests | 09:51 |
| opendevreview | Merged openstack/neutron stable/2024.1: Revert "[HA] Do not add initial state change delay in HA router" https://review.opendev.org/c/openstack/neutron/+/937879 | 10:06 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Set always the GW LRP "gateway_mtu" option https://review.opendev.org/c/openstack/neutron/+/937026 | 10:15 |
| opendevreview | Merged openstack/ovn-octavia-provider master: Add Tobiko to gate jobs https://review.opendev.org/c/openstack/ovn-octavia-provider/+/936341 | 10:51 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Use the Neutron port ID in ``UpdateLSwitchPortQosOptionsCommand`` https://review.opendev.org/c/openstack/neutron/+/937682 | 14:02 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: WIP == Isolate test_ovn_db_sync.TestOvnNbSync.* tests https://review.opendev.org/c/openstack/neutron/+/937992 | 14:12 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: WIP == Isolate test_ovn_db_sync.TestOvnNbSync.* tests https://review.opendev.org/c/openstack/neutron/+/937992 | 14:13 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Allow network owner reader to get subnets https://review.opendev.org/c/openstack/neutron/+/900236 | 14:20 |
| ralonsoh | ykarel, I've pushed 937992 to test (again) the issue with test_ovn_db_sync.TestOvnNbSync.* | 14:20 |
| ralonsoh | I've seen more errors related to the DB | 14:20 |
| ralonsoh | this time different, but still errors related to the DB | 14:21 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Read the environment variable EVENTLET_MONKEYPATCH https://review.opendev.org/c/openstack/neutron/+/937535 | 14:24 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Catch when the process does not exist when killing it https://review.opendev.org/c/openstack/neutron/+/935046 | 14:34 |
| ralonsoh | haleyb, hello! If you have some minutes, please check https://review.opendev.org/c/openstack/neutron/+/936838 | 14:35 |
| ralonsoh | thanks! | 14:35 |
| haleyb | ralonsoh: hi! will look between meetings here | 14:37 |
| opendevreview | Jakub Libosvar proposed openstack/ovn-bgp-agent master: WIP: Introduce multinode tempest job https://review.opendev.org/c/openstack/ovn-bgp-agent/+/936968 | 14:56 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: common: add is_monked_patched as helper to eventlet_utils https://review.opendev.org/c/openstack/neutron/+/937908 | 15:02 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: common: fix wait_until_true to support native thread https://review.opendev.org/c/openstack/neutron/+/937843 | 15:02 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: ovs: remove the usage of eventlet in the OVS agent https://review.opendev.org/c/openstack/neutron/+/937765 | 15:02 |
| ykarel | ralonsoh, ack | 15:08 |
| opendevreview | Merged openstack/neutron master: Fast exit when initially creating tunnel allocations https://review.opendev.org/c/openstack/neutron/+/936813 | 15:20 |
| opendevreview | Merged openstack/neutron stable/2024.2: Revert "[HA] Do not add initial state change delay in HA router" https://review.opendev.org/c/openstack/neutron/+/937878 | 15:22 |
| haleyb | ralonsoh: i just had one question on that change, added comment | 16:15 |
| opendevreview | Max proposed openstack/ovsdbapp master: fix: add NB LRP name index https://review.opendev.org/c/openstack/ovsdbapp/+/938004 | 16:39 |
| opendevreview | Jakub Libosvar proposed openstack/ovn-bgp-agent master: WIP: Introduce multinode tempest job https://review.opendev.org/c/openstack/ovn-bgp-agent/+/936968 | 16:48 |
| opendevreview | Jakub Libosvar proposed openstack/ovn-bgp-agent master: WIP: Introduce multinode tempest job https://review.opendev.org/c/openstack/ovn-bgp-agent/+/936968 | 18:20 |
| opendevreview | Jakub Libosvar proposed openstack/ovn-bgp-agent master: WIP: Introduce multinode tempest job https://review.opendev.org/c/openstack/ovn-bgp-agent/+/936968 | 19:56 |
| opendevreview | Jakub Libosvar proposed openstack/ovn-bgp-agent master: WIP: Introduce multinode tempest job https://review.opendev.org/c/openstack/ovn-bgp-agent/+/936968 | 20:19 |
| opendevreview | Brian Haley proposed openstack/neutron master: WIP: Remove false-positive ACLs in OVN DB sync https://review.opendev.org/c/openstack/neutron/+/937299 | 22:22 |
| *** haleyb is now known as haleyb|out | 22:47 | |
| opendevreview | Jakub Libosvar proposed openstack/ovn-bgp-agent master: WIP: Introduce multinode tempest job https://review.opendev.org/c/openstack/ovn-bgp-agent/+/936968 | 22:58 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!