Tuesday, 2025-01-21

opendevreviewSahid Orentino Ferdjaoui proposed openstack/neutron master: async_process: fix potential race condition with respawn  https://review.opendev.org/c/openstack/neutron/+/93962707:40
opendevreviewSahid Orentino Ferdjaoui proposed openstack/neutron master: async_process: remove usage of eventlet for AsyncProcess  https://review.opendev.org/c/openstack/neutron/+/93934807:40
opendevreviewSahid Orentino Ferdjaoui proposed openstack/neutron master: ovs: reimplement signals handling  https://review.opendev.org/c/openstack/neutron/+/93932107:40
opendevreviewSahid Orentino Ferdjaoui proposed openstack/neutron master: common: fix wait_until_true to support native thread  https://review.opendev.org/c/openstack/neutron/+/93784307:40
opendevreviewSahid Orentino Ferdjaoui proposed openstack/neutron master: ovs: remove the usage of eventlet in the OVS agent  https://review.opendev.org/c/openstack/neutron/+/93776507:40
lajoskatonaykarel, slaweq: Hi, could you please take a look to these do improvement patches for tap-as-a-service: https://review.opendev.org/q/topic:%22taas_driver_docs%22 , thanks in advance08:00
slaweqlajoskatona hi, sure thing08:00
opendevreviewMerged openstack/neutron stable/2023.2: functional: Handle ovsdb monitor returning inserts in different checks  https://review.opendev.org/c/openstack/neutron/+/93949708:54
ralonsohbcafarel, hi! can you check https://review.opendev.org/c/openstack/neutron/+/937968?08:55
ralonsohthanks!08:55
bcafarelsure thing, looking08:55
bcafareldone, sorry I forgot to follow up on that one!08:57
ralonsohbcafarel, no problem at all08:59
ralonsohthanks!08:59
ralonsohhey folks, please check https://review.opendev.org/q/topic:%22bug/2087942%22. These patches are small and quick to review08:59
ralonsohand the CI is passing08:59
sahidralonsoh: o/ any you have a second check to https://review.opendev.org/c/openstack/neutron/+/939321 I may have missed something, if you can perhaps share some pointers? Ny understanding is that we want to transfert the signal to the thread that is handling the ovs agent process09:22
ralonsohsahid, let me check09:27
ralonsohsahid, some kind of testing would be needed there09:27
ralonsohyou can't deliver this patch without testing09:28
opendevreviewRodolfo Alonso proposed openstack/neutron master: Check existence of GW port before trying to delete it  https://review.opendev.org/c/openstack/neutron/+/93945109:34
sahidralonsoh: yes sure that makes sense, at least i'm in the good direction, thank you09:42
sahidbtw good catch regarding the async process issue 09:43
sahidI have responded to your comment regarding the try..catch, i though it was a nice way to handle it but  i will let you reply :-)09:44
opendevreviewSahid Orentino Ferdjaoui proposed openstack/neutron master: ml2/ovs: change log level from DEBUG to INFO for port deletion during binding  https://review.opendev.org/c/openstack/neutron/+/92990910:05
opendevreviewSlawek Kaplonski proposed openstack/neutron-tempest-plugin master: Add scenario tests for the Vlan QinQ feature  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/93777810:31
opendevreviewEduardo Olivares proposed x/whitebox-neutron-tempest-plugin master: Add config dict dataplane_podified_services  https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/93951910:57
opendevreviewEduardo Olivares proposed x/whitebox-neutron-tempest-plugin master: Add config dict dataplane_podified_services  https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/93951910:57
opendevreviewMerged openstack/neutron stable/2023.2: Allow network owner reader to get subnets  https://review.opendev.org/c/openstack/neutron/+/93796811:04
opendevreviewRodolfo Alonso proposed openstack/neutron master: WIP == [OVN] ``PortBindingUpdateUpEvent``  https://review.opendev.org/c/openstack/neutron/+/93934511:43
opendevreviewRodolfo Alonso proposed openstack/neutron master: DNM - Test "neutron-ovn-tempest-ipv6-only-ovs*" with WSGI  https://review.opendev.org/c/openstack/neutron/+/93934611:43
opendevreviewSlawek Kaplonski proposed openstack/neutron-lib master: [API-REF] Add clarification about QinQ and transparent vlan differences  https://review.opendev.org/c/openstack/neutron-lib/+/93970811:44
ralonsohslaweq, some comments in https://review.opendev.org/q/topic:%22bug/2095323%2212:18
ralonsohif you want to address them12:18
slaweqthx, I will address them after the meeting I will start in few minutes12:27
slaweqralonsoh please also check https://review.opendev.org/c/openstack/neutron/+/939248 when You will have a minute12:29
lajoskatonahaleyb: I will be late from the meeting most probably12:31
ralonsohslaweq, sure12:32
ralonsohslaweq, did you test 939248? just asking13:24
opendevreviewMichel Nederlof proposed openstack/ovn-bgp-agent master: Fix cleanup of rules per evpn device  https://review.opendev.org/c/openstack/ovn-bgp-agent/+/92781613:37
*** svinota_ is now known as svinota13:43
opendevreviewSlawek Kaplonski proposed openstack/neutron master: QinQ implementation for the ML2/OVN backend  https://review.opendev.org/c/openstack/neutron/+/93763313:44
slaweqralonsoh with this patch I was able to run this upgrade-check tool, without this patch it is failing13:46
slaweqso yes, I did manually some test but I didn't really test cases where this table exists or not exists in the db13:47
ralonsohperfect13:48
gaudenz__ralonsoh, slaweq, haleyb, fungi: The conntrackd fixup change which removes the GPL script is ready for another review since some days. I think I fixed the issue which caused some tests to fail previously. Would be great if you could have a look so we can get this merged and solve the GPL issue. https://review.opendev.org/c/openstack/neutron/+/93880013:58
ralonsohgaudenz__, sure, after the meeting13:59
gaudenz__No worries.13:59
*** gaudenz__ is now known as gaudenz13:59
haleyb#startmeeting networking14:00
opendevmeetMeeting started Tue Jan 21 14:00:43 2025 UTC and is due to finish in 60 minutes.  The chair is haleyb. Information about MeetBot at http://wiki.debian.org/MeetBot.14:00
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.14:00
opendevmeetThe meeting name has been set to 'networking'14:00
haleybPing list: bcafarel, elvira, frickler, mlavalle, mtomaska, obondarev, slaweq, tobias-urdin, ykarel, lajoskatona, jlibosva, averdagu, amotoki, haleyb, ralonsoh14:00
mlavalle\o14:00
ralonsohhello14:01
svinotaralonsoh, I couldn't login to edit the wiki page, so if there will be one-two minute for an ad-hoc announce in the end of the meeting, would be nice.14:01
ralonsohralonsoh, sure, there will be time14:01
slaweqo/14:01
rubasovo/14:01
lajoskatonao/14:01
haleybsvinota: there will be an on-demand time at the end, thanks for attending14:01
svinotathanks14:02
haleyb#announcements14:02
haleybWe are in week R-10 if my math is correct14:02
haleyb#link https://releases.openstack.org/epoxy/schedule.html14:02
opendevreviewRodolfo Alonso proposed openstack/neutron master: WIP == [eventlet-deprecation] Remove the usage of eventlet in the Neutron API  https://review.opendev.org/c/openstack/neutron/+/93865914:03
haleybin R-6 we will have non-client library freeze14:03
opendevreviewRodolfo Alonso proposed openstack/neutron master: DNM - Test "neutron-ovn-tempest-ipv6-only-ovs*" with WSGI  https://review.opendev.org/c/openstack/neutron/+/93260114:03
haleybin R-5 will be E-3 milestone14:03
bcafarellate o/14:03
haleybReminder: If you have a topic for the drivers meeting on Friday, please add it to the wiki @ https://wiki.openstack.org/wiki/Meetings/NeutronDrivers14:05
haleybAnd of course a reminder to use the priorities dashboard for patches in the "ready to merge" state14:05
haleybI sent the Linuxbridge removal email to the ML last week, and it seems people have accepted is was the right thing to do14:06
slaweqgreat :) thx haleyb for doing this finally14:07
haleybthere was a related email from Cern regarding knowledge sharing14:07
lajoskatonagood question if heores will move the code to x/ namsspace, let's see:-)14:07
haleybi will not hold my breath on that, i would hope we can spend any time helping with migration questions and OVN adoption14:08
lajoskatona+1 for pushing toward migration14:09
haleybi did not have any other announcements14:09
haleyb#topic Bugs14:10
haleyblajoskatona was deputy last week14:10
haleyb#link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/JIRUHN7HMJUDB7KB55UYUZ5Q2IK2MC6X/14:10
haleybthere were quite a few14:10
opendevreviewSahid Orentino Ferdjaoui proposed openstack/neutron master: async_process: fix potential race condition with respawn  https://review.opendev.org/c/openstack/neutron/+/93962714:11
opendevreviewSahid Orentino Ferdjaoui proposed openstack/neutron master: async_process: remove usage of eventlet for AsyncProcess  https://review.opendev.org/c/openstack/neutron/+/93934814:11
opendevreviewSahid Orentino Ferdjaoui proposed openstack/neutron master: ovs: reimplement signals handling  https://review.opendev.org/c/openstack/neutron/+/93932114:11
opendevreviewSahid Orentino Ferdjaoui proposed openstack/neutron master: common: fix wait_until_true to support native thread  https://review.opendev.org/c/openstack/neutron/+/93784314:11
opendevreviewSahid Orentino Ferdjaoui proposed openstack/neutron master: ovs: remove the usage of eventlet in the OVS agent  https://review.opendev.org/c/openstack/neutron/+/93776514:11
haleyblajoskatona: were there any bugs you wanted to highlight?14:11
lajoskatonanot really there were few without owners14:12
lajoskatonabut as I remember the high prio ones are under control14:12
haleybok14:13
lajoskatonahttps://bugs.launchpad.net/neutron/+bug/209484214:13
lajoskatonathis one in the high list without owner14:13
lajoskatona"ovs.db.error.Error: ovsdb error: 0 values when type requires between 1 and 9223372036854775807" when router is attached to network without subnet14:13
haleybthat looks familiar, or at least we just fixed something related in quota14:13
haleybat least it looks like the quota one, but it wasn't in ovsdb of course14:14
ralonsohthe cause is explained in the description14:14
haleyboh, maybe this is related to a fix ralonsoh had regarding gw_port without an IP address?14:14
ralonsohwe can't create a GW port wihtout subnet in the GW network14:14
ralonsohhaleyb, I don't have anything yet14:15
ralonsohthere is a quick attempt https://review.opendev.org/c/openstack/neutron/+/93925314:15
ralonsohbut this is something that needs to be tested14:15
haleybralonsoh: ah, that was it, it was ihrachys not you, sorry i mis-remembered14:16
lajoskatonait is visible in zuul logs also not just for master14:17
lajoskatonasee the opensearch logs I added as comment14:17
haleyback, i guess we can wait until that change is updated to see if it fixes it14:18
lajoskatona+114:18
haleybanother one ihar filed was14:18
haleyb#link https://bugs.launchpad.net/neutron/+bug/209484614:19
haleybtrunk driver functional failure14:19
haleybaccording to codesearch there was only 1 failure so far, but it looks like something that shouldn't have happened14:20
ralonsohit happened only once14:20
ralonsohand this is a functional test14:20
ralonsohI would reduce the importance, but I'll keep it in the list14:20
haleybyes, let's see if it happens again first14:21
lajoskatonait is medium at the moment due to low frequesncy, but we can dereease14:21
haleybsure14:22
haleybthere was another medium i started to take a look at14:22
haleyb#link https://bugs.launchpad.net/neutron/+bug/209509714:22
haleyb500 error creating resource with emoji for name14:22
haleybit happens for any string field, for example, description too14:23
lajoskatonathat's a funny one, but really happens14:23
haleybmy original thought was to just fix the network db code, but now i think we should change neutron-lib to add a validator for such fields14:23
haleybbut i wanted to get others thoughts on that14:23
ralonsohan extra check for printable chars is ok14:24
lajoskatonahaleyb: yes a validator sounds good14:24
ralonsohit won't break anything using "normal" chars14:24
ralonsohand with the n-lib validator, we'll have a printable exception, not error 50014:24
haleyband i'm not sure it's a medium as i originally tagged it, since it only affects the caller14:24
ralonsohthe only problem here is to return a valid exception14:25
haleybyes, a quick hack of the network db code and i could get a 400,14:25
ralonsohnot 50014:25
ralonsohso I would lower the priority14:25
haleybi.e. InvalidInput such exception14:25
ralonsohexactly14:25
lajoskatonanova gives back 400 for such input for example14:26
ralonsohas it should be, this is a bad request, not a server error14:26
lajoskatonaexactly, so good to have simlar response from Neutron API also14:27
slaweq++ for some 4xx error in such case14:27
haleybi took it and can work on a validator14:28
haleybnext unowned bug14:28
haleyb#link https://bugs.launchpad.net/neutron/+bug/209518514:28
haleybihrachys was busy last week14:29
haleybtest_legacy_router_conntrack_helper failure14:29
lajoskatonaeven on Friday I got a bunch of bugs from Him :-)14:29
haleybhe gave a possible solution, but would need more investigation14:29
lajoskatonathis one was seen in zuul logs only on stable/14:29
ralonsohlast sentence of the description is a clue for a fix14:30
ralonsohso I would give a try14:30
lajoskatonabut as he wrote can happen on master14:30
opendevreviewIhar Hrachyshka proposed openstack/neutron master: QinQ implementation for the ML2/OVN backend  https://review.opendev.org/c/openstack/neutron/+/93763314:30
haleybright, just needs an owner if someone can take it14:30
haleybthe last bug without owner is14:31
haleyb#link https://bugs.launchpad.net/neutron/+bug/209509114:31
haleybykarel_: you filed this, does it happen frequently enough to debug it further?14:32
lajoskatonayesterday we discussed on the CI meeting with ykarel and slwaq and mlavalle14:32
ykarel_haleyb, happened only once14:32
haleybah14:32
ykarel_mlavalle, will be working on alternative14:33
* haleyb was out yesterday as it was a holiday14:33
*** ykarel_ is now known as ykarel14:33
haleybok, as long as someone is working on it14:33
haleybany other bugs to discuss?14:33
haleybykarel is the deputy this week, mtomaska next week14:34
haleybis that ok?14:34
haleybi will follow-up later so we can move along with other items14:35
haleyb#topic community goals14:35
mlavallehaleyb ykarel I'll keep an eye on that failure14:36
haleyback14:36
haleyblajoskatona: seems forward progress on neutronclient deprecation?14:36
haleyb#link https://review.opendev.org/q/topic:%22bug/1999774%2214:36
ykarelhaleyb, ack for deputy, thx for the reminder14:36
lajoskatonanot this week, I had no time for it14:36
haleybwell, at least both open changes have zuul +1, i will take a look at the second in the chain14:37
lajoskatonathanks14:37
haleyband evenlet deprecation14:38
ralonsohthis is not going well14:38
ralonsohI can't identify what is happening14:38
ralonsohinitially the hash ring manager was a possible culprit14:38
ralonsohbut since last week (we merged a patch), the timeouts are increased and we don't have idle nodes14:39
ralonsohnow the problem is, sometimes, that the events are processed in different order, DB locks and other problems I can't explain14:39
ralonsohI'm trying to open the related bugs14:40
ralonsohand I'm trying with this:14:40
ralonsohhttps://review.opendev.org/c/openstack/neutron/+/93865914:40
ralonsohthere are many places in the code (python-ovs, ovsdbapp, Neutron, etc) where the libraries have workarounds for eventlet14:40
ralonsohso I'14:40
ralonsohsorry14:40
ralonsohso I would prefer testing Neutron API without eventlet14:41
opendevreviewMerged x/whitebox-neutron-tempest-plugin master: Add config dict dataplane_podified_services  https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/93951914:41
ralonsohbecause this is actually the expected behaviour14:41
ralonsohand that's all (I'm overwhelmed right now)14:41
haleybralonsoh: what else can we do to help? i know ihrachys was helping last week, and i can look at things this week14:43
haleybbut i realize it is very complicated14:43
ralonsohhaleyb, first of all, trying to check what is happening in CI patches like https://review.opendev.org/c/openstack/neutron/+/932601/12?tab=change-view-tab-header-zuul-results-summary14:43
ralonsoheven without eventlet, I see common errors (ports not active, PG not present, etc)14:44
ralonsohso, for example, to take one CI log and try to squeeze it to find what was the problem14:44
haleyback, and i would ask anyone with spare cycles to take a look so we come up with some ideas14:45
lajoskatonaI ry to allocate some time to it, anyway the n-d-r issue is also there with the same root 14:46
haleybthanks14:46
haleybi want to move to on-demand as svinota is here14:46
haleyb#topic on-demand14:46
ralonsohsvinota, hello!14:46
svinotahello14:46
svinotaand thanks for your time :)14:47
haleybthanks for pyroute2 :)14:47
lajoskatonaWelcome :-)14:47
svinotaso shortly, the pyroute2 project is undergoing a big (to say the least) shift in the core14:47
svinotaI continue to support the old synchronous API, it will not be dropped14:48
svinotabut from 0.9.1 the core is based on asyncio14:48
svinotarelated docs: https://docs.pyroute2.org/asyncio.html14:48
svinotaalso: https://docs.pyroute2.org/iproute_intro.html14:48
svinotathanks to ykarel , I start to monitor the zuul as well14:49
svinotaso I hope not to miss major compatibility errors; beside of that I run own integration tests14:50
svinotaI don't know how this shift can affect the neutron core, so as I said — I keep the sync API anyways14:51
svinotabut if the project can provide something more within the async paradigm — let me know14:51
haleybis there a point in time where we should start migrating things? will there be gotcha's, etc?14:52
svinotathat's all folks :) thanks14:52
ralonsohthis is a bug change, of course. We use pyroute "only" in the agents (luckily, because asyncio is not compatible with wsgi)14:52
ralonsohand most of the pyrouter calls are executed inside a root daemon (privsep)14:52
ralonsohso we'll need to refactor the non-admin calls and the admin calls independetly14:52
ralonsohcan both coexist right now?14:53
svinota0.9.1 is scheduled for the beginning of February, but as I see, the current neutron code works with the master branch14:53
svinotaand the master branch is already async-enabled14:53
ralonsohperfect14:53
svinotaso I believe no migration is required14:53
ralonsohbut eventually this is the desired implementation, right?14:54
svinotaasync is the first class citizen now, so the sync code is simply a set of wrappers around it; so you can use either variant, totally up to you, both will coexist14:55
ralonsohperfect14:55
svinotain the best case the library users should not notice any change14:55
slaweqsvinota thx for sharing all of that with us14:55
svinota(it took like some months to fix it in that way, but I hope it pays back)14:56
slaweqbut neutron is not the only user of the pyroute2 in openstack14:56
ralonsohos-vif, right14:56
slaweqdid you maybe had any chance to test it with other projects as well?14:56
ralonsohand other projects in networking14:56
slaweqoctavia I see is using it a bit14:56
svinotaI tested kuryr. Octavia should work as before.14:57
ralonsohso this should be on us to create pyroute-master CI jobs14:57
slaweqhttps://codesearch.opendev.org/?q=pyroute2&i=nope&literal=nope&files=&excludeFiles=&repos=14:57
slaweqI see many of them really14:57
svinotaI forgot os-vif, thanks14:57
slaweqI don't want you to test them all on your own14:57
haleybralonsoh: we do have functional and fullstack pyroute2 master jobs14:57
svinotaslaweq, thanks for the link. I will try to go through the list and see if we can include them in the integration testing14:58
slaweqbut maybe you could send email to the openstack-discuss ML to raise awarness of this change in different teams14:58
ralonsohhaleyb, yes but for all projects?14:58
haleybralonsoh: i only know of neutron14:58
lajoskatonaperhaps some common tempest integration job? 14:58
lajoskatonathat an cover a lot of projects14:58
slaweqsvinota thx a lot, if you will have any questions about it, please ping me :)14:59
svinotaslaweq, 10x14:59
ralonsohok, let's create a bug in LP to handle this15:00
ralonsohto create the needed CI jobs for pyroute-master15:00
ralonsohfunctional or tempest15:00
ralonsohand, of course, svinota thanks a lot15:00
lajoskatonaand advertise it on mail list to have a flag for all users not just for us15:00
svinotathanks, colleagues15:01
lajoskatonasvinota thanks for the good cooperation and for the news :-)15:01
haleybyes, thanks a lot svinota!15:01
haleybdoes anyone want to file the bug, etc? or is this on the PTL :)15:02
lajoskatonaalways the one who asks :-)15:02
slaweqhaha15:02
haleybok, i have to drop for another meeting15:02
haleybthanks everyone for attending, have a good week15:03
slaweqo/15:03
haleyb#endmeeting15:03
opendevmeetMeeting ended Tue Jan 21 15:03:14 2025 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:03
opendevmeetMinutes:        https://meetings.opendev.org/meetings/networking/2025/networking.2025-01-21-14.00.html15:03
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/networking/2025/networking.2025-01-21-14.00.txt15:03
opendevmeetLog:            https://meetings.opendev.org/meetings/networking/2025/networking.2025-01-21-14.00.log.html15:03
ralonsohbye15:03
lajoskatonabye!15:03
opendevreviewSlawek Kaplonski proposed openstack/neutron master: Don't change original target dict by the OwnerCheck policy rule  https://review.opendev.org/c/openstack/neutron/+/93962415:49
opendevreviewSlawek Kaplonski proposed openstack/neutron master: Make API policies for tags to be working with resource attributes  https://review.opendev.org/c/openstack/neutron/+/93813515:49
opendevreviewRodolfo Alonso proposed openstack/neutron master: DNM - Test "neutron-ovn-tempest-ipv6-only-ovs*" with WSGI  https://review.opendev.org/c/openstack/neutron/+/93260116:55
opendevreviewMerged openstack/neutron-lib master: [API-REF] Add clarification about QinQ and transparent vlan differences  https://review.opendev.org/c/openstack/neutron-lib/+/93970817:05
opendevreviewRodolfo Alonso proposed openstack/neutron master: WIP - Do not check if the subnet is in use before deleting it  https://review.opendev.org/c/openstack/neutron/+/93973517:22
opendevreviewRodolfo Alonso proposed openstack/neutron master: DNM - Test "neutron-ovn-tempest-ipv6-only-ovs*" with WSGI  https://review.opendev.org/c/openstack/neutron/+/93973617:25
opendevreviewMerged openstack/neutron stable/2024.2: tests: Don't assume update outside ovs txn notifies separately from create  https://review.opendev.org/c/openstack/neutron/+/93964217:44
*** eandersson0 is now known as eandersson17:51
vsaienkoplease review small patch https://review.opendev.org/c/openstack/neutron/+/939075 that change code duplication by using is_cidr_host from utils19:08
froyothx tt_ for debugging info about the OVN NB LB entries dissapearing, don't hesitate to contact if you need more help! 19:59
opendevreviewMerged openstack/neutron master: [Functional job] Switch test order to avoid memory issue  https://review.opendev.org/c/openstack/neutron/+/93964720:41
opendevreviewBrian Haley proposed openstack/neutron master: Optionally configure IPv6 metadata address  https://review.opendev.org/c/openstack/neutron/+/92649722:39
opendevreviewMerged openstack/neutron master: Bump mypy to 1.14.1 (the latest)  https://review.opendev.org/c/openstack/neutron/+/93885423:44

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