| opendevreview | OpenStack Proposal Bot proposed openstack/neutron-vpnaas-dashboard master: Imported Translations from Zanata https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/955257 | 03:24 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/neutron-tempest-plugin master: Use plugin architecture to register options https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/955192 | 05:04 |
| opendevreview | Dr. Jens Harbott proposed openstack/neutron stable/2024.2: [DNM] Check designate issue with oslo.utils uc revert https://review.opendev.org/c/openstack/neutron/+/951494 | 08:27 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: [ML2/OVN] Validate allowed address pairs and distributed ports https://review.opendev.org/c/openstack/neutron/+/954570 | 08:53 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Use ALLOWED_ADDRESS_PAIR resources constant from neutron_lib https://review.opendev.org/c/openstack/neutron/+/955488 | 08:54 |
| opendevreview | Bence Romsics proposed openstack/neutron unmaintained/yoga: [unmaintained-only] Do not use flow_group_id https://review.opendev.org/c/openstack/neutron/+/955585 | 11:20 |
| opendevreview | Bence Romsics proposed openstack/neutron unmaintained/xena: [unmaintained-only] Do not use flow_group_id https://review.opendev.org/c/openstack/neutron/+/955586 | 11:20 |
| opendevreview | Bence Romsics proposed openstack/neutron unmaintained/wallaby: [unmaintained-only] Do not use flow_group_id https://review.opendev.org/c/openstack/neutron/+/955587 | 11:20 |
| opendevreview | Bence Romsics proposed openstack/neutron unmaintained/victoria: [unmaintained-only] Do not use flow_group_id https://review.opendev.org/c/openstack/neutron/+/955588 | 11:20 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [eventlet-removal] Don't use eventlet in the unit tests https://review.opendev.org/c/openstack/neutron/+/952258 | 12:57 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [eventlet-removal] Don't use eventlet in the unit tests https://review.opendev.org/c/openstack/neutron/+/952258 | 12:58 |
| opendevreview | Merged openstack/neutron stable/2025.1: [FT] Isolate randomly failing address scope tests https://review.opendev.org/c/openstack/neutron/+/955466 | 12:59 |
| mlavalle | #startmeeting networking | 13:00 |
| opendevmeet | Meeting started Tue Jul 22 13:00:04 2025 UTC and is due to finish in 60 minutes. The chair is mlavalle. Information about MeetBot at http://wiki.debian.org/MeetBot. | 13:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 13:00 |
| opendevmeet | The meeting name has been set to 'networking' | 13:00 |
| mlavalle | Ping list: bcafarel, elvira, frickler, mlavalle, mtomaska, slaweq, tobias-urdin, ykarel, lajoskatona, jlibosva, averdagu, haleyb, ralonsoh | 13:00 |
| slaweq | o/ | 13:00 |
| lajoskatona | o/ | 13:00 |
| bcafarel | o/ | 13:00 |
| elvira | o/ hi | 13:00 |
| ykarel | o/ | 13:00 |
| mtomaska | o/ | 13:01 |
| ralonsoh | hello | 13:01 |
| mlavalle | #announcements | 13:01 |
| rubasov | o/ | 13:02 |
| cbuggy | o/ | 13:02 |
| mlavalle | Hi everybody. As you might remember, our fearless leader is on PTO this week | 13:02 |
| mlavalle | We are currently in Week R-10 of Flamingo | 13:03 |
| mlavalle | Our next milestone in this development cycle will be Flamingo-3, week of August 25th | 13:03 |
| mlavalle | Final 2025.2 Flamingo release: October 3rd, 2025 | 13:04 |
| mlavalle | #link https://releases.openstack.org/flamingo/schedule.html | 13:05 |
| mlavalle | The next OpenInfra PTG will take place October 27-31, 2025 and registration for the event is now open | 13:05 |
| mlavalle | #link https://ptg.openinfra.dev/ | 13:06 |
| mlavalle | Reminder: this Friday the drivers meeting is cancelled but if you have topics for next week, please add it to the wiki @ https://wiki.openstack.org/wiki/Meetings/NeutronDrivers | 13:07 |
| mlavalle | that was all the announcements i had, any others? | 13:08 |
| mlavalle | ok, moving on | 13:09 |
| mlavalle | #topic bugs | 13:09 |
| mlavalle | Last week's deputy was sahid, but I didn't find his report. Did anybody see it? | 13:10 |
| lajoskatona | nope | 13:10 |
| ralonsoh | hmmm my bad, I should have checked it on monday | 13:11 |
| mlavalle | that's ok. let's spend a few minutes triaging some bugs here | 13:11 |
| ralonsoh | anyway, we have several new ones | 13:11 |
| ralonsoh | that are not assigned | 13:12 |
| ralonsoh | OVN: Intermittent metadata failures for SR-IOV VMs | 13:12 |
| ralonsoh | #link https://bugs.launchpad.net/neutron/+bug/2117078 | 13:12 |
| ralonsoh | I tried to quick triage this bug | 13:12 |
| mlavalle | I see that | 13:13 |
| ralonsoh | but to debug this issue we would need some logs | 13:13 |
| ralonsoh | in particular when the port is bound and when it tries to retrieve the metadata | 13:13 |
| ralonsoh | I'll ask for all of them, if possible | 13:13 |
| mlavalle | ok, you follow up that one | 13:14 |
| mlavalle | thanks | 13:14 |
| ralonsoh | next one | 13:14 |
| ralonsoh | neutron-openvswitch-agent crashes on start | 13:14 |
| ralonsoh | #link https://bugs.launchpad.net/neutron/+bug/2117153 | 13:14 |
| ralonsoh | This folk is using an old version | 13:14 |
| ralonsoh | and it stating that cannot reproduce it with a newer version | 13:15 |
| ralonsoh | (if I'm not wrong) | 13:15 |
| mlavalle | yeap | 13:15 |
| mlavalle | that's what he says | 13:15 |
| ralonsoh | so I would just suggest to update os-ken and test again | 13:15 |
| ralonsoh | I'll update the LP bug | 13:15 |
| mlavalle | sounds good | 13:16 |
| ralonsoh | (but to be honest, I didn't find any revelant patch in os-ken) | 13:16 |
| ralonsoh | next one | 13:16 |
| ralonsoh | [eventlet-removal] Add H999 hacking check to ban eventlet imports | 13:16 |
| ralonsoh | #link https://bugs.launchpad.net/neutron/+bug/2117373 | 13:16 |
| ralonsoh | I'll assign this one to me | 13:16 |
| mlavalle | that is under control by you | 13:16 |
| ralonsoh | but this is for everyone | 13:16 |
| ralonsoh | I mean | 13:16 |
| ralonsoh | for any networking project that removes eventlet | 13:17 |
| ralonsoh | it is needed to do something like this: | 13:17 |
| ralonsoh | https://review.opendev.org/c/openstack/networking-sfc/+/955295 | 13:17 |
| mlavalle | I see what you mean | 13:17 |
| mlavalle | undestood | 13:17 |
| ralonsoh | next one | 13:17 |
| ralonsoh | [neutron-tempest-plugin] Test ``test_create_router_update_external_gateways`` failing | 13:17 |
| lajoskatona | I ill check stadiums this week I hope for this one | 13:17 |
| ralonsoh | lajoskatona, thanks! | 13:17 |
| ralonsoh | #link https://bugs.launchpad.net/neutron/+bug/2117383 | 13:18 |
| ralonsoh | I opened this one because I saw this error | 13:18 |
| ralonsoh | but only once | 13:18 |
| ralonsoh | but that could be a problem in the testing FW | 13:18 |
| ralonsoh | Details: {'type': 'IpAddressAlreadyAllocated', 'message': 'IP address 172.24.5.33 already allocated in subnet 8936b536-5feb-4f28-ae4d-6733056bf18c', 'detail': ''} | 13:18 |
| ralonsoh | but, to be honest, I did see this problem once | 13:19 |
| ralonsoh | next one | 13:19 |
| ralonsoh | test_subport_delete random failure | 13:19 |
| ralonsoh | #link https://bugs.launchpad.net/neutron/+bug/2117405 | 13:19 |
| ralonsoh | I told ykarel to assign this one to me | 13:19 |
| mlavalle | I can follow the one for the tempest test | 13:19 |
| ralonsoh | mlavalle, thanks! | 13:19 |
| ralonsoh | so I assigned 2117405 to me | 13:20 |
| ykarel | thx ralonsoh i forgot to assign that | 13:20 |
| ralonsoh | no problem at all | 13:20 |
| ralonsoh | next one | 13:20 |
| ralonsoh | Horizon shows all prefix lengths when creating subnet from subnet pool | 13:20 |
| ralonsoh | #link https://bugs.launchpad.net/neutron/+bug/2116927 | 13:20 |
| ralonsoh | to be honest, I don't know if that is a problem in horizon or neutron | 13:21 |
| ralonsoh | Neutron is failing as expected | 13:21 |
| lajoskatona | I think it is irealted to my work in Horizon so I am on it | 13:22 |
| lajoskatona | I commented, but will assign it to myself | 13:22 |
| ralonsoh | so maybe, after this patch, horizon is not enforcing the subnet mask | 13:22 |
| mlavalle | thanks lajoskatona | 13:22 |
| ralonsoh | subnet range | 13:22 |
| ralonsoh | lajoskatona, thanks for taking care | 13:22 |
| ralonsoh | last one is already assigned, Bence sent some patches to um branches: https://bugs.launchpad.net/neutron/+bug/2117477 | 13:23 |
| ralonsoh | and that's all | 13:23 |
| mlavalle | and the last LP in the list is already owned by rubas | 13:23 |
| mlavalle | rubasov: | 13:23 |
| mlavalle | we are caught up | 13:23 |
| mlavalle | any other bugs to discuss? | 13:24 |
| yusufgungor_ | yes @mlavalle we have | 13:25 |
| yusufgungor_ | can i talk about it? | 13:25 |
| mlavalle | it would be better if you open a report in Launchpad | 13:25 |
| mlavalle | that way the conversation is recorded in the report and everybody can follow it | 13:26 |
| yusufgungor_ | yes there exist a launchpad report, actually we want to talk about the review of that bug | 13:26 |
| mlavalle | what's the report and the associated patch? | 13:27 |
| mlavalle | go ahead | 13:27 |
| yusufgungor_ | a bug fix for neutron bgp dynamic routing : https://bugs.launchpad.net/neutron/+bug/2100752 | 13:28 |
| yusufgungor_ | Lajos Katona (@lajoskatona) and Jens Harbott (@frickler) reviewed it already. | 13:28 |
| yusufgungor_ | 13:28 | |
| yusufgungor_ | @frickler deferred to other reviewers after his review. | 13:28 |
| yusufgungor_ | We think it is a critical bug and should be fixed and want some guidance to progress it. | 13:28 |
| yusufgungor_ | https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/953428 | 13:28 |
| ralonsoh | The main problem with this patch is that almost nobody can test it and that we don't know the code well to review it | 13:29 |
| lajoskatona | I asked some test coverage for the issue, is that possible in our current CI? (sorry I have to check the details again as it was weeks ago) | 13:29 |
| yusufgungor_ | @lajoskatona we also discussed the test coverage issue with @frickler, but it will be over engineering. it is really simple fix which only add one more filter while getting the port from db | 13:31 |
| yusufgungor_ | local test scenarios also written under review discussion | 13:33 |
| lajoskatona | yes from that perspective you are right that the patch itself is quite simple filtering for the case when the port is migrated | 13:34 |
| mlavalle | so if we find a way to test the fix in our CI, could we move ahead with this review? | 13:36 |
| mlavalle | the size of the fix doesn't preclude the need for testing it | 13:37 |
| yusufgungor_ | when getting the ports from DB, now are adding a new filter "ML2PortBinding.status == lib_consts.ACTIVE" | 13:37 |
| yusufgungor_ | I think the main question is should bgp advertise a port ip with not ACTIVE status? I think we should not. | 13:37 |
| yusufgungor_ | @mlavalle you are right, but in this patch we are not adding a new feature or complex fix. Existing test coverage could find the problem if any exist on this patch | 13:38 |
| mlavalle | under that assumption, code coverage would diminish over time constantly | 13:40 |
| lajoskatona | +1 | 13:41 |
| mlavalle | here's the situation: | 13:41 |
| lajoskatona | Perhaps we can just make sure that the above assertion (port.status==ACTIVE to be advertised the IP) add to existing tests in neutron-tempest-plugin | 13:42 |
| mlavalle | on one hand we don't have in the community all the expertise to review this code with confidence | 13:42 |
| yusufgungor_ | @mlavalle so we can say every bug fix has a responsibility to increase the test coverage even minor ones | 13:43 |
| yusufgungor_ | so can we say neutron dynamic routing deprecated? | 13:43 |
| mlavalle | on the other, if we can make everybody confident that we are not introducing side effects with some sort of testing, we could move forward | 13:43 |
| mlavalle | yusufgungor_: we are a community and we try as much as we can to help each other. help us to help you | 13:44 |
| yusufgungor_ | @mlavalle i know, thanks for your great efforts. Do you know any other guys who are using neutron bgp dynamic routing? we may let them to test locally? | 13:45 |
| mlavalle | I'll initiate a thread in the mailing list about this issue. Let's see if somebody offers help | 13:46 |
| lajoskatona | ack, thanks | 13:47 |
| yusufgungor_ | @mlavalle thanks. we also can write a test, but since we will be the ones writing the test, it won't be very meaningful when it comes to testing the test. | 13:47 |
| mlavalle | let us see your test proposal anyways | 13:48 |
| mlavalle | for now, let's move on | 13:48 |
| lajoskatona | +1, tests can help to better understand the situation, so I hope it will help | 13:48 |
| yusufgungor_ | as we discussed with @frickler under review, the test will be much more complex than this small fix. Since there isn't enough developer to understand this fix, it won't be possible to decide whether the test is sufficient and correct. | 13:49 |
| mlavalle | this week's deputy is jlibosva. he is not online but I will make sure he is aware | 13:50 |
| mlavalle | #topic community goals | 13:50 |
| mlavalle | any updates from neutronclient, lajoskatona | 13:51 |
| mlavalle | ? | 13:51 |
| lajoskatona | no, just the bug from Horizon we discussed earlier | 13:51 |
| mlavalle | thanks | 13:52 |
| mlavalle | how about eventlet, ralonsoh ? | 13:52 |
| ralonsoh | yes | 13:52 |
| ralonsoh | I'm fighting with https://review.opendev.org/c/openstack/neutron/+/952258 | 13:52 |
| ralonsoh | I've managed to execute all UTs withoyt eventlet | 13:52 |
| ralonsoh | there are some UTs skipped... | 13:52 |
| ralonsoh | we can manage this later | 13:52 |
| ralonsoh | but the main problem is zuul/CI | 13:52 |
| ralonsoh | I cannot pass the needed variables to FT CI to run it with eventlet | 13:53 |
| ralonsoh | https://zuul.opendev.org/t/openstack/status?change=952258 | 13:53 |
| ralonsoh | I'll push a new PS to limit the execution of FT to a couple of tests | 13:53 |
| ralonsoh | and I'll ask qa folks about this | 13:53 |
| ralonsoh | I'm must be doing something wrong in tox | 13:54 |
| ralonsoh | that's all I have | 13:54 |
| mlavalle | thanks for the update and hard work on this topic | 13:54 |
| mlavalle | #topic on-demand | 13:55 |
| mlavalle | any other topics we should discuss today? | 13:55 |
| cardoe | There's a patch that Ironic would really like to see landed that I wanted to mention. | 13:55 |
| mlavalle | cardoe: ok, which one? | 13:55 |
| cardoe | https://review.opendev.org/c/openstack/neutron/+/945497 | 13:56 |
| mlavalle | it's in my pile. I'll review it today | 13:56 |
| ralonsoh | ah ok, let me check again it | 13:56 |
| mlavalle | anything else? | 13:58 |
| mlavalle | ok, have a great week | 13:59 |
| mlavalle | #endmeeting | 13:59 |
| opendevmeet | Meeting ended Tue Jul 22 13:59:27 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 13:59 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/networking/2025/networking.2025-07-22-13.00.html | 13:59 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/networking/2025/networking.2025-07-22-13.00.txt | 13:59 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/networking/2025/networking.2025-07-22-13.00.log.html | 13:59 |
| ralonsoh | nye | 13:59 |
| lajoskatona | Bye | 14:00 |
| yusufgungor_ | bye, thanks everyone 🙏 | 14:00 |
| mlavalle | yusufgungor_: thank you for working on this issue | 14:01 |
| slaweq | o/ | 14:01 |
| yusufgungor_ | @mlavalle my pleasure | 14:02 |
| cardoe | ralonsoh: I updated my ovn-db-sync with your requested release note update so it needs another re-review. | 14:06 |
| ralonsoh | cardoe, let me check | 14:07 |
| cardoe | You mentioned it'll need to be backported so I'll do that as each one lands. How far back would you like me to go? 2024.1? | 14:07 |
| ralonsoh | up the latest stable branch | 14:08 |
| ralonsoh | that will be 2024.1, yes, because 2023.2 was marked as EOL | 14:09 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [eventlet-removal] Don't use eventlet in the unit tests https://review.opendev.org/c/openstack/neutron/+/952258 | 14:33 |
| daidv | Hi mlavalle , some weeks ago, we discussed about Distributed DNS Forwarder spec, if you have some free time, could you review this spec for me: https://review.opendev.org/c/openstack/neutron-specs/+/952737 | 16:24 |
| daidv | I want to move on to implementation based on that spec | 16:25 |
| opendevreview | Merged openstack/neutron stable/2024.2: [FT] Isolate randomly failing address scope tests https://review.opendev.org/c/openstack/neutron/+/955467 | 16:48 |
| opendevreview | Merged openstack/neutron master: Update report state rpc API https://review.opendev.org/c/openstack/neutron/+/945497 | 19:31 |
| opendevreview | Merged openstack/neutron-specs master: Add spec for Distributed DNS Forwarder https://review.opendev.org/c/openstack/neutron-specs/+/952737 | 22:15 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!