14:00:10 <lujinluo> #startmeeting neutron_upgrades
14:00:11 <openstack> Meeting started Thu Aug 30 14:00:10 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:12 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
14:00:14 <openstack> The meeting name has been set to 'neutron_upgrades'
14:00:19 <lujinluo> o/
14:00:43 <lujinluo> ok, I will wait for my friends first
14:01:41 <lujinluo> o/
14:01:51 <mlavalle> o/
14:02:04 <lujinluo> hi! mlavalle
14:02:14 <mlavalle> hi lujinluo
14:02:18 <lujinluo> njohnston: are you around for the meeting?
14:02:30 <mlavalle> lujinluo: you still in Japan?
14:02:47 <lujinluo> no, I am in China during the past two months
14:02:56 <lujinluo> Will be flying to the US tmr
14:03:18 <njohnston> good morning
14:03:29 <lujinluo> good morning! njohnston
14:03:37 <njohnston> sorry I am double booked today, but I will do my best :-)
14:03:44 <mlavalle> me too
14:03:49 <njohnston> where to the US are you travelling lujinluo?
14:04:01 <lujinluo> ack, i will try to finish this meeting as soon as possible
14:04:15 <lujinluo> San Francisco \o/
14:04:43 <njohnston> Excellent!  It's a lovely city.  Going for business or pleasure?
14:04:55 <lujinluo> Going to school actually, haha
14:05:10 <njohnston> Excellent!
14:05:12 <lujinluo> I plan to pursue another graduate degree
14:06:05 <lujinluo> So back to the agenda: we have 3 AIs from last week
14:06:17 <lujinluo> 1. njohnston to talk to zzzeek of decorators in engine facade pattern
14:06:33 <lujinluo> do you have any updates on that njohnston?
14:07:10 <njohnston> I did have a chance to talk to him a bit, but I need to get a meeting with him for a deeper dive.  He did give me a few tips, and is willing to review code.
14:07:24 <lujinluo> excellent!
14:07:31 <mlavalle> is he going to Denver?
14:07:42 <njohnston> Unsure, I will check on that.
14:08:01 <mlavalle> we might have a face to face meeting with him
14:08:28 <lujinluo> Yeah, that would help a lot
14:08:57 <mlavalle> I'll even buy him a beer if he devotes us a session
14:09:31 <lujinluo> Haha
14:09:37 <njohnston> :-)
14:10:05 <lujinluo> I will follow the discussion on etherpad, :-)
14:10:24 <lujinluo> 2. lujinluo Add decorators to google tracking spreadsheet
14:11:12 <lujinluo> I reviewed the Neutron code base and I think db_api.retry_session_if_inactive is the only place where we need to move to OVO
14:11:22 <lujinluo> So I updated it on #link https://docs.google.com/spreadsheets/d/1Mz7V40GSdcUH_aBoNWjsFaNRp4wf-duz1GFWligiNXc/edit#gid=1051788848
14:12:03 <lujinluo> Line 131
14:12:30 <lujinluo> 3. lujinluo to see if neutron-tempest-plugin-scenario-linuxbridge failure is transient in https://review.openstack.org/#/c/544206/
14:12:53 <lujinluo> I rechecked 544206, and it passed Zuul \o/
14:14:07 <lujinluo> I have a silly question though. Previsouly we can comment "check experimental" to see the results of experimental tests. That did not seem to work anymore
14:14:34 <lujinluo> How can I see the results of experimental tests? mlavalle and njohnston do you have any suggestions?
14:14:48 <njohnston> "check experimental" should work, I'll give it a try
14:14:55 <mlavalle> yeah
14:15:00 <njohnston> I used it not too long ago
14:15:45 <lujinluo> i see. then maybe it was due to I commented "recheck" first so it triggered ordinary Zuul first.
14:15:50 <njohnston> checking it on https://review.openstack.org/580450
14:16:45 <lujinluo> I see. thanks
14:16:49 <njohnston> I see that change in the experimental queue in zuul now, so it works :-)
14:17:18 <lujinluo> Cool. understood
14:17:33 <lujinluo> Then let's move to the patches
14:17:40 <lujinluo> #topic OVO patches
14:17:59 <lujinluo> #link https://review.openstack.org/#/q/status:open+project:openstack/neutron+branch:master+topic:bp/adopt-oslo-versioned-objects-for-db
14:18:18 <lujinluo> #link Tag OVO https://review.openstack.org/#/c/565773/
14:18:45 <lujinluo> Lajos addressed slaweq's comment of "register_hook"
14:19:19 <lujinluo> The patch seems fine to me now. Please help review it when available
14:19:37 <njohnston> will do
14:20:14 <lujinluo> thanks
14:20:24 <mlavalle> `/me added it to his pile
14:20:29 <mlavalle> LOL
14:20:42 <lujinluo> #link https://review.openstack.org/#/c/537320/ Port OVO in db/external_net_db
14:20:44 * mlavalle added it to his pile
14:21:21 <lujinluo> I respinned earlier. There are several failed UTs. Need to double check it is related
14:22:28 <lujinluo> #link https://review.openstack.org/#/c/544206/ Port Binding OVO
14:23:21 <lujinluo> It passed all the tests. But I need to see how it behaves in experimental tests
14:24:12 <lujinluo> #link https://review.openstack.org/#/c/382037/ Port Binding Level OVO
14:24:33 <lujinluo> njohnston and I gave +1. mlavalle could you please take a look when available?
14:24:49 <mlavalle> yes, added to my pile
14:24:59 <lujinluo> It is slaweq's patch. So we may need another +2. Let me add some more cors
14:25:02 <lujinluo> *cores
14:26:59 <lujinluo> OK, added Brian and Akihiro
14:27:11 <lujinluo> All the rest patches are not updated since next week
14:27:21 <lujinluo> #topic open discussion
14:28:20 <lujinluo> Just FYI, the new engine facade switch patch mlavalle mentioned last week. is it this one #link https://review.openstack.org/#/c/545501/ ?
14:29:06 <mlavalle> yes it is
14:29:28 <lujinluo> understood. will keep an eye on it!
14:30:03 <mlavalle> let me make a comment on it
14:30:46 <lujinluo> just added you to the patch :) njohnston
14:31:18 <mlavalle> the patch was based on this queries:
14:31:31 <mlavalle> http://codesearch.openstack.org/?q=session.begin&i=nope&files=&repos=neutron
14:31:41 <mlavalle> http://codesearch.openstack.org/?q=autonested_transaction&i=nope&files=&repos=neutron
14:31:49 <njohnston> thank you lujinluo!
14:32:29 <mlavalle> I just ran these queries prior to the meeting
14:32:45 <lujinluo> understood.
14:32:48 <mlavalle> and my impresion is that the results are shorter than when I put together that patch
14:33:32 <lujinluo> haha, this is good
14:33:36 <mlavalle> so the patch might reflect a "worst case scenario"
14:34:48 <mlavalle> I think really the approach should be, instead of a big patch, decompose it in managebale pieces based on the queires
14:35:21 <lujinluo> agree
14:35:21 <mlavalle> and I am available to help
14:35:46 <lujinluo> Will try to help too
14:35:51 <lujinluo> :-)
14:37:07 <lujinluo> So if no one has anything else, let's call it a day! So mlavalle and njohnston can concentrate on the other task! :)
14:37:34 <njohnston> Thanks lujinluo - have safe travels!
14:37:45 <lujinluo> thank you njohnston !
14:38:27 <lujinluo> #endmeeting