| opendevreview | Merged openstack/neutron master: [OVN] Release note for ``neutron-ovn-maintenance-worker`` process https://review.opendev.org/c/openstack/neutron/+/923198 | 06:26 |
|---|---|---|
| opendevreview | Brian Haley proposed openstack/neutron master: Release note for ``neutron-periodic-workers`` process https://review.opendev.org/c/openstack/neutron/+/923199 | 06:35 |
| ralonsoh | I always forget that: what is the channel for the devstack project? | 06:39 |
| opendevreview | Merged openstack/neutron stable/2023.1: Add debug information to ``MacvtapAgentTestCase.test_get_all_devices`` https://review.opendev.org/c/openstack/neutron/+/923399 | 06:46 |
| frickler | ralonsoh: #openstack-qa | 06:52 |
| ralonsoh | thanks | 06:52 |
| opendevreview | Merged openstack/neutron stable/2023.1: Return empty BpInfo if missing binding:profile https://review.opendev.org/c/openstack/neutron/+/923476 | 08:28 |
| opendevreview | Merged openstack/neutron master: Release note for ``neutron-periodic-workers`` process https://review.opendev.org/c/openstack/neutron/+/923199 | 08:28 |
| opendevreview | Merged openstack/neutron stable/2024.1: Change to use selectin for RBACs in SubnetPool DB load strategy https://review.opendev.org/c/openstack/neutron/+/923495 | 08:44 |
| opendevreview | Darin Chakalov proposed openstack/neutron-fwaas master: Enable FWaaS DVR setup support https://review.opendev.org/c/openstack/neutron-fwaas/+/922963 | 08:45 |
| opendevreview | Merged openstack/neutron stable/2023.2: Change to use selectin for RBACs in SubnetPool DB load strategy https://review.opendev.org/c/openstack/neutron/+/923496 | 08:59 |
| Ali_Safari | Hi all, can anyone take look at these backports? | 09:26 |
| Ali_Safari | https://review.opendev.org/c/openstack/ovn-octavia-provider/+/921383 | 09:26 |
| Ali_Safari | https://review.opendev.org/c/openstack/ovn-octavia-provider/+/921384 | 09:26 |
| Ali_Safari | https://review.opendev.org/c/openstack/ovn-octavia-provider/+/921385 | 09:27 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [DHCP] Lock the execution of ``_dhcp_ready_ports_loop`` https://review.opendev.org/c/openstack/neutron/+/923625 | 10:07 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [DHCP] Reduce to one single thread the event processing https://review.opendev.org/c/openstack/neutron/+/923626 | 10:07 |
| opendevreview | Merged openstack/ovn-octavia-provider stable/2024.1: Don't create an OVSDB connection per API request https://review.opendev.org/c/openstack/ovn-octavia-provider/+/921383 | 10:23 |
| opendevreview | Merged openstack/ovn-octavia-provider stable/2023.2: Don't create an OVSDB connection per API request https://review.opendev.org/c/openstack/ovn-octavia-provider/+/921384 | 10:29 |
| opendevreview | Merged openstack/ovn-octavia-provider stable/2023.1: Don't create an OVSDB connection per API request https://review.opendev.org/c/openstack/ovn-octavia-provider/+/921385 | 10:29 |
| opendevreview | Merged openstack/neutron stable/2023.2: [RBAC] Update the subnet policies https://review.opendev.org/c/openstack/neutron/+/923489 | 10:33 |
| opendevreview | Merged openstack/neutron master: [FT] Add a timeout for the NB/SB connection stop method https://review.opendev.org/c/openstack/neutron/+/923550 | 10:34 |
| opendevreview | Merged openstack/neutron stable/2024.1: Add the port "fixed_ips" information in the DHCP RPC https://review.opendev.org/c/openstack/neutron/+/923561 | 10:34 |
| opendevreview | Merged openstack/neutron stable/2023.2: Add the port "fixed_ips" information in the DHCP RPC https://review.opendev.org/c/openstack/neutron/+/923562 | 10:34 |
| opendevreview | Merged openstack/neutron stable/2023.1: Add the port "fixed_ips" information in the DHCP RPC https://review.opendev.org/c/openstack/neutron/+/923563 | 10:34 |
| opendevreview | Merged openstack/neutron stable/2023.1: Increase timeout in test_get_all_devices() https://review.opendev.org/c/openstack/neutron/+/923400 | 10:34 |
| opendevreview | Darin Chakalov proposed openstack/neutron-fwaas master: Enable FWaaS DVR setup support https://review.opendev.org/c/openstack/neutron-fwaas/+/922963 | 10:41 |
| *** ykarel__ is now known as ykarel | 10:54 | |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [DHCP] Reduce to one single thread the event processing https://review.opendev.org/c/openstack/neutron/+/923626 | 11:03 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [DHCP] Skip metadata proxy timeout in unit testing https://review.opendev.org/c/openstack/neutron/+/923633 | 11:25 |
| ralonsoh | haleyb|out, hi! please check the following n-lib patches. Once merged, I'll propose a new release | 11:32 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron-lib/+/921649 | 11:32 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron-lib/+/923240 | 11:32 |
| ralonsoh | thanks! | 11:32 |
| ralonsoh | lajoskatona, ^ | 11:33 |
| lajoskatona | ralonsoh: checking | 12:00 |
| racosta | Hey ralonsoh, good morning/afternoon. Can you please take a look at my last comments here: https://review.opendev.org/c/openstack/neutron/+/907489 | 12:25 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Adding manager role support https://review.opendev.org/c/openstack/neutron/+/923578 | 12:30 |
| ralonsoh | racosta, let me check | 12:38 |
| *** haleyb|out is now known as haleyb | 14:00 | |
| opendevreview | Brian Haley proposed openstack/neutron-lib master: Don't add tenant_id to all resources during create request https://review.opendev.org/c/openstack/neutron-lib/+/921649 | 14:25 |
| haleyb | ralonsoh: i just had a relnote update for that ^^ but it depends on three neutron changes that have not merged | 14:26 |
| ralonsoh | right, this one will be more complex | 14:26 |
| ralonsoh | anyway, we can wait a bit | 14:26 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Adding manager role support https://review.opendev.org/c/openstack/neutron/+/923578 | 14:52 |
| otherwiseguy | haleyb: my apologies again on the nested issue. Not sure how much work deleting the NAT entries at the right times is going to be. | 15:18 |
| otherwiseguy | Another reason I really wish the 0.0.0.0/0 thing worked, but talking to the OVN folks, seems like pretty much no chance of that for backport since the thing that currently *kind of* makes it work is a new feature just merged a month ago. | 15:20 |
| haleyb | otherwiseguy: when i tried many many patches ago, un-applying the patch and re-running the sync tool seemed to remove the extra rules. so maybe i just missed something when i added the flag | 15:20 |
| otherwiseguy | haleyb: commented on the signal handler backport, the patch currently bumps the version to the stable/2024.1 release of oslo.service which has the backport (which I think is what you suggested). Looks like the version referenced for jammy was oslo.service from yoga even though jammy supports up to bobcat. I won't be heartbroken if we can't work out backports for the fix, but it would be nice. | 15:23 |
| otherwiseguy | haleyb: what are the cases where the NAT should be deleted via actions taken by API requests? Removing route connecting routers? Removing subnet from router? | 15:24 |
| haleyb | otherwiseguy: using a backported stable version should be doable, i just have not got to my review folder yet after vaction :( | 15:25 |
| otherwiseguy | No problem at all. I'm not in a huge hurry at all. I also felt bad about reviewing and finding the delete issue right before the 4th. After clicking submit I thought "I should have sent that Monday." :p | 15:26 |
| haleyb | otherwiseguy: and i haven't completely read your response yet to the patch, so don't have context for above, but it should only be updating the snat entries imo, not changing anything else | 15:27 |
| haleyb | hot dogs, apple pie, fireworks, and OVN, what a fourth! :) | 15:27 |
| otherwiseguy | Basically SNAT entries get added when they are supposed to (at least in the simple case from running your reproducer from LP), but they just never get deleted even after the nested router and its interfaces are deleted. | 15:29 |
| otherwiseguy | So the NAT table would grow indefinitely, etc. | 15:30 |
| haleyb | otherwiseguy: oh, that's not the problem i thought it was | 15:30 |
| otherwiseguy | So I'm not exactly sure what all of the API actions that could occur that would make a router no longer nested, and therefor no longer needing a NAT entry. Removing the route that connects the routers? Deleting the subnet from the nested router, etc. | 15:32 |
| otherwiseguy | So not sure how much work keeping the NAT entries in sync w/ the actual deployed routers is going to be. | 15:33 |
| otherwiseguy | But it seems like it could be at least a little messy. | 15:34 |
| haleyb | deleting the interface from the parent "upstream" subnet should make the router "unfound", i didn't look at the routes | 15:35 |
| haleyb | also removing the subnet from the nested router | 15:35 |
| otherwiseguy | The code looked like the only time it deleted NAT entries is if "enable_snat=False" was passed to ...some function (looking) | 15:36 |
| otherwiseguy | update_nat_rules() | 15:36 |
| * haleyb can look after meeting(s) | 15:37 | |
| opendevreview | Merged openstack/neutron master: Checking pci_slot to avoid changing staus to BUILD forever https://review.opendev.org/c/openstack/neutron/+/923467 | 15:42 |
| opendevreview | Merged openstack/neutron-lib master: api-ref: Add delete port bindings https://review.opendev.org/c/openstack/neutron-lib/+/923240 | 16:47 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!