14:01:07 #startmeeting networking 14:01:08 Meeting started Tue Jun 4 14:01:07 2019 UTC and is due to finish in 60 minutes. The chair is slaweq. Information about MeetBot at http://wiki.debian.org/MeetBot. 14:01:09 hi 14:01:10 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 14:01:12 The meeting name has been set to 'networking' 14:01:14 Hi 14:01:16 o/ 14:01:28 hi 14:02:07 o/ 14:02:08 hello 14:02:23 #topic Announcements 14:02:46 o/ 14:02:49 Milestone Train-1 is this week 14:02:58 #link https://releases.openstack.org/train/schedule.html 14:03:33 mlavalle: do we need to release new versions this week then? 14:03:43 o/ 14:03:58 yes, that's the agreement we did last ccyle 14:04:04 release stable branches 14:04:17 that's what I thought 14:04:33 so amotoki and mlavalle You will do this, right? 14:04:43 haleyb did that last time 14:04:46 * haleyb arrives after being in nickserv detention 14:05:14 i actually started stable patches, but think there are one or two more things to merge in each branch 14:05:14 haleyb: will you create the patches for stable branches this time around? 14:05:25 cool, thanks! 14:05:40 so yes, i can do it, i'll ping people about getting reviews done on stragglers 14:05:54 haleyb: thx 14:05:57 :-) 14:06:20 next one then 14:06:28 Just a reminder: Open Infrastructure Summit CFP Open - Deadline: July 2 14:06:35 link: http://lists.openstack.org/pipermail/openstack-discuss/2019-May/006262.html 14:07:13 so if You want to have propose some talk for next Summit, it's time to prepare proposal :) 14:07:51 another reminder, is someone missed it on previous meeting: Photos from recent Denver PTG are available at: 14:07:58 #link https://www.dropbox.com/sh/fydqjehy9h5y728/AAC1gIc5bJwwNd5JkcQ6Pqtra/Neutron?dl=0&subfolder_nav_tracking=1 14:08:21 and last reminder 14:08:23 Train PTG summary: 14:08:30 #link http://lists.openstack.org/pipermail/openstack-discuss/2019-May/006408.html 14:08:49 any other announcements? 14:09:07 none from me 14:09:59 ok, lets move on then 14:10:07 #topic Blueprints 14:10:30 This is what we have planned for T-1: 14:10:36 #link https://launchpad.net/neutron/+milestone/train-1 14:11:18 on this topic I have to say that last week we completed https://blueprints.launchpad.net/neutron/+spec/smart-nic-ovs 14:11:48 I just wanted to ask if we need anything else for that one 14:11:51 thx mlavalle 14:11:59 this is the patch that completed the feature: https://review.opendev.org/#/c/586252/ 14:12:12 it merged last week 14:12:40 on https://blueprints.launchpad.net/neutron/+spec/enginefacade-switch 14:13:17 we are making good progress 14:13:25 we are merging patches 14:13:39 can we add this one to train-1 also https://bugs.launchpad.net/neutron/+bug/1458890 14:13:40 Launchpad bug 1458890 in neutron "[RFE] Add segment support to Neutron" [Wishlist,Triaged] - Assigned to Carl Baldwin (carl-baldwin) 14:13:40 and there are several more in the pipeline 14:14:03 based on changes for routed networks to support multiple segments 14:14:17 we can also levereage same to enhance sriov 14:14:33 thanks to ralonsoh and njohnston for their hard work on this 14:15:08 mlavalle: yes, I saw couple of patches merged/close to merge related to this one 14:15:22 and it looks that it's going pretty easy so far :) 14:15:28 (too easy :P) 14:15:40 we're plucking the low hanging fruit first :-) 14:15:43 well, I think we are starting to hit the difficult ones 14:16:14 here's an example for last night: https://review.opendev.org/#/c/662869/ 14:16:40 so don't let your guard down and stay ready for a long fight 14:16:47 ;-) 14:16:49 :) 14:16:50 amen brother 14:17:00 thx You all for working on this 14:17:33 munimeha1: regarding multiple segments in routed networks, we already have that as a target: https://blueprints.launchpad.net/neutron/+spec/multiple-segment-per-network-per-host 14:17:43 munimeha1: according to Your question, I think that we can only add Your rfe to T-2 as T-1 is this week :) 14:17:48 mlavalle: what do You think? 14:18:02 ok T-2 is fine 14:18:04 thanks 14:18:24 munimeha1: but look at the blueprint that I just mentioned ^^^^ 14:18:30 sure 14:18:36 that is what you are talking about, right? 14:18:45 yes 14:19:01 so it is already targeted for T-1. it is going to slip to T-2 14:19:19 bt you can help revieiwing the spec: https://review.opendev.org/#/c/657170 14:19:24 sure 14:19:28 which I ask others to do as well 14:19:31 speaking about https://blueprints.launchpad.net/neutron/+spec/multiple-segment-per-network-per-host there is patch https://review.opendev.org/#/c/657170 waiting for review also 14:19:38 it is my goal for this week 14:19:55 slaweq: right. thanks for bringing it up 14:20:05 You were faster mlavalle :) 14:20:25 Did you notice this one is related https://review.opendev.org/#/c/656885/ 14:21:24 munimeha1: yes, that one is a debt that Nova has owed to us for a long time 14:21:27 and don't forget the bug for routed networks: https://bugs.launchpad.net/neutron/+bug/1828543 14:21:28 Launchpad bug 1828543 in neutron "Routed provider networks: placement API handling errors" [Medium,New] - Assigned to Lajos Katona (lajos-katona) 14:21:59 lajoskatona: thanks. I owe you a response to an email. Haven't forgotten 14:22:20 * mlavalle was feeling bad this morning while walking the dog 14:22:21 mlavalle: thanks, now in keystoneauth there is a fix for exception handling 14:22:48 so I will update this BP https://blueprints.launchpad.net/neutron/+spec/multiple-segment-per-network-per-host to include links to all those things there, ok for You? 14:22:51 mlavalle: so I continue to see what is missing to make this working again and add tests 14:23:05 cool, thanks 14:23:07 slaweq: ok for me, thanks 14:23:15 ok, I will do it after the meeting 14:23:20 lets move on then 14:23:28 we have also BP https://blueprints.launchpad.net/neutron/+spec/event-notifier-ironic 14:23:42 and we have already merged https://review.opendev.org/#/c/658787/ 14:23:52 which looks like the only needed thing on our side 14:24:02 do You know if we can consieder it as done maybe? 14:24:10 or what else is missing on our side for it 14:24:31 hang on... 14:25:14 I think that's it 14:25:42 we finished that one 14:25:50 great, so mlavalle can You update BP and set proper status for it? 14:25:58 yes I will 14:26:03 thx a lot mlavalle 14:26:15 ok, lets move on 14:26:20 and I will also go over the PTG summary this week 14:26:36 there are other BPs in there that we need to add to the dashboard 14:26:53 thx mlavalle 14:27:05 #topic Bugs 14:27:29 last week amotoki was our bug deputy 14:27:35 he sent summary http://lists.openstack.org/pipermail/openstack-discuss/2019-June/006850.html 14:28:22 I would especially want to ask db experts (looking at You njohnston :)) to look at https://bugs.launchpad.net/neutron/+bug/1831009 14:28:23 Launchpad bug 1831009 in neutron "Improper close connection to database leading to mysql/mariadb block connection." [Undecided,New] 14:29:16 is this reported against master branch? 14:29:37 oh, I see he is reporting the neutron version 14:29:42 no, it's Neutron-server: openstack-neutron-13.0.2-1.el7.noarch 14:30:36 that's rocky 14:30:44 yes, I think so :) 14:31:49 we also have 2 bugs related to api performance: https://bugs.launchpad.net/neutron/+bug/1830679 and https://bugs.launchpad.net/neutron/+bug/1830630 14:31:51 Launchpad bug 1830679 in neutron "Security groups RBAC cause a major performance degradation" [High,New] 14:31:52 Launchpad bug 1830630 in neutron "Get external networks too slowly because it would join subnet and rbac" [Medium,New] 14:32:08 both are not assigned so would be good if someone would have time to take a look 14:32:54 any other bugs You want to talk about today? 14:34:08 ok, I get it as no :) 14:34:17 this week our bug deputy is haleyb 14:34:23 * mlavalle will try to take a look at some of the bugs mentioned above ^^^^ 14:34:35 and next week it will be manjeets 14:34:45 slaweq: thanks for the reminder 14:35:00 mlavalle: will You contact manjeets or should I ask him if he will be able to do it? 14:35:04 haleyb: yw :) 14:35:26 slaweq: go ahead and ping him, please 14:35:33 sure, I will do 14:35:54 ok, next topic then 14:35:56 #topic neutron-lib 14:36:00 boden: hi :) 14:36:02 hi 14:36:31 there's a neutron-lib release patch out for review https://review.opendev.org/#/c/661839/ as far as I'm concerned it's good to go 14:37:01 * mlavalle just +1ed it 14:37:02 would be nice to get that out as I think some folks are waiting for it 14:37:04 ok thanks! 14:37:26 yeah prometheanfire was very vocal last week 14:37:38 the other topic is in regards to reviews for non-neutron networking projects (sfc, l2gw, etc.) 14:37:50 wondering if we can look to expand the +2 powers in some of these 14:37:56 perhaps the neutron core team 14:38:19 boden: that was added to "on demand" section :) 14:38:23 but we can talk about it now 14:38:35 thx for bringing this up 14:39:20 boden, mlavalle, slaweq: that was discussed during ptg as well, and with mlavalle we planned to have a meeting with l2gw ptl 14:39:21 is this for neutron-lib consumption patches? 14:39:36 I have a question, we have decided to use neutron-lib master branch for neutron CI, or not? 14:39:53 mlavalle yes and no... in general its not easy to land patches in some gates due to lack of reviews.. these could be neutron-lib related or not 14:40:50 yes, I agree with boden here 14:41:02 do me a favor. Send me an email with the list of projects 14:41:24 and I'll make sure all the core team members have +2 rights in them 14:41:34 sometimes we have some "trivial" changes e.g. related to jobs definitions or some "mechanical" patches in stadium projects and there is no many people who can +2 those patches 14:42:13 mlavalle cool thanks 14:42:22 liuyulong we never got anywhere on that discussion 14:42:23 mlavalle: IMHO all stadium projects should be in this list 14:42:29 sounds good, recent example is the rehoming of tempest plugins 14:42:42 I will also start a personal routines to go over the pending patches in those projects 14:42:54 yeah, but it's more than stadium 14:43:29 lajoskatona: yes, I will ping ricardo today 14:43:50 mlavalle: thanks, 14:44:13 mlavalle: otherwise the idea to add +2 right for cores is good. 14:44:27 I mean for stadium projects 14:44:56 we can start with stadium I guess and go from there... if other projects crop up we can discuss expanding +2 rights I suppose 14:45:08 I think you can just include the neutron-cores group in the list of cores for each of the stadium projects... IIRC that is how the stable cores work for stadium projects (at least fwaas) 14:45:13 in my understanding, neutron-release team members have an appripriate right for stadium projects, but it is not true for third-party prijects. 14:45:30 sounds good boden 14:45:45 nothing else from me... just business as usual 14:45:58 ok, thx boden for updates 14:46:40 anything else regarding neutron-lib? 14:46:51 or we can move on? 14:47:01 nothing from me 14:47:19 ok, lets move on then 14:47:23 last topic 14:47:33 #topic On demand agenda 14:47:47 we already discussed one of things from the list 14:47:53 there is also one from njohnston 14:47:58 "Cleaning old OVO compatibility code." 14:48:02 go on njohnston 14:48:11 oh, I think we talked about that last week; my apologies for not cleaning it off the agenda 14:48:21 njohnston: no problem :) 14:48:33 so I will clean it today to not forget next time 14:48:39 thanks 14:48:51 anything else You want to talk about today? 14:49:07 if not, I will give You couple of minutes back :) 14:49:07 I don't have anything else 14:49:17 Hold on 14:49:46 I have nothing special. I still thinks njohnston's topic is worth checked by everyone https://review.opendev.org/#/c/661995/ 14:50:07 One minute about that neutron-lib dev 14:50:21 njohnston: I owe you an update on https://review.opendev.org/#/c/661995/ 14:50:47 thanks amotoki and tidwellr for great feedback on that 14:51:23 the change is to define a policy as regards OVO object compatibility code in such a way that we respect the upgrade windows for distributions; please take a look and let us know what you thing 14:51:25 *think 14:51:52 njohnston: I can add this to the review, but as long as we suport N->N+2 or greater I think that will be acceptable 14:52:13 njohnston: you're welcome. thanks for leading this coordination :) 14:52:14 Everytime if neutron touch a constant or a utils function, we will try to move it to neutron-lib, and then neutron dev was blocked by such procedure. 14:52:46 So it neutron CI does not run on master neutron-lib, this will happen again. 14:52:52 liuyulong: we aren't blocked but a bit slowered down I would say :) 14:53:14 liuyulong: it is true if it affects other stadium projects other than neturon. is my understanding right? 14:53:55 amotoki, not exactly, event a constant is only used by neutron itself. 14:54:00 at one point we were discussing something like that topic here https://review.opendev.org/#/c/602748/. but no one really had any interest 14:55:08 there are pros and cons to using neutron-lib master vs. released... if someone wants to kick the tires on this topic again go ahead, but I probably don't have the bandwidth 14:55:15 liuyulong: if a constant or function is used only by neutron, it doesn't have to be sent to the lib 14:55:19 So, I'd like to say, if we should ensure neutron happy first, then do that move. 14:55:21 liuyulong: if the neutron repo is an only consumer of some constant, there is no need for neutron-lib in pricinpal. 14:55:38 ^ true 14:55:42 amotoki is right 14:56:09 in fact, if that util function or constant is only used by neutron, it shouldn't go to the lib 14:56:23 Yes, agress 14:56:25 yep, that was the idea 14:56:26 agree 14:57:27 API definitions are exceptions as we would like to define the neutron API as a whole of the neutron stadium. All others are to share something in stadium projects. 14:58:42 However, if a function is only used by neutron alone temporarily, IMO, it should implement in neutron first. 14:59:16 liuyulong: you'ree right 14:59:17 ok, we are running out of time here 14:59:25 thx for attending 14:59:29 o/ 14:59:32 o/ 14:59:33 o/ 14:59:35 and have a great week! 14:59:36 o/ 14:59:38 #endmeeting