| opendevreview | Miro Tomaska proposed openstack/neutron master: [WIP]get IPs from the external_ids column as that is more robust https://review.opendev.org/c/openstack/neutron/+/938767 | 04:11 |
|---|---|---|
| opendevreview | Miro Tomaska proposed openstack/neutron master: [WIP]get IPs from the external_ids column as that is more robust https://review.opendev.org/c/openstack/neutron/+/938767 | 04:12 |
| opendevreview | yatin proposed openstack/neutron master: [DNM] functional pyroute2 master https://review.opendev.org/c/openstack/neutron/+/938641 | 05:41 |
| ricolin | Hi dear Neutron team, can we please get some more review on https://review.opendev.org/q/topic:%22add-ovn-db-sync-utils%22 once you got time, review been stuck there for a while now. Many thanks | 07:05 |
| opendevreview | Vasyl Saienko proposed openstack/neutron master: Install DVR src to mac flow for AAPs https://review.opendev.org/c/openstack/neutron/+/938657 | 07:14 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [eventlet-deprecation] Nova notifier retrieves local thread https://review.opendev.org/c/openstack/neutron/+/938651 | 07:53 |
| ralonsoh | ykarel, slaweq ^^ folks, this patch had +2+w but there was a trivial merge conflict. Please, check it if you have time | 07:53 |
| ralonsoh | thanks | 07:53 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: WIP == [eventlet-deprecation] Remove the usage of eventlet in the Neutron API https://review.opendev.org/c/openstack/neutron/+/938659 | 07:54 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: DNM - Test "neutron-ovn-tempest-ipv6-only-ovs*" with WSGI https://review.opendev.org/c/openstack/neutron/+/932601 | 08:00 |
| sahid_ | o/ I have issue regarding Depends-On looks like my change on os-ken is not used: https://review.opendev.org/c/openstack/neutron/+/937765 ansy chance that I get some help? | 08:35 |
| sahid_ | https://zuul.opendev.org/t/openstack/build/4f86ce8c9b804692bd051e97b002489c/log/controller/logs/screen-q-agt.txt?severity=3 | 08:36 |
| sahid_ | we should have a warning from os-ken | 08:37 |
| sahid_ | lajoskatona: you may have an idea? | 08:39 |
| ralonsoh | sahid_, the CI job needs to have os-ken as a dependency | 09:15 |
| ralonsoh | let me check one | 09:15 |
| ralonsoh | for example: neutron-ovs-tempest-with-os-ken-master | 09:16 |
| ralonsoh | you can create a Neutron patch with only one job in the "check" queue | 09:16 |
| ralonsoh | something like this: https://review.opendev.org/c/openstack/neutron/+/932601 (but with one job only that should be neutron-ovs-tempest-with-os-ken-master) | 09:17 |
| ralonsoh | then, if you push this patch for Neutron with the depends-on pointing to the os-ken patch, the CI will checkout the specific commit | 09:17 |
| ralonsoh | slaweq, I've commented on your mail related to vlan_transparency. I also see your topic in the drivers agenda for the next meeting | 09:18 |
| ralonsoh | I think this is going to be approved, makes sense: the config variable is redundant | 09:18 |
| lajoskatona | sahid_: sorry, I just read your message, but ralonsoh is right you need this job: https://opendev.org/openstack/neutron/src/branch/master/zuul.d/tempest-singlenode.yaml#L292 neutron-ovs-tempest-with-os-ken-master | 09:23 |
| sahid_ | lajoskatona: ralonsoh: cool thank you I will make a try | 09:29 |
| 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 | 09:33 |
| opendevreview | yatin proposed openstack/neutron master: [DNM] functional pyroute2 master https://review.opendev.org/c/openstack/neutron/+/938641 | 09:35 |
| opendevreview | yatin proposed openstack/neutron master: [DNM] functional pyroute2 master https://review.opendev.org/c/openstack/neutron/+/938641 | 09:37 |
| ralonsoh | svinota was very fast fixing the pyroute2 issues | 09:53 |
| ralonsoh | mnaser, hello! I think you'll like to check https://review.opendev.org/c/openstack/neutron/+/937026 | 09:54 |
| ralonsoh | slaweq, ^ if you have a couple of mins | 09:54 |
| ralonsoh | ykarel, hey! I'm checking with Neutron API (without eventlet) and 4 workers | 10:28 |
| ralonsoh | I see some weird phenomenon | 10:29 |
| ralonsoh | for example, in https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_005/932601/8/check/neutron-ovn-tempest-ipv6-only-ovs-release-wsgi-19/005b97e/controller/logs/screen-neutron-api.txt | 10:29 |
| ralonsoh | when a port is deleted, I see LogicalSwitchPortUpdateDownEvent -> LogicalSwitchPortUpdateUpEvent-> LogicalSwitchPortUpdateDownEvent one after the other, in a few milliseconds | 10:30 |
| ralonsoh | some examples | 10:30 |
| ralonsoh | https://paste.opendev.org/show/bPJGbWFrOfP3FfnEY97r/ | 10:31 |
| ralonsoh | it doesn't make any sense | 10:31 |
| ralonsoh | and this is a big problem: Neutron is trying to delete and bind this port at the same time | 10:31 |
| ralonsoh | of course, the binding process fails | 10:31 |
| ralonsoh | I'm reporting this issue in LP, but this could be an OVN issue | 10:32 |
| opendevreview | Vasyl Saienko proposed openstack/neutron master: Install DVR src to mac flow for AAPs https://review.opendev.org/c/openstack/neutron/+/938657 | 10:41 |
| opendevreview | Merged openstack/ovn-octavia-provider master: Add Octavia OVN DBs sync cmd https://review.opendev.org/c/openstack/ovn-octavia-provider/+/925747 | 11:01 |
| ykarel | ralonsoh, ohkk so a new issue then | 11:20 |
| ralonsoh | yes... I think so pffffff | 11:20 |
| ralonsoh | already reported in LP | 11:21 |
| ykarel | thx | 11:22 |
| ykarel | ralonsoh, checked one of those port across https://paste.opendev.org/show/bLz8hbsxATMOhX6oA50K/ | 11:43 |
| ykarel | something made port to be deleted ->added->deleted on ovs vswitchd side | 11:44 |
| ykarel | and that lead to the notifications(expected due to the ops on ovs side) on neutron side | 11:44 |
| ralonsoh | let me check that | 11:46 |
| ykarel | also the port you added not related to the failed test case in that job, right? | 11:47 |
| opendevreview | Vasyl Saienko proposed openstack/neutron master: Install DVR src to mac flow for AAPs https://review.opendev.org/c/openstack/neutron/+/938657 | 11:48 |
| opendevreview | Vasyl Saienko proposed openstack/neutron master: Install DVR src to mac flow for AAPs https://review.opendev.org/c/openstack/neutron/+/938657 | 11:49 |
| slaweq | ykarel ralonsoh lajoskatona hi, please review https://review.opendev.org/c/openstack/neutron/+/938640 as bug reporter confirmed that this fixed the issue for them, CI is also green so I think this is good to go now | 11:54 |
| ralonsoh | ykarel, no, is not related, but this behaviour is what is triggering the issues (most of the times) | 11:56 |
| ralonsoh | ykarel, there are plenty of errors like this one in the logs | 11:56 |
| ralonsoh | slaweq, sure, let me check | 11:56 |
| ykarel | ralonsoh, you mean port getting deleted, added, deleted normal on ovs side? | 11:56 |
| ralonsoh | no no, this is not normal, I don't know why this is happening | 11:57 |
| ykarel | ok | 11:57 |
| ralonsoh | of course this is not an error in neutron | 11:57 |
| ralonsoh | nor in OVS | 11:57 |
| ykarel | yeap but that's triggering the notifications in neutron and then race and port not found in nb db | 11:57 |
| ralonsoh | nova is only deleting the port, so far | 11:58 |
| ralonsoh | why OVN controller is releasing the port, claiming it again and releasing it for a second time? | 11:59 |
| ralonsoh | I don't think this behaviour is correct | 11:59 |
| ykarel | isn't ^ due to ovs-vswitchd ? | 12:00 |
| ykarel | del->add-> del | 12:00 |
| ralonsoh | yes | 12:00 |
| ralonsoh | you are right | 12:00 |
| ralonsoh | are we doing something weird in os-vif? let me check | 12:01 |
| opendevreview | Vasyl Saienko proposed openstack/neutron master: Install DVR src to mac flow for AAPs https://review.opendev.org/c/openstack/neutron/+/938657 | 12:06 |
| vsaienko_ | Hello lajoskatona can you please check I've fixed your comments https://review.opendev.org/c/openstack/neutron/+/936807 | 12:10 |
| 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 | 12:35 |
| 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 | 12:35 |
| opendevreview | Merged openstack/neutron master: [eventlet-deprecation] Nova notifier retrieves local thread https://review.opendev.org/c/openstack/neutron/+/938651 | 12:42 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider master: Remove join on helper request daemon thread https://review.opendev.org/c/openstack/ovn-octavia-provider/+/938797 | 12:44 |
| lajoskatona | vsaienko_: sure, checking | 12:52 |
| vsaienko_ | thank you | 13:03 |
| vsaienko_ | neutron team, please review https://review.opendev.org/c/openstack/neutron/+/938657 patch that adds missing flows for AAP entries in DVR environment. In the bug description I've attached terraform reproducer for this issue | 13:14 |
| opendevreview | Gaudenz Steinlin proposed openstack/neutron master: Fixup conntrackd support https://review.opendev.org/c/openstack/neutron/+/938800 | 13:21 |
| ralonsoh | lajoskatona, hello! do you mind checking https://review.opendev.org/c/openstack/neutron/+/937026? Some folks want to have it and the bug has been there for a long time | 13:42 |
| ralonsoh | thanks in advance! | 13:42 |
| 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 | 13:59 |
| *** haleyb|out is now known as haleyb | 14:37 | |
| haleyb | ptl is back :) | 14:43 |
| ralonsoh | welcome! | 14:44 |
| haleyb | hopefully there weren't too many fires, i have a lot of email | 14:44 |
| 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 | 14:47 |
| lajoskatona | haleyb: :-) | 15:49 |
| haleyb | lajoskatona: o/ | 15:52 |
| ralonsoh | haleyb, if you have time https://review.opendev.org/c/openstack/neutron/+/938319 | 15:54 |
| ralonsoh | (again, welcome! hehehe) | 15:54 |
| haleyb | ralonsoh: nothing like a short and easy one to warm my brain up :) i'll take a look | 15:58 |
| noonedeadpunk | hey folks! I'm struggling a bit with understanding IPv6 setup for OVN with ovn-bgp-agent right now. We're already running OVS with IPv6 through dragent, and now OVN with ipv4 and ovn-bgp-agent. But somehow ipv6 there a bit overwhelming for me... | 16:06 |
| noonedeadpunk | it could be partially due to running ovn-bgp-agent only on ovn gateway nodes (which are standalone and != computes) | 16:06 |
| lajoskatona | haleyb: another thing we waited for you and your authority see the meeting logs: https://meetings.opendev.org/meetings/networking/2025/networking.2025-01-07-14.00.log.html#l-151 | 16:06 |
| lajoskatona | haleyb: this is for the self wf+1 patch https://review.opendev.org/c/openstack/neutron/+/936235 | 16:07 |
| noonedeadpunk | but in short, I've enabled expose_ipv6_gua_tenant_networks, and according to the doc agent on net node should be enough (with nbdb driver and underlay expose method). | 16:07 |
| noonedeadpunk | though I don't see any announcement of ipv6 tenant network | 16:08 |
| noonedeadpunk | but also - with dragent we had small /64 for L3 router specifically, which was added to "public" vlan (l2). and all tenants were given /64 subnet from /48 subnet pool | 16:09 |
| haleyb | lajoskatona: yes, i noticed that right before i went on break, will read the meeting notes, thanks for reminder | 16:09 |
| noonedeadpunk | is the concept is the same with `expose_ipv6_gua_tenant_networks`? as it somehow felt that networks can be directly advertised without a need for this small /64 on router (and eventually without router itself?) | 16:10 |
| opendevreview | Gaudenz Steinlin proposed openstack/neutron master: Fixup conntrackd support https://review.opendev.org/c/openstack/neutron/+/938800 | 16:10 |
| haleyb | ralonsoh: i had a comment on that change btw, just added | 16:11 |
| lajoskatona | haleyb: yeah, I saw your comment on the patch, good catch | 16:12 |
| lajoskatona | haleyb: we can have another round of discussion with everybody back, and perhaps trying to have Liu as well | 16:13 |
| opendevreview | Merged openstack/neutron master: Setup DVR router ARP for AAP with correct MAC https://review.opendev.org/c/openstack/neutron/+/936807 | 16:14 |
| haleyb | lajoskatona: yes. and i can understand his frustration maybe need to come up with some options | 16:17 |
| opendevreview | Serhii Ivanov proposed openstack/neutron-specs master: Add `Agent Startup State Tracking` spec https://review.opendev.org/c/openstack/neutron-specs/+/935724 | 16:20 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider master: Remove join on helper request daemon thread https://review.opendev.org/c/openstack/ovn-octavia-provider/+/938797 | 16:39 |
| opendevreview | Merged openstack/ovn-octavia-provider master: Add octavia_client with openstacksdk https://review.opendev.org/c/openstack/ovn-octavia-provider/+/930692 | 16:44 |
| frickler | noonedeadpunk: if you find out anything, I'd be interested in answers, too ;) for now I'll just note that running dragent with OVN works fine for me with IPv6 (no BGP for v4) | 16:57 |
| noonedeadpunk | frickler: yeah, I can recall you saying that some time ago (like year or so lol) | 16:58 |
| noonedeadpunk | so I'm accounting fallback to dragent as an option -thanks! | 16:58 |
| noonedeadpunk | just our net folks really want to have BGP for v4 | 16:59 |
| noonedeadpunk | and align approach for v4 and v6 | 16:59 |
| noonedeadpunk | and v4 I kind of got working, with couple of patches that pending review. | 17:04 |
| noonedeadpunk | (except single vrf for now) | 17:04 |
| opendevreview | Gaudenz Steinlin proposed openstack/neutron master: Fixup conntrackd support https://review.opendev.org/c/openstack/neutron/+/938800 | 17:13 |
| opendevreview | Vasyl Saienko proposed openstack/neutron master: Install DVR src to mac flow for AAPs https://review.opendev.org/c/openstack/neutron/+/938657 | 17:58 |
| opendevreview | Brian Haley proposed openstack/neutron master: Consume neutron-lib Context class project_id change https://review.opendev.org/c/openstack/neutron/+/936845 | 18:01 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: Remove dibbler code from l3 agent and elsewhere https://review.opendev.org/c/openstack/neutron/+/934283 | 22:05 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: Remove dibbler code from l3 agent and elsewhere https://review.opendev.org/c/openstack/neutron/+/934283 | 22:10 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: Clean up win32 specific tags in requirements.txt https://review.opendev.org/c/openstack/neutron/+/938847 | 22:12 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: Remove is_bsd skips in test suite https://review.opendev.org/c/openstack/neutron/+/938849 | 22:14 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: WIP Remove linuxbridge driver https://review.opendev.org/c/openstack/neutron/+/927216 | 22:27 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: WIP Remove linuxbridge driver https://review.opendev.org/c/openstack/neutron/+/927216 | 22:27 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: tests: Remove unnecessary hasattr() by always defining attribute https://review.opendev.org/c/openstack/neutron/+/938850 | 22:30 |
| opendevreview | Jakub Libosvar proposed openstack/ovn-bgp-agent master: WIP: Introduce multinode tempest job https://review.opendev.org/c/openstack/ovn-bgp-agent/+/936968 | 22:33 |
| 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 | 22:43 |
| opendevreview | Brian Haley proposed openstack/neutron master: Fix some pylint issues in the test tree https://review.opendev.org/c/openstack/neutron/+/938853 | 22:54 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: Bump mypy to 1.14.1 (the latest) https://review.opendev.org/c/openstack/neutron/+/938854 | 23:14 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: Enable mypy pre-commit hook https://review.opendev.org/c/openstack/neutron/+/938855 | 23:14 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron-lib master: Enable mypy pre-commit hook https://review.opendev.org/c/openstack/neutron-lib/+/938856 | 23:21 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron-lib master: Bump mypy to 1.14.1 (the latest) https://review.opendev.org/c/openstack/neutron-lib/+/938857 | 23:21 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: doc: Remove description of CI jobs from documentation https://review.opendev.org/c/openstack/neutron/+/938858 | 23:30 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!