14:00:14 <lujinluo> #startmeeting neutron_upgrades
14:00:16 <openstack> Meeting started Thu Jun  7 14:00:14 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:17 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
14:00:20 <openstack> The meeting name has been set to 'neutron_upgrades'
14:00:29 <lujinluo> o/
14:00:57 <lujinluo> ok, i will wait for a while for people to join me.
14:01:06 <njohnston> o/
14:01:19 <TuanVu> Hi Luo-san
14:01:24 <TuanVu> Hi njohnston
14:01:24 <lujinluo> hi njohnston ! so happy that i am not alone!
14:01:26 <lujinluo> hi TuanVu
14:01:33 <mlavalle> o/
14:01:40 <lujinluo> hello mlavalle :)
14:01:42 <njohnston> :-)
14:01:49 <TuanVu> Hi mlavalle
14:01:53 <lujinluo> ok, i think we have gather enough peopl
14:02:02 <lujinluo> we do not have any AIs from last wee
14:02:06 <lujinluo> *week
14:02:07 <mlavalle> Hi lujinluo, TuanVu, njohnston
14:02:14 <lujinluo> so let's jump to OVO
14:02:18 <lujinluo> #topic OVO
14:02:47 <lujinluo> https://review.openstack.org/#/q/topic:bp/adopt-oslo-versioned-objects-for-db+(status:open+)
14:03:09 <lujinluo> https://review.openstack.org/#/c/549168/ the first one is about Router OVO
14:03:30 <lujinluo> but it just got updated around 30 mins ago. Zuul has not said anything yet
14:04:16 <lujinluo> hungpv: do you have any specific updates? or shall we wait for the result of Zuul first?
14:04:31 <hungpv> yes, I took some comments from you
14:04:49 <hungpv> the only one I can think of right now
14:05:29 <hungpv> https://review.openstack.org/#/c/549168/10/neutron/db/l3_db.py
14:05:46 <hungpv> L444-445
14:06:06 <hungpv> with context.session.begin(subtransactions=True):
14:06:09 <hungpv> context.session.refresh(router)
14:06:20 * mlavalle would like to say a couple of things in the Open Agenda section of the meeting
14:06:35 <hungpv> I wanted to replace it with something like router.refresh()
14:06:45 <hungpv> is it correct though?
14:06:58 <lujinluo> is 'router' now an OVO object?
14:07:08 <hungpv> yes, it is
14:07:23 <lujinluo> then you should use router.update() to update changed fields
14:07:32 <hungpv> ok, i see
14:07:52 <hungpv> so can you comeback tomorrow anfd give me your comments on it?
14:08:16 <lujinluo> yes, and i also saw some other places need to be addressed. will review tmr
14:08:39 <hungpv> thank you
14:08:47 <hungpv> so that's all from me
14:08:59 <lujinluo> ok, next one is https://review.openstack.org/#/c/565358/
14:09:13 <lujinluo> some of us have rebased it a couple of times
14:09:30 <TuanVu> yeah, because my network patch is based on it
14:09:52 <lujinluo> but every time it fails fullstack test
14:10:08 <lujinluo> i will debug it tmr. it seems related to the changes made by the patch
14:10:26 <TuanVu> that's great!
14:10:31 <TuanVu> thank you in advance, Luo
14:11:18 <lujinluo> next is https://review.openstack.org/#/c/507772/
14:11:43 <TuanVu> latest update is in my newest comment
14:11:57 <TuanVu> it looks pretty green now, thanks to your help, Luo-san :)
14:12:04 <lujinluo> i saw you have already updated the patch!
14:12:19 <TuanVu> yeah, as your suggestion
14:12:37 <TuanVu> however, there are still 2 failed Tempest Tests
14:12:43 <TuanVu> and fullstack test
14:12:52 <TuanVu> looks like there's problem with network mtu
14:13:08 <lujinluo> it seems the remaining issues are the ones mentioned by slawek last week
14:13:17 <lujinluo> anyway good to know you made progress!
14:13:37 <TuanVu> thanks :)
14:14:15 <lujinluo> ok, i will check the mtu issue tmr as well. hopefully we can find something useful
14:14:27 <TuanVu> that's awesome!
14:14:53 <TuanVu> your suggestion have been helping me a lot so far
14:15:02 <lujinluo> good to know :)
14:15:17 <lujinluo> we do not have any other updates from the remaining patches
14:15:28 <lujinluo> #topic open discussion
14:15:48 <lujinluo> mlavalle: you mentioned you have something to share?
14:15:56 <mlavalle> yeah
14:16:05 <lujinluo> please
14:16:14 <mlavalle> I have two points:
14:16:32 <mlavalle> 1) This bug was filed a couple of days ago https://bugs.launchpad.net/neutron/+bug/1775220
14:16:33 <openstack> Launchpad bug 1775220 in neutron "Unit test neutron.tests.unit.objects.test_ports.PortBindingLevelDbObjectTestCase. test_get_objects_queries_constant fails often" [High,Confirmed] - Assigned to Miguel Lavalle (minsel)
14:17:04 <mlavalle> We are finding that this test fails sometimes
14:17:12 <mlavalle> re-checking takes care of it
14:17:45 <mlavalle> I assigned it to myself to not loose track of it
14:17:59 <mlavalle> Would any of you help fixing it?
14:18:30 <lujinluo> i think this is the test case that bothered you for a long time TuanVu
14:18:59 <lujinluo> could you take a look at it, if possible? i think you might be familiar with it. i will also help for sure
14:19:00 <TuanVu> hmm ... let me see
14:20:05 <TuanVu> honestly, I didn't know how to fix it. It was a great thank to the patch from Ihar
14:20:36 <lujinluo> ok, i will check it then. mlavalle
14:20:43 <mlavalle> thanks!
14:21:05 <mlavalle> second thing is in regards to https://review.openstack.org/#/c/414251/
14:21:30 <mlavalle> Yesterday I sent this message to the ML: http://lists.openstack.org/pipermail/openstack-dev/2018-June/131252.html
14:21:55 <mlavalle> I don't want 414251 to break other projects
14:22:07 <mlavalle> so I sent the mail and talking to many of them directly
14:22:38 <mlavalle> What I wanted to say here is that you should expect 414251 to merge towards the end of next week
14:22:50 <mlavalle> I know one of your patches depends on it
14:22:59 <lujinluo> understood. yes, the portbinding one
14:23:21 <mlavalle> it would be better if you rebased that patch on top of https://review.openstack.org/#/c/571041
14:23:55 <mlavalle> This one pluralizes the names of port_binding and the binding attribute in the Port OVO
14:24:10 <mlavalle> which lujinluo reocmmended me to do in one of her reviews ;-)
14:24:30 <lujinluo> haha, got it.
14:24:36 <lujinluo> will do it in next week
14:24:41 <mlavalle> cool
14:24:51 <mlavalle> that's all I had for today ;-)
14:25:01 <lujinluo> good.
14:25:02 <njohnston> I have one minor question.  I was looking at some of the changes in my area that need to be made, and some of them are implementing join functionality, for example http://git.openstack.org/cgit/openstack/neutron/tree/neutron/plugins/ml2/drivers/l2pop/db.py#n77 - I was wondering if there were examples of how others have handled this sort of thing.
14:25:46 <lujinluo> njohnston: unfortunately we have not figured out a general way to handle joins
14:26:07 <lujinluo> that's why they have been pushed away for so long
14:26:29 <mlavalle> njohnston: any suggestions?
14:27:25 <lujinluo> yeah, any suggestions would be welcomed
14:28:10 <lujinluo> we discussed it once, and it may end up to be case-by-case by overriding get_object in each OVO objects. but we have tried any out yet
14:28:21 <njohnston> I have only flights of fancy as of yet, I will see if I can dig in to it sometime soon and get a real sense of how the data is being massaged.
14:28:40 <lujinluo> cool
14:28:53 <mlavalle> cool, thanks. Very often that's how one gets to be an expert on something
14:29:01 <njohnston> That is my hope :-)
14:29:04 <mlavalle> asking a questions and having to dig deeper
14:29:28 <lujinluo> i am memo-ing this ^
14:30:03 <mlavalle> The first expert on something most likely started that way
14:30:41 <lujinluo> i have one announcement. i am taking PTO from June 13th to June 25th. So i will not be able to chair the next two weekly meetings
14:30:56 <lujinluo> shall we cancel them or maybe someone else would like to chair?
14:31:02 <njohnston> I could help out
14:31:10 <lujinluo> thank you! njohnston
14:31:10 <njohnston> I have experience chairing meetings
14:31:12 <mlavalle> great njohnston!
14:31:25 <lujinluo> we love you!
14:31:42 <mlavalle> have fun lujinluo
14:31:53 <njohnston> Am I remembering correctly I need to be on the list of chairs in order to be able to use the startmeeting command?
14:32:06 <lujinluo> not exactly i think
14:32:33 <lujinluo> i used to help Ihar chair meetings while i was not listed as chairs at that moment
14:33:33 <njohnston> ok good :-)
14:33:34 <lujinluo> mlavalle: correct? (asking for double check
14:33:59 <mlavalle> I think that's the case
14:34:10 <mlavalle> if you start the meeting, you are the chair
14:34:57 <lujinluo> so basically anyone can start the meeting
14:35:02 <mlavalle> yes
14:35:07 <lujinluo> cool!
14:35:18 <lujinluo> then thank you for the next two weeks! njohnston
14:35:26 <njohnston> Happy to help
14:35:34 <njohnston> Enjoy your vacation!
14:35:41 <lujinluo> thank you!
14:35:54 <lujinluo> does anyone have anything else?
14:36:13 <TuanVu> none from me
14:36:57 <lujinluo> ok then. let's call it a day
14:37:01 <njohnston> nope
14:37:07 <lujinluo> #endmeeting