18:00:46 <bh526r_> #startmeeting gluon
18:00:47 <openstack> Meeting started Wed Dec 14 18:00:46 2016 UTC and is due to finish in 60 minutes.  The chair is bh526r_. Information about MeetBot at http://wiki.debian.org/MeetBot.
18:00:48 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
18:00:50 <openstack> The meeting name has been set to 'gluon'
18:00:59 <bh526r_> hello everyone
18:01:07 <georgk> #info Georg Kunz
18:01:09 <bh526r_> #topic Roll Call
18:01:09 <jinli> Hi all
18:01:11 <georgk> Hi
18:01:14 <bh526r_> #info Bin Hu
18:01:19 <bh526r_> Hi Georg
18:01:21 <jinli> #info JinLi
18:01:43 <krenczewski> #info Kamil Renczewski
18:01:50 <bh526r_> Hi Kamil
18:02:00 <krenczewski> Hello
18:02:04 <bh526r_> #topic Admin Update
18:02:27 <bh526r_> #info not so much update
18:02:57 <bh526r_> #info Release team has announced a method for projects to set create stable/ocata branch
18:03:46 <bh526r_> #info we need to follow the steps to create stable/ocata branch later on, when our development is more complete and more stable
18:04:34 <bh526r_> #info And to tag the release, of course
18:04:56 <bh526r_> #info we can worry about it in new year
18:05:05 <bh526r_> #topic Gluon Tasks
18:05:21 <bh526r_> #info Task list for informational purpose
18:05:35 <bh526r_> #link https://wiki.openstack.org/wiki/Gluon/Tasks-Ocata
18:05:48 <bh526r_> #link https://etherpad.openstack.org/p/ocata-gluon-work-plan
18:06:07 <bh526r_> #link https://etherpad.openstack.org/p/ocata-nova-neutron-session
18:06:32 <bh526r_> #topic Status Update and Patch Review
18:06:56 <bh526r_> #info (1) Authentication and Access Control
18:07:12 <bh526r_> #link https://review.openstack.org/#/c/398577/
18:07:59 <bh526r_> #info Ian provided comments, and Kamal uploaded a new patchset 4
18:08:32 <bh526r_> I think it is quite stable, I suggest we move on and merge it.
18:08:49 <bh526r_> Any other comments or suggestions?
18:09:41 <bh526r_> If no objection, it is approved
18:10:14 <bh526r_> #info This patch is approved, and will be merged
18:11:16 <bh526r_> #info (2) OpenContrail's Mechanism Driver
18:11:39 <bh526r_> #link https://review.openstack.org/#/c/402071/
18:12:04 <alraddarla_> #info Darla Ahlert
18:12:06 <alraddarla_> sorry i am late
18:12:33 <bh526r_> #info Kamil submitted new patchset with README.md
18:13:26 <bh526r_> #info Bin is still verifying if it works with Gluon
18:14:09 <bh526r_> #info And comments regarding if using extended plugin method is a better way than mechanism driver
18:14:40 <bh526r_> #info And comments regarding tempoary home v.s. permanent home
18:15:01 <bh526r_> #info It seems that more time is needed to finish the review
18:15:54 <bh526r_> #info (3) Testing
18:15:57 <bh526r_> #link https://review.openstack.org/#/c/388821/
18:16:49 <bh526r_> Jin, have you addressed Paul's comments?
18:17:08 <jinli> will reply to him today
18:17:27 <bh526r_> e.g. Line 36: raise -> is raised
18:18:28 <jinli> Oh, for that one, I already made changes for him
18:18:40 <jinli> I updated new patches
18:19:28 <bh526r_> I just checked new patchset 4, it doesn't seem changed
18:19:36 <bh526r_> And line 43, 47 and 52
18:19:46 <bh526r_> Those are not changed in patchset 4
18:20:26 <jinli> I am checking
18:21:17 <bh526r_> #info Jin will double check and make sure the comments on Line 36, 43, 47, 52 etc., by Paul will be addressed
18:22:00 <bh526r_> #info Once those are addressed, we can approve it.
18:22:33 <bh526r_> #link https://review.openstack.org/#/c/393935/
18:23:39 <bh526r_> #info it sits there for a while, Jin submitted patchset 14 that addressed PEP8 issue
18:25:41 <bh526r_> If no more comments or objection, I suggest to approve it
18:26:51 <bh526r_> Jin, I have one question:
18:26:59 <jinli> ys
18:27:17 <jinli> yes
18:27:54 <bh526r_> You added an "import" in gluon/api/baseObject.py in your patchset 14 submitted on Nov 30
18:28:57 <bh526r_> Tom had a patch https://review.openstack.org/#/c/399051/, which was merged on Dec 6
18:29:37 <bh526r_> In his patch, gluon/api/baseObject.py was also modified quite a lot
18:30:47 <bh526r_> Because the patch 399051 was merged on Dec 6, it means your patch on Nov 30 is NOT based on most recent code base of gluon/api/baseObject.py.
18:30:50 <bh526r_> Right?
18:31:00 <jinli> let me do a recheck and make sure the test case are still ok
18:31:04 <jinli> thanks Bin
18:31:42 <bh526r_> Sounds good, please do a recheck, and also make sure your patch is based on most recent code base, e.g. gluon/api/baseObject.py
18:33:10 <jinli> srue
18:33:12 <jinli> sure
18:33:21 <bh526r_> #info Jin will recheck test his patch and make sure (1) test case is still ok based on most recent code base (2) changes on gluon/api/baseObject.py is on most recent code base
18:34:43 <bh526r_> #info (4) ODL Shim Layer
18:34:59 <bh526r_> #link https://review.openstack.org/#/c/407388/
18:35:47 <bh526r_> #info Kamal gave some comments, and +2
18:36:22 <bh526r_> #info It seems that in new ODL release, the API has slightly changed, so the adaptor also needs to be re-worked
18:37:28 <bh526r_> #info Also Kamal commented that it needs to use configure file instead of hardcoded
18:38:05 <bh526r_> #info e.g. in shim.conf, add a [Open Daylight] section for those configuration needed for ODL
18:39:46 <bh526r_> #info So this patch needs update to address (1) new adaptor for ODL APIs that are changed (2) configruation file support (3) and other comments in review
18:40:24 <bh526r_> any other comments?
18:41:11 <bh526r_> #info Georg takes the action to address those issues and upload new patchset :)
18:41:49 <bh526r_> #info (5) Developer Documentation
18:41:52 <bh526r_> #link https://review.openstack.org/#/c/395031/
18:42:07 <bh526r_> #info Looks like it was approved and merged yesterday
18:42:48 <bh526r_> #info congratulations, team. We had one merged :)
18:43:23 <bh526r_> #info (6) Bug Fixes
18:43:33 <bh526r_> #link https://review.openstack.org/#/c/410837/
18:43:46 <bh526r_> #info this is a new patch Darla submitted this morning
18:44:13 <bh526r_> #info Tin and Bin reviewed it, and gave a couple of suggestions
18:44:28 <alraddarla_> #info Darla will be updating that with the comments later today. Wanted to see if anyone else had any other comments first
18:44:47 <bh526r_> any other comments from everyone?
18:46:08 <bh526r_> Thank you Darla
18:46:50 <bh526r_> #info (7) Nova Enhancement
18:47:03 <bh526r_> #link https://review.openstack.org/#/c/390513/
18:47:38 <bh526r_> #info Ian is working with Nova team on this patch
18:47:52 <bh526r_> #info There are some comments and discussions
18:48:01 <bh526r_> #info so work in progress
18:48:37 <bh526r_> #topic Next Steps
18:49:20 <bh526r_> #info There are several actions through the review above, for Jin, Georg, and Darla to take
18:49:43 <bh526r_> #info Hopefully we will have new patchset, and ready to approve and merge
18:50:01 <bh526r_> #topic AOB
18:50:24 <krenczewski> I have a question regarding OpenContrail’s Mechanism Driver
18:50:33 <bh526r_> Do you have holiday plan? I am thinking of whether we keep or cancel the meeting on Dec 21 and Dec 28.
18:50:37 <bh526r_> Ok, sure
18:51:01 <krenczewski> Bin, in summary of yesterday's meeting you wrote about discussion between me, Nachi, Ian and Tom
18:51:09 <bh526r_> Yes
18:51:23 <krenczewski> Do you know if someone initiated something?
18:51:47 <bh526r_> Let me initiate this email thread today so that we can keep the ball rolling
18:51:55 <krenczewski> OK, thanks
18:52:11 <bh526r_> Sure.
18:53:15 <bh526r_> What is everyone's opinion about the meeting on Dec 21 and Dec 28?
18:53:49 <krenczewski> Dec 21 is Ok, but 28 is problematic :-)
18:54:03 <bh526r_> I see.
18:54:24 <bh526r_> So let's keep next week's meeting on Dec 21, and cancel the one on Dec 28
18:54:39 <bh526r_> Does it work for everyone?
18:54:52 <krenczewski> +1
18:55:13 <bh526r_> Great.
18:55:20 <alraddarla_> Works for me
18:55:42 <bh526r_> #info Group agreed to keep the meeting next week (Dec 21), but cancel the meeting on Dec 28 to enjoy holidays
18:56:13 <bh526r_> Hay Ian is here
18:56:46 <bh526r_> That's all from me for today
18:57:24 <bh526r_> If nothing else, let's adjourn the meeting
18:57:31 <bh526r_> #info meeting adjourned
18:57:36 <bh526r_> Thank you everyone
18:57:42 <bh526r_> #endmeeting