Monday, 2023-10-02

Continuity__ralonsoh: WRT https://bugs.launchpad.net/neutron/+bug/2035332 happy to put you in touch with my engineer, to discuss the issues we are seeing. 08:03
*** Continuity__ is now known as Continuity08:03
Continuityor if you have any additional requirements, can get the LP updated08:17
opendevreviewFelix Huettner proposed openstack/neutron master: fix netns deletion of broken namespaces  https://review.opendev.org/c/openstack/neutron/+/89625109:41
opendevreviewArkady Shtempler proposed openstack/neutron-tempest-plugin master: Set segmentation ID outside the ranges assigned to project network  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/89450310:15
*** Continuity__ is now known as Continuity10:24
*** damian___921839 is now known as damian___9218310:59
opendevreviewFelix Huettner proposed openstack/neutron master: ovn-l3 scheduler: calculate load of chassis per priority  https://review.opendev.org/c/openstack/neutron/+/89365311:13
opendevreviewFelix Huettner proposed openstack/neutron master: ovn-l3: reschedule lower priorities  https://review.opendev.org/c/openstack/neutron/+/89365411:13
opendevreviewFelix Huettner proposed openstack/neutron master: ovn-l3: reschedule priorities on new chassis  https://review.opendev.org/c/openstack/neutron/+/89365511:13
opendevreviewFelix Huettner proposed openstack/neutron master: ovn-l3 router scheduler: reproduce scheduling issue  https://review.opendev.org/c/openstack/neutron/+/89365611:13
opendevreviewFelix Huettner proposed openstack/neutron master: ovn-l3: implement caching for Scheduler  https://review.opendev.org/c/openstack/neutron/+/89365711:13
opendevreviewFelix Huettner proposed openstack/neutron master: ovn-l3: try to keep routers at current chassis  https://review.opendev.org/c/openstack/neutron/+/89365811:13
opendevreviewFelix Huettner proposed openstack/neutron master: ovn-l3: value the higher priorities when scheduling  https://review.opendev.org/c/openstack/neutron/+/89365911:13
bbezakHi - it looks like manual ovn sync removes metadata port when there is already ovn lb health monitor port in the network - https://bugs.launchpad.net/neutron/+bug/203809112:44
bbezakI'm wondering if ovn octavia provider should use different owner for health monitor port then metadata port12:44
ralonsohfroyo, didn't you change the port type for this ^^^12:45
ralonsoh?12:45
ralonsoh(maybe I'm wrong)12:45
froyoralonsoh, yeah12:45
froyoralonsoh: you are correct12:45
ralonsohis that backported up to Yoga?12:45
froyosure, until wallaby indeed12:45
ralonsohbbezak, please check the version you are using (of octavia)12:46
froyo^^  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/874273 12:47
bbezakyeah, I've seen this bug. let me triple check my octavia version :)12:47
ralonsohslaweq, please, if you have 30 secs: https://review.opendev.org/c/openstack/neutron/+/89692612:51
ralonsohlajoskatona, ^12:51
ralonsohthen I'll backport it up to Wallaby12:51
opendevreviewRodolfo Alonso proposed openstack/neutron stable/2023.2: [OVN] Add the default condition check in ``PortBindingChassisEvent``  https://review.opendev.org/c/openstack/neutron/+/89701912:52
opendevreviewRodolfo Alonso proposed openstack/neutron stable/2023.1: [OVN] Add the default condition check in ``PortBindingChassisEvent``  https://review.opendev.org/c/openstack/neutron/+/89702012:52
opendevreviewRodolfo Alonso proposed openstack/neutron stable/zed: [OVN] Add the default condition check in ``PortBindingChassisEvent``  https://review.opendev.org/c/openstack/neutron/+/89704112:53
bbezakfroyo: well, my HM port is named ovn-lb-hm-*, so I this patch is already there12:54
opendevreviewMerged openstack/os-vif stable/2023.2: Update TOX_CONSTRAINTS_FILE for stable/2023.2  https://review.opendev.org/c/openstack/os-vif/+/89406212:55
froyobbezak yeah that is a good check, so I will take a look to your new launchpad, looks related to neutron-sync tool doing something weird ...12:56
bbezakyeah, thx froyo, last comment in the bug from mnasiadka is useful12:57
froyoack12:57
ltomasbobbezak, froyo: I think the problem is now different... what ralonsoh was referring to was the patch to avoid using the metadata port for ovn-lb (by creating a different one), now it seems that this different one is not being used properly12:57
lajoskatonaralonsoh: checking12:58
ltomasbobbezak, froyo so, probably the neutron filters need to be updated to not use that port12:58
froyoltomasbo, looks like searching by network:distributed owner, but anyway I will take a look to check it12:59
opendevreviewRodolfo Alonso proposed openstack/neutron stable/xena: [OVN] Add the default condition check in ``PortBindingChassisEvent``  https://review.opendev.org/c/openstack/neutron/+/89704313:07
opendevreviewRodolfo Alonso proposed openstack/neutron stable/yoga: [OVN] Add the default condition check in ``PortBindingChassisEvent``  https://review.opendev.org/c/openstack/neutron/+/89704413:07
opendevreviewRodolfo Alonso proposed openstack/neutron stable/wallaby: [OVN] Add the default condition check in ``PortBindingChassisEvent``  https://review.opendev.org/c/openstack/neutron/+/89704513:07
mnasiadkafroyo: I don't know if it's not easier to change the device owner in ovn-octavia-provider to something like network:lbaas13:08
froyomnasiadka, yeah it could be a good solution no use the network:distributed, replacing by octavia:XXXX or network:XXXX13:10
mnasiadkafroyo: because most probably the network:distributed is used in dhcp entries for metadata, neutron-metadata-agent itself and the sync ;)13:10
froyo^^ https://github.com/openstack/ovn-octavia-provider/blob/983ee0c4f31413904b2f1c2b3a655a32cba70401/ovn_octavia_provider/helper.py#L14313:11
froyoI will push a patch in some minutes13:11
mnasiadkafroyo: thanks :)13:15
opendevreviewLajos Katona proposed openstack/tap-as-a-service master: docs: Fix releasenote building  https://review.opendev.org/c/openstack/tap-as-a-service/+/89695613:16
opendevreviewArkady Shtempler proposed openstack/neutron-tempest-plugin master: Set segmentation ID outside the ranges assigned to project network  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/89450313:29
*** haleyb_out is now known as haleyb13:33
haleybralonsoh: just wanted to verify we should take mnaser's backport in 2023.2, https://review.opendev.org/c/openstack/releases/+/896971/1/deliverables/bobcat/neutron.yaml is one commit out of date14:18
ralonsohhaleyb, which one?14:19
mnaseri would argue trunks should be shipped broken if its not there14:19
haleyb[OVN] Match LSP_TYPE_VIRTUAL in PortBindingUpdateVirtualPortsEvent14:19
mnasers/should/would/14:19
haleyblet me find the change14:19
haleybhttps://review.opendev.org/c/openstack/neutron/+/89680314:19
ralonsohyes, let me check the hashes 14:19
ralonsohhaleyb, let me update the releases patch14:21
haleybralonsoh: ack, and i didn't check all the other hashes yet in a meeting will check after14:21
ralonsohI think the other hashes are ok, I've checked them now14:26
ralonsohbut a second pair of eyes is always welcome14:26
opendevreviewLajos Katona proposed openstack/networking-bgpvpn master: DNM: test master  https://review.opendev.org/c/openstack/networking-bgpvpn/+/89705115:04
opendevreviewMiro Tomaska proposed openstack/neutron master: OVN Metadata handle process execeptions  https://review.opendev.org/c/openstack/neutron/+/89098615:07
opendevreviewRodolfo Alonso proposed openstack/neutron master: [OVN] Set the Neutron port status based on "lsp.up" and "lsp.enabled"  https://review.opendev.org/c/openstack/neutron/+/89693916:23
mnaserhas anyone seen issues with port forwarding + ovn over the past little bit?16:50
mnaseri have a case where traffic isnt showing up to the vm tap... load_balancer is there, it's also assigned to the logical_switch at this point too16:50
mnaserbut i dont see a `Logical_Switch_Port` anywhere with the fip that is being forwarded16:53
mnaserhrm, i see it in Logical_Router too17:00
admin1are you able to see it in the flows  mnaser 18:18
mnaseradmin1: yep.  i can see flows for it too18:18
admin1the use case is ovn lb  pointing to backend pools not working ? 18:19
opendevreviewBrian Haley proposed openstack/neutron master: DNM: Always stop dhclient in fullstack tests  https://review.opendev.org/c/openstack/neutron/+/89683918:31
racosta[OVN LB + ARP issue] maybe this link can help you mnaser: https://github.com/ovn-org/ovn/issues/124  /  https://mail.openvswitch.org/pipermail/ovs-dev/2022-April/393627.html18:32
mnaserracosta: interesting, running 22.03.2 here, let me see18:34
opendevreviewBrian Haley proposed openstack/neutron master: DNM: Always stop dhclient in fullstack tests  https://review.opendev.org/c/openstack/neutron/+/89683918:43
opendevreviewBrian Haley proposed openstack/neutron master: DNM: Always stop dhclient in fullstack tests  https://review.opendev.org/c/openstack/neutron/+/89683919:24
opendevreviewBrian Haley proposed openstack/neutron master: DNM: Always stop dhclient in fullstack tests  https://review.opendev.org/c/openstack/neutron/+/89683921:36
opendevreviewTerry Wilson proposed openstack/ovsdbapp master: Handle events with conditions and match_fn  https://review.opendev.org/c/openstack/ovsdbapp/+/89709022:50

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