| *** jayaanand_ is now known as jayaanand | 01:24 | |
| opendevreview | Rico Lin proposed openstack/neutron-vpnaas stable/2024.2: Use write connection for get_vpn_services_on_host https://review.opendev.org/c/openstack/neutron-vpnaas/+/943519 | 02:54 |
|---|---|---|
| opendevreview | Rico Lin proposed openstack/neutron-vpnaas stable/2024.1: Use write connection for get_vpn_services_on_host https://review.opendev.org/c/openstack/neutron-vpnaas/+/943520 | 02:55 |
| opendevreview | Rico Lin proposed openstack/neutron-vpnaas stable/2023.2: Use write connection for get_vpn_services_on_host https://review.opendev.org/c/openstack/neutron-vpnaas/+/943521 | 02:57 |
| *** dmellado0755393736 is now known as dmellado075539373 | 06:43 | |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [QoS] QoS rule check also considers the direction https://review.opendev.org/c/openstack/neutron/+/943373 | 06:51 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: osdb: remove usage of eventlet in dbhandler https://review.opendev.org/c/openstack/neutron/+/940983 | 07:14 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: ofswitch: remove usage of eventlet timeout https://review.opendev.org/c/openstack/neutron/+/940982 | 07:14 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: ovs: remove usage of loopingcall.FixedIntervalLoopingCall https://review.opendev.org/c/openstack/neutron/+/941429 | 07:14 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: ovs: reimplement signals handling https://review.opendev.org/c/openstack/neutron/+/939321 | 07:14 |
| 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 | 07:14 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: DNM - Test errors in ``neutron-functional-with-pyroute2-master`` https://review.opendev.org/c/openstack/neutron/+/942615 | 08:31 |
| opendevreview | OpenStack Release Bot proposed openstack/neutron-lib stable/2025.1: Update .gitreview for stable/2025.1 https://review.opendev.org/c/openstack/neutron-lib/+/943561 | 09:16 |
| opendevreview | OpenStack Release Bot proposed openstack/neutron-lib stable/2025.1: Update TOX_CONSTRAINTS_FILE for stable/2025.1 https://review.opendev.org/c/openstack/neutron-lib/+/943562 | 09:16 |
| opendevreview | OpenStack Release Bot proposed openstack/neutron-lib master: Update master for stable/2025.1 https://review.opendev.org/c/openstack/neutron-lib/+/943563 | 09:16 |
| opendevreview | OpenStack Release Bot proposed openstack/os-ken stable/2025.1: Update .gitreview for stable/2025.1 https://review.opendev.org/c/openstack/os-ken/+/943564 | 09:16 |
| opendevreview | OpenStack Release Bot proposed openstack/os-ken stable/2025.1: Update TOX_CONSTRAINTS_FILE for stable/2025.1 https://review.opendev.org/c/openstack/os-ken/+/943565 | 09:16 |
| opendevreview | OpenStack Release Bot proposed openstack/os-ken master: Update master for stable/2025.1 https://review.opendev.org/c/openstack/os-ken/+/943566 | 09:16 |
| opendevreview | OpenStack Release Bot proposed openstack/ovsdbapp stable/2025.1: Update .gitreview for stable/2025.1 https://review.opendev.org/c/openstack/ovsdbapp/+/943567 | 09:16 |
| opendevreview | OpenStack Release Bot proposed openstack/ovsdbapp stable/2025.1: Update TOX_CONSTRAINTS_FILE for stable/2025.1 https://review.opendev.org/c/openstack/ovsdbapp/+/943568 | 09:16 |
| opendevreview | OpenStack Release Bot proposed openstack/ovsdbapp master: Update master for stable/2025.1 https://review.opendev.org/c/openstack/ovsdbapp/+/943569 | 09:16 |
| opendevreview | OpenStack Release Bot proposed openstack/python-neutronclient stable/2025.1: Update .gitreview for stable/2025.1 https://review.opendev.org/c/openstack/python-neutronclient/+/943570 | 09:16 |
| opendevreview | OpenStack Release Bot proposed openstack/python-neutronclient stable/2025.1: Update TOX_CONSTRAINTS_FILE for stable/2025.1 https://review.opendev.org/c/openstack/python-neutronclient/+/943571 | 09:16 |
| opendevreview | OpenStack Release Bot proposed openstack/python-neutronclient master: Update master for stable/2025.1 https://review.opendev.org/c/openstack/python-neutronclient/+/943572 | 09:16 |
| opendevreview | Merged openstack/ovsdbapp master: Update master for stable/2025.1 https://review.opendev.org/c/openstack/ovsdbapp/+/943569 | 09:36 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [QoS] QoS rule check also considers the direction https://review.opendev.org/c/openstack/neutron/+/943373 | 09:50 |
| opendevreview | Merged openstack/python-neutronclient master: Update master for stable/2025.1 https://review.opendev.org/c/openstack/python-neutronclient/+/943572 | 09:51 |
| opendevreview | Merged openstack/os-ken master: Update master for stable/2025.1 https://review.opendev.org/c/openstack/os-ken/+/943566 | 09:55 |
| opendevreview | Merged openstack/os-ken stable/2025.1: Update .gitreview for stable/2025.1 https://review.opendev.org/c/openstack/os-ken/+/943564 | 09:55 |
| opendevreview | Merged openstack/python-neutronclient stable/2025.1: Update .gitreview for stable/2025.1 https://review.opendev.org/c/openstack/python-neutronclient/+/943570 | 09:58 |
| opendevreview | Merged openstack/python-neutronclient stable/2025.1: Update TOX_CONSTRAINTS_FILE for stable/2025.1 https://review.opendev.org/c/openstack/python-neutronclient/+/943571 | 09:58 |
| opendevreview | Merged openstack/os-ken stable/2025.1: Update TOX_CONSTRAINTS_FILE for stable/2025.1 https://review.opendev.org/c/openstack/os-ken/+/943565 | 09:59 |
| opendevreview | Merged openstack/neutron-lib master: Update master for stable/2025.1 https://review.opendev.org/c/openstack/neutron-lib/+/943563 | 10:00 |
| opendevreview | Merged openstack/neutron-lib stable/2025.1: Update .gitreview for stable/2025.1 https://review.opendev.org/c/openstack/neutron-lib/+/943561 | 10:00 |
| opendevreview | Merged openstack/neutron-lib stable/2025.1: Update TOX_CONSTRAINTS_FILE for stable/2025.1 https://review.opendev.org/c/openstack/neutron-lib/+/943562 | 10:00 |
| opendevreview | Merged openstack/ovsdbapp stable/2025.1: Update .gitreview for stable/2025.1 https://review.opendev.org/c/openstack/ovsdbapp/+/943567 | 10:05 |
| opendevreview | Merged openstack/ovsdbapp stable/2025.1: Update TOX_CONSTRAINTS_FILE for stable/2025.1 https://review.opendev.org/c/openstack/ovsdbapp/+/943568 | 10:05 |
| opendevreview | Merged openstack/networking-bagpipe unmaintained/yoga: [CI] Remove periodic stable jobs from unmaintained branch https://review.opendev.org/c/openstack/networking-bagpipe/+/943367 | 10:50 |
| opendevreview | Merged openstack/neutron master: ml2/sriov: bind port on segment that subnet belongs https://review.opendev.org/c/openstack/neutron/+/941345 | 11:07 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/neutron master: [OVN] Do not supply gateway_port if it's not binded to chassis https://review.opendev.org/c/openstack/neutron/+/931495 | 11:39 |
| opendevreview | Renjing Xiao proposed openstack/neutron-tempest-plugin master: Cover both enabled and disabled cases in nested snat validation test https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/940906 | 11:39 |
| opendevreview | Renjing Xiao proposed openstack/neutron-tempest-plugin master: Cover both enabled and disabled cases in nested snat validation test https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/940906 | 11:42 |
| opendevreview | Merged openstack/networking-bagpipe unmaintained/zed: [CI] Remove periodic stable jobs from unmaintained branch https://review.opendev.org/c/openstack/networking-bagpipe/+/943020 | 11:59 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: WIP == [OVN] Make the LSP up/down events more robust https://review.opendev.org/c/openstack/neutron/+/943404 | 12:40 |
| opendevreview | Michel Nederlof proposed openstack/ovn-bgp-agent master: Change order of removing tenant routes. https://review.opendev.org/c/openstack/ovn-bgp-agent/+/943594 | 13:32 |
| opendevreview | Lajos Katona proposed openstack/neutron master: [eventlet-removal] Remove the usage of eventlet in the SR-IOV agent https://review.opendev.org/c/openstack/neutron/+/942884 | 14:04 |
| opendevreview | Renjing Xiao proposed openstack/neutron-tempest-plugin master: Cover both enabled and disabled cases in nested snat validation test https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/940906 | 14:36 |
| haleyb | atmark: that should work as the rpc messages are versioned | 14:48 |
| opendevreview | Merged x/whitebox-neutron-tempest-plugin master: Timeout before getting logs https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/943410 | 14:52 |
| lajoskatona | haleyb, ralonsoh: Hi, regarding eventlet deprecation, I just realized that there is the macvtap agent also, with quick grep I can't see direct eventlet or such dependency, but to tell the truth I am dumb for it totally | 14:53 |
| ralonsoh | lajoskatona, yes, I think I documented that in the eventlet deprecation document | 15:07 |
| ralonsoh | lajoskatona, let me check one thing | 15:07 |
| ralonsoh | lajoskatona, it is using oslo.service | 15:08 |
| ralonsoh | launcher = service.launch(cfg.CONF, agent, restart_method='mutate') | 15:08 |
| ralonsoh | for now we have one single implementation for this, that is the eventlet one | 15:08 |
| ralonsoh | https://github.com/openstack/oslo.service/blob/master/oslo_service/backend/eventlet/service.py | 15:09 |
| ralonsoh | so we need to wait | 15:09 |
| ralonsoh | in any case, if I didn't open a LP bug for this, I'll do it now | 15:09 |
| haleyb | thanks ralonsoh | 15:10 |
| haleyb | ralonsoh: i did have another question for you on db contractions - i thought we typically didn't do it, but i've seen a couple of reviews doing it, maybe i'm mis-remembering | 15:10 |
| haleyb | for exmaple, https://review.opendev.org/c/openstack/neutron-fwaas/+/929501 | 15:10 |
| ralonsoh | haleyb, we don't do since 2017, I think | 15:12 |
| ralonsoh | in that cases, we use the upgrade process adding the exception | 15:12 |
| haleyb | yeah, i couldn't find any comment in the docs, just shared memory | 15:13 |
| ralonsoh | we are dropping a table and that's forbidden in the upgrade phase | 15:13 |
| ralonsoh | one sec | 15:13 |
| ralonsoh | haleyb, in any case, this is in Neutron | 15:14 |
| ralonsoh | we don't block that for other projects | 15:15 |
| ralonsoh | from the docs | 15:15 |
| ralonsoh | * if possible, contract migrations should be minimized or avoided to reduce | 15:15 |
| ralonsoh | the time when API endpoints must be down during database upgrade. | 15:15 |
| ralonsoh | this is why we stop doing that | 15:15 |
| haleyb | ralonsoh: ack, that comment is so non-obvious that you shouldn't do them. but i guess these changes are good since they are not in neutron | 15:20 |
| zigo | Hi there! If I activate json logging, neutron-bgp-agent just dies. Is this a known bug? | 15:34 |
| ralonsoh | haleyb, but we need to move the agent away from the eventlet directory at some point | 15:40 |
| ralonsoh | so I'll register the bug | 15:40 |
| ralonsoh | zigo, no that I'm aware, please report a LP bug with some traces | 15:40 |
| ralonsoh | and how to reproduce it | 15:40 |
| zigo | Well, no traces, because I believe the issue is in the logging. | 15:40 |
| zigo | Is there a specific project for dynamic-routing, or just neutron? | 15:41 |
| haleyb | ralonsoh: right, can we move it even if it's calling service.launch()? | 15:41 |
| zigo | Maybe the issue is in os-ken? | 15:46 |
| vsaienko | ralonsoh: hello, please review https://review.opendev.org/c/openstack/neutron/+/939961 I've addressed your comments (this is small patch to the docs) | 15:48 |
| * zigo tries something | 15:48 | |
| ralonsoh | haleyb, no unless we remove how the agent is launched | 15:49 |
| ralonsoh | I'll just wait until we have oslo_service service module with threading backend | 15:49 |
| ralonsoh | zigo, no idea, I would need to see the logs | 15:49 |
| ralonsoh | vsaienko, I'll check it | 15:49 |
| vsaienko | thanks! | 15:50 |
| opendevreview | Merged openstack/neutron-fwaas master: Install runtime/test dependencies for venv target https://review.opendev.org/c/openstack/neutron-fwaas/+/943116 | 15:50 |
| ralonsoh | vsaienko, but I don't understand | 15:50 |
| ralonsoh | why did you remove the section where you specify that is needed to define different MACs per node? | 15:51 |
| ralonsoh | that's the quid of this section | 15:51 |
| vsaienko | I've added it back https://review.opendev.org/c/openstack/neutron/+/939961/5/doc/source/install/ovn/manual_install.rst#113 | 15:51 |
| vsaienko | |for each physnet with unique MAC ... | 15:52 |
| ralonsoh | ah ah sorry | 15:52 |
| ralonsoh | I misread it, thanks! | 15:52 |
| zigo | https://bugs.launchpad.net/neutron/+bug/2101077 | 15:52 |
| vsaienko | I'm sorry I'm not a native english speaker | 15:52 |
| ralonsoh | me neither | 15:53 |
| opendevreview | Takashi Kajinami proposed openstack/neutron-fwaas master: Drop v1 to v2 migration https://review.opendev.org/c/openstack/neutron-fwaas/+/929501 | 15:59 |
| opendevreview | Merged openstack/networking-bagpipe master: Drop removed tables for linux bridge plugin https://review.opendev.org/c/openstack/networking-bagpipe/+/943115 | 16:58 |
| opendevreview | Terry Wilson proposed openstack/neutron master: Avoid race condition by using persist_uuid https://review.opendev.org/c/openstack/neutron/+/923971 | 17:00 |
| frickler | zigo: added some questions. fwiw n-d-r is still a part of neutron proper, as opposed to some other neutron-* named repos | 17:07 |
| opendevreview | Michel Nederlof proposed openstack/ovn-bgp-agent master: Move the check for allowed address scopes to the appropriate place https://review.opendev.org/c/openstack/ovn-bgp-agent/+/943626 | 17:48 |
| opendevreview | Michel Nederlof proposed openstack/ovn-bgp-agent master: Change order of removing tenant routes. https://review.opendev.org/c/openstack/ovn-bgp-agent/+/943594 | 17:52 |
| opendevreview | Mark Loza proposed openstack/neutron unmaintained/2023.1: Revert "[HA] Do not add initial state change delay in HA router" https://review.opendev.org/c/openstack/neutron/+/943630 | 18:32 |
| opendevreview | Mark Loza proposed openstack/neutron unmaintained/zed: Revert "[HA] Do not add initial state change delay in HA router" https://review.opendev.org/c/openstack/neutron/+/943631 | 18:34 |
| opendevreview | Mark Loza proposed openstack/neutron unmaintained/yoga: Revert "[HA] Do not add initial state change delay in HA router" https://review.opendev.org/c/openstack/neutron/+/943632 | 18:36 |
| opendevreview | Merged x/whitebox-neutron-tempest-plugin master: Test verifies BZ#2214566/OSPRH-13533 doesn't regress https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/941930 | 18:46 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2024.2: [QoS] QoS rule check also considers the direction https://review.opendev.org/c/openstack/neutron/+/943644 | 21:23 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2024.1: [QoS] QoS rule check also considers the direction https://review.opendev.org/c/openstack/neutron/+/943645 | 21:23 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2023.2: [QoS] QoS rule check also considers the direction https://review.opendev.org/c/openstack/neutron/+/943646 | 21:23 |
| opendevreview | Merged openstack/neutron unmaintained/2023.1: Revert "[HA] Do not add initial state change delay in HA router" https://review.opendev.org/c/openstack/neutron/+/943630 | 22:22 |
| *** haleyb is now known as haleyb|out | 22:48 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!