Tuesday, 2023-08-08

opendevreviewLewis Denny proposed openstack/neutron master: Add max limit to agent_down_time  https://review.opendev.org/c/openstack/neutron/+/88937300:06
opendevreviewMerged openstack/neutron master: hash-ring: Retry all DB operations if inactive  https://review.opendev.org/c/openstack/neutron/+/89024501:33
opendevreviewTakashi Kajinami proposed openstack/neutron master: Fix missing oslo.versionedobjects library option  https://review.opendev.org/c/openstack/neutron/+/89071606:06
amorinhey team, would you mind pinging me and aprats when neutron meeting starts? We'd like to talk about https://bugs.launchpad.net/neutron/+bug/202972207:06
slaweqlajoskatona hi, can You maybe check https://review.opendev.org/c/openstack/neutron/+/889153 when You will have few minutes? Thx in advance07:19
opendevreviewwangjiaqi proposed openstack/neutron master: Cleanup py27 support  https://review.opendev.org/c/openstack/neutron/+/89072807:25
opendevreviewwangjiaqi proposed openstack/neutron-lib master: Cleanup py27 support  https://review.opendev.org/c/openstack/neutron-lib/+/89072907:27
lajoskatonaslaweq: checking07:28
opendevreviewwangjiaqi proposed openstack/neutron-tempest-plugin master: Cleanup py27 support  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/89073007:28
opendevreviewwangjiaqi proposed openstack/os-ken master: Cleanup py27 support  https://review.opendev.org/c/openstack/os-ken/+/89073508:49
opendevreviewSlawek Kaplonski proposed openstack/neutron-tempest-plugin master: DNM Just testing some sql queries  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/89041909:14
opendevreviewSlawek Kaplonski proposed openstack/neutron master: [OVN] Add baremetal support without Neutron DHCP agent for IPv6  https://review.opendev.org/c/openstack/neutron/+/89068310:08
opendevreviewMerged openstack/neutron master: Add NET_OWNER_MEMBER and NET_OWNER_READER policy rules  https://review.opendev.org/c/openstack/neutron/+/88915310:11
opendevreviewMerged openstack/neutron stable/2023.1: [OVN] Hash Ring: Better handle Neutron worker failures  https://review.opendev.org/c/openstack/neutron/+/89039210:41
opendevreviewMerged openstack/neutron stable/zed: [OVN] Hash Ring: Better handle Neutron worker failures  https://review.opendev.org/c/openstack/neutron/+/89039310:41
opendevreviewMerged openstack/neutron stable/yoga: [OVN] Hash Ring: Better handle Neutron worker failures  https://review.opendev.org/c/openstack/neutron/+/89039410:41
opendevreviewMerged openstack/neutron stable/xena: [OVN] Hash Ring: Better handle Neutron worker failures  https://review.opendev.org/c/openstack/neutron/+/89039510:41
opendevreviewLajos Katona proposed openstack/tap-as-a-service master: Doc: add documentation for usage and flow examples for OVS  https://review.opendev.org/c/openstack/tap-as-a-service/+/82838211:44
racostaHey folks, if you have some time, please check: https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/888787 12:35
*** elvira2 is now known as elvira12:50
opendevreviewBrian Haley proposed openstack/neutron master: Remove some invalid text from the network component doc  https://review.opendev.org/c/openstack/neutron/+/89079013:08
opendevreviewMerged openstack/neutron stable/zed: dvr: Avoid installing non-dvr openflow rule on startup  https://review.opendev.org/c/openstack/neutron/+/89022913:11
opendevreviewMerged openstack/neutron stable/2023.1: dvr: Avoid installing non-dvr openflow rule on startup  https://review.opendev.org/c/openstack/neutron/+/89022813:11
opendevreviewMerged openstack/neutron stable/wallaby: dvr: Avoid installing non-dvr openflow rule on startup  https://review.opendev.org/c/openstack/neutron/+/89023113:11
*** elvira1 is now known as elvira13:11
opendevreviewTobias Fischer proposed openstack/neutron master: Add more logging for lrp scheduling on gtws  https://review.opendev.org/c/openstack/neutron/+/89079513:40
*** obondarev_ is now known as obondarev13:56
slaweq#startmeeting networking14:00
opendevmeetMeeting started Tue Aug  8 14:00:17 2023 UTC and is due to finish in 60 minutes.  The chair is slaweq. 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
mlavalleo/14:00
lajoskatonao/14:00
slaweqping bcafarel, elvira, frickler, mlavalle, mtomaska, obondarev, slawek, tobias-urdin, ykarel, lajoskatona, jlibosva, averdagu, amotoki14:00
slaweqo/14:00
frickler\o14:00
obondarevo/14:00
rubasovo/14:00
ykarelo/14:00
haleybo/14:00
slaweqralonsoh is still off so I will chair today's meeting again14:00
bcafarelo/14:01
slaweqI think we can start as we have pretty many folks there already14:01
slaweq#topic announcements14:01
sahido/14:01
slaweqI just have regular reminder about release schedule14:01
slaweqBobcat / 2023.2 schedule: https://releases.openstack.org/bobcat/schedule.html14:01
slaweqNext milestone is Aug 21-Aug 25 - Final release for non-client libraries14:02
slaweqand also PTL and TC nominations will start in 3 weeks (also Aug 21-Aug 25)  - please consider preparing Your nomination if You are interested in one of those positions14:02
slaweqthat's all announcements from me today14:03
slaweqdo You have anything else?14:03
slaweqI guess this means "no" :)14:04
slaweq#topic bugs14:04
slaweqlucasagomes was bug deputy last week but he forgot to sent summary email14:04
slaweqI just pinged him about it before this meeting so please expect email before EOD today14:04
lucasagomessorry folks, I will submit it soon!14:05
slaweqin the meantime do You have any bugs You would like to discuss now?14:05
slaweqor maybe lucasagomes have already something what would like to bring up here?14:05
frickleramorin wanted to discuss one14:05
lucasagomesI just started looking into the list, if I find something in the meantime I will bring it up14:05
amorinhello!14:05
fricklerhttps://bugs.launchpad.net/neutron/+bug/202972214:06
amorinthanks for the ping :)14:06
slaweqhi amorin14:06
amorinyup, we discoverd something when using DVR14:06
amorinsome ip rules are set in order to reach the external gateway of the subnet14:06
amorinin a 2 routers architecture, with custom routing rules, it prevent the routing to reach the gw correctly14:07
amorinthis is not affected no DVR14:07
amorinthis is not affecting* no DVR14:07
amorinwe were wondering if someone know why, for DVR, we are using the ip rule to reach the gw14:07
amorinand a different routing table14:07
amorininstead of the default one14:07
slaweqI see that  haleyb already started looking at it and I guess that he and maybe obondarev are the best guys to ask why it was originally done like that14:09
obondarevthere is a great set of articles describing DVR mechanism by Assaf Muller: https://assafmuller.com/category/dvr/ - amorin you may find answers there 14:09
amorinack, will read that14:10
haleybit's been a while, but there are typically ip rules to deal with scoping of addresses.14:10
obondarevcurrently I don't remember this detail on ip rule, sorry14:10
amorinwe implemented a patch to add an extra rule, it works14:10
amorinbut if the original idea of ip rule is not needed anymore, maybe we can totally get rid of it14:11
haleybas i mentioned in the patch, it could be a certain order of things shows the bug, sometimes we just don't account for things correctly14:11
haleybi think the rules are needed to account for address scopes14:12
slaweqamorin so You can probably read Assaf's blog posts about it and we can continue that discussion in the LP and gerrit, is that ok for You?14:14
haleybwe can just follow-up in the patch and bug, but i am following it14:14
amorinsounds good, thanks14:14
slaweqthx amorin for bringing it up here and thx haleyb for taking care of that LP14:15
slaweqok, I think we can move on14:15
slaweq#topic community_goals14:15
slaweqConsistent and Secure Default RBAC14:15
slaweqPatch https://review.opendev.org/c/openstack/neutron/+/886724 should be ready to review now - it still requires neutron-lib bump in the upper-contraints but other than that it looks ok14:16
slaweqso please take a look at it if You will have few minutes14:16
slaweqI would like to hopefully get it merged in this cycle14:16
slaweqnext one14:17
slaweqNeutron client deprecation14:17
slaweqlajoskatona any updates on this one?14:17
lajoskatonanot much, I worked on the sfc part14:17
lajoskatonathis is for SDK: https://review.opendev.org/c/openstack/neutron/+/88672414:17
lajoskatonathe usual etherpad: https://etherpad.opendev.org/p/python-neutronclient_deprecation14:18
lajoskatonaplease check it, the patches without a 'tick' are something to review14:18
lajoskatonathis one for fwaas is ready: https://review.opendev.org/c/openstack/python-neutronclient/+/880629 as SKd was released for it14:19
slaweqlajoskatona I think You gave wrong link "for SDK" :)14:19
lajoskatonasorry, my mind was faster :-)14:19
slaweq:)14:19
lajoskatonaotherwise I started to look into Horizon, and started to play to have SDK behind it14:20
lajoskatonathat's it for this topic from me14:20
slaweqthx14:20
slaweqI will finally try to find time to review those patches this week14:20
slaweqwith that I think we can move on14:21
lajoskatonathanks in advance14:21
slaweqto the next topic, which is14:21
slaweq#topic on_demand14:21
slaweqdo You have any other neutron related topics to discuss today?14:21
mlavallenope14:21
fricklerI have a question about IPv6 metadata14:21
slaweqfrickler sure14:21
fricklerI see the tests are only done for OVS, and in my local test it doesn't work for OVN, is that a known gap?14:22
slaweqyes, it is14:22
slaweqOVN metadata agent don't support it (yet)14:22
fricklerso it is missing in the gap document14:23
slaweqI have it on my todo list but not with high priority for now14:23
slaweqright, I think we forgot about gaps document14:23
slaweqto add it there14:23
fricklerok, I can do a patch, need to add some DNS gaps, too, it seems14:24
frickleralso haleyb do you still plan to followup with https://review.opendev.org/c/openstack/neutron/+/876903 ?14:24
slaweqthx frickler 14:24
haleybfrickler: i need to abandon that and work on a different fix, that didn't work :(14:24
haleyband we don't need to switch to the ec2 one14:25
opendevreviewLucas Alvares Gomes proposed openstack/neutron master: [OVN] ovn-db-sync check for router port differences  https://review.opendev.org/c/openstack/neutron/+/89079914:25
fricklerwell not switch but add it as an option I though would be a good idea?14:25
frickler*thought14:26
haleybi think the problem was that there is no dhcp response value for host routes in ipv6, so it doesn't work14:26
haleyband there was an issue with dnsmasq and addressing, just making sure one host at a time configures the address should be enough14:27
fricklerbut do you need a specific route? wouldn't the default route from the RA be enough?14:27
haleybnot when you have an isolated subnet, which is where the dnsmasq issue got into play14:27
racostaIn the IPv6-only case, the VM generates an LLA address automatically, and this local scope address is not known by ovn-southbound and neutron. At this point we have a hard time! In the current architecture, the metadata makes a proxy and uses the local address of the VM to find the corresponding port to forward and receive the traffic (Port_Binding table) - OVN case.14:28
haleybwe decided to just fix it a different way14:28
fricklerah, ok, maybe split those topics, then14:28
haleybi'll try and work on that in the next few weeks14:29
fricklero.k., I'll wait for that, thx14:30
slaweqok, thank You both for bringing it up here and working on it14:31
slaweqI also have 2 quick topics for today on demand section14:31
slaweqfirst one, which I wanted to discuss also with ralonsoh but as he's not here yet, I will ask whole team here:14:32
slaweqDo we want to add neutron-core to the openstacksdk/OSC "service-core" group? 14:32
slaweqSee https://lists.openstack.org/pipermail/openstack-discuss/2023-August/034608.html for more details14:32
slaweqexample patch in project-config: https://review.opendev.org/q/topic:add-cinder-core14:32
lajoskatonagood idea14:32
slaweqIMO it would be good idea to add neutron-core team to that SDK/OSC group also but I wanted to know Your opinion about it14:32
mlavallegood idea14:33
fricklerwe have some individuals already added I think14:33
obondarev+1 to add14:33
slaweqfrickler yes, I think that I am there and amotoki 14:34
slaweqbut still having all neutron-cores in that service-core group would be good IMO14:34
frickleryes, I'm all for it, just wanted to mention that it could simplify the list a bit14:34
slaweqthx14:35
slaweqok, I will propose such patch and also ask ralonsoh to review it as Neutron PTL14:35
slaweqand now second topic14:35
slaweqAs You probably noticed it's that time of the year where user survey questions are going to be prepared for next surver. We have list of current questions, see https://lists.openstack.org/pipermail/openstack-discuss/2023-August/034596.html for details14:36
slaweqand we have time until 18th of Aug to propose some changes there in Neutron related questions14:37
slaweqI would like to ask You all to check what's there currently and maybe we can discuss about it on the next week's team meeting if we want to change something there14:37
slaweqwill that work for You?14:37
lajoskatona+114:38
fricklercan someone copy the questions into an etherpad?14:39
slaweqfrickler yes, I will prepare etherpad with those questions today or tomorrow morning14:39
mlavallewill look at it14:39
slaweqand will send link to it in this channel14:39
lajoskatonathanks slaweq14:39
fricklercool, thx14:39
slaweqand with that we came to the end of the agenda for today14:40
slaweqif You don't have any other topics for today, I will give You some time back14:40
mlavalle\o/14:41
slaweqand please remember about CI meeting in 19 minutes14:41
slaweqit will be on video this week14:41
lajoskatonaack14:41
mlavalleack14:41
slaweq#endmeeting14:41
opendevmeetMeeting ended Tue Aug  8 14:41:19 2023 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)14:41
opendevmeetMinutes:        https://meetings.opendev.org/meetings/networking/2023/networking.2023-08-08-14.00.html14:41
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/networking/2023/networking.2023-08-08-14.00.txt14:41
opendevmeetLog:            https://meetings.opendev.org/meetings/networking/2023/networking.2023-08-08-14.00.log.html14:41
slaweqo/14:41
bcafarelnoted14:41
lajoskatonao/14:41
mlavalleo/14:41
rubasovo/14:42
amorino/14:43
slaweq#startmeeting neutron_ci15:01
opendevmeetMeeting started Tue Aug  8 15:01:05 2023 UTC and is due to finish in 60 minutes.  The chair is slaweq. Information about MeetBot at http://wiki.debian.org/MeetBot.15:01
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:01
opendevmeetThe meeting name has been set to 'neutron_ci'15:01
mtomaskao/15:01
slaweqping bcafarel, lajoskatona, mlavalle, mtomaska, ralonsoh, ykarel, jlibosva, elvira15:01
slaweqThis will be video meeting this time: https://meetpad.opendev.org/neutron-ci-meetings15:01
slaweqGrafana dashboard: https://grafana.opendev.org/d/f913631585/neutron-failure-rate?orgId=115:02
slaweq#topic Actions from previous meetings15:03
slaweqralonsoh to check failed neutron.tests.fullstack.test_l3_agent.TestHAL3Agent.test_keepalived_multiple_sighups_does_not_forfeit_primary test15:03
slaweq#action ralonsoh to check failed neutron.tests.fullstack.test_l3_agent.TestHAL3Agent.test_keepalived_multiple_sighups_does_not_forfeit_primary test15:03
slaweqykarel to check broken centos periodic jobs15:03
slaweq    https://bugs.launchpad.net/neutron/+bug/202933515:04
slaweq    https://review.opendev.org/c/openstack/devstack/+/89028015:04
slaweq    https://review.opendev.org/#/q/Icd99f467d47aaafaaf3ee8f2a3c4da08842cb67215:04
opendevreviewLucas Alvares Gomes proposed openstack/neutron master: [OVN] ovn-db-sync check for router port differences  https://review.opendev.org/c/openstack/neutron/+/89079915:06
slaweqmtomaska to check failing neutron-functional-with-sqlalchemy-master periodic job15:08
mtomaskahttps://opendev.org/openstack/neutron/src/branch/master/neutron/db/migration/cli.py#L8415:08
slaweq#action mtomaska to check failing neutron-functional-with-sqlalchemy-master periodic job15:14
slaweq#topic Stable branches15:14
slaweq#topic Stadium projects15:14
slaweq#action lajoskatona will send DNM patch for neutron-dynamic-routing to check jobs15:17
slaweq#topic Grafana15:17
slaweq#topic Rechecks15:19
opendevreviewLucas Alvares Gomes proposed openstack/neutron master: [OVN] ovn-db-sync check for router port differences  https://review.opendev.org/c/openstack/neutron/+/89079915:22
slaweq#topic Tempest/Scenario15:29
slaweqhttps://cfc27c554a994f35be4e-76bf72ffc642f12cb8e7d8393148d522.ssl.cf5.rackcdn.com/874797/41/gate/neutron-ovn-tempest-ipv6-only-ovs-release/8162e79/testr_results.html15:30
mtomaskayep installing master deps right now15:32
slaweq#endmeeting15:34
opendevmeetMeeting ended Tue Aug  8 15:34:17 2023 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:34
opendevmeetMinutes:        https://meetings.opendev.org/meetings/neutron_ci/2023/neutron_ci.2023-08-08-15.01.html15:34
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/neutron_ci/2023/neutron_ci.2023-08-08-15.01.txt15:34
opendevmeetLog:            https://meetings.opendev.org/meetings/neutron_ci/2023/neutron_ci.2023-08-08-15.01.log.html15:34
opendevreviewLajos Katona proposed openstack/neutron-dynamic-routing master: DNM: Test patch only  https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/89080615:36
apratsHello there ! Regarding #2029722 I followed obondarev advice and looked at the blog. I didn't find an explanation of why there is this ip rule. One of the explanation I could come up with is that you don't want to route traffic through the snat if it doesn't know the way to route the packet back to the other subnet. With this in mind, my patch seems relevant, only adding an ip rule if when the route exists. 15:38
fricklerlajoskatona: is there anything up with n-d-r or just a general check?16:01
haleybaprats: ack, thanks. So the first router didn't know of the routes to the subnet behind the second router?16:29
haleybcan you just put the steps you did to recreate this in the bug? for example the 'openstack...' commands16:30
lajoskatonafrickler: Hi, just a failure in the periodic job, see: https://zuul.openstack.org/buildset/ee086b0e391f4272aadd9acea19adc4f16:30
lajoskatonafrickler: and to see if it was an accident or something to take care16:30
fricklerlajoskatona: seems the failure was reproduced, looks docker related. I can take a closer look tomorrow, thx for spotting that issue17:13
apratshaleyb: ack, I will add the commands and a snippet of my tf. On my last try, all the routers had the routes as described in the bug before enabling the gateway on Router Ext. 17:37
lajoskatonafrickler: thanks, tomorrow I am on PTO, but after that if need help or out of time  just ping me18:02
opendevreviewAdam proposed openstack/neutron master: Handle cases where FIP rules are missing from host  https://review.opendev.org/c/openstack/neutron/+/89082719:17
opendevreviewBrian Haley proposed openstack/neutron master: Update OVN db sync code log messages  https://review.opendev.org/c/openstack/neutron/+/89083221:12
opendevreviewMerged openstack/neutron stable/yoga: [OVN] Prevent Trunk creation/deletion with parent port bound  https://review.opendev.org/c/openstack/neutron/+/88760321:34
opendevreviewMerged openstack/neutron master: Add extra router attributes for ECMP and BFD  https://review.opendev.org/c/openstack/neutron/+/87479723:40

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