| *** markvoelker has joined #openstack-meeting-5 | 01:06 | |
| *** ricolin has joined #openstack-meeting-5 | 01:18 | |
| *** markvoelker has quit IRC | 01:40 | |
| *** jayahn has quit IRC | 02:29 | |
| *** srwilkers has quit IRC | 02:29 | |
| *** markvoelker has joined #openstack-meeting-5 | 02:37 | |
| *** markvoelker has quit IRC | 03:11 | |
| *** markvoelker has joined #openstack-meeting-5 | 04:08 | |
| *** yamahata has joined #openstack-meeting-5 | 04:20 | |
| *** yamahata has quit IRC | 04:24 | |
| *** yamahata has joined #openstack-meeting-5 | 04:24 | |
| *** iyamahat has joined #openstack-meeting-5 | 04:26 | |
| *** markvoelker has quit IRC | 04:41 | |
| *** markvoelker has joined #openstack-meeting-5 | 05:38 | |
| *** lujinluo has joined #openstack-meeting-5 | 06:01 | |
| *** lujinluo has quit IRC | 06:10 | |
| *** lujinluo has joined #openstack-meeting-5 | 06:12 | |
| *** markvoelker has quit IRC | 06:12 | |
| *** markvoelker has joined #openstack-meeting-5 | 07:09 | |
| *** markvoelker has quit IRC | 07:42 | |
| *** skazi has quit IRC | 07:43 | |
| *** markvoelker has joined #openstack-meeting-5 | 08:39 | |
| *** markvoelker has quit IRC | 09:13 | |
| *** lujinluo has quit IRC | 09:20 | |
| *** markvoelker has joined #openstack-meeting-5 | 10:10 | |
| *** markvoelker has quit IRC | 10:43 | |
| *** markvoelker has joined #openstack-meeting-5 | 11:40 | |
| *** markvoelker has quit IRC | 12:14 | |
| *** dims_ is now known as dims | 12:37 | |
| *** mjturek has joined #openstack-meeting-5 | 13:09 | |
| *** lujinluo has joined #openstack-meeting-5 | 13:36 | |
| *** njohnston has joined #openstack-meeting-5 | 13:40 | |
| *** d|k has quit IRC | 13:58 | |
| *** ianychoi has quit IRC | 13:59 | |
| lujinluo | #startmeeting neutron_upgrades | 14:00 |
|---|---|---|
| openstack | Meeting started Thu Aug 23 14:00:07 2018 UTC and is due to finish in 60 minutes. The chair is lujinluo. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:00 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
| *** openstack changes topic to " (Meeting topic: neutron_upgrades)" | 14:00 | |
| openstack | The meeting name has been set to 'neutron_upgrades' | 14:00 |
| njohnston | o/ | 14:00 |
| lujinluo | o/ | 14:00 |
| *** ianychoi has joined #openstack-meeting-5 | 14:00 | |
| lujinluo | just the two of us? let's give others some time | 14:00 |
| njohnston | +1 | 14:00 |
| *** mlavalle has joined #openstack-meeting-5 | 14:02 | |
| mlavalle | o/ | 14:02 |
| *** hongbin has joined #openstack-meeting-5 | 14:02 | |
| lujinluo | o/ | 14:02 |
| lujinluo | ok, let's get started first. If Tuan and Hung come, they can join right away | 14:03 |
| *** hungpv has joined #openstack-meeting-5 | 14:04 | |
| njohnston | sounds good | 14:04 |
| lujinluo | We have 2 AIs from last week | 14:04 |
| lujinluo | 1. njohnston Comment on https://review.openstack.org/#/c/565773/5/neutron/services/tag/tag_plugin.py@70 with location for decorator relocation | 14:04 |
| njohnston | Yes, I did that :-) | 14:04 |
| lujinluo | yeah, thanks | 14:04 |
| lujinluo | I also commented on that. Since we have a lot of places using that decorator | 14:05 |
| lujinluo | I wonder if we should move it to objects/db/api.py instead of moving to each specific objects, i.e. objects/tag.py | 14:05 |
| njohnston | yeah. So I have an action item on my side to talk with zzzeek about the engine facade pattern, to see how we should approach things while not breaking that | 14:06 |
| njohnston | Perhaps he can give a recommendation on this | 14:06 |
| lujinluo | i see. let me add a quick AI of this week then | 14:06 |
| lujinluo | #action njohnston to talk to zzzeek of decorators in engine facade pattern | 14:07 |
| njohnston | thanks! | 14:07 |
| slaweq | hi | 14:08 |
| lujinluo | The second AI we have is me to add decorators to the googl spread sheet | 14:08 |
| lujinluo | slaweq: hi! | 14:08 |
| lujinluo | The Google spread sheet is here https://docs.google.com/spreadsheets/d/1Mz7V40GSdcUH_aBoNWjsFaNRp4wf-duz1GFWligiNXc/edit#gid=1051788848 | 14:08 |
| lujinluo | but I have not added it yet, so sorry! | 14:08 |
| lujinluo | Will move this AI to my this week's AI | 14:08 |
| lujinluo | #action lujinluo Add decorators to google tracking spreadsheet | 14:09 |
| lujinluo | Then let's move to the patches | 14:09 |
| lujinluo | #topic OVO | 14:09 |
| *** openstack changes topic to "OVO (Meeting topic: neutron_upgrades)" | 14:09 | |
| lujinluo | #link https://review.openstack.org/#/q/topic:bp/adopt-oslo-versioned-objects-for-db+(status:open) | 14:10 |
| lujinluo | #link https://review.openstack.org/565773 Tag OVO | 14:10 |
| lujinluo | this is the one where we had the decorators discussion | 14:10 |
| njohnston | I'm inclined to say let's exclude the decorator from scope for now, and handle decorators separately once that picture becomes clearer | 14:11 |
| lujinluo | besides the decorator comment and commit msg comment left by slaweq, i am fine with the change | 14:11 |
| lujinluo | +1 | 14:11 |
| lujinluo | let me add a quick comment to Lajos then | 14:12 |
| njohnston | Sorry, I just commented :-) | 14:12 |
| lujinluo | haha, no worries and thanks! | 14:12 |
| lujinluo | #link https://review.openstack.org/#/c/544206/ Port Binding OVO | 14:13 |
| lujinluo | I worked on this patch this week and addressed Manjeet's comments from long time ago. | 14:14 |
| njohnston | I think the zuul failure is a transient | 14:14 |
| lujinluo | But I saw neutron-tempest-plugin-scenario-linuxbridge failed in the recent two patches | 14:14 |
| njohnston | I have seen that issue in other changes as well though | 14:15 |
| njohnston | not to say they are definitely not related to the change, but there is a strong chance it is unrelated | 14:15 |
| lujinluo | i see. then i will recheck tmr, and see how Zuul reacts | 14:16 |
| lujinluo | #action lujinluo to see if neutron-tempest-plugin-scenario-linuxbridge failure is transient in https://review.openstack.org/#/c/544206/ | 14:16 |
| lujinluo | #link https://review.openstack.org/#/c/561834/ Segment OVO | 14:17 |
| lujinluo | this one has received a +2 from slaweq | 14:17 |
| lujinluo | mlavalle: could you please take a look if it is ready? | 14:17 |
| mlavalle | lujinluo: sure, will look today | 14:17 |
| lujinluo | thanks | 14:17 |
| mlavalle | in my pile | 14:17 |
| lujinluo | #link https://review.openstack.org/#/c/579053/ Use OVO in neutron/ipam/subnet_alloc.py | 14:18 |
| mlavalle | in fact, he pinged me a little more than a week ago, but with my trip to China, I didn't have time | 14:18 |
| mlavalle | I meant 561834 | 14:19 |
| lujinluo | Haha, I see. stein just opened, so we still have a lot of time | 14:19 |
| lujinluo | For 579053, njohnston, i am sorry I made a mistake last week | 14:20 |
| lujinluo | update() always return None.. thus regressions occur | 14:20 |
| njohnston | I see | 14:20 |
| njohnston | So I should undo the change you suggested in your comment on PS3 | 14:21 |
| lujinluo | thus regardless of update() succeeds or not, RetryRequest is triggered | 14:21 |
| lujinluo | yes | 14:21 |
| njohnston | OK, I will do that, and then try and figure out what is going wrong with the UTs | 14:21 |
| lujinluo | thanks. i will also follow it :) | 14:21 |
| lujinluo | #link https://review.openstack.org/#/c/382037/ Port Binding Level OVO | 14:22 |
| njohnston | this one gets a +1 from me | 14:23 |
| lujinluo | I reviewed it last week, and it seems the latest patch addressed hongbin's comments about comments | 14:23 |
| hongbin | o/ | 14:24 |
| slaweq | yes, and release note | 14:24 |
| slaweq | I would be very happy if that could be merged finally :) | 14:24 |
| hongbin | the patch looks very good now | 14:24 |
| mlavalle | I'll get back to it today, then | 14:24 |
| lujinluo | i see. i am fine with it then, just +1ed | 14:25 |
| mlavalle | added to today's pile | 14:25 |
| lujinluo | thanks mlavalle! | 14:25 |
| slaweq | thx mlavalle :) | 14:25 |
| mlavalle | I made my first comment to that patch almost two years ago.... LOL | 14:25 |
| lujinluo | how time flies LOL | 14:26 |
| lujinluo | #link https://review.openstack.org/#/c/549168/ Router OVO | 14:26 |
| lujinluo | The author, Hung, is not here today, aslo no updates from last week either.. | 14:27 |
| njohnston | hungpv is online | 14:27 |
| lujinluo | oh, let me try ping him | 14:27 |
| lujinluo | hungpv: hi | 14:27 |
| njohnston | hmm perhaps he is not at his computer | 14:28 |
| hungpv | Hi guys. Sorry, i been busy with internal task this week | 14:28 |
| njohnston | hi! | 14:28 |
| lujinluo | hi! | 14:28 |
| hungpv | Haven't got time to update this yet | 14:29 |
| lujinluo | ok, anyway i will leave the comment we gave at our last meeting to the patch. hopefully you will have more time for it this week :) | 14:29 |
| lujinluo | That's all the patches we have for now. | 14:30 |
| lujinluo | #topic open discussion | 14:30 |
| *** openstack changes topic to "open discussion (Meeting topic: neutron_upgrades)" | 14:30 | |
| lujinluo | #link https://releases.openstack.org/rocky/schedule.html | 14:30 |
| njohnston | I just wanted to say, we only have 9 patches in the topic, 2 of which I could see getting merged in the next few days | 14:30 |
| lujinluo | A gentle reminder that we are at the final RC week of Rocky now | 14:30 |
| mlavalle | yes, we will cut a rc-2 later today | 14:31 |
| mlavalle | and that is going to be the end of Rocky | 14:31 |
| lujinluo | njohnston: yes | 14:31 |
| njohnston | I plan on creating a few more changes, and I also hope to drum up some help at the PTG, so hopefully things look more lively in terms of our activity :-) | 14:31 |
| lujinluo | njohnston: thank you!! | 14:31 |
| lujinluo | Just in case you may find the tracking spread sheet helpful, the link is here | 14:32 |
| lujinluo | #link https://docs.google.com/spreadsheets/d/1Mz7V40GSdcUH_aBoNWjsFaNRp4wf-duz1GFWligiNXc/edit#gid=1051788848 | 14:32 |
| mlavalle | one thing that I wanted to raise is the convergence of the OVO effort with the new engine facade adoption | 14:33 |
| mlavalle | possible convergence I meant | 14:33 |
| mlavalle | with the adoption of OVO, it seems to me we are reducing our dependence in the old engina facade | 14:33 |
| mlavalle | is that impression correct? | 14:33 |
| lujinluo | yes | 14:33 |
| mlavalle | on the other hand the new engine facade adoption effort is really stalled on its own | 14:34 |
| mlavalle | so since this team is combing the DB layer access anyway, amybe we should merge the efforts | 14:34 |
| mlavalle | maybe^^^^ | 14:34 |
| mlavalle | would that make sense? | 14:35 |
| njohnston | mlavalle: Where is the engine facade work tracked? | 14:35 |
| * njohnston thinks it is a great idea | 14:35 | |
| mlavalle | njohnston: that's the problem, nobody is really tracking it anymore | 14:35 |
| lujinluo | agree | 14:35 |
| lujinluo | #link https://blueprints.launchpad.net/neutron/+spec/enginefacade-switch BP of new engine facade | 14:35 |
| *** hungpv has quit IRC | 14:36 | |
| njohnston | well this team is very good at tracking! | 14:36 |
| mlavalle | I made an effort early in Rocky. I can resucitate a patch I pushed at that time | 14:36 |
| lujinluo | yeah, it seems it has been stalled for a while | 14:36 |
| mlavalle | not necessarily because the patch is mergable | 14:36 |
| mlavalle | but is was sort of an inverntory of wehere we still have old engine facade stuff | 14:37 |
| mlavalle | we can start from there | 14:37 |
| njohnston | sounds great! | 14:37 |
| mlavalle | I can work on it this coming weekend | 14:37 |
| lujinluo | it is a good idea | 14:37 |
| mlavalle | refresh it and I'll show it to this team during the next meeting | 14:38 |
| lujinluo | we can track its status in this meeting in the future | 14:38 |
| mlavalle | great | 14:38 |
| mlavalle | I'll take the first step then this weekend | 14:38 |
| lujinluo | thanks! mlavalle | 14:38 |
| mlavalle | bases on what we discover, it might become a topic in the PTG | 14:39 |
| mlavalle | based^^^ | 14:39 |
| lujinluo | yes! that's what I was thinking | 14:39 |
| lujinluo | #Link https://etherpad.openstack.org/p/neutron-stein-ptg | 14:39 |
| mlavalle | That's all I had | 14:39 |
| lujinluo | So at the PTG, what's from upgrades subteam is 1) more help wanted; 2) convergence of the efforts of new engine facade | 14:41 |
| mlavalle | nice! | 14:41 |
| njohnston | yes | 14:41 |
| lujinluo | So I will add these two points to the etherpad, but feel free to edit njohnston mlavalle :) | 14:41 |
| njohnston | and if the topic of using the SQLAlchemy "baked queries" functionality comes up for speeding up sql query construction, I will argue that it naturally belongs in the OVO objects | 14:41 |
| mlavalle | ok | 14:41 |
| lujinluo | njohnston: great | 14:41 |
| njohnston | nothing else from me :-) | 14:43 |
| mlavalle | I lied.... | 14:43 |
| mlavalle | I have one final question | 14:44 |
| mlavalle | I've been creating our Stein-1 dashboard: https://launchpad.net/neutron/+milestone/stein-1 | 14:44 |
| mlavalle | and the OVO blueprint is listed with Slow progress | 14:45 |
| mlavalle | but it seems to me we have good progress | 14:45 |
| mlavalle | are you guys ok if I change the status? | 14:45 |
| lujinluo | yes, please | 14:45 |
| mlavalle | cool | 14:46 |
| lujinluo | we have covered most of the objects by now | 14:46 |
| njohnston | we should update the completion percentage | 14:46 |
| mlavalle | now it's I am telling the truth.... I don't have anything else for today's meeting ;-) | 14:46 |
| njohnston | I hope that 26% complete is an understatement | 14:46 |
| lujinluo | hmm, this 26% bothers me now. | 14:50 |
| lujinluo | But from the objects part, we have covered most of them. The rest is mainly joins that we intentionally put away. | 14:51 |
| njohnston | I assume the 26% is manually computed since it has been constant despite the changes I have introduced | 14:51 |
| njohnston | and it just has not been updated | 14:51 |
| njohnston | did we decide to defer action on the RBAC objects? | 14:52 |
| njohnston | that probably predates me | 14:52 |
| lujinluo | No, we did not decide to defer it. Just no one has enough bandwidth for it | 14:52 |
| njohnston | OK so if I get a volunteer in Denver I can give it to them :-) | 14:53 |
| lujinluo | exactly | 14:53 |
| lujinluo | ok, does anyone have anything else? | 14:54 |
| lujinluo | if not, we can call it a day | 14:54 |
| njohnston | nothing here, thanks for a productive meeting | 14:54 |
| lujinluo | thanks for being here njohnston :) | 14:55 |
| lujinluo | have a good day/night my friends | 14:55 |
| lujinluo | see you all next week | 14:55 |
| lujinluo | #endmeeting | 14:55 |
| *** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings/" | 14:55 | |
| openstack | Meeting ended Thu Aug 23 14:55:48 2018 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:55 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/neutron_upgrades/2018/neutron_upgrades.2018-08-23-14.00.html | 14:55 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/neutron_upgrades/2018/neutron_upgrades.2018-08-23-14.00.txt | 14:55 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/neutron_upgrades/2018/neutron_upgrades.2018-08-23-14.00.log.html | 14:55 |
| *** mlavalle has left #openstack-meeting-5 | 14:59 | |
| *** njohnston has left #openstack-meeting-5 | 15:01 | |
| *** lujinluo has quit IRC | 15:05 | |
| *** ricolin has quit IRC | 16:24 | |
| *** ricolin has joined #openstack-meeting-5 | 16:42 | |
| *** iyamahat has quit IRC | 17:30 | |
| *** yamahata has quit IRC | 17:57 | |
| *** ricolin has quit IRC | 18:07 | |
| *** iyamahat has joined #openstack-meeting-5 | 18:11 | |
| *** yamahata has joined #openstack-meeting-5 | 18:37 | |
| *** mjturek has quit IRC | 18:49 | |
| *** mjturek has joined #openstack-meeting-5 | 19:04 | |
| *** mjturek has quit IRC | 19:32 | |
| *** mjturek has joined #openstack-meeting-5 | 19:38 | |
| *** radeks has joined #openstack-meeting-5 | 19:56 | |
| *** mjturek has quit IRC | 20:01 | |
| *** radeks has quit IRC | 20:12 | |
| *** radeks has joined #openstack-meeting-5 | 20:12 | |
| *** mjturek has joined #openstack-meeting-5 | 20:26 | |
| *** iyamahat_ has joined #openstack-meeting-5 | 20:53 | |
| *** iyamahat has quit IRC | 20:56 | |
| *** mjturek has quit IRC | 21:05 | |
| *** iyamahat__ has joined #openstack-meeting-5 | 21:15 | |
| *** iyamahat_ has quit IRC | 21:18 | |
| *** radeks has quit IRC | 21:28 | |
| *** ricolin has joined #openstack-meeting-5 | 23:29 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!