| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Make scheduling of unhosted gateways aware of current transaction https://review.opendev.org/c/openstack/neutron/+/874760 | 08:07 |
|---|---|---|
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Add helper for retrieving LR associated with LRP https://review.opendev.org/c/openstack/neutron/+/873698 | 08:07 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Apply soft anti-affinity for LRs with multiple LRPs when scheduling https://review.opendev.org/c/openstack/neutron/+/873699 | 08:07 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Add support for enable_default_route_bfd attribute https://review.opendev.org/c/openstack/neutron/+/878543 | 08:07 |
| opendevreview | Lajos Katona proposed openstack/neutron master: Add tap_mirror to extension to OVN supported extensions https://review.opendev.org/c/openstack/neutron/+/905840 | 09:32 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2023.2: [OVN] A LRP in an external tunnelled network has no chassis https://review.opendev.org/c/openstack/neutron/+/909191 | 09:40 |
| ralonsoh | slaweq, hello! if you have a couple of mins: https://review.opendev.org/c/openstack/neutron/+/908986 | 09:42 |
| ralonsoh | thanks! | 09:42 |
| ralonsoh | in any case, I would like mlavalle to check this patch too | 09:42 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: WIP === [OVN] Logical Router pinned chassis scheduler https://review.opendev.org/c/openstack/neutron/+/909194 | 10:02 |
| slaweq | <bbezak> "slaweq: change of yours looks..." <- thx for testing it. I still need to fix small issue in fullstack job and we should be good to go with it then | 13:53 |
| haleyb | so i think i will just cancel drivers meeting as i didn't see any response in the one bug https://bugs.launchpad.net/neutron/+bug/2052906 | 14:01 |
| lajoskatona | haleyb: ack | 14:02 |
| haleyb | and nothing on the agenda | 14:02 |
| opendevreview | Merged openstack/neutron stable/2023.1: Change SG rules backref load method to "joined" https://review.opendev.org/c/openstack/neutron/+/908587 | 14:04 |
| slaweq | thx haleyb for info and have a great weekend then :) | 14:05 |
| haleyb | slaweq: you too, i'm going to get some coffee now then :) | 14:06 |
| slaweq | enjoy :) | 14:06 |
| *** tobias-urdin7 is now known as tobias-urdin | 14:07 | |
| ralonsoh | have a nice weekend! | 14:09 |
| haleyb | ralonsoh: you too! | 14:10 |
| haleyb | ralonsoh: did you have a chance to look at https://bugs.launchpad.net/ubuntu/+source/neutron/+bug/2051935 ? | 14:11 |
| * haleyb will be offline for a little bit... | 14:11 | |
| ralonsoh | let me check it | 14:11 |
| *** tobias-urdin0 is now known as tobias-urdin | 14:20 | |
| haleyb | ralonsoh: so you can't ping the 10.0.0.1 IP ? | 14:46 |
| ralonsoh | I can yes but I can't ping to the external GW port IP | 14:46 |
| ralonsoh | I've tested with centralized and DVR routers | 14:47 |
| ralonsoh | (in ML2/OVS) | 14:47 |
| haleyb | ralonsoh: so ml2/ovs or ovn? your note in the bug says ovn | 14:48 |
| ralonsoh | yes but you said this is a regression from OVS | 14:48 |
| ralonsoh | it is not | 14:48 |
| haleyb | my customer is running ml2/ovs with this setup and it works | 14:49 |
| ralonsoh | I've done this twice | 14:49 |
| ralonsoh | there should be something else | 14:49 |
| haleyb | ralonsoh: i will reproduce again. regardless i think it's a bug - the role of a gateway is to nat traffic, it shouldn't be selective based on source IP | 14:54 |
| ralonsoh | but the router NATs traffic from internal network CIDRs to external GW | 14:55 |
| ralonsoh | the traffic the router is receiving belongs to another CIDR | 14:55 |
| ralonsoh | your change is breaking the isolation of router NATs | 14:55 |
| haleyb | in this case it is an internal network cidr as it is configured via the internal interface | 14:55 |
| ralonsoh | the traffic is coming from other CIDR the external routers doesn't know | 14:56 |
| haleyb | ralonsoh: how is it different from a corporate network that has a single external gateway with a /22 comprised of many /24 subnets? it does nat for them all not just the one it has an IP on | 14:58 |
| ralonsoh | no no, I'm not talking about a specific configuration | 14:58 |
| ralonsoh | what I'm saying is that a router should not have any knowledge of other CIDRs not connected to a router port | 14:59 |
| ralonsoh | and there is another caveat in your implementation: it is considering one nested router | 14:59 |
| ralonsoh | but we can have 10, for example | 14:59 |
| haleyb | ralonsoh: i can add more and test, but it should work since it will recursively fine the cidrs. the alternative is to add a single snat rule for 0.0.0.0/0 | 15:00 |
| ralonsoh | but again, I can't make it work in ML2/OVS | 15:01 |
| ralonsoh | I'll repeat with another environment | 15:01 |
| ralonsoh | but so far this is not working in ML2/OVS | 15:02 |
| haleyb | ok, thanks | 15:02 |
| ralonsoh | and there are not tests in tempest | 15:02 |
| ralonsoh | there is a test checking connectivity between two routers but not using GW networks | 15:02 |
| haleyb | i'll ask our ops team as they built and deployed this ml2/ovs cloud | 15:02 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: WIP === [OVN] Logical Router pinned chassis scheduler https://review.opendev.org/c/openstack/neutron/+/909194 | 15:09 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: [S-RBAC] Change policies for port's binding:profile field https://review.opendev.org/c/openstack/neutron/+/909075 | 15:12 |
| opendevreview | Arnaud Morin proposed openstack/neutron master: Allow restoration of tun_ofports on agent restart https://review.opendev.org/c/openstack/neutron/+/860270 | 15:43 |
| opendevreview | Merged openstack/neutron master: [OVN] Add ``ProviderResourceAssociation`` to any ML2/OVN router https://review.opendev.org/c/openstack/neutron/+/908986 | 16:01 |
| haleyb | ralonsoh: i just did an ml2/ovs deployment using my instructions from the bug and it works, ping 8.8.8.8 | 16:48 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Make scheduling of unhosted gateways aware of current transaction https://review.opendev.org/c/openstack/neutron/+/874760 | 16:49 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Add helper for retrieving LR associated with LRP https://review.opendev.org/c/openstack/neutron/+/873698 | 16:49 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Apply soft anti-affinity for LRs with multiple LRPs when scheduling https://review.opendev.org/c/openstack/neutron/+/873699 | 16:49 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Add support for enable_default_route_bfd attribute https://review.opendev.org/c/openstack/neutron/+/878543 | 16:49 |
| opendevreview | Merged openstack/ovn-bgp-agent master: Add documentation about NB DB driver https://review.opendev.org/c/openstack/ovn-bgp-agent/+/903407 | 16:55 |
| opendevreview | Merged openstack/neutron-lib master: rpc: fix some typos https://review.opendev.org/c/openstack/neutron-lib/+/909039 | 18:37 |
| haleyb | slaweq: if you're still around can you look at https://review.opendev.org/c/openstack/neutron/+/902841 ? going through my list of old patches | 18:50 |
| opendevreview | Vasyl Saienko proposed openstack/neutron master: Drop hardcoded dhcp and l3 agent type https://review.opendev.org/c/openstack/neutron/+/908764 | 19:03 |
| opendevreview | Merged openstack/neutron-lib master: rpc: explain how get_notifier arguments actually work https://review.opendev.org/c/openstack/neutron-lib/+/909040 | 19:23 |
| opendevreview | Merged openstack/neutron-lib master: rpc: don't explicitly pass access_policy https://review.opendev.org/c/openstack/neutron-lib/+/909041 | 19:23 |
| opendevreview | Brian Haley proposed openstack/neutron master: [ovn] Apply soft anti-affinity for LRs with multiple LRPs when scheduling https://review.opendev.org/c/openstack/neutron/+/873699 | 20:23 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: WIP: Add documentation for aa-l3-gw-multihoming https://review.opendev.org/c/openstack/neutron/+/899402 | 20:28 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!