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