17:59:29 <flwang> #startmeeting zaqar
17:59:30 <openstack> Meeting started Mon May 30 17:59:29 2016 UTC and is due to finish in 60 minutes.  The chair is flwang. Information about MeetBot at http://wiki.debian.org/MeetBot.
17:59:31 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
17:59:33 <openstack> The meeting name has been set to 'zaqar'
17:59:50 <flwang> #topic roll call
18:00:13 <flwang> o/
18:00:21 <Eva-i> o/
18:01:25 <flwang> vkmc: ?
18:02:10 <flwang> #topic code review
18:02:26 <vkmc> o/
18:02:47 <flwang> though i don't want to be pushy, i'm always doing that, what a shame ;)
18:02:49 <flwang> http://stackalytics.com/report/contribution/zaqar-group/30
18:02:56 <flwang> our review performance is low
18:04:04 <flwang> Total reviews: 122 (4.1 per day) Total reviewers: 17 (0.2 per reviewer per day) Total reviews by core team: 104 (3.5 per day) Core team size: 8 (0.4 per core per day)
18:04:12 <Eva-i> yes... I did less reviews lately
18:05:32 <flwang> if somebody's patch isn't reviewed after a long time, he/she will be very sad, especially when he/she is the PTL of the project ;)
18:05:41 <Eva-i> oh
18:06:10 <flwang> because that means he/she is losing somebody he/she loved :)
18:06:45 <flwang> anyway, i assume you guys got my  point, pls review review review
18:06:57 <flwang> at least 1 per day
18:07:08 <Eva-i> yes
18:07:26 <flwang> and pls keep cruising in irc channel
18:08:51 <flwang> ok, now some critical/important patches i would like to highlight
18:08:58 <flwang> 1. https://review.openstack.org/302479  tempest gate job
18:09:06 <flwang> pls pls pls review it
18:09:33 <flwang> we take a long time to prepare it and i think it's ready now
18:09:43 <flwang> and it's very important for us
18:10:35 <Eva-i> alright, I'll will review it after the meeting
18:11:40 <flwang> Eva-i: cool, thanks
18:11:55 <flwang> and this one PATCH(update) queue https://review.openstack.org/#/c/294368/
18:12:06 <flwang> Eva-i: i need your help on that one
18:12:21 <flwang> since you're one of the original reviewers
18:12:26 <Eva-i> okay, that one too
18:12:45 <Eva-i> it's hard patch
18:13:40 <flwang> Eva-i: yep, i see.
18:14:28 <flwang> and the api ref patch
18:14:31 <flwang> # link https://review.openstack.org/321344
18:15:26 <flwang> above link is the basic architecture, and you can see the style http://docs-draft.openstack.org/44/321344/6/check/gate-zaqar-api-ref/4c19fdc//api-ref/build/html/versions/index.html
18:16:12 <flwang> after we complete all the work, we will add a publish job so that our api ref can be published on docs.openstack.org
18:17:13 <Eva-i> the style... I can see only "API versions" section on that page
18:18:11 <Eva-i> for the full api-ref content I should go somewhere else?
18:19:09 <flwang> http://docs-draft.openstack.org/44/321344/6/check/gate-zaqar-api-ref/4c19fdc//api-ref/build/html/
18:19:16 <flwang> but i haven't done all the content
18:19:43 <flwang> i need some help to add contents
18:19:46 <Eva-i> When I click "Messaging Service API v2 (CURRENT)" link, I see no useful content
18:20:06 <Eva-i> just a blank section
18:20:21 <flwang> Eva-i: yep, since i haven't done anything at that part
18:20:25 <Eva-i> Aha, okay
18:20:28 <flwang> versions is just a sample
18:20:47 <flwang> flwang is working on queues,  kevin_zheng is working on messages and claims, wxy is working on subscriptions, wanghao is working on pools and flavors
18:21:35 <flwang> i hope we can merge the basic frame if we're happy with that
18:22:18 <Eva-i> So this init commit is ready for reviews? Alright, let's add content later.
18:23:02 <flwang> Eva-i: yep, it's ready to go
18:24:26 <flwang> ok, those are the patches i want to highlight
18:24:39 <flwang> Eva-i: vkmc: flaper87: anything else?
18:24:51 <Eva-i> not from me
18:25:08 <vkmc> not from me either
18:25:27 <flwang> ok, cool
18:25:37 <flwang> #topic specs/features
18:26:08 <flwang> do you guys have a chance review the 'subscription confirmation' spec?
18:26:27 <flwang> #link https://review.openstack.org/284555
18:27:01 <Eva-i> probably, but no promise
18:27:28 <flwang> i mean have you reviewed it?
18:27:50 <Eva-i> I revieved it few times in the start
18:28:08 <Eva-i> I don't know how it looks now
18:28:29 <Eva-i> need to check
18:28:42 <flwang> Eva-i: yep, i see. for me, it's almost ready and i hope we can get it in asap, and we can polish the details when doing the code reivew
18:29:34 <Eva-i> okay
18:30:00 <flwang> Eva-i: pls focus on the api part
18:30:35 <Eva-i> flwang: do you like how api part is described in the spec?
18:30:42 <flwang> based on current design, only one new api endpoint will be added,   which looks like this PUT /v2/queues/{queue_name}/subscriptions/{subscriptions_id}/confirm?signature=xxx
18:30:58 <flwang> Eva-i: no, i have commented that
18:31:27 <flwang> i would like to see a clear api desgin, which includes    request, api url and response
18:31:35 <flwang> and the possible HTTP code
18:34:09 <Eva-i> flwang: I would like too
18:37:34 <Eva-i> flwang: except http code. Http code will be described in patch.
18:37:50 <Eva-i> or we end polishing http code.
18:38:48 <flwang> Eva-i: that's fine
18:39:33 <flwang> anything else before you give it another shot?
18:40:17 <flwang> ok, next one
18:40:21 <Eva-i> ok...
18:40:22 <flwang> #link  Mistral notification   https://review.openstack.org/318202
18:41:11 <flwang> i really like this spec, since it's opening a windows to trigger the other services of openstack
18:42:26 <flwang> that said, we can implement a Senlin subscriber support after the Mistral one
18:45:56 <flwang> comments?
18:46:34 <flwang> i had a lot offline chat with Zane Bitter as well
18:47:09 <flwang> i think that spec is very clear and ready to go
18:47:44 <Eva-i> I will read it
18:48:22 <Eva-i> I guess today will be review day for me
18:49:09 <flwang> everyday is a review day :D
18:49:29 <flwang> in other words, review is the daily work for a core reviewer :)
18:49:38 <Eva-i> yeah
18:50:05 <flwang> ok, let's move on
18:50:06 <Eva-i> I just wanted to say, I'll not complete install guide today
18:50:19 <flwang> Eva-i: haha, no worries
18:50:19 <Eva-i> And spend my time for reviews
18:50:55 <flwang> would you mind talking about the install guide status? given the next item 'deprecate v1.1' is still in planning, i mean no spec for review
18:51:34 <flwang> Eva-i: from the patch, i saw you already figured out how to do the install guide, right?
18:51:45 <Eva-i> Install guide is working, but it's not in a format defined by a doc team. We will host it in our repo.
18:51:57 <Eva-i> flwang: yes
18:52:14 <Eva-i> *the format
18:55:38 <Eva-i> I'll just modify it to comply with doc team specs and it will be ready to go.
18:59:23 <flwang> ok
18:59:40 <flwang> we can disscuss in zaqar channel
18:59:47 <flwang> thank you for joining the meeting guys
18:59:55 <flwang> Eva-i: thank you!
19:00:02 <flwang> #endmeeting