| opendevreview | Merged openstack/neutron master: [OVN] Set always the GW LRP "gateway_mtu" option https://review.opendev.org/c/openstack/neutron/+/937026 | 00:27 |
|---|---|---|
| opendevreview | Merged openstack/neutron-lib master: Enable mypy for pep8 tox target https://review.opendev.org/c/openstack/neutron-lib/+/921484 | 01:23 |
| opendevreview | Merged openstack/neutron master: Support Guru Meditation Report(GMR) in agents https://review.opendev.org/c/openstack/neutron/+/932281 | 01:31 |
| opendevreview | Merged openstack/neutron master: Bump hacking to 7.0.0 https://review.opendev.org/c/openstack/neutron/+/934701 | 01:31 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron-lib master: Deprecate is_bsd and remove its use in test suite https://review.opendev.org/c/openstack/neutron-lib/+/938852 | 01:44 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: WIP Remove linuxbridge driver https://review.opendev.org/c/openstack/neutron/+/927216 | 01:45 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: Remove linuxbridge driver https://review.opendev.org/c/openstack/neutron/+/927216 | 01:46 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: Remove dibbler code from l3 agent and elsewhere https://review.opendev.org/c/openstack/neutron/+/934283 | 01:48 |
| opendevreview | Brian Haley proposed openstack/neutron master: Fix some pylint issues in the test tree https://review.opendev.org/c/openstack/neutron/+/938853 | 03:12 |
| opendevreview | Dong Ma proposed openstack/ovn-bgp-agent master: Add the support of create kubernetes resource https://review.opendev.org/c/openstack/ovn-bgp-agent/+/937457 | 03:23 |
| opendevreview | Dong Ma proposed openstack/ovn-bgp-agent master: Add the support of create kubernetes resource https://review.opendev.org/c/openstack/ovn-bgp-agent/+/937457 | 04:02 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Synchronize the network segment range initialization https://review.opendev.org/c/openstack/neutron/+/938319 | 06:43 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [eventlet-deprecation] Implement a metadata proxy without eventlet https://review.opendev.org/c/openstack/neutron/+/937545 | 06:59 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [eventlet-deprecation] Use non-eventlet metadata proxy in OVN metadata agent https://review.opendev.org/c/openstack/neutron/+/938393 | 07:00 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [eventlet-deprecation] Replace ``eventlet.spawn_n`` usage https://review.opendev.org/c/openstack/neutron/+/938541 | 08:25 |
| ralonsoh | slaweq, hello! One quick review, an optimization requested by Terry in the previous patch: https://review.opendev.org/c/openstack/neutron/+/937682 | 08:27 |
| ralonsoh | thanks in advance | 08:27 |
| ralonsoh | ah, and the upper patch (that has already 2 +2): https://review.opendev.org/c/openstack/neutron/+/938120/6 | 08:28 |
| ralonsoh | bcafarel, hello! If you have 1 min: https://review.opendev.org/q/topic:%22bug/2038646%22 | 08:29 |
| ralonsoh | thanks in advance | 08:29 |
| slaweq | ralonsoh I just +2 it but You need other +2 as well, maybe ykarel or lajoskatona can check it | 08:31 |
| slaweq | and then approve both patches | 08:31 |
| ralonsoh | thanks! | 08:31 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/os-ken master: dnm: remove patch https://review.opendev.org/c/openstack/os-ken/+/938868 | 08:33 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: dnm: just to trigger CI https://review.opendev.org/c/openstack/neutron/+/938487 | 08:34 |
| ralonsoh | slaweq, https://review.opendev.org/c/openstack/neutron/+/937372 n-lib 3.17 is already available | 08:35 |
| ralonsoh | if you update the patch, I'll +2 it | 08:35 |
| ralonsoh | (ping me once is ready) | 08:36 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: dnm: just to trigger CI https://review.opendev.org/c/openstack/neutron/+/938487 | 08:38 |
| opendevreview | Merged openstack/ovsdbapp master: Add bfd parameter for lr_route_add https://review.opendev.org/c/openstack/ovsdbapp/+/936877 | 09:57 |
| *** ralonsoh_ is now known as ralonsoh | 09:58 | |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/os-ken master: hub: use native as default implementation https://review.opendev.org/c/openstack/os-ken/+/938337 | 10:19 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/os-ken master: hub: add equivalents for eventlet functions using native https://review.opendev.org/c/openstack/os-ken/+/938875 | 10:19 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/os-ken master: hub: implement listen as native https://review.opendev.org/c/openstack/os-ken/+/938876 | 10:19 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/os-ken master: dnm: remove patch https://review.opendev.org/c/openstack/os-ken/+/938868 | 10:20 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: common: fix wait_until_true to support native thread https://review.opendev.org/c/openstack/neutron/+/937843 | 10:28 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: ovs: remove the usage of eventlet in the OVS agent https://review.opendev.org/c/openstack/neutron/+/937765 | 10:28 |
| bcafarel | ralonsoh: 2023.2 has one missing assertRaises compared to 2024.1, is it intentional? https://review.opendev.org/c/openstack/neutron/+/937968/2/neutron/tests/unit/conf/policies/test_subnet.py#768 | 11:06 |
| ralonsoh | bcafarel, let me check | 11:06 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2023.2: Allow network owner reader to get subnets https://review.opendev.org/c/openstack/neutron/+/937968 | 11:09 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Check if the LRP exists in ``check_provider_distributed_ports`` https://review.opendev.org/c/openstack/neutron/+/938889 | 12:36 |
| opendevreview | Merged openstack/neutron master: Make sure that policy enforcer is initialized before use https://review.opendev.org/c/openstack/neutron/+/938640 | 12:52 |
| opendevreview | Merged openstack/neutron master: Remove is_bsd skips in test suite https://review.opendev.org/c/openstack/neutron/+/938849 | 12:52 |
| opendevreview | Merged openstack/neutron master: [OVN] Use the Neutron port ID in ``UpdateLSwitchPortQosOptionsCommand`` https://review.opendev.org/c/openstack/neutron/+/937682 | 12:53 |
| opendevreview | Merged openstack/neutron master: [OVN] Improve ``_ovn_qos_rule`` method, removing unnecesary argument https://review.opendev.org/c/openstack/neutron/+/938120 | 12:53 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: dnm: just to trigger CI https://review.opendev.org/c/openstack/neutron/+/938487 | 12:56 |
| opendevreview | Merged openstack/neutron stable/2024.2: Allow network owner reader to get subnets https://review.opendev.org/c/openstack/neutron/+/937966 | 12:59 |
| opendevreview | Merged openstack/neutron stable/2024.1: Allow network owner reader to get subnets https://review.opendev.org/c/openstack/neutron/+/937967 | 13:00 |
| mlavalle | haleyb: are we meeting today? | 13:59 |
| haleyb | mlavalle: i did see one new item from slaweq in the on-demand agenda | 14:00 |
| haleyb | slaweq: did you want to discuss your vlan_transparent proposal today? | 14:00 |
| slaweq | haleyb I added it there before xmas break but this week I also sent email about it to the ML - we can discuss it there as well, it's nothing urgent | 14:01 |
| slaweq | and if we want to discuss it within the drivers team, I think it would be better to do it in 1 or 2 weeks so that first I can have some feedback from the community on ML | 14:01 |
| haleyb | slaweq: ack. i am still behind on some email will look on ML | 14:01 |
| slaweq | but I see also topic from noonedeadpunk in the agenda | 14:02 |
| haleyb | So we do not need to meet today, just a reminder there are some in-progress specs | 14:02 |
| mlavalle | yeap that was the agreement during the weekly meeting | 14:02 |
| mlavalle | haleyb: ok, will look at them. have a nice weekend. you too slaweq | 14:03 |
| lajoskatona | o/, have a nice weekend | 14:03 |
| mlavalle | you too lajoskatona | 14:03 |
| jamesdenton__ | hello all. quick question (hopefully) - for a given HA_Chassis_Group, what could cause the ha_chassis list to remain unpopulated? | 14:03 |
| haleyb | slaweq: i thought we were just going to treat that ovn-bgp-agent one as a bug, but i don't exactly remember | 14:04 |
| slaweq | haleyb me neighter :) I just noticed it now in the agenda, maybe it is just leftover | 14:04 |
| ralonsoh | jamesdenton__, that you don't have GW chassis | 14:04 |
| haleyb | i will let noonedeadpunk respond and can just discuss next week if needed | 14:05 |
| ralonsoh | jamesdenton__, or the AZs don't match the chassis ones | 14:05 |
| jamesdenton__ | hmm ok, lemme take a peek. thank you | 14:05 |
| slaweq | sounds good for me, thx haleyb | 14:06 |
| slaweq | and have a nice weekend :) | 14:06 |
| lajoskatona | slaweq, ralonsoh: a question regarding ovs initialization, I see in logs issues when ovs-agent tries to initialize ovsdb values. | 14:07 |
| ralonsoh | lajoskatona, do you have a link to a CI? | 14:07 |
| lajoskatona | slaweq, ralonsoh: example is in CI logs which was suprise for me yesterday: https://zuul.opendev.org/t/openstack/build/fc2035fd67f24c87a4ad070fefdcc703/log/controller/logs/screen-q-agt.txt#561-565 | 14:07 |
| lajoskatona | I saw it in my local devstack also but thought some mistake or issue in my devstack | 14:08 |
| ralonsoh | (it is loading very slow) | 14:09 |
| lajoskatona | it seems that sometimes ovs-agent cant create and set values in the Manager table (ovs-vsctl: multiple rows in Manager match "ptcp:6640:127.0.0.1"\n) | 14:09 |
| lajoskatona | ralonsoh: yes it is not the raw view and that is lsow but I can point to specific lines :-) | 14:09 |
| ralonsoh | pffff can you copy/paste the exact timestamp? | 14:10 |
| ralonsoh | is not loading at all | 14:10 |
| ralonsoh | hold on, I have it | 14:10 |
| lajoskatona | and the first thing was what I lookd due to some downstream issue is the inactivyty_probe timeout value and realiyed that we set for Manager and Controller tables, but I found that in my devstack for example ovs-agent was not able to set this value in Manager table due to the issue in logs | 14:11 |
| lajoskatona | ralonsoh: Jan 08 14:52:18.437125 just to have it | 14:11 |
| ralonsoh | lajoskatona, what is the issue you have in the logs? | 14:14 |
| lajoskatona | ralonsoh: the strange is that only this debug log (Jan 08 14:52:18.455956 np0039512747 neutron-openvswitch-agent[63181]: DEBUG oslo.privsep.daemon [-] privsep: reply[cdb9b623-b580-4bd9-9349-ed6a566f7b16]: (4, ('', 'ovs-vsctl: multiple rows in Manager match "ptcp:6640:127.0.0.1"\n')) {{(pid=65448) _call_back /opt/stack/data/venv/lib/python3.12/site-packages/oslo_privsep/daemon.py:498}} ) | 14:15 |
| lajoskatona | 566 | 14:15 |
| lajoskatona | but in my local devstack for example ovs-agent can't set the inactivity_timeout in the Manager table, and in one of our downstream env we have similar issue (that is on older branch so I have to double check with them) | 14:16 |
| ralonsoh | but I think you have a problem in your table | 14:17 |
| ralonsoh | from the OVS documentation | 14:17 |
| ralonsoh | target: string (must be unique within table) | 14:17 |
| lajoskatona | but the strange for me is that it seems that the agent fails to create the manager and than just goes over, and after restart it happens again | 14:17 |
| ralonsoh | lajoskatona, let me check manually if we can create a duplicate | 14:18 |
| lajoskatona | ralonsoh: you mean the pctp.... string? | 14:18 |
| ralonsoh | lajoskatona, yes, check http://www.openvswitch.org//ovs-vswitchd.conf.db.5.pdf | 14:19 |
| ralonsoh | (section "manager table") | 14:19 |
| ralonsoh | this is why this is so strange to have multiple manager rows matching the same string | 14:19 |
| lajoskatona | ralonsoh: yes | 14:19 |
| lajoskatona | but in this case the CI job also runs to the same issue? | 14:20 |
| lajoskatona | and if it fails to create as the target is already there, shouldn't the ovs-agent retry with set to refresh the values, like after ovs-agent restart? | 14:22 |
| * noonedeadpunk looking thourgh | 14:22 | |
| noonedeadpunk | I eventually wanted some eyes/reviews on ovn-bgp one | 14:23 |
| noonedeadpunk | and struggling to have any for a while now | 14:23 |
| lajoskatona | oonedeadpunk: if you can you can set the review priority on the patch is that is ready for review, if you can't copy it here and I can try increase priority on it | 14:24 |
| jamesdenton__ | ralonsoh thanks for the tip on the AZ - ovn-cms-options="enable-chassis-as-gw,availability-zones=nova" was set, but my networks did not specify an AZ | 14:26 |
| noonedeadpunk | #link https://review.opendev.org/c/openstack/neutron/+/931495 | 14:28 |
| noonedeadpunk | #link https://review.opendev.org/c/openstack/neutron/+/936643 | 14:29 |
| noonedeadpunk | ugh, sorry | 14:29 |
| noonedeadpunk | wrong ones :D | 14:29 |
| ralonsoh | lajoskatona, we don't have something like "--may-exist" for ovs-vsctl create | 14:29 |
| noonedeadpunk | #link https://review.opendev.org/c/openstack/ovn-bgp-agent/+/935801 | 14:29 |
| noonedeadpunk | #link https://review.opendev.org/c/openstack/ovn-bgp-agent/+/937362 | 14:29 |
| ralonsoh | lajoskatona, so maybe we need to catch the error and retry without creating the register | 14:29 |
| noonedeadpunk | I don't see review prioirty there lajoskatona | 14:30 |
| ralonsoh | noonedeadpunk, in the reply button | 14:30 |
| noonedeadpunk | only CR and W | 14:31 |
| noonedeadpunk | I believe this is only cor core reviewers | 14:31 |
| lajoskatona | noonedeadpunk: no it's realy not there, I can check the config in the depths of some of the project-config things | 14:32 |
| noonedeadpunk | in gerrit acls, yeah | 14:32 |
| sahid | lajoskatona: ralonsoh: o/ any chance to have a look in it when you have a moment? | 14:33 |
| sahid | https://review.opendev.org/c/openstack/neutron/+/937765 | 14:33 |
| lajoskatona | ralonsoh: I check it, if that can work, the thing that it happens silently with a debug line was confusing | 14:33 |
| ralonsoh | lajoskatona, I'm checking now what is returned and if there is any kind of error | 14:33 |
| sahid | I have made the change to use os-ken-master ci but that does not look to work I guess I have made something wrong | 14:33 |
| ralonsoh | sahid, yes, I'll check it | 14:34 |
| sahid | ralonsoh: thanks a lot | 14:34 |
| sahid | ralonsoh: oh wait I was looking into the wrong ci job | 14:35 |
| sahid | sorry | 14:36 |
| lajoskatona | ralonsoh: do you think a bug is necessary for this? | 14:39 |
| opendevreview | Bodo Petermann proposed openstack/neutron master: Define OvnPeriodicWorker that has OVN IDL enabled https://review.opendev.org/c/openstack/neutron/+/938905 | 14:39 |
| ralonsoh | lajoskatona, yes, I think so | 14:40 |
| ralonsoh | we are not setting the inactivity timeout if the register is already created | 14:40 |
| lajoskatona | ralonsoh: yes that was my local observation and red flag also | 14:46 |
| lajoskatona | noonedeadpunk: the ACL change for bgp-agent (and ovsdbapp) https://review.opendev.org/c/openstack/project-config/+/938908 , with these I think all the Neutron projects will have this as I quickly checked | 14:47 |
| ralonsoh | lajoskatona, ^ there are many empty spaces in the patches | 14:48 |
| ralonsoh | https://review.opendev.org/c/openstack/project-config/+/938908/1/gerrit/acls/openstack/ovsdbapp.config#16 | 14:49 |
| lajoskatona | ralonsoh: thanks, seems I have to check my vim.cfg file.... | 14:49 |
| ralonsoh | also I see something extra that is needed | 14:50 |
| ralonsoh | from previous patches: https://review.opendev.org/c/openstack/project-config/+/929776/4/gerrit/acls/openstack/tap-as-a-service.config | 14:50 |
| ralonsoh | [submit-requirement "Review-Priority"] | 14:50 |
| ralonsoh | also I think you need to use tabs, not spaces | 14:51 |
| noonedeadpunk | lajoskatona: there're trailing whitespaces in files fwiw | 14:51 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2023.2: Allow network owner reader to get subnets https://review.opendev.org/c/openstack/neutron/+/937968 | 14:57 |
| lajoskatona | noonedeadpunk, ralonsoh: yeah, finally I have tabs everywhere, and it is a reminder for me why I hate tabs :-) | 15:06 |
| noonedeadpunk | haha | 15:07 |
| ralonsoh | lajoskatona, shouldn't you add the [submit-requirement "Review-Priority"] section? | 15:07 |
| ralonsoh | same as in https://review.opendev.org/c/openstack/project-config/+/929776/4/gerrit/acls/openstack/tap-as-a-service.config | 15:08 |
| lajoskatona | ralonsoh: ahh, I see it in neutron.config also | 15:09 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/2024.2: Make sure that policy enforcer is initialized before use https://review.opendev.org/c/openstack/neutron/+/938911 | 15:16 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/2024.1: Make sure that policy enforcer is initialized before use https://review.opendev.org/c/openstack/neutron/+/938912 | 15:19 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/2023.2: Make sure that policy enforcer is initialized before use https://review.opendev.org/c/openstack/neutron/+/938913 | 15:19 |
| opendevreview | Bodo Petermann proposed openstack/neutron-vpnaas master: Use OvnPeriodicWorker for VPN agent check https://review.opendev.org/c/openstack/neutron-vpnaas/+/938915 | 15:26 |
| opendevreview | Rodolfo Alonso proposed openstack/ovsdbapp master: Add ``QoSUpdateCommand`` command https://review.opendev.org/c/openstack/ovsdbapp/+/938179 | 15:31 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/os-ken master: hub: implement threads related functions https://review.opendev.org/c/openstack/os-ken/+/938673 | 15:37 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/os-ken master: hub: add equivalents for eventlet functions using native https://review.opendev.org/c/openstack/os-ken/+/938875 | 15:37 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/os-ken master: hub: implement listen as native https://review.opendev.org/c/openstack/os-ken/+/938876 | 15:37 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/os-ken master: hub: use native as default implementation https://review.opendev.org/c/openstack/os-ken/+/938337 | 15:37 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: WIP Make API policies for tags to be working with resource attributes https://review.opendev.org/c/openstack/neutron/+/938135 | 15:53 |
| opendevreview | Jakub Libosvar proposed openstack/ovn-bgp-agent master: Introduce multinode tempest job https://review.opendev.org/c/openstack/ovn-bgp-agent/+/936968 | 15:55 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: dnm: just to trigger CI https://review.opendev.org/c/openstack/neutron/+/938487 | 16:12 |
| opendevreview | Jakub Libosvar proposed openstack/ovn-bgp-agent master: Introduce multinode tempest job https://review.opendev.org/c/openstack/ovn-bgp-agent/+/936968 | 16:12 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: common: fix wait_until_true to support native thread https://review.opendev.org/c/openstack/neutron/+/937843 | 16:13 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: ovs: remove the usage of eventlet in the OVS agent https://review.opendev.org/c/openstack/neutron/+/937765 | 16:13 |
| opendevreview | Jakub Libosvar proposed openstack/ovn-bgp-agent master: Introduce multinode tempest job https://review.opendev.org/c/openstack/ovn-bgp-agent/+/936968 | 16:27 |
| opendevreview | Merged openstack/neutron-lib master: Deprecate is_bsd and remove its use in test suite https://review.opendev.org/c/openstack/neutron-lib/+/938852 | 17:33 |
| opendevreview | Bodo Petermann proposed openstack/neutron-vpnaas master: Use OvnPeriodicWorker for VPN agent check https://review.opendev.org/c/openstack/neutron-vpnaas/+/938915 | 17:36 |
| opendevreview | Merged openstack/neutron-lib master: Enable mypy pre-commit hook https://review.opendev.org/c/openstack/neutron-lib/+/938856 | 17:50 |
| opendevreview | Merged openstack/neutron-lib master: Bump mypy to 1.14.1 (the latest) https://review.opendev.org/c/openstack/neutron-lib/+/938857 | 17:50 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/os-ken master: hub: implement threads related functions https://review.opendev.org/c/openstack/os-ken/+/938673 | 18:42 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/os-ken master: hub: add equivalents for eventlet functions using native https://review.opendev.org/c/openstack/os-ken/+/938875 | 18:42 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/os-ken master: hub: implement listen as native https://review.opendev.org/c/openstack/os-ken/+/938876 | 18:42 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/os-ken master: hub: use native as default implementation https://review.opendev.org/c/openstack/os-ken/+/938337 | 18:42 |
| jamesdenton__ | johnsom do you know offhand if octavia+ovn can be configured to source nat traffic towards the pool members? | 20:38 |
| johnsom | jamesdenton__: no, I don’t think so | 20:42 |
| jamesdenton__ | thank you. seems to present real client IP but does NAT accordingly (router or floating ip) | 20:43 |
| jamesdenton__ | it's working, just looking for different behavior. thanks again! | 20:46 |
| opendevreview | Brian Haley proposed openstack/neutron master: Remove unused classes in functional tests https://review.opendev.org/c/openstack/neutron/+/938962 | 21:47 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!