Thursday, 2023-12-14

opendevreviewhuanghailun proposed openstack/neutron master: Add allowed_address_pairs add/remove atomic operations  https://review.opendev.org/c/openstack/neutron/+/90189502:46
opendevreviewMiro Tomaska proposed openstack/neutron master: DHCP agent get ovn dhcp port from server  https://review.opendev.org/c/openstack/neutron/+/90357202:49
opendevreviewhuanghailun proposed openstack/neutron master: Add allowed_address_pairs add/remove atomic operations  https://review.opendev.org/c/openstack/neutron/+/90189507:00
opendevreviewRodolfo Alonso proposed openstack/neutron stable/yoga: [DNM] Testing error in "neutron-functional-with-uwsgi-fips" stable/yoga  https://review.opendev.org/c/openstack/neutron/+/90359208:25
opendevreviewMerged openstack/ovn-octavia-provider master: Remove unnecessary ceilometer service overrides  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/90255409:18
opendevreviewLucas Alvares Gomes proposed openstack/neutron stable/2023.1: [OVN] Enhanced external port scheduling  https://review.opendev.org/c/openstack/neutron/+/90351009:31
opendevreviewRodolfo Alonso proposed openstack/neutron stable/yoga: [DNM] Testing error in "neutron-functional-with-uwsgi-fips" stable/yoga  https://review.opendev.org/c/openstack/neutron/+/90359209:48
opendevreviewhuanghailun proposed openstack/neutron master: Add allowed_address_pairs add/remove atomic operations  https://review.opendev.org/c/openstack/neutron/+/90189509:48
opendevreviewRodolfo Alonso proposed openstack/neutron stable/yoga: [DNM] Testing error in "neutron-functional-with-uwsgi-fips" stable/yoga  https://review.opendev.org/c/openstack/neutron/+/90359211:04
opendevreviewLucas Alvares Gomes proposed openstack/neutron master: Fix IGMP inconsistency across drivers  https://review.opendev.org/c/openstack/neutron/+/90175311:48
opendevreviewLuis Tomas Bolivar proposed openstack/ovn-bgp-agent master: Add documentation about NB DB driver  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/90340712:00
opendevreviewLuis Tomas Bolivar proposed openstack/ovn-bgp-agent master: Fix watcher logic for exposing OVN LB with t he NB DB driver  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/90340812:02
ContinuityIs anyone aware of anyone working on OVN+DVR and VLAN tenant networks.12:05
Continuityhttps://bugs.launchpad.net/neutron/+bug/203533212:05
ContinuityWe have found a workaround of sorts listed in the bug above, but would be good to understand if anyone is looking at OVN+DVR and VLAN networks12:05
fricklerContinuity: seems that that bug was missed by the usual bug triaging. ralonsoh maybe you can take a look?12:18
ralonsohI was doing that now12:18
ralonsohContinuity, please check https://review.opendev.org/c/openstack/neutron/+/87125212:18
Continuity:D12:18
Continuitycheers12:18
fricklerjinx ;)12:18
ContinuitySo we have that patched deployed, we still end up in the same situation as explained in the bug i linked12:19
ralonsohContinuity, if you check the commit message, the 3rd bullet point12:20
ralonsoh"reside-on-redirect-chassis to False, but with ovn-chassis-mac-mappings12:20
ralonsoh  configured"12:20
ralonsohthis is for VLAN networks with DVR12:20
ralonsohltomasbo, ^^ some help here?12:21
ralonsohah, https://review.opendev.org/c/openstack/neutron/+/878450 should be considered too (but I see this flag in the bug report)12:22
Continuityyeah this is the issue i have had, its a bit of a mess, and hard to work out :D12:23
ralonsohdo you have the mac mappings configured (in each compute node)?12:23
Continuityyes12:23
Continuityi believe so12:23
ralonsohnow it is not possible (with DVR) to mix vlan/flat and tunnelled networks12:23
ralonsohthe configuration combinations are strict on this12:23
ContinuitySo we have a combo that works, but requires a change after the networks are deployed12:24
Continuitywhich is explained in the bug i linked above12:24
Continuityi mean it might be working through luck.... :D12:24
ralonsohContinuity, why do you change the chassis priority?12:26
ralonsohthis is done by Neutron12:26
ralonsohshould not be modified manually12:26
ralonsoh"New routers with vlan networks attached don't create all three ports on the controllers." What does it means (sorry, i don't get it)12:27
ContinuitySo each of the controllers should have a chassis_gateway created, sometimes (not sure why) those chassis_gateways were not created correctly on all three controllers12:28
Continuitythen the next issue we had after that was that the ha-chassis groups didnt match the chassis_gateways12:29
Continuitywhich is why we had to modify them to get them into the correct order12:29
Continuitythe first two issues may have been related to patches that have been fixed since we were troubleshooting this12:29
ralonsohwhat patches?12:30
ralonsohbecause what you are saying is that the OVN L3 scheduler is maybe not working fine12:30
ralonsoh(and that's a problem for sure)12:30
ralonsohI would focus first on find why, when you create a LRP, the OVN L3 scheduler is not creatign the corresponding 3 Gateway_Chassis registers12:31
Continuityyes, we have had an issues with the OVN L3 scheduler, but since we had those issues, we have patched, and I remember seeing patches that addressed those issues12:32
Continuityralonsoh: i will try building some new networks in the next day or so, and test that 12:32
ralonsohwhat patches? just to know12:32
Continuityer.... :D12:32
ralonsohif possible, please add extra logs in the code12:32
ralonsohand print them, please12:32
Continuitywill do12:32
ralonsohthat could help us a lot12:32
Continuityif we are hacing issues with the OVN l3 scheduler Ill open another bug report and place that data in there, and split the issues in to smaller bugs, rather than it all being in oen big one.12:34
stephenfinslaweq: ralonsoh: Could one of you sanity check https://review.opendev.org/c/openstack/python-openstackclient/+/903672 when you have time?12:34
ralonsohContinuity, perfect12:34
ralonsohstephenfin, let me check12:34
Continuitynp12:34
stephenfinralonsoh: Thanks12:43
stephenfinI also recently rewrote the SG docs for nova. You can find them at https://docs.openstack.org/nova/latest/user/security-groups.html Not remotely urgent but if you have time to take a skim through and sanity check those also it would be helpful to all12:43
ykarelContinuity, your external network is also of vlan type?13:10
Continuityykarel: yes13:11
ykarelContinuity, thx for confirmation13:12
opendevreviewLuis Tomas Bolivar proposed openstack/ovn-bgp-agent master: Fix watcher logic for exposing OVN LB with t he NB DB driver  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/90340813:46
ltomasbosorry ralonsoh I was having lunch and then meeting... reading now13:49
ltomasboContinuity, ralonsoh there was an issue with FIPs at core ovn when redirect-type=bridge was used13:53
ralonsohltomasbo, is there a LP bug? because so far, I though that option was working13:54
ralonsohso is it possible right now to use VLAN + DVR?13:54
ltomasboContinuity, perhaps related to this one? https://bugzilla.redhat.com/show_bug.cgi?id=200712013:54
ltomasbothis was fixed, but pernaps it is not in the ovn version used13:55
ralonsohokidoki13:55
ltomasboContinuity, ralonsoh: from current experience, the real distribution of provider tenant vlan networks is with the redirect-type=bridge flag, the reside-on-redirect-chassis is still centralized, the different between true and false is if it goes tunneled or not to the gateway port14:01
Continuitythanks14:02
opendevreviewLucas Alvares Gomes proposed openstack/neutron master: Fix IGMP inconsistency across drivers  https://review.opendev.org/c/openstack/neutron/+/90175314:11
*** haleyb|out is now known as haleyb14:24
opendevreviewLuis Tomas Bolivar proposed openstack/ovn-bgp-agent master: Add documentation about NB DB driver  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/90340714:34
opendevreviewRodolfo Alonso proposed openstack/neutron stable/yoga: [stable-only] Workaround in ``TestPortBindingChassisEvent``  https://review.opendev.org/c/openstack/neutron/+/90359214:55
ralonsohbcafarel, ^^ if you have time15:18
ralonsohykarel, 15:18
ykarelralonsoh, thx, you can drop those Log imports too15:21
ralonsohahh yes, sorry15:26
opendevreviewRodolfo Alonso proposed openstack/neutron stable/yoga: [stable-only] Workaround in ``TestPortBindingChassisEvent``  https://review.opendev.org/c/openstack/neutron/+/90359215:26
opendevreviewHarald Jensås proposed openstack/neutron stable/2023.2: [OVN] DB sync host/physnet - filter on agent_type  https://review.opendev.org/c/openstack/neutron/+/90368215:29
opendevreviewHarald Jensås proposed openstack/neutron stable/2023.1: [OVN] DB sync host/physnet - filter on agent_type  https://review.opendev.org/c/openstack/neutron/+/90368315:29
*** jamesdenton_ is now known as jamesdenton15:37
opendevreviewLuis Tomas Bolivar proposed openstack/ovn-bgp-agent master: Add documentation about NB DB driver  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/90340715:56
opendevreviewRodolfo Alonso proposed openstack/neutron master: Improve the SG RPC callback ``security_group_info_for_ports``  https://review.opendev.org/c/openstack/neutron/+/90370715:59
opendevreviewLuis Tomas Bolivar proposed openstack/ovn-bgp-agent master: Fix watcher logic for exposing OVN LB with the NB DB driver  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/90340815:59
opendevreviewRodolfo Alonso proposed openstack/neutron master: Improve the SG RPC callback ``security_group_info_for_ports``  https://review.opendev.org/c/openstack/neutron/+/90370716:00
opendevreviewMerged openstack/ovn-bgp-agent master: Fix watcher logic for exposing OVN LB with the NB DB driver  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/90340816:17
opendevreviewMerged openstack/neutron stable/zed: get_hosts_mapped_with_segments add filter agt_type  https://review.opendev.org/c/openstack/neutron/+/90179019:16
opendevreviewHarald Jensås proposed openstack/neutron stable/zed: [OVN] DB sync host/physnet - filter on agent_type  https://review.opendev.org/c/openstack/neutron/+/90368521:16
*** dmellado2 is now known as dmelladoo21:55
*** dmelladoo is now known as dmellado21:58
opendevreviewMerged openstack/neutron master: [OVN] Fix UT assertion  https://review.opendev.org/c/openstack/neutron/+/90358422:31

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!