opendevreview | Merged openstack/neutron master: [OVN] Document Network Availability Zones https://review.opendev.org/c/openstack/neutron/+/788040 | 00:00 |
---|---|---|
opendevreview | Merged openstack/neutron master: Add pagging and sorting support for "network_ip_availability" https://review.opendev.org/c/openstack/neutron/+/806404 | 00:42 |
opendevreview | liuyulong proposed openstack/neutron-lib master: Leave *_ALL_TABLES to Neutron https://review.opendev.org/c/openstack/neutron-lib/+/807224 | 01:04 |
opendevreview | Merged openstack/neutron master: Make OVN driver validate Geneve max_header_size https://review.opendev.org/c/openstack/neutron/+/714716 | 02:08 |
ralonsoh | slaweq, https://review.opendev.org/c/openstack/neutron/+/806889 | 07:05 |
ralonsoh | trivial review | 07:05 |
ralonsoh | if you have time | 07:05 |
slaweq | ralonsoh: +W'ed | 07:08 |
slaweq | thx | 07:08 |
ralonsoh | thanks | 07:08 |
opendevreview | Rodolfo Alonso proposed openstack/neutron stable/wallaby: Add port status to information dictionary in agent RPC https://review.opendev.org/c/openstack/neutron/+/807253 | 07:13 |
opendevreview | Oleg Bondarev proposed openstack/neutron master: Ensure net dict has provider info on precommit delete https://review.opendev.org/c/openstack/neutron/+/807243 | 07:33 |
opendevreview | Slawek Kaplonski proposed openstack/neutron stable/train: Update arp entry of snat port on qrouter ns https://review.opendev.org/c/openstack/neutron/+/807244 | 07:34 |
opendevreview | Slawek Kaplonski proposed openstack/neutron stable/wallaby: Skip DVR binding for ports with invalid OFPORT https://review.opendev.org/c/openstack/neutron/+/807254 | 07:39 |
opendevreview | Slawek Kaplonski proposed openstack/neutron stable/victoria: Skip DVR binding for ports with invalid OFPORT https://review.opendev.org/c/openstack/neutron/+/807255 | 07:39 |
opendevreview | Merged openstack/neutron master: Include oslo.cache options in neutron.conf https://review.opendev.org/c/openstack/neutron/+/805547 | 08:08 |
opendevreview | Rodolfo 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/+/801712 | 08:19 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN][Placement] Add support for minimum bandwidth QoS rules https://review.opendev.org/c/openstack/neutron/+/786441 | 08:19 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN][Placement] Drive binding by placement allocation https://review.opendev.org/c/openstack/neutron/+/786478 | 08:19 |
opendevreview | Maxim Korezkij proposed openstack/neutron-vpnaas master: Added strongswan supported diffie-helllman-groups https://review.opendev.org/c/openstack/neutron-vpnaas/+/802714 | 08:31 |
opendevreview | Oleg Bondarev proposed openstack/neutron master: Local IP RPC server-agent interface https://review.opendev.org/c/openstack/neutron/+/807116 | 08:37 |
opendevreview | Kevin Li proposed openstack/neutron master: update subport status when trunk/subport create/delete is triggerred https://review.opendev.org/c/openstack/neutron/+/807076 | 09:02 |
opendevreview | Maxim Korezkij proposed openstack/neutron-vpnaas master: Added strongswan supported diffie-helllman-groups https://review.opendev.org/c/openstack/neutron-vpnaas/+/802714 | 09: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 work | 09:44 |
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 work | 09:55 |
ignaziocassano | ther l3 agent log on the controllers where router namespace does not exist report: https://paste.openstack.org/show/808573/ | 10:08 |
obondarev | ignaziocassano, hi, do you have OVS agents on all controllers? | 10:37 |
obondarev | need to figure out why you don't have br-int on those 2 | 10:38 |
ignaziocassano | obondarev: I solved deploying again openvswitch.....it was not present on two controllers. Now it works | 11:09 |
ignaziocassano | obondarev: I do not know why it did not deploy it in the first kolla deployment | 11:10 |
zigo | I 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 |
zigo | Can someone help? | 13:39 |
zigo | slaweq: ralonsoh: ? | 13:39 |
ralonsoh | zigo, https://review.opendev.org/c/openstack/neutron-lib/+/806129 | 13:41 |
ralonsoh | you need this | 13:41 |
zigo | Oh... thanks !@ | 13:41 |
slaweq | ralonsoh++ | 13:41 |
zigo | ralonsoh: Are you sure there's anything about oslo.metrics in Rocky ?!? | 13:43 |
zigo | I trust you, but this feels kind of weird. | 13:44 |
ralonsoh | that was the kind of error this patch solved | 13:44 |
ralonsoh | I'll try that locally | 13:44 |
zigo | zigo@k:zbuz>_ ~/salsa/openstack/libs/python-neutron-lib/python-neutron-lib python-neutron-lib (debian/rocky)$ patch -p1 --dry-run <Disable_oslo.messaging_metrics.patch | 13:45 |
zigo | checking file neutron_lib/tests/unit/test_rpc.py | 13:45 |
zigo | Hunk #1 succeeded at 287 (offset -1 lines). | 13:45 |
zigo | Hunk #2 FAILED at 448. | 13:45 |
zigo | 1 out of 2 hunks FAILED | 13:45 |
ralonsoh | this test is passing both in py27 and py36 | 13:49 |
ralonsoh | slaweq, https://bugs.launchpad.net/neutron/+bug/1942179 is public | 14:02 |
ralonsoh | will you push the patches? | 14:02 |
slaweq | ralonsoh: sure, give me 5 minutes | 14:02 |
lajoskatona | zigo: 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.messaging | 14:05 |
zigo | Yeah, weird... | 14:05 |
zigo | I tried what ralonsoh said, currently running Neutron unit tests... | 14:05 |
ralonsoh | could be something different, of course | 14:05 |
ralonsoh | what version of oslo.messaging do you have? | 14:05 |
zigo | Package version is 8.1.4-1+deb10u1 | 14:06 |
opendevreview | Slawek Kaplonski proposed openstack/neutron master: Don't use singleton in routes.middleware.RoutesMiddleware https://review.opendev.org/c/openstack/neutron/+/807335 | 14:11 |
slaweq | ralonsoh: ^^ | 14:11 |
slaweq | lets see how happy zuul will be with it :) | 14:11 |
zigo | ralonsoh: The patch didn't help much, I have 380 failures instead of 480 ... :/ | 14:21 |
ralonsoh | how are you executing the UTs? | 14:21 |
zigo | ralonsoh: 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 |
ralonsoh | why don't use use tox? we create the environment with the correct library versions | 14:29 |
ralonsoh | did you check the versions you have installed in your env? | 14:29 |
zigo | ralonsoh: Tox does pip stuff, that's not what I'm doing, I'm doing Debian packaging, which means no network env. | 14:29 |
ralonsoh | so you need to check the versions | 14:29 |
zigo | ralonsoh: The versions are all taken from packages as well. | 14:30 |
ralonsoh | with the versions installed by tox, the UTs pass | 14:30 |
ralonsoh | what are the errors? | 14:30 |
zigo | I can give you the output of pip freeze if you need it... | 14:30 |
ralonsoh | ok | 14:30 |
zigo | ralonsoh: Example failure: https://paste.opendev.org/show/808581/ | 14:32 |
zigo | Let me get you the pip freeze output now... | 14:32 |
zigo | FYI, 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 |
zigo | ralonsoh: There you go: https://paste.opendev.org/show/808582/ | 14:34 |
zigo | The 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 |
zigo | I probably could bissect it, but it will take forever (too many commits...). | 14:35 |
ralonsoh | zigo, please, use the sqlalchemy version recommended by https://github.com/openstack/requirements/blob/stable/rocky/upper-constraints.txt | 14:39 |
ralonsoh | this is 1.2.10, not 1.2.18 | 14:40 |
zigo | ralonsoh: I can't do that, that's a Debian release that is 2 years old that we're talking about ! | 14:40 |
ralonsoh | then I can't help you on this. This is why we set upper-constraints | 14:41 |
zigo | ralonsoh: Do you have the smallest idea how distributions are working ? :) | 14:42 |
zigo | Nv mind, thanks a lot for your help. | 14:42 |
ralonsoh | I do, I participate in RHEL distribution | 14:42 |
zigo | Well, in RHEL, they pretend that OpenStack is alone and they package whatever OpenStack likes. That's CHEATING ! :) | 14:43 |
ralonsoh | not at all | 14:43 |
zigo | Anyways, I will go back to packaging 13.0.7 + a few patches, as it built well. | 14:43 |
ralonsoh | we use containers to install the needed version for each service | 14:43 |
zigo | Maybe I'll just bissect and try to find out. | 14:43 |
opendevreview | Merged openstack/neutron master: Add port status to information dictionary in agent RPC https://review.opendev.org/c/openstack/neutron/+/806889 | 15:07 |
opendevreview | Lajos Katona proposed openstack/neutron-lib master: Exception for BFD: BfdMonitorInUse https://review.opendev.org/c/openstack/neutron-lib/+/807354 | 15:13 |
opendevreview | Rodolfo Alonso proposed openstack/neutron-lib master: Set "cache_ok=False" in all "TypeDecorator" derived classes https://review.opendev.org/c/openstack/neutron-lib/+/807360 | 15:45 |
opendevreview | Rodolfo Alonso proposed openstack/neutron-lib master: Set "cache_ok=True" in all "TypeDecorator" derived classes https://review.opendev.org/c/openstack/neutron-lib/+/807360 | 15:47 |
*** hjensas is now known as hjensas|afk | 16:29 | |
haleyb | ralonsoh: 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 anything | 17:14 |
ralonsoh | haleyb, this is a but weird. Normal response for "update" command is 200 | 17:18 |
ralonsoh | as any other object in Neutron | 17:18 |
ralonsoh | (checking the neutron-lib api right now) | 17:19 |
haleyb | ralonsoh: yes, that's what i thought, i don't see a HTTPAccepted anywhere | 17:38 |
haleyb | there are others that list 202 as well, so maybe just some typos | 17:42 |
haleyb | ralonsoh: 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 |
opendevreview | Brian Haley proposed openstack/neutron-lib master: Fix some api-ref typos https://review.opendev.org/c/openstack/neutron-lib/+/807390 | 18:31 |
gregraka | (d | 19:37 |
opendevreview | Merged openstack/networking-ovn stable/train: ovn: Consider all router ports in is_lsp_router_port() https://review.opendev.org/c/openstack/networking-ovn/+/806870 | 20:43 |
opendevreview | Ihar Hrachyshka proposed openstack/neutron master: ovn: use stateless NAT rules for FIPs https://review.opendev.org/c/openstack/neutron/+/804807 | 22:42 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!