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