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 Continuity | 08:03 | |
Continuity | or if you have any additional requirements, can get the LP updated | 08:17 |
opendevreview | Felix Huettner proposed openstack/neutron master: fix netns deletion of broken namespaces https://review.opendev.org/c/openstack/neutron/+/896251 | 09:41 |
opendevreview | Arkady 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/+/894503 | 10:15 |
*** Continuity__ is now known as Continuity | 10:24 | |
*** damian___921839 is now known as damian___92183 | 10:59 | |
opendevreview | Felix Huettner proposed openstack/neutron master: ovn-l3 scheduler: calculate load of chassis per priority https://review.opendev.org/c/openstack/neutron/+/893653 | 11:13 |
opendevreview | Felix Huettner proposed openstack/neutron master: ovn-l3: reschedule lower priorities https://review.opendev.org/c/openstack/neutron/+/893654 | 11:13 |
opendevreview | Felix Huettner proposed openstack/neutron master: ovn-l3: reschedule priorities on new chassis https://review.opendev.org/c/openstack/neutron/+/893655 | 11:13 |
opendevreview | Felix Huettner proposed openstack/neutron master: ovn-l3 router scheduler: reproduce scheduling issue https://review.opendev.org/c/openstack/neutron/+/893656 | 11:13 |
opendevreview | Felix Huettner proposed openstack/neutron master: ovn-l3: implement caching for Scheduler https://review.opendev.org/c/openstack/neutron/+/893657 | 11:13 |
opendevreview | Felix Huettner proposed openstack/neutron master: ovn-l3: try to keep routers at current chassis https://review.opendev.org/c/openstack/neutron/+/893658 | 11:13 |
opendevreview | Felix Huettner proposed openstack/neutron master: ovn-l3: value the higher priorities when scheduling https://review.opendev.org/c/openstack/neutron/+/893659 | 11:13 |
bbezak | Hi - 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/2038091 | 12:44 |
bbezak | I'm wondering if ovn octavia provider should use different owner for health monitor port then metadata port | 12:44 |
ralonsoh | froyo, didn't you change the port type for this ^^^ | 12:45 |
ralonsoh | ? | 12:45 |
ralonsoh | (maybe I'm wrong) | 12:45 |
froyo | ralonsoh, yeah | 12:45 |
froyo | ralonsoh: you are correct | 12:45 |
ralonsoh | is that backported up to Yoga? | 12:45 |
froyo | sure, until wallaby indeed | 12:45 |
ralonsoh | bbezak, please check the version you are using (of octavia) | 12:46 |
froyo | ^^ https://review.opendev.org/c/openstack/ovn-octavia-provider/+/874273 | 12:47 |
bbezak | yeah, I've seen this bug. let me triple check my octavia version :) | 12:47 |
ralonsoh | slaweq, please, if you have 30 secs: https://review.opendev.org/c/openstack/neutron/+/896926 | 12:51 |
ralonsoh | lajoskatona, ^ | 12:51 |
ralonsoh | then I'll backport it up to Wallaby | 12:51 |
opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2023.2: [OVN] Add the default condition check in ``PortBindingChassisEvent`` https://review.opendev.org/c/openstack/neutron/+/897019 | 12:52 |
opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2023.1: [OVN] Add the default condition check in ``PortBindingChassisEvent`` https://review.opendev.org/c/openstack/neutron/+/897020 | 12:52 |
opendevreview | Rodolfo Alonso proposed openstack/neutron stable/zed: [OVN] Add the default condition check in ``PortBindingChassisEvent`` https://review.opendev.org/c/openstack/neutron/+/897041 | 12:53 |
bbezak | froyo: well, my HM port is named ovn-lb-hm-*, so I this patch is already there | 12:54 |
opendevreview | Merged openstack/os-vif stable/2023.2: Update TOX_CONSTRAINTS_FILE for stable/2023.2 https://review.opendev.org/c/openstack/os-vif/+/894062 | 12:55 |
froyo | bbezak 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 |
bbezak | yeah, thx froyo, last comment in the bug from mnasiadka is useful | 12:57 |
froyo | ack | 12:57 |
ltomasbo | bbezak, 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 properly | 12:57 |
lajoskatona | ralonsoh: checking | 12:58 |
ltomasbo | bbezak, froyo so, probably the neutron filters need to be updated to not use that port | 12:58 |
froyo | ltomasbo, looks like searching by network:distributed owner, but anyway I will take a look to check it | 12:59 |
opendevreview | Rodolfo Alonso proposed openstack/neutron stable/xena: [OVN] Add the default condition check in ``PortBindingChassisEvent`` https://review.opendev.org/c/openstack/neutron/+/897043 | 13:07 |
opendevreview | Rodolfo Alonso proposed openstack/neutron stable/yoga: [OVN] Add the default condition check in ``PortBindingChassisEvent`` https://review.opendev.org/c/openstack/neutron/+/897044 | 13:07 |
opendevreview | Rodolfo Alonso proposed openstack/neutron stable/wallaby: [OVN] Add the default condition check in ``PortBindingChassisEvent`` https://review.opendev.org/c/openstack/neutron/+/897045 | 13:07 |
mnasiadka | froyo: I don't know if it's not easier to change the device owner in ovn-octavia-provider to something like network:lbaas | 13:08 |
froyo | mnasiadka, yeah it could be a good solution no use the network:distributed, replacing by octavia:XXXX or network:XXXX | 13:10 |
mnasiadka | froyo: 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#L143 | 13:11 |
froyo | I will push a patch in some minutes | 13:11 |
mnasiadka | froyo: thanks :) | 13:15 |
opendevreview | Lajos Katona proposed openstack/tap-as-a-service master: docs: Fix releasenote building https://review.opendev.org/c/openstack/tap-as-a-service/+/896956 | 13:16 |
opendevreview | Arkady 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/+/894503 | 13:29 |
*** haleyb_out is now known as haleyb | 13:33 | |
haleyb | ralonsoh: 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 date | 14:18 |
ralonsoh | haleyb, which one? | 14:19 |
mnaser | i would argue trunks should be shipped broken if its not there | 14:19 |
haleyb | [OVN] Match LSP_TYPE_VIRTUAL in PortBindingUpdateVirtualPortsEvent | 14:19 |
mnaser | s/should/would/ | 14:19 |
haleyb | let me find the change | 14:19 |
haleyb | https://review.opendev.org/c/openstack/neutron/+/896803 | 14:19 |
ralonsoh | yes, let me check the hashes | 14:19 |
ralonsoh | haleyb, let me update the releases patch | 14:21 |
haleyb | ralonsoh: ack, and i didn't check all the other hashes yet in a meeting will check after | 14:21 |
ralonsoh | I think the other hashes are ok, I've checked them now | 14:26 |
ralonsoh | but a second pair of eyes is always welcome | 14:26 |
opendevreview | Lajos Katona proposed openstack/networking-bgpvpn master: DNM: test master https://review.opendev.org/c/openstack/networking-bgpvpn/+/897051 | 15:04 |
opendevreview | Miro Tomaska proposed openstack/neutron master: OVN Metadata handle process execeptions https://review.opendev.org/c/openstack/neutron/+/890986 | 15:07 |
opendevreview | Rodolfo 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/+/896939 | 16:23 |
mnaser | has anyone seen issues with port forwarding + ovn over the past little bit? | 16:50 |
mnaser | i 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 too | 16:50 |
mnaser | but i dont see a `Logical_Switch_Port` anywhere with the fip that is being forwarded | 16:53 |
mnaser | hrm, i see it in Logical_Router too | 17:00 |
admin1 | are you able to see it in the flows mnaser | 18:18 |
mnaser | admin1: yep. i can see flows for it too | 18:18 |
admin1 | the use case is ovn lb pointing to backend pools not working ? | 18:19 |
opendevreview | Brian Haley proposed openstack/neutron master: DNM: Always stop dhclient in fullstack tests https://review.opendev.org/c/openstack/neutron/+/896839 | 18: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.html | 18:32 |
mnaser | racosta: interesting, running 22.03.2 here, let me see | 18:34 |
opendevreview | Brian Haley proposed openstack/neutron master: DNM: Always stop dhclient in fullstack tests https://review.opendev.org/c/openstack/neutron/+/896839 | 18:43 |
opendevreview | Brian Haley proposed openstack/neutron master: DNM: Always stop dhclient in fullstack tests https://review.opendev.org/c/openstack/neutron/+/896839 | 19:24 |
opendevreview | Brian Haley proposed openstack/neutron master: DNM: Always stop dhclient in fullstack tests https://review.opendev.org/c/openstack/neutron/+/896839 | 21:36 |
opendevreview | Terry Wilson proposed openstack/ovsdbapp master: Handle events with conditions and match_fn https://review.opendev.org/c/openstack/ovsdbapp/+/897090 | 22:50 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!