Friday, 2021-09-03

opendevreviewMerged openstack/neutron master: [OVN] Document Network Availability Zones  https://review.opendev.org/c/openstack/neutron/+/78804000:00
opendevreviewMerged openstack/neutron master: Add pagging and sorting support for "network_ip_availability"  https://review.opendev.org/c/openstack/neutron/+/80640400:42
opendevreviewliuyulong proposed openstack/neutron-lib master: Leave *_ALL_TABLES to Neutron  https://review.opendev.org/c/openstack/neutron-lib/+/80722401:04
opendevreviewMerged openstack/neutron master: Make OVN driver validate Geneve max_header_size  https://review.opendev.org/c/openstack/neutron/+/71471602:08
ralonsohslaweq, https://review.opendev.org/c/openstack/neutron/+/80688907:05
ralonsohtrivial review07:05
ralonsohif you have time07:05
slaweqralonsoh: +W'ed07:08
slaweqthx07:08
ralonsohthanks07:08
opendevreviewRodolfo Alonso proposed openstack/neutron stable/wallaby: Add port status to information dictionary in agent RPC  https://review.opendev.org/c/openstack/neutron/+/80725307:13
opendevreviewOleg Bondarev proposed openstack/neutron master: Ensure net dict has provider info on precommit delete  https://review.opendev.org/c/openstack/neutron/+/80724307:33
opendevreviewSlawek Kaplonski proposed openstack/neutron stable/train: Update arp entry of snat port on qrouter ns  https://review.opendev.org/c/openstack/neutron/+/80724407:34
opendevreviewSlawek Kaplonski proposed openstack/neutron stable/wallaby: Skip DVR binding for ports with invalid OFPORT  https://review.opendev.org/c/openstack/neutron/+/80725407:39
opendevreviewSlawek Kaplonski proposed openstack/neutron stable/victoria: Skip DVR binding for ports with invalid OFPORT  https://review.opendev.org/c/openstack/neutron/+/80725507:39
opendevreviewMerged openstack/neutron master: Include oslo.cache options in neutron.conf  https://review.opendev.org/c/openstack/neutron/+/80554708:08
opendevreviewRodolfo Alonso proposed openstack/neutron master: [OVN][Placement] Add a SB Chassis event to track changes in BW config  https://review.opendev.org/c/openstack/neutron/+/80171208:19
opendevreviewRodolfo Alonso proposed openstack/neutron master: [OVN][Placement] Add support for minimum bandwidth QoS rules  https://review.opendev.org/c/openstack/neutron/+/78644108:19
opendevreviewRodolfo Alonso proposed openstack/neutron master: [OVN][Placement] Drive binding by placement allocation  https://review.opendev.org/c/openstack/neutron/+/78647808:19
opendevreviewMaxim Korezkij proposed openstack/neutron-vpnaas master: Added strongswan supported diffie-helllman-groups  https://review.opendev.org/c/openstack/neutron-vpnaas/+/80271408:31
opendevreviewOleg Bondarev proposed openstack/neutron master: Local IP RPC server-agent interface  https://review.opendev.org/c/openstack/neutron/+/80711608:37
opendevreviewKevin Li proposed openstack/neutron master: update subport status when trunk/subport create/delete is triggerred  https://review.opendev.org/c/openstack/neutron/+/80707609:02
opendevreviewMaxim Korezkij proposed openstack/neutron-vpnaas master: Added strongswan supported diffie-helllman-groups  https://review.opendev.org/c/openstack/neutron-vpnaas/+/80271409:13
ignaziocassano Hello Everyone, I am facing an issue with router-ha in kolla wallaby. In my neutron.conf I inserted l3_ha = True max_l3_agents_per_router = 3 l3_ha_net_cidr = 169.254.192.0/18 but router ha does not work. The router namespace is presente only on one of the three contrllers and if the controller goes down touting stop to work09:44
ignaziocassanoHello Everyone, I am facing an issue with router-ha in kolla wallaby. In my neutron.conf I inserted l3_ha = True max_l3_agents_per_router = 3 l3_ha_net_cidr = 169.254.192.0/18 but router ha does not work. The router namespace is presente only on one of the three contrllers and if the controller goes down touting stop to work09:55
ignaziocassanother l3 agent log on the controllers where router namespace does not exist report: https://paste.openstack.org/show/808573/10:08
obondarevignaziocassano, hi, do you have OVS agents on all controllers?10:37
obondarevneed to figure out why you don't have br-int on those 210:38
ignaziocassanoobondarev: I solved  deploying again openvswitch.....it was not present on two controllers. Now it works11:09
ignaziocassanoobondarev: I do not know why it did not deploy it in the first kolla deployment11:10
zigoI get 400+ unit test failures trying to rebuild Neutron stable/rocky for Buster (re: the last CVE). All like this: https://paste.opendev.org/show/808578/13:39
zigoCan someone help?13:39
zigoslaweq: ralonsoh: ?13:39
ralonsohzigo, https://review.opendev.org/c/openstack/neutron-lib/+/80612913:41
ralonsohyou need this13:41
zigoOh... thanks !@13:41
slaweqralonsoh++13:41
zigoralonsoh: Are you sure there's anything about oslo.metrics in Rocky ?!?13:43
zigoI trust you, but this feels kind of weird.13:44
ralonsohthat was the kind of error this patch solved13:44
ralonsohI'll try that locally13:44
zigozigo@k:zbuz>_ ~/salsa/openstack/libs/python-neutron-lib/python-neutron-lib python-neutron-lib (debian/rocky)$ patch -p1 --dry-run <Disable_oslo.messaging_metrics.patch13:45
zigochecking file neutron_lib/tests/unit/test_rpc.py13:45
zigoHunk #1 succeeded at 287 (offset -1 lines).13:45
zigoHunk #2 FAILED at 448.13:45
zigo1 out of 2 hunks FAILED13:45
ralonsohthis test is passing both in py27 and py3613:49
ralonsohslaweq, https://bugs.launchpad.net/neutron/+bug/1942179 is public14:02
ralonsohwill you push the patches?14:02
slaweqralonsoh: sure, give me 5 minutes14:02
lajoskatonazigo: strange that You have metrics related errors on rocky as the patch for metrics (https://review.opendev.org/c/openstack/oslo.messaging/+/761848 ) is only from 12.9.0 of oslo.messaging14:05
zigoYeah, weird...14:05
zigoI tried what ralonsoh said, currently running Neutron unit tests...14:05
ralonsohcould be something different, of course14:05
ralonsohwhat version of oslo.messaging do you have?14:05
zigoPackage version is 8.1.4-1+deb10u114:06
opendevreviewSlawek Kaplonski proposed openstack/neutron master: Don't use singleton in routes.middleware.RoutesMiddleware  https://review.opendev.org/c/openstack/neutron/+/80733514:11
slaweqralonsoh: ^^14:11
slaweqlets see how happy zuul will be with it :)14:11
zigoralonsoh: The patch didn't help much, I have 380 failures instead of 480 ... :/14:21
ralonsohhow are you executing the UTs?14:21
zigoralonsoh: PYTHON=python3.7 python3-stestr run --subunit neutron\.tests\.unit(?!.*test_wsgi\.TestWSGIServer\.test_start_random_port_with_ipv6.*|.*agent\.dhcp\.test_agent\.TestDhcpAgent\.test_dhcp_agent_manager.*|.*test_wsgi\.TestWSGIServer\.test_app.*|.*test_wsgi\.TestWSGIServer\.test_disable_ssl.*)14:28
ralonsohwhy don't use use tox? we create the environment with the correct library versions14:29
ralonsohdid you check the versions you have installed in your env?14:29
zigoralonsoh: Tox does pip stuff, that's not what I'm doing, I'm doing Debian packaging, which means no network env.14:29
ralonsohso you need to check the versions14:29
zigoralonsoh: The versions are all taken from packages as well.14:30
ralonsohwith the versions installed by tox, the UTs pass14:30
ralonsohwhat are the errors?14:30
zigoI can give you the output of pip freeze if you need it...14:30
ralonsohok14:30
zigoralonsoh: Example failure: https://paste.opendev.org/show/808581/14:32
zigoLet me get you the pip freeze output now...14:32
zigoFYI, now if there's unit test failures, my packaging runs pip freeze automatically ... But not in the Buster package (yet), so got to manually trigger it.14:33
zigoralonsoh: There you go: https://paste.opendev.org/show/808582/14:34
zigoThe thing is, rebuilding the 13.0.7 tag worked without a glitch, so there's something wrong between 13.0.7 and the tip of stable/rocky ...14:35
zigoI probably could bissect it, but it will take forever (too many commits...).14:35
ralonsohzigo, please, use the sqlalchemy version recommended by https://github.com/openstack/requirements/blob/stable/rocky/upper-constraints.txt14:39
ralonsohthis is 1.2.10, not 1.2.1814:40
zigoralonsoh: I can't do that, that's a Debian release that is 2 years old that we're talking about !14:40
ralonsohthen I can't help you on this. This is why we set upper-constraints14:41
zigoralonsoh: Do you have the smallest idea how distributions are working ? :)14:42
zigoNv mind, thanks a lot for your help.14:42
ralonsohI do, I participate in RHEL distribution14:42
zigoWell, in RHEL, they pretend that OpenStack is alone and they package whatever OpenStack likes. That's CHEATING ! :)14:43
ralonsohnot at all14:43
zigoAnyways, I will go back to packaging 13.0.7 + a few patches, as it built well.14:43
ralonsohwe use containers to install the needed version for each service14:43
zigoMaybe I'll just bissect and try to find out.14:43
opendevreviewMerged openstack/neutron master: Add port status to information dictionary in agent RPC  https://review.opendev.org/c/openstack/neutron/+/80688915:07
opendevreviewLajos Katona proposed openstack/neutron-lib master: Exception for BFD: BfdMonitorInUse  https://review.opendev.org/c/openstack/neutron-lib/+/80735415:13
opendevreviewRodolfo Alonso proposed openstack/neutron-lib master: Set "cache_ok=False" in all "TypeDecorator" derived classes  https://review.opendev.org/c/openstack/neutron-lib/+/80736015:45
opendevreviewRodolfo Alonso proposed openstack/neutron-lib master: Set "cache_ok=True" in all "TypeDecorator" derived classes  https://review.opendev.org/c/openstack/neutron-lib/+/80736015:47
*** hjensas is now known as hjensas|afk16:29
haleybralonsoh: qq on a bug in qos api - https://bugs.launchpad.net/neutron/+bug/1942448 - are we supposed to return a 202 ?  i didn't think we did for anything17:14
ralonsohhaleyb, this is a but weird. Normal response for "update" command is 20017:18
ralonsohas any other object in Neutron17:18
ralonsoh(checking the neutron-lib api right now)17:19
haleybralonsoh: yes, that's what i thought, i don't see a HTTPAccepted anywhere17:38
haleybthere are others that list 202 as well, so maybe just some typos17:42
haleybralonsoh: i don't mind doing the patch as it seems there are other typos in a few files, just didn't know if qos was "special" :)17:56
opendevreviewBrian Haley proposed openstack/neutron-lib master: Fix some api-ref typos  https://review.opendev.org/c/openstack/neutron-lib/+/80739018:31
gregraka(d19:37
opendevreviewMerged openstack/networking-ovn stable/train: ovn: Consider all router ports in is_lsp_router_port()  https://review.opendev.org/c/openstack/networking-ovn/+/80687020:43
opendevreviewIhar Hrachyshka proposed openstack/neutron master: ovn: use stateless NAT rules for FIPs  https://review.opendev.org/c/openstack/neutron/+/80480722:42

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