| ralonsoh | haleyb, I was on PTO yesterday. I'm aware that I'm the bug deputy, thanks! | 07:14 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/ovn-octavia-provider master: Drop redundant N347 check https://review.opendev.org/c/openstack/ovn-octavia-provider/+/972303 | 07:15 |
| opendevreview | Lajos Katona proposed openstack/ovn-octavia-provider master: Add periodic-weekly queue for ovn-octavia-provider https://review.opendev.org/c/openstack/ovn-octavia-provider/+/972325 | 07:22 |
| zigo | Hi. | 08:32 |
| zigo | Bastian Blank just announced that iptables-legacy support is being removed from Debian (in favor of nftables). Does this affect Neutron in some ways? Does Neutron gained support for nft? | 08:32 |
| lajoskatona | zigo: Hi, I see this as last patch for using nftables via the iptables wrapper from ralonsoh: https://review.opendev.org/c/openstack/neutron/+/971049 | 08:36 |
| opendevreview | Merged openstack/ovn-octavia-provider master: Change test code to use project_id key https://review.opendev.org/c/openstack/ovn-octavia-provider/+/972253 | 08:38 |
| opendevreview | Lajos Katona proposed openstack/neutron master: [eventlet-removal] Handle systemctl stop for DHCP agent https://review.opendev.org/c/openstack/neutron/+/967902 | 08:53 |
| zigo | Cool, thanks ! | 08:55 |
| opendevreview | Lajos Katona proposed openstack/os-ken master: [eventlet]: change events to real threading events https://review.opendev.org/c/openstack/os-ken/+/970277 | 08:56 |
| ralonsoh | zigo, does it mean the iptables API is no longer supported? | 09:00 |
| zigo | ralonsoh: It means that /usr/bin/iptables will be using nftables, just like if one set the set-alternative, I believe. But the userland binary isn't the point: the kernel compat flag will not be on, so iptables (from the kernel point of view) wont be available. | 09:01 |
| opendevreview | Marcin Wilk proposed openstack/neutron master: Use SB DB for checking if an OVN agent is alive https://review.opendev.org/c/openstack/neutron/+/971876 | 09:02 |
| ralonsoh | zigo, does it mean the iptables API is no longer supported? | 09:46 |
| zigo | Neutron uses /usr/bin/iptables, right ? | 09:46 |
| ralonsoh | yes, the legacy API, we still use the iptables commands with nft | 09:46 |
| zigo | Then that will continue to work, just /usr/bin/iptables will be using nftables, I guess. | 09:47 |
| ralonsoh | ok then | 09:47 |
| ralonsoh | I tried some years ago to change the iptables code and implement a nfttables equivalent | 09:47 |
| ralonsoh | after 1 year, I abandoned this patch because I didn't find what I was doing wrong | 09:48 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Test FT LP bug 2137467 https://review.opendev.org/c/openstack/neutron/+/972435 | 09:48 |
| opendevreview | mkarpin proposed openstack/neutron stable/2025.1: wip test https://review.opendev.org/c/openstack/neutron/+/972440 | 09:56 |
| opendevreview | mkarpin proposed openstack/neutron stable/2025.1: wip test https://review.opendev.org/c/openstack/neutron/+/972440 | 09:57 |
| zigo | ralonsoh: What I remember was wrong, was the parsing of the nftables output instead of iptables-legacy. Everything else just worked. | 09:58 |
| zigo | If that has been fixed, then all is ok. | 09:58 |
| zigo | Also, maybe if using OVS as firewall instead of iptables, then nobody cares? | 09:58 |
| ralonsoh | zigo, we use iptables for (1) hybrid FW (as you said, OVS+iptables), (2) the L3 service (everything related to ML2/OVS L3 uses iptables) and (3) some minor uses (fix hash in OVN metadata with DPDK) | 10:00 |
| zigo | Ok. | 10:00 |
| opendevreview | Pavlo Shchelokovskyy proposed openstack/neutron stable/2025.1: [metadata] cleanup response headers https://review.opendev.org/c/openstack/neutron/+/972451 | 10:55 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add ``path_separator`` to alembic configuration https://review.opendev.org/c/openstack/neutron/+/972453 | 11:06 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Fix list of migration entry points for the ovn-db-sync tool https://review.opendev.org/c/openstack/neutron/+/972454 | 11:17 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-fwaas master: Fix unit and functiona CI jobs https://review.opendev.org/c/openstack/neutron-fwaas/+/972195 | 11:50 |
| ralonsoh | lajoskatona, slaweq haleyb I've proposed https://review.opendev.org/c/openstack/releases/+/972457. That would be needed for stadium project since the tenant_id -> project_id changes | 11:50 |
| ralonsoh | I'm currently working in the n-sfc ones | 11:50 |
| lajoskatona | ralonsoh: thanks | 11:59 |
| opendevreview | Rodolfo Alonso proposed openstack/networking-sfc master: WIP == Replace "tenant_id" with "project_id" in testing https://review.opendev.org/c/openstack/networking-sfc/+/972459 | 12:02 |
| opendevreview | Ashish Gupta proposed openstack/neutron-tempest-plugin master: Enable ip_allocation extension in neutron-tempest-plugin-openvswitch job https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/963287 | 12:13 |
| opendevreview | Rodolfo Alonso proposed openstack/networking-sfc master: WIP == Replace "tenant_id" with "project_id" in testing https://review.opendev.org/c/openstack/networking-sfc/+/972459 | 12:27 |
| slaweq | lajoskatona ykarel haleyb hi, please check https://review.opendev.org/c/openstack/neutron-fwaas/+/972195 when you will have a minute, it fixes neutron-fwaas ci jobs | 13:04 |
| ykarel | done, thx | 13:10 |
| lajoskatona | slaweq: checking | 13:17 |
| haleyb | ralonsoh: sure, will look a little later. there is also https://review.opendev.org/c/openstack/neutron/+/972034 which continues the changing, trying to keep all the reviews on the not-too-big side | 13:39 |
| ralonsoh | haleyb, so we can wait for this new patch | 13:40 |
| haleyb | oh yeah, it can wait | 13:40 |
| ralonsoh | in any case, the stadium projects need the new version at least for testing | 13:40 |
| ralonsoh | once released, we would need to bump the requirements of all of them | 13:41 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add ``path_separator`` to alembic configuration https://review.opendev.org/c/openstack/neutron/+/972453 | 13:47 |
| opendevreview | Merged openstack/neutron-fwaas master: Fix unit and functiona CI jobs https://review.opendev.org/c/openstack/neutron-fwaas/+/972195 | 13:51 |
| opendevreview | Rodolfo Alonso proposed openstack/networking-sfc master: Replace "tenant_id" with "project_id" in testing https://review.opendev.org/c/openstack/networking-sfc/+/972459 | 14:06 |
| opendevreview | Takashi Kajinami proposed openstack/neutron-fwaas master: Remove leftover of FWaaS v1 https://review.opendev.org/c/openstack/neutron-fwaas/+/972485 | 14:18 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: Test FT LP bug 2137467 https://review.opendev.org/c/openstack/neutron/+/972435 | 14:18 |
| opendevreview | Takashi Kajinami proposed openstack/neutron-fwaas master: Remove maintenance status warning https://review.opendev.org/c/openstack/neutron-fwaas/+/972486 | 14:22 |
| opendevreview | Rodolfo Alonso proposed openstack/networking-sfc master: Replace "tenant_id" with "project_id" in testing https://review.opendev.org/c/openstack/networking-sfc/+/972459 | 14:24 |
| ralonsoh | folks ^^ all tests are passing now | 14:24 |
| opendevreview | Renjing Xiao proposed x/whitebox-neutron-tempest-plugin master: Add neutron agents health check https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/971059 | 14:28 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Bump oslo.config to 10.2.0 https://review.opendev.org/c/openstack/neutron/+/972491 | 14:34 |
| ralonsoh | haleyb, slaweq please check https://review.opendev.org/c/openstack/neutron/+/972453. FT job is not longer printing the deprecation warnings | 15:27 |
| opendevreview | Merged openstack/neutron master: Change test code to use project_id key https://review.opendev.org/c/openstack/neutron/+/972034 | 15:40 |
| *** manpreetk_ is now known as manpreetk | 17:30 | |
| opendevreview | Merged openstack/neutron master: Fix list of migration entry points for the ovn-db-sync tool https://review.opendev.org/c/openstack/neutron/+/972454 | 17:33 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: WIP == Detach the stdout and stderr after killing a external process https://review.opendev.org/c/openstack/neutron/+/972538 | 18:52 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Bump os-ken to 4.1.0 https://review.opendev.org/c/openstack/neutron/+/972540 | 18:57 |
| opendevreview | Lajos Katona proposed openstack/os-ken master: [eventlet]: change events to real threading events https://review.opendev.org/c/openstack/os-ken/+/970277 | 19:02 |
| opendevreview | Lajos Katona proposed openstack/neutron master: WIP: [eventlet]: Make ovs-agent stop again https://review.opendev.org/c/openstack/neutron/+/967105 | 19:13 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: Test FT LP bug 2137467 https://review.opendev.org/c/openstack/neutron/+/972435 | 19:44 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: func: Improve BGP IDL lock tests https://review.opendev.org/c/openstack/neutron/+/972547 | 19:44 |
| opendevreview | Merged openstack/neutron master: Add ``path_separator`` to alembic configuration https://review.opendev.org/c/openstack/neutron/+/972453 | 19:58 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: func: Remove MySQL test case from OVN base class https://review.opendev.org/c/openstack/neutron/+/971472 | 20:21 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: func: Remove MySQL test case from OVN base class https://review.opendev.org/c/openstack/neutron/+/971472 | 20:28 |
| opendevreview | Merged openstack/ovn-octavia-provider master: Drop redundant N347 check https://review.opendev.org/c/openstack/ovn-octavia-provider/+/972303 | 20:36 |
| opendevreview | Brian Haley proposed openstack/neutron master: Change agent and Ipam code to use project_id https://review.opendev.org/c/openstack/neutron/+/972589 | 21:54 |
| opendevreview | Brian Haley proposed openstack/neutron master: Change tenant_network_types to project_network_types https://review.opendev.org/c/openstack/neutron/+/972591 | 22:43 |
| opendevreview | Brian Haley proposed openstack/neutron master: Change tenant_network_types to project_network_types https://review.opendev.org/c/openstack/neutron/+/972591 | 23:05 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!