| opendevreview | liuyulong proposed openstack/neutron-tempest-plugin master: Skip nested snat validation for DVR https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/937443 | 00:49 |
|---|---|---|
| ricolin | Hi team, greatfully appricated if we can have more review on https://review.opendev.org/q/topic:%22add-ovn-db-sync-utils%22 please | 03:39 |
| opendevreview | yatin proposed openstack/neutron master: [DNM] wsgi switch higher workers https://review.opendev.org/c/openstack/neutron/+/936272 | 03:49 |
| opendevreview | yatin proposed openstack/neutron master: [DNM] wsgi switch higher workers eventlet https://review.opendev.org/c/openstack/neutron/+/937367 | 03:51 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Test devstack patch https://review.opendev.org/c/openstack/neutron/+/937519 | 07:56 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-tempest-plugin master: Test devstack patch https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/937520 | 07:58 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Improve initial hash ring setup https://review.opendev.org/c/openstack/neutron/+/936428 | 08:30 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Fast exit when initially creating tunnel allocations https://review.opendev.org/c/openstack/neutron/+/936813 | 08:30 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Fast exit when initially creating tunnel allocations https://review.opendev.org/c/openstack/neutron/+/936813 | 08:32 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Create a OVN hash ring maintenance thread per worker https://review.opendev.org/c/openstack/neutron/+/936838 | 08:32 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Reduce the OVN hash ring touch interval https://review.opendev.org/c/openstack/neutron/+/937351 | 08:32 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Do not fail if the ACL does not exists in the deletion https://review.opendev.org/c/openstack/neutron/+/934409 | 08:32 |
| lajoskatona | ralonsoh, ykarel: Hi, if you have 1 minute for vpnaas: https://review.opendev.org/c/openstack/neutron-vpnaas/+/937022 :-) | 08:59 |
| ralonsoh | sure | 08:59 |
| ralonsoh | done | 09:01 |
| lajoskatona | ralonsoh: thanks | 09:10 |
| ralonsoh | ykarel, hi, I think we have a serious problem. I was wrong and the devstack@neutron-rpc-server.service service is failing with the newer eventlet version | 09:14 |
| ralonsoh | I'm seeing the same errors as with eventlet | 09:15 |
| ralonsoh | but to be honest, that should be hitting any other project too | 09:15 |
| ralonsoh | frickler, ^ | 09:15 |
| frickler | ralonsoh: so just to be sure, that means that "switch to wsgi" doesn't really help? | 09:16 |
| ralonsoh | correct | 09:16 |
| frickler | :( | 09:18 |
| ykarel | ralonsoh, ohh | 09:19 |
| opendevreview | Merged openstack/neutron-vpnaas master: functional: Fix failing due to Noble and pip restricitons https://review.opendev.org/c/openstack/neutron-vpnaas/+/937022 | 09:33 |
| ralonsoh | frickler, I've identified the patch in eventlet ("Upgrade RLocks as last thing we do") | 09:41 |
| ralonsoh | this is breaking oslo.messaging | 09:41 |
| ralonsoh | ok, I'm going to take this conversation to the eventlet channnel and talk to Herve | 09:44 |
| ralonsoh | frickler, ykarel solved | 10:06 |
| ralonsoh | we are multi-monkey_patching | 10:06 |
| ralonsoh | we are calling this method twice | 10:06 |
| ralonsoh | this is as easy as adding a global variable to control that | 10:06 |
| ralonsoh | pushing a patch in 1 minute | 10:06 |
| frickler | I heard yesterday that calling monkey_patch multiple times should be fine | 10:12 |
| ralonsoh | let me push the patch and update the requirements one with this one | 10:19 |
| ralonsoh | but I'm testing now and works fine | 10:19 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Prevent calling "monkey_patch" more than once https://review.opendev.org/c/openstack/neutron/+/937527 | 10:24 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Prevent calling "monkey_patch" more than once https://review.opendev.org/c/openstack/neutron/+/937527 | 10:28 |
| opendevreview | Dong Ma proposed openstack/ovn-bgp-agent master: Update patch with comments https://review.opendev.org/c/openstack/ovn-bgp-agent/+/937530 | 10:52 |
| ralonsoh | lajoskatona, slaweq ykarel please we need https://review.opendev.org/c/openstack/neutron/+/937527 ASAP | 10:52 |
| ralonsoh | the requirements patch https://review.opendev.org/c/openstack/requirements/+/933257 tempest job seems to be working | 10:53 |
| slaweq | ralonsoh +2 | 10:58 |
| ykarel | ralonsoh, lgtm | 11:16 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: QinQ API extension implementation in the Neutron server https://review.opendev.org/c/openstack/neutron/+/937372 | 11:20 |
| opendevreview | Dong Ma proposed openstack/ovn-bgp-agent master: WIP: Add the support of create kubernetes resource https://review.opendev.org/c/openstack/ovn-bgp-agent/+/937457 | 11:20 |
| frickler | ralonsoh: nice. I still see some failures in the tempest job, but those seem to be related to https://review.opendev.org/c/openstack/tempest/+/937416, which hopefully should merge soon | 11:23 |
| ralonsoh | cool, so from the Neutron side we are good (for now) | 11:24 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: QinQ API extension implementation in the Neutron server https://review.opendev.org/c/openstack/neutron/+/937372 | 11:53 |
| lajoskatona | ralonsoh: ack, looking | 12:13 |
| opendevreview | Dong Ma proposed openstack/ovn-bgp-agent master: WIP: Add the support of create kubernetes resource https://review.opendev.org/c/openstack/ovn-bgp-agent/+/937457 | 12:22 |
| opendevreview | Dong Ma proposed openstack/ovn-bgp-agent master: WIP: Add the support of create kubernetes resource https://review.opendev.org/c/openstack/ovn-bgp-agent/+/937457 | 12:52 |
| opendevreview | Dong Ma proposed openstack/ovn-bgp-agent master: WIP: Add the support of create kubernetes resource https://review.opendev.org/c/openstack/ovn-bgp-agent/+/937457 | 13:10 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Improve initial hash ring setup https://review.opendev.org/c/openstack/neutron/+/936428 | 13:37 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Fast exit when initially creating tunnel allocations https://review.opendev.org/c/openstack/neutron/+/936813 | 13:38 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Create a OVN hash ring maintenance thread per worker https://review.opendev.org/c/openstack/neutron/+/936838 | 13:38 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Reduce the OVN hash ring touch interval https://review.opendev.org/c/openstack/neutron/+/937351 | 13:38 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Do not fail if the ACL does not exists in the deletion https://review.opendev.org/c/openstack/neutron/+/934409 | 13:38 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: DNM - Test "neutron-ovn-tempest-ipv6-only-ovs*" with 4 workers https://review.opendev.org/c/openstack/neutron/+/936429 | 13:38 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Read the environment variable EVENTLET_MONKEYPATCH https://review.opendev.org/c/openstack/neutron/+/937535 | 14:10 |
| opendevreview | Brian Haley proposed openstack/neutron master: WIP: Remove false-positive ACLs in OVN DB sync https://review.opendev.org/c/openstack/neutron/+/937299 | 14:38 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-lib master: Add VlanQinqDriverError exception https://review.opendev.org/c/openstack/neutron-lib/+/937543 | 15:51 |
| slaweq | haleyb hi, regarding my comment in https://review.opendev.org/c/openstack/neutron-lib/+/935731 - please take a look and let me know if that makes sense for you and if it is clear now :) | 15:54 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [WIP] Metadata proxy server without eventlet https://review.opendev.org/c/openstack/neutron/+/937545 | 16:03 |
| opendevreview | Merged openstack/neutron master: Set IP/MAC address on VPNaaS gateway port (OVN) https://review.opendev.org/c/openstack/neutron/+/936850 | 16:36 |
| 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:06 |
| -opendevstatus- NOTICE: Gerrit will undergo a short restart to pick up some bugfixes for the 3.10 release that we upgraded to. | 19:26 | |
| haleyb | slaweq: thanks, I think i understand now, will work on it | 19:42 |
| haleyb | noonedeadpunk: had a question about the item you added to the drivers meeting agenda, https://bugs.launchpad.net/ovn-bgp-agent/+bug/2088558 - it almost doesn't seem like an RFE but just a bug. Do you just need to get eyes on the change? | 19:48 |
| noonedeadpunk | haleyb: yeah. pretty much and if that actually a bug or I just misunderstood things | 22:09 |
| noonedeadpunk | as it's around for a month without any feedback, so wanted to get some attention | 22:10 |
| haleyb | noonedeadpunk: maybe ping someone like jlibosva when he's around as he does more ovn-bgp-agent work | 22:13 |
| noonedeadpunk | I also have a more fresh one for which I actually didn't create a bug report even so far: https://review.opendev.org/c/openstack/ovn-bgp-agent/+/937362 | 22:18 |
| noonedeadpunk | but that's a different story - it's brand new so to say | 22:19 |
| *** haleyb is now known as haleyb|out | 23:07 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!