18:01:01 <bh526r> #startmeeting gluon
18:01:02 <openstack> Meeting started Wed Aug 23 18:01:01 2017 UTC and is due to finish in 60 minutes.  The chair is bh526r. Information about MeetBot at http://wiki.debian.org/MeetBot.
18:01:03 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
18:01:05 <openstack> The meeting name has been set to 'gluon'
18:01:22 <bh526r> #topic Roll Call
18:01:37 <bh526r> #info Bin Hu
18:01:47 <krenczewski> #info Kamil Renczewski
18:01:50 <bh526r> #info Jin Li
18:01:55 <bh526r> Hi Kamil
18:02:05 <krenczewski> Hi Bin
18:02:33 <bh526r> I sent you and some others a proposal of demo scenario
18:02:50 <bh526r> We can discuss it after we finish discussing patches
18:03:06 <krenczewski> Ok
18:03:39 <bh526r> #topic Admin Update
18:04:19 <bh526r> #info Bin will attend PTG in Denver.
18:04:56 <bh526r> #info No other admin update
18:05:10 <bh526r> #info Pike Release Status
18:05:27 <bh526r> #topic Pike Release Status
18:05:48 <bh526r> #info Bin cut the stable/pike branch
18:06:31 <bh526r> #info Bin has installed Gluon stable/pike branch with Devstack stable/pike branch
18:06:55 <bh526r> #info Sanity check seems good for Gluon and Proton
18:07:48 <bh526r> #info Documentation update is needed for updated command line. Bin is working on it.
18:08:21 <bh526r> #info Once remaining outstanding patches of policy are approved and merged, Bin will submit documentation patch
18:09:35 <bh526r> One issue is that, in Devstack stable/pike, etcd3 is started by default. Gluon is using etcd-2.3.6. So there are issues of competing ports of etcd cluster
18:10:13 <bh526r> Current solution is to disable etcd3 after installing Devstack stable/pike, then we can move on as before
18:10:59 <bh526r> We need to look into using etcd3 in Gluon after Pike release, then we can reuse etcd cluster started by Devstack
18:11:57 <bh526r> Alternatively, we can look into using different ports / a different etcd cluster for Gluon backends.
18:12:16 <bh526r> This will be the work after Pike release
18:13:04 <bh526r> #info We need to look into resolving etcd3 v.s. etcd 2.3.6 after Pike Release
18:13:35 <bh526r> #info Now let's look at outstanding 3 patches
18:14:13 <bh526r> #info Patch 1: Gluon object-level authorization
18:14:23 <bh526r> #link https://review.openstack.org/#/c/482752/
18:14:50 <bh526r> #info Jin and Bin went over the code. All look good.
18:14:58 <bh526r> #info Existing comments were resolved.
18:15:08 <bh526r> I suggest group to approve this patch
18:15:43 <bh526r> #info Group approved this patch 482757. Bin will take the action to merge it, and cherrypick to stable/pike
18:16:18 <bh526r> #info Patch 2: Policies in yaml
18:16:30 <bh526r> #link https://review.openstack.org/#/c/486254/
18:16:47 <bh526r> Jin and I are sitting together to review the code.
18:27:21 <bh526r> #info models/net-l3vpn/api.yaml: (1) add tenant-id for VpnAfConfig to track which project creates this object.; (2) change network_owner to owner
18:32:36 <bh526r> #info managers/manager_base.py: Line 113 "def create_ports(self, api_class, values):": put comments there indicating that the code now is hard-coded to create default interface when creating ports. We need a TO-DO to change the hardcoded code to support all types of YAML.
18:33:10 <bh526r> #info This TO-DO is beyond Pike
18:41:04 <bh526r> #info: policies/base.py: doc/samples/policy.jason is no longer needed. We need to add a comment withint this policy.json, indicating that the first part is moved to code, and the rest are defined in YAML. This file is no longer needed, but for historical record.
18:42:15 <bh526r> #info policies/net-l3vpn.py: already comment there that it is no longer needed, because polocy is defined in YAML. This is also for historical record.
18:45:16 <bh526r> sorry Kamil, we may not have time to go over PoC scenario draft
18:45:34 <krenczewski> OK, lets move this to next week
18:45:50 <bh526r> because we want to finsh review of Pike patches
18:45:55 <bh526r> sorry for this Kamil
18:46:12 <bh526r> Meanwhile, if you have comments, we can discuss in email
18:46:40 <bh526r> Then next week, we can hopefully finish the discussion of draft PoC scenario
18:47:35 <krenczewski> Currently there is much of work to do before we can say what we could show on summt
18:47:50 <krenczewski> So probably next week would be to early for me
18:53:11 <bh526r> Sure, thank you Kamil
18:54:22 <bh526r> #info requirements.txt: this is not needed. So it needs to be deleted in the patchset
18:55:25 <bh526r> #info: all others look good. Code themselves look good.
18:55:51 <bh526r> #info Jin will update the comments and YAML based on above review, and upload a new patchset.
18:56:48 <bh526r> #info Group agrees to approve the new patchset, and Bin will take the action to merge it
18:57:18 <bh526r> #info Patch 3: Function delete_port() takes exactly 3 arguments (4 given) in handle_port_delete()
18:57:28 <bh526r> #link https://review.openstack.org/#/c/496167/
18:58:20 <bh526r> #info It is being reviewed and there are review comments. It is expected that author will further work on it. It will be approved if everything is fine after review. Bin will take the action to merge after approval
18:58:28 <bh526r> #topic AOB
18:58:42 <bh526r> #info Nothing else. Meeting adjourned.
18:59:00 <bh526r> #info Team will work offline to finalize those patches and documentation this week
18:59:08 <bh526r> #endmeeting