| opendevreview | Merged openstack/neutron stable/wallaby: [OVN] Cleanup old Hash Ring node entries https://review.opendev.org/c/openstack/neutron/+/893144 | 02:04 |
|---|---|---|
| slaweq | ralonsoh hi, I just commented in https://review.opendev.org/c/openstack/neutron/+/884474, please check it and let me know if this is ok for You :) | 07:18 |
| *** ralonsoh_ooo is now known as ralonsoh | 07:18 | |
| ralonsoh | slaweq, sure, let me check | 07:18 |
| frickler | ralonsoh: did you see https://bugs.launchpad.net/neutron/+bug/2033508 ? looks to me like a regression caused by your patch | 07:31 |
| ralonsoh | frickler, I'll check it later today | 07:32 |
| frickler | ty | 07:33 |
| opendevreview | Merged openstack/neutron stable/2023.1: [FT] Make explicit the "publish" call check in "test_port_forwarding" https://review.opendev.org/c/openstack/neutron/+/893168 | 08:01 |
| opendevreview | Merged openstack/neutron stable/zed: [FT] Make explicit the "publish" call check in "test_port_forwarding" https://review.opendev.org/c/openstack/neutron/+/893169 | 08:03 |
| opendevreview | Merged openstack/neutron stable/yoga: [FT] Make explicit the "publish" call check in "test_port_forwarding" https://review.opendev.org/c/openstack/neutron/+/893170 | 08:42 |
| opendevreview | Merged openstack/neutron master: [UT] Reduce the binding retry loop in ``L3HATestFramework`` https://review.opendev.org/c/openstack/neutron/+/889071 | 08:45 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron master: [OVN] Fix rate and burst for stateless security groups https://review.opendev.org/c/openstack/neutron/+/892648 | 08:46 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron master: [OVN] Fix rate and burst for stateless security groups https://review.opendev.org/c/openstack/neutron/+/892648 | 08:49 |
| ralonsoh | slaweq, lajoskatona https://review.opendev.org/q/I5f11b568bab4518617409b7c3a02c7222df80a92, if you have time | 09:08 |
| lajoskatona | ralonsoh: checking | 09:09 |
| slaweq | ralonsoh done | 09:27 |
| opendevreview | Dr. Jens Harbott proposed openstack/neutron master: ovn: document fragmentation / pmtud gaps https://review.opendev.org/c/openstack/neutron/+/892839 | 09:28 |
| ralonsoh | thanks folks | 09:29 |
| opendevreview | Merged openstack/neutron stable/xena: [FT] Make explicit the "publish" call check in "test_port_forwarding" https://review.opendev.org/c/openstack/neutron/+/893171 | 09:48 |
| zigo | Hi ! | 09:56 |
| zigo | In neutronclient, I have a failure during test discovery: | 09:56 |
| zigo | File "/<<PKGBUILDDIR>>/neutronclient/tests/unit/osc/v2/networking_bgpvpn/fakes.py", line 20, in <module> | 09:56 |
| zigo | from openstack.network.v2 import bgpvpn as _bgpvpn | 09:56 |
| zigo | ImportError: cannot import name 'bgpvpn' from 'openstack.network.v2' (/usr/lib/python3/dist-packages/openstack/network/v2/__init__.py) | 09:56 |
| zigo | From where is bgpvpn comming from? I tried adding python3-networking-bgpvpn as build-depends, but that's not doing it ... | 09:57 |
| zigo | ralonsoh: slaweq: Does one of you know? | 09:58 |
| frickler | zigo: openstack. should be python3-openstacksdk | 09:59 |
| zigo | Oh ok, thanks. Maybe I need a versioned build-depends on it then, trying. | 09:59 |
| ralonsoh | zigo, that module was released in the last version of sdk | 10:00 |
| frickler | I think that bgpvpn was added very recently there. might not be released yet? or need the latest one | 10:00 |
| ralonsoh | it is released | 10:00 |
| zigo | Yeah, then probably, we need a openstacksdk >= 1.4.0 in the requirements. | 10:00 |
| frickler | would make sense to add that in neutronclient upstream I guess, yes | 10:01 |
| zigo | Let me rebuild and I'll open the PR if it fixes. | 10:01 |
| ralonsoh | it is in pythonclient, although the requirement was bumped last week | 10:02 |
| zigo | This fixed it! :) | 10:02 |
| opendevreview | Thomas Goirand proposed openstack/python-neutronclient master: Add openstacksdk>=1.4.0 as test-requirements.txt https://review.opendev.org/c/openstack/python-neutronclient/+/893346 | 10:05 |
| zigo | frickler: ralonsoh: ^ There you go ... :) | 10:05 |
| * frickler is a big fan of alphabetic ordering, but otherwise that's exactly what I'd have suggested, ack | 10:07 | |
| opendevreview | Thomas Goirand proposed openstack/python-neutronclient master: Add openstacksdk>=1.4.0 as test-requirements.txt https://review.opendev.org/c/openstack/python-neutronclient/+/893346 | 10:07 |
| zigo | frickler: Ok, done ! :) | 10:07 |
| zigo | So, with this new release, /usr/bin/neutron is gone? | 10:08 |
| frickler | iiuc yes | 10:10 |
| zigo | I'll have to learn the new commands ... :) | 10:10 |
| ralonsoh | zigo, please check https://review.opendev.org/c/openstack/python-neutronclient/+/893346. Update the requirements.txt file too | 10:11 |
| ralonsoh | I don't know how https://review.opendev.org/c/openstack/python-neutronclient/+/875728 passed the CI | 10:12 |
| zigo | ralonsoh: I do. This was a transitive dependency, so it was pulled with other requirements. | 10:13 |
| frickler | ralonsoh: openstack tests use what is defined in upper-constraints | 10:13 |
| zigo | In my case (Debian packaging) it pulled the version from unstable, ie: antelope, not the last release like with the OpenStack CI. | 10:13 |
| ralonsoh | right, we are not enforcing an upper version in the projects, just the min one | 10:14 |
| zigo | Hint: let's rework the lower-bound dependency check to catch these ... :) | 10:14 |
| zigo | Let me rework my PR. | 10:14 |
| frickler | the lower constraints jobs were dropped because nobody had time to keep them in good shape. so if you have a lot of spare time, that's a good idea :D | 10:15 |
| zigo | Actually, it was dropped because pip was just bad and not doing the right thing, so it was almost impossible to work it out. | 10:16 |
| zigo | global-reqs patch: https://review.opendev.org/c/openstack/requirements/+/893349 | 10:18 |
| hjensas | Good morning, Is there a way to configure multiple bridge mappings for OVN in devstack? (I need baremetal:br-bm in addition to public:br-ex)? | 10:19 |
| opendevreview | Thomas Goirand proposed openstack/python-neutronclient master: Add openstacksdk>=1.1.0 as test-requirements.txt https://review.opendev.org/c/openstack/python-neutronclient/+/893346 | 10:19 |
| hjensas | AFICT devstack does - sudo ovs-vsctl set open . external-ids:ovn-bridge-mappings=$PHYSICAL_NETWORK:$PUBLIC_BRIDGE - i.e only one bridge? | 10:20 |
| ralonsoh | hjensas, external_ids:ovn-bridge-mappings=physnet1:br-eth0,physnet2:br-eth1 | 10:21 |
| ralonsoh | https://man7.org/linux/man-pages/man8/ovn-controller.8.html | 10:22 |
| hjensas | ralonsoh: thanks, I assume devstack does not support that yet. I can try to figure out a patch to add it. | 10:23 |
| rammses | hi i did this https://docs.openstack.org/neutron/yoga/admin/config-bgp-floating-ip-over-l2-segmented-network.html#the-bgp-agent configuration for a test | 10:24 |
| rammses | i noticed that "announce tenant networks option" on bgp speaker is not working out of the box, i needed to add the "tenant network" to address scope | 10:24 |
| rammses | so it gets announced by bgp speaker | 10:24 |
| rammses | Is this a bug ? | 10:24 |
| ralonsoh | hjensas, I think it does but, to be honest, I'm not sure | 10:24 |
| ralonsoh | ltomasbo, ^^ can you check this? | 10:25 |
| hjensas | looks like it set up ovn-bridge-mappings=$PHYSICAL_NETWORK:$PUBLIC_BRIDGE - guess we can add an OVN_BRIDGE_MAPPINGS that works like OVS_BRIDGE_MAPPINGS and default that to $PHYSICAL_NETWORK:$PUBLIC_BRIDGE | 10:26 |
| ltomasbo | but this is with NDR, right? | 10:26 |
| ltomasbo | the reason (if memory works) for the address scope is to avoid overlapping IPs | 10:27 |
| ralonsoh | hjensas, right, the ovn_agent plugin is only configuring one bridge | 10:28 |
| hjensas | ralonsoh: ack, thanks. I'll start working on a devstack change. :) | 10:33 |
| bbezak | Hi, We're getting a neutron-ovs-agent errors with iptables-restore - stable/yoga,ml2/ovs - interestingly only on c8s, rl8 operating systems. I was able to replicate it locally on vm: | 10:49 |
| bbezak | https://paste.opendev.org/show/bChNEGSL52bI3R7At4b1/ | 10:49 |
| bbezak | https://9b32f0b8cfc44eb5fd33-150df88d14e6b0a30b936772a21a5ab1.ssl.cf5.rackcdn.com/893204/1/check/kayobe-overcloud-rocky8/29fd785/primary/kolla/neutron/neutron-openvswitch-agent.txt | 10:49 |
| bbezak | it happens in kayobe CI | 10:50 |
| bbezak | I stand corrected, we're using c8 stream containers on both hosts r8 and c8s. So maybe that is caused by recent updated iptables in c8s - iptables-1.8.5-8.el8.x86_64 | 11:00 |
| frickler | rammses: that is not a bug, but how neutron-dynamic-routing works. note that in that doc, the tenant subnet is created from the same provider-subnet-pool, assuring the same address scope is getting used | 11:03 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-dynamic-routing master: Bump Neutron to 23.0.0.0b3 https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/893189 | 11:04 |
| ralonsoh | frickler, ^^ both LP bugs, we need to fix them in one patch | 11:05 |
| ralonsoh | to pass the CI | 11:05 |
| frickler | ralonsoh: ack, will wait for CI results, but lgtm in general | 11:08 |
| opendevreview | Merged openstack/neutron master: [OVN] Add the 'uplink-status-propagation' extension to ML2/OVN https://review.opendev.org/c/openstack/neutron/+/892895 | 11:37 |
| elvira | Hi! I proposed https://review.opendev.org/c/openstack/neutron/+/892648. If someone has some spare time to take a look I would appreciate it! Maybe slaweq or lucasagomes? o/ | 11:56 |
| lucasagomes | Sure will take a look | 12:11 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron master: [OVN] Fix rate and burst for stateless security groups https://review.opendev.org/c/openstack/neutron/+/892648 | 13:04 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron master: [OVN] Fix rate and burst for stateless security groups https://review.opendev.org/c/openstack/neutron/+/892648 | 13:10 |
| opendevreview | Merged openstack/neutron stable/yoga: [OVN] Add the 'uplink-status-propagation' extension to ML2/OVN https://review.opendev.org/c/openstack/neutron/+/892898 | 13:27 |
| opendevreview | Merged openstack/neutron stable/xena: [OVN] Add the 'uplink-status-propagation' extension to ML2/OVN https://review.opendev.org/c/openstack/neutron/+/892899 | 13:27 |
| opendevreview | Merged openstack/neutron stable/xena: [OVN] Cleanup old Hash Ring node entries https://review.opendev.org/c/openstack/neutron/+/893143 | 13:27 |
| bbezak | looks like c8 stream iptables 1.8.5 is indeed broken, I've raised a bug https://bugzilla.redhat.com/show_bug.cgi?id=2236501, similar as this case: https://www.mail-archive.com/ubuntu-bugs@lists.ubuntu.com/msg5852087.html | 13:40 |
| rammses | Frickler, thank you for explanation | 13:43 |
| ralonsoh | bbezak, that version of version of openstack are you running? | 13:50 |
| ralonsoh | bbezak, this could be related to https://review.opendev.org/c/openstack/neutron/+/844589 | 13:51 |
| opendevreview | Merged openstack/neutron stable/wallaby: [OVN] Add the 'uplink-status-propagation' extension to ML2/OVN https://review.opendev.org/c/openstack/neutron/+/892900 | 13:53 |
| opendevreview | yatin proposed openstack/neutron stable/wallaby: [FT] Make explicit the "publish" call check in "test_port_forwarding" https://review.opendev.org/c/openstack/neutron/+/893374 | 13:56 |
| ykarel | ralonsoh, seen in wallaby too, so pushed ^ resolving merge conflict | 13:57 |
| ralonsoh | checking now | 13:57 |
| bbezak | ralonsoh, it is failing on yoga, xena, wallaby - failing only on c8 stream with iptables 1.8.5. iptables 1.8.4 works fine though | 14:02 |
| ralonsoh | bbezak, right, I read again the bug and as you pointed out, this could be this missing patch in iptbles | 14:02 |
| ralonsoh | not a neutron bug | 14:03 |
| frickler | CI should really move from centos to rockylinux whereever possible IMO | 14:21 |
| hjensas | ralonsoh: re ovn-bridge-mappings - https://review.opendev.org/c/openstack/devstack/+/893401 works for me, follows pattern used with openvswitch agent and I think won't break anything. | 14:52 |
| ralonsoh | hjensas, thanks, I'll add to my review pile (maybe tomorrow morning) | 14:53 |
| hjensas | ralonsoh: thanks! | 14:53 |
| ralonsoh | slaweq, if you have 1 min: | 16:35 |
| ralonsoh | * https://review.opendev.org/q/I2370ea2f96e2e31dbd43bf232a63394388e6945f | 16:35 |
| ralonsoh | * https://review.opendev.org/q/I81390af2ea2fc384423518b84de3acf7adaf9193 | 16:35 |
| ralonsoh | thanks! | 16:35 |
| ralonsoh | or haleyb ^ (thanks in advance) | 16:36 |
| opendevreview | Andrew Bogott proposed openstack/neutron-tempest-plugin master: Allow updating of address pair on shared network https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/890248 | 16:49 |
| opendevreview | Andrew Bogott proposed openstack/neutron master: port policies: Allow port creation/updating in shared networks https://review.opendev.org/c/openstack/neutron/+/886214 | 17:26 |
| opendevreview | Andrew Bogott proposed openstack/neutron master: port policies: Allow port creation/updating in shared networks https://review.opendev.org/c/openstack/neutron/+/886214 | 17:27 |
| opendevreview | Andrew Bogott proposed openstack/neutron master: port policies: Allow port creation/updating in shared networks https://review.opendev.org/c/openstack/neutron/+/886214 | 17:34 |
| opendevreview | Merged openstack/neutron master: Default SG rules - use new rules templates to create rules for SGs https://review.opendev.org/c/openstack/neutron/+/884474 | 19:05 |
| opendevreview | Merged openstack/neutron master: Default SG rules template - Update related docs and add release note https://review.opendev.org/c/openstack/neutron/+/887664 | 19:05 |
| opendevreview | Merged openstack/neutron stable/2023.1: [OVN] Add the 'uplink-status-propagation' extension to ML2/OVN https://review.opendev.org/c/openstack/neutron/+/892896 | 19:05 |
| opendevreview | Merged openstack/neutron stable/wallaby: [FT] Make explicit the "publish" call check in "test_port_forwarding" https://review.opendev.org/c/openstack/neutron/+/893374 | 19:06 |
| opendevreview | Merged openstack/neutron stable/yoga: [OVN] Skip the port status UP update during a live migration https://review.opendev.org/c/openstack/neutron/+/892887 | 19:06 |
| opendevreview | Merged openstack/neutron stable/yoga: [OVN][Trunk] Set the subports correct host during live migration https://review.opendev.org/c/openstack/neutron/+/892888 | 19:06 |
| opendevreview | Merged openstack/neutron stable/xena: [OVN] Skip the port status UP update during a live migration https://review.opendev.org/c/openstack/neutron/+/892885 | 19:06 |
| opendevreview | Merged openstack/neutron stable/xena: [OVN][Trunk] Set the subports correct host during live migration https://review.opendev.org/c/openstack/neutron/+/892886 | 19:06 |
| opendevreview | Merged openstack/neutron stable/wallaby: [OVN] Skip the port status UP update during a live migration https://review.opendev.org/c/openstack/neutron/+/892889 | 19:06 |
| opendevreview | Merged openstack/neutron stable/zed: [OVN] Add the 'uplink-status-propagation' extension to ML2/OVN https://review.opendev.org/c/openstack/neutron/+/892897 | 21:00 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!