14:01:07 <ralonsoh> #startmeeting networking
14:01:07 <opendevmeet> Meeting started Tue May 16 14:01:07 2023 UTC and is due to finish in 60 minutes.  The chair is ralonsoh. Information about MeetBot at http://wiki.debian.org/MeetBot.
14:01:07 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
14:01:07 <opendevmeet> The meeting name has been set to 'networking'
14:01:09 <ralonsoh> hello all
14:01:09 <bcafarel> o/
14:01:10 <sahid_> o/
14:01:11 <obondarev> hi
14:01:14 <isabek> Hi
14:01:19 <rubasov> o/
14:02:09 <ralonsoh> let's start
14:02:20 <ralonsoh> #topic announcements
14:02:28 <ralonsoh> #link https://releases.openstack.org/bobcat/schedule.html
14:02:38 <ralonsoh> last week was the milestone 1
14:02:54 <lajoskatona> o/
14:03:07 <ralonsoh> and we released 23.0.0.0b1
14:03:17 <ralonsoh> but, of course, I had an mistake
14:03:25 <ralonsoh> I used an old hash
14:03:30 <ralonsoh> please check https://review.opendev.org/c/openstack/releases/+/883242
14:03:37 <ralonsoh> that is using the correct one this time
14:03:40 <ralonsoh> (and sorry)
14:03:59 <lajoskatona> blame the reviewers :-)
14:04:09 <ralonsoh> and as usual, Vancouver etherpad
14:04:11 <ralonsoh> #link https://etherpad.opendev.org/p/neutron-vancouver-2023
14:04:24 <ralonsoh> please remember we had the forum session on Tuesday this week
14:04:50 <ralonsoh> and we have a new episode in openinfra
14:04:53 <ralonsoh> #link https://openinfra.dev/live/#all-episodes
14:04:58 <ralonsoh> "May the OpenInfra Force Be With You: Preview of the OpenInfra Summit!"
14:05:44 <ralonsoh> and last but not least, n-lib 3.6.0
14:05:46 <ralonsoh> #link https://review.opendev.org/c/openstack/releases/+/881231
14:06:01 <ralonsoh> anything else missing?
14:06:16 <mlavalle> no CI meeting today
14:06:28 <ralonsoh> exactly, there is no CI meeting today
14:06:44 <ralonsoh> so please, during the next section raise any concern related to CI/bugs/etc
14:06:50 <ralonsoh> #topic bugs
14:07:03 <ralonsoh> last week I forgot that, so let's do it first
14:07:08 <ralonsoh> This week elvira is the deputy, next week will be ykarel.
14:07:12 <ralonsoh> ack^ ?
14:07:35 <ralonsoh> (I'll ping them after the meeting)
14:07:49 <ralonsoh> last week report is from isabek
14:07:52 <ralonsoh> #link https://lists.openstack.org/pipermail/openstack-discuss/2023-May/033666.html
14:08:02 <ralonsoh> there are 3 pending bugs to be assigned
14:08:11 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/2019314
14:08:33 <ralonsoh> it seems to me that is a low hanging fruit
14:08:44 <ralonsoh> but should be tested in the CI
14:09:46 <lajoskatona> yes, perhaps the repoter can propose the patch, if not even I an push it to test the idea
14:10:46 <ralonsoh> I was trying to find smething similar in the OVS agent
14:10:53 <ralonsoh> that doesn't use this common code
14:10:57 <ralonsoh> and we don't have it
14:11:12 <ralonsoh> so maybe is something that is affecting OVS, LB and SRIOC
14:11:33 <ralonsoh> please, check it if you have time
14:11:49 <lajoskatona> I try to check it later this week/early next week
14:11:54 <ralonsoh> thanks a lot
14:11:59 <ralonsoh> next one
14:12:01 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/2018967
14:12:20 <ralonsoh> I think this could be an issue with shared resources between tests
14:12:25 <ralonsoh> but not confirmed
14:12:52 <ralonsoh> actually this error is called from a clenaup method
14:13:05 <ralonsoh> so maybe (just maybe) we can catch the exception
14:13:25 <ralonsoh> but it could be better not to share resources between tests to avoid that
14:13:39 <opendevreview> Frode Nordahl proposed openstack/neutron master: [ovn] Implement support for external-gateway-multihoming extension  https://review.opendev.org/c/openstack/neutron/+/874199
14:14:03 <ralonsoh> again, if you have time, please check that bug
14:14:12 <lajoskatona> we shall ask zhouhenglc to check it
14:14:24 <ralonsoh> I'll do it in the bug
14:14:27 <ralonsoh> thanks!
14:14:48 <ralonsoh> the last one
14:14:51 <opendevreview> Slawek Kaplonski proposed openstack/neutron-specs master: Add "used_in_non_default_sg" attribute to the default SG rules API  https://review.opendev.org/c/openstack/neutron-specs/+/883267
14:14:51 <opendevreview> Slawek Kaplonski proposed openstack/neutron-specs master: Add "remote_address_group_id" attribute to the default SG rules API  https://review.opendev.org/c/openstack/neutron-specs/+/883268
14:14:56 <ralonsoh> #link https://bugs.launchpad.net/neutron/+bug/2019012
14:15:02 <ralonsoh> I'm currently testing that in an env
14:15:18 <ralonsoh> but I'm testing it with master, not victoria
14:15:30 <ralonsoh> if that is reproducible in master, I'll try to debug the issue
14:15:44 <ralonsoh> I'll update the bug once I have something
14:16:07 <ralonsoh> and that's all I have today in this topic
14:16:14 <ralonsoh> sorry, one more thing
14:16:26 <ralonsoh> (because we won't have CI meeting)
14:16:31 <ralonsoh> the issues with sRBAC
14:16:38 <ralonsoh> lajoskatona, has proposed a series of patches
14:16:58 <lajoskatona> this one: https://review.opendev.org/q/topic:bug%252F2019097
14:17:30 <ralonsoh> I think we'll need to wait until the releases patch is merged
14:17:36 <lajoskatona> it's not serious just create net or similar in tests as admin
14:17:36 <ralonsoh> but all of them look fine
14:17:53 <lajoskatona> exactly, I hope all will be green
14:18:12 <ralonsoh> I'll keep an eye on the releases patch and once we have it, I'll recheck your patches
14:18:22 <ralonsoh> and thanks for the quick response
14:18:25 <lajoskatona> thanks
14:19:10 <ralonsoh> ok, let's continue then
14:19:15 <ralonsoh> #topic specs
14:19:17 <mlavalle> I have a topic in bugs
14:19:22 <ralonsoh> please, go on
14:19:30 <mlavalle> that I added to the agenda: https://bugs.launchpad.net/neutron/+bug/2019449
14:19:49 <mlavalle> I filed it this past Friday and prposed a test fix for it
14:19:59 <mlavalle> https://review.opendev.org/c/openstack/neutron/+/883047
14:20:07 <opendevreview> Slawek Kaplonski proposed openstack/neutron master: Update api extension for default sg rules API  https://review.opendev.org/c/openstack/neutron/+/883269
14:20:09 <mlavalle> This patch fixes the problem
14:20:15 <lajoskatona> +1, this was introduced by me
14:20:30 <lajoskatona> thanks for it mlavalle
14:20:44 <mlavalle> however, I left in the LP report a couple of questions that I would like to discuss before going this route
14:21:01 <mlavalle> no need to discuss here. We can have the conversation in LP and Gerrit
14:21:07 <mlavalle> that's all
14:21:36 <ralonsoh> what was the patch introducing that?
14:21:40 <lajoskatona> mlavalle: I check the questions in launchpad
14:21:43 <lajoskatona> https://review.opendev.org/q/I207ecf7954b06507e03cb16b502ceb6e2807e0e7
14:21:51 <lajoskatona> ralonsoh: ----^
14:22:29 <ralonsoh> ok, then we'll need to backport it up to Xena
14:22:47 <mlavalle> yes, the fix would have to be backported
14:22:59 <ralonsoh> ok, cool, good catch
14:23:02 <mlavalle> but let's discuss a bit
14:23:10 <mlavalle> before going this route
14:23:27 <mlavalle> that's all
14:23:33 <lajoskatona> +1
14:23:44 <ralonsoh> ok
14:24:10 <ralonsoh> let's continue with the topic specs then
14:24:16 <ralonsoh> #link https://review.opendev.org/c/openstack/neutron-specs/+/882151
14:24:20 <ralonsoh> ERSPAN for tap-as-a-service
14:24:41 <ralonsoh> some reviews done but seems almost ready
14:24:57 <lajoskatona> thanks for the reviews
14:25:00 <ralonsoh> but please, more reviewers are welcome to the specs
14:25:29 * mlavalle will review today
14:25:33 <ralonsoh> thanks
14:25:40 <ralonsoh> the next one
14:25:42 <ralonsoh> #link https://review.opendev.org/c/openstack/neutron-specs/+/882272
14:25:50 <ralonsoh> just one review, thank obondarev
14:26:04 <obondarev> sure
14:26:06 <ralonsoh> again, please spend some time reviewing the specs
14:26:17 * mlavalle will review between today and tomorrow
14:26:35 <ralonsoh> I'll ping you on Friday morning/afternoon just as a heads-up for the spec reviews
14:26:49 <ralonsoh> the last one is
14:26:51 <ralonsoh> #link https://review.opendev.org/c/openstack/nova-specs/+/859290
14:27:03 <ralonsoh> I have it open but I didn't finish
14:27:11 <ralonsoh> it is needed for the Neutron related code
14:27:26 <ralonsoh> this one, from the Neutron point of view, is easy to review
14:27:57 <mlavalle> also added to my pile. will go trhough it this week
14:28:27 <ralonsoh> Oh my
14:28:36 <ralonsoh> I see slawek sent two new specs 13 mins ago
14:28:46 <ralonsoh> (I updated the list 1 hour ago)
14:28:48 <ralonsoh> #link https://review.opendev.org/c/openstack/neutron-specs/+/883267
14:28:58 <ralonsoh> #link https://review.opendev.org/c/openstack/neutron-specs/+/883268
14:29:16 <ralonsoh> ahhh no, these are just updates
14:29:26 <mlavalle> LOL
14:29:47 * mlavalle also has a topic related to specs
14:29:51 <elvira> ralonsoh: ack for the CI deputy role, missed the ping before
14:30:05 <ralonsoh> ok, because these specs where not implemented in 2023.1, I think we should move them to 2023.2
14:30:12 <ralonsoh> I'll ask slawek to do this
14:30:51 <ralonsoh> elvira, np, thanks
14:31:07 <ralonsoh> ok, something else in this topic?
14:31:10 <mlavalle> yes
14:31:32 <mlavalle> this spec https://specs.openstack.org/openstack/neutron-specs/specs/2023.1/metadata-rate-limit.html was approved in 2023.1
14:32:01 <mlavalle> I've continued its implementation: https://review.opendev.org/c/openstack/neutron/+/858879
14:32:28 <mlavalle> I've addressed rubasov's and mtomaska's comments, thanks!
14:32:44 <ges> Thanks for taking over, I hadn't have time to dig more into it.
14:32:49 <ralonsoh> ok, that development was abandoned sometime ago
14:32:50 <mlavalle> also haleyb pointed earlier today some minor changes needed
14:32:55 <ralonsoh> mlavalle, thanks for restoring it
14:33:05 <haleyb> o/
14:33:20 <mlavalle> I'll address haleyb's comments today . Thanks!
14:33:44 <mlavalle> after that I would like more eyes on this code, so we can merge it in the near future
14:33:48 <mlavalle> that's all
14:34:04 <ralonsoh> ok, thanks
14:34:39 <ralonsoh> ok, let's move then
14:34:49 <ralonsoh> #topic community_goals
14:34:56 <ralonsoh> #1) Consistent and Secure Default RBAC
14:35:21 <ralonsoh> so far, all issues related to sRBAC in Neutron code have been addresses
14:35:24 <ralonsoh> addresed*
14:35:31 <ralonsoh> there are some pending backports
14:35:38 <ralonsoh> #link https://review.opendev.org/q/topic:bug%252F2018727
14:35:43 <ralonsoh> #link https://review.opendev.org/q/topic:bug%252F2018585
14:36:25 <ralonsoh> and ad commented before, the patches from lajoskatona
14:36:31 <ralonsoh> #link https://review.opendev.org/q/topic:bug%252F2019097
14:36:41 <bcafarel> I think I catched up on my backport backlog during the meeting :)
14:36:48 <ralonsoh> cool
14:37:15 <ralonsoh> next week slawek will update on the service-to-service role community effort
14:37:39 <mlavalle> I can also help with the backports backlog
14:37:46 <ralonsoh> he has started identifying the calls performed by Nova and other services
14:38:12 <ralonsoh> and that's the update in this topic, next week slawek will provide better info
14:38:23 <ralonsoh> next one
14:38:26 <ralonsoh> 2) Neutron client deprecation
14:38:32 <ralonsoh> lajoskatona, please
14:38:52 <lajoskatona> this week only some bug fixes due to the recent neutronclient release
14:38:58 <lajoskatona> https://review.opendev.org/q/topic:bug%252F2003861
14:39:18 <lajoskatona> and partly this one for sfc: https://review.opendev.org/c/openstack/networking-sfc/+/883078
14:39:31 <lajoskatona> had to squash with the RBAC fix
14:39:43 <ralonsoh> yeah, this one fixing both problems
14:39:45 <ralonsoh> thanks!
14:39:57 <lajoskatona> that's it for the neutronclient topic for this week
14:40:07 <ralonsoh> thank you!
14:40:31 <opendevreview> Alexander Shishebarov proposed openstack/neutron master: Fix not working use_random_fully config option  https://review.opendev.org/c/openstack/neutron/+/883239
14:40:33 <ralonsoh> and that's all for today
14:40:35 <sahid_> lajoskatona: i started to work on it for nova https://review.opendev.org/c/openstack/nova/+/882714
14:40:38 <ralonsoh> #topic on_demand
14:41:05 <sahid_> (just a quick headsuo, sorry for disrupting)
14:41:14 <lajoskatona> sahid_: cool, thanks, I will check your patch
14:41:17 <ralonsoh> sahid_, please add nova folks to the patch
14:41:30 <ralonsoh> and ping them in nova channel too, just to speed up the review
14:41:37 <sahid_> ack
14:41:39 <ralonsoh> and, of course, thanks for this!
14:41:49 <sahid_> sure :-)
14:41:56 <ralonsoh> I have one quick topic
14:42:08 <ralonsoh> #link https://review.opendev.org/c/openstack/neutron/+/883120
14:42:57 <ralonsoh> the patch I pushed before was stopping any fullstack tst
14:43:22 <ralonsoh> I need to find a way to solve it with sqlalchemy 2.0
14:43:39 <ralonsoh> but please, be aware that for 10 days, we didn't execute any fullstack test
14:43:56 <lajoskatona> nice
14:44:09 <ralonsoh> actually the revert is now failing because of two tests
14:44:29 <ralonsoh> so I need to investigate both, the failing tests with the current code and sqlalchemy 2.0
14:44:49 <ralonsoh> anyway, something else you want to comment here?
14:45:09 <ges> I have a little something
14:45:11 <ralonsoh> sure
14:45:32 <ges> We identified that when deleting a security group, all the agents are querying the rpc server for the SG's rules
14:46:13 <ges> this seems related to a callback that tries to clean-up the resource cache
14:46:35 <ralonsoh> sorry, what agents?
14:46:43 <ralonsoh> OVS and LB only
14:47:18 <ges> Well, we use different kind of agents in prod here, but I think that'd be OVS at least
14:47:26 <ges> anything that subscribes to SG updates
14:48:09 <ges> I came up with a patch to avoid this behavior here: https://review.opendev.org/c/openstack/neutron/+/883235
14:50:24 <ralonsoh> did you check that with remote groups?
14:50:41 <opendevreview> Alexander Shishebarov proposed openstack/neutron master: Fix not working use_random_fully config option  https://review.opendev.org/c/openstack/neutron/+/883239
14:51:09 <ralonsoh> ges, we can discuss and comment on the patch in gerrit
14:51:21 <ralonsoh> thanks for raising this issue
14:51:35 <ges> I think gerrit is the better place, too, just wanted to make you aware of this.
14:51:41 <ralonsoh> thanks
14:51:48 <ralonsoh> anything else?
14:52:25 <ralonsoh> thank you all for attending
14:52:30 <ralonsoh> #endmeeting