00:01:03 <cyeoh> #startmeeting nova api 00:01:04 <openstack> Meeting started Fri Feb 13 00:01:03 2015 UTC and is due to finish in 60 minutes. The chair is cyeoh. Information about MeetBot at http://wiki.debian.org/MeetBot. 00:01:05 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 00:01:07 <openstack> The meeting name has been set to 'nova_api' 00:01:21 <cyeoh> Hi, anyone around this week? 00:01:31 <alex_xu> o/ 00:01:42 <gmann> hi 00:02:00 <oomichi> hi 00:02:12 <alex_xu> long time no meeting together ! 00:02:20 <cyeoh> heh yea! 00:02:37 <cyeoh> ok ok, lets get started 00:02:40 <oomichi> alex_xu: yeah, but we can talk anytime on irc:) 00:02:55 <cyeoh> #topic release criteria for microversions and v2.1 00:03:00 <alex_xu> oomichi: yeah, but meeting == party 00:03:10 <oomichi> alex_xu: +1 00:03:11 <cyeoh> #link https://etherpad.openstack.org/p/v2_1_ReleaseCriteria 00:04:04 <cyeoh> now I think we have the patch merged that we needed for tempest run with neutron to run successfully? 00:04:23 <oomichi> cyeoh: yeah, successfully. 00:04:40 <oomichi> I tested WIP patch for doing it. 00:04:49 <oomichi> https://review.openstack.org/#/c/139903/ 00:04:55 <cyeoh> cool :-) So we just need to merge 151564 then 00:05:18 <oomichi> yeah, the time is comming :) 00:06:07 <gmann> yes, i will update that soon 00:06:14 <oomichi> gmann: thanks ! 00:06:25 <cyeoh> I think the baremetal nodes patches are in good shape and should merge pretty soon 149125 and 154714 00:06:55 <oomichi> cyeoh: yeah, I agree. 00:07:01 <cyeoh> I was *very* surprised to discover there were not api sample tests for os-baremental-nodes though 00:07:28 <oomichi> because the API is not so famous. 00:07:43 <cyeoh> yea, and it just slipped through review. 00:08:09 <cyeoh> Park has done the manual check of v2 versus v2.1 code for anything merged to v2 since October 00:08:42 <cyeoh> so I think we're fine there. There might be one or two things we've missed but between tempest tests and manual checks I think its sufficent 00:08:49 <alex_xu> +1 for Park's work 00:09:02 <oomichi> cyeoh: yeah, that is good work 00:09:12 <cyeoh> ... can only do so much and we'll pick up anything else as a bug report 00:09:26 <gmann> yea, that is good thing 00:09:38 <cyeoh> so I'm not sure when to merge 149948 00:10:05 <oomichi> cyeoh: need to merge before kilo-3 00:10:10 <cyeoh> we don't have doc yet for writing a v3 plugin (but I hope to be able to start that today) 00:10:11 <oomichi> that is my hope :) 00:10:36 <oomichi> the doc would be a good input. 00:10:39 <cyeoh> we can definitely merge v2.1 before microversions 00:11:13 <oomichi> cyeoh: +1 00:11:26 <cyeoh> so how about we concentrate on getting the patches merged mentioned above 00:11:47 <cyeoh> and aim to merge 153935 Monday/Tue next week? 00:12:16 <cyeoh> the only thing I'm not sure about is some doc around nova client 00:12:17 <oomichi> cyeoh: I agree 00:12:46 <cyeoh> so by default v2.1 will be export on /v21 00:12:47 <alex_xu> cyeoh: it's already merged? 00:12:49 <gmann> 153935 is merged 00:13:18 <cyeoh> oh sorry I mean 149948 00:13:26 <cyeoh> the EXPERIMENTAL->CURRENT setting 00:14:19 <oomichi> cyeoh: +1 00:14:44 <cyeoh> so my one question is how we recommend trying out v2.1? 00:15:05 <cyeoh> do we just say use /v2.1 00:15:14 <cyeoh> or try to describe what devstack does? 00:15:27 <cyeoh> or both? Either way I think operators will need some guidance 00:15:36 <oomichi> or $ nova --service-type computev21 list 00:15:39 <oomichi> on devstack 00:16:39 <alex_xu> cyeoh: may provide a way let devstack configure v2.1 as default 00:17:06 <oomichi> alex_xu: oh, that is a good idea :) 00:17:15 <alex_xu> cyeoh: some configuration can make the v2.1 export as /v2 00:17:29 <alex_xu> oomichi: thanks 00:17:49 <oomichi> alex_xu: devstack has already contained the option for switching v2.1 as /v2 00:17:54 <cyeoh> yea I guess we should just describe a few different ways so there is something that will work depending on the operator environment setup 00:18:11 <oomichi> we can just change True as the default value on devstack. 00:18:11 <alex_xu> oomichi: already! cool 00:18:21 <cyeoh> I'll put in on the release critera list 00:18:40 <oomichi> alex_xu: yeah, the option was necessary for running v2.1 job on the gate. 00:18:58 <alex_xu> oomichi: oops, yea, right, I missed that 00:19:34 <cyeoh> oomichi: do you have a link to where in devstack you need to set a value to True? 00:19:37 <gmann> then we might not need V21 jobs :) 00:20:33 <oomichi> cyeoh: yeah, I will write the link on the etherpad. 00:20:42 <cyeoh> oomichi, thanks! 00:22:11 <cyeoh> I think thats it for v2.1, for microversions I see there is the proposed header name change 00:22:40 <cyeoh> I dont think I really care, but if everyone else is going for project name I'm ok with that 00:22:59 <cyeoh> #link https://review.openstack.org/#/c/153183/1/specs/kilo/approved/api-microversions.rst 00:23:00 <oomichi> cyeoh: thanks :) 00:23:11 <cyeoh> does anyone have any objections to it? 00:24:44 <alex_xu> i'm ok 00:25:06 <cyeoh> ok I guess we need to look around for another specs +2 00:25:09 <gmann> fine for me. 00:26:12 <cyeoh> I guess there will be a bit of test churn due to it 00:27:19 <oomichi> cyeoh: test churn? 00:27:39 <cyeoh> updating unittests to send the right header? 00:28:08 <cyeoh> I think it might be hardcoded in a few places 00:28:11 <cyeoh> (not sure) 00:28:22 <oomichi> cyeoh: yes, we need to update them at many places. 00:28:47 <cyeoh> #topic microversions 00:29:26 <cyeoh> so this is the api change we've been lining up as the first microversion: https://review.openstack.org/#/c/140313/ 00:30:03 <cyeoh> I'd appreciate any reviews on it and the patch its dependent on (though there will be another update in a couple of hours) 00:30:11 * alex_xu network is pretty slow... 00:30:58 <cyeoh> the only other microversion release criteria change I'm aware of is novaclient support 00:31:02 <oomichi> cyeoh: the patch needs FFE and I will support it, will you? 00:31:14 <alex_xu> cyeoh: I can't open it, I guess it's keypair one? is it get ffe? 00:31:20 <cyeoh> 140313 needs a ffe? 00:31:48 <cyeoh> alex_xu: yes 00:31:52 <oomichi> cyeoh: yeah, the author sent a FFE request mail yesterday. 00:32:14 <cyeoh> oomichi: yes I'll support it 00:32:29 <oomichi> cyeoh: nice, so we can push it. 00:32:53 <cyeoh> yep and we need some change in order to get the first microversion ;-) 00:33:04 <alex_xu> cyeoh: +1 00:34:27 <cyeoh> did anyone have anything else on v2.1 or microversions? 00:34:40 <alex_xu> nothing from me 00:34:52 <oomichi> that is all from me :) 00:35:12 <cyeoh> #topic api policy improvement 00:35:20 <gmann> cyeoh: regarding the sample files structure for microversion i n 140313 00:35:33 <cyeoh> gmann: sure, go ahead 00:36:13 <gmann> cyeoh: should we keep that in parallel to V2 sample file as we are thinking to merge V3 nd V2 sample file 00:36:45 <gmann> in 140313 i saw it is inside V3/ 00:37:22 <oomichi> gmann: that is a nice point. 00:37:45 <cyeoh> gmann: for the moment I think its ok, we can do the reorg post things going live 00:37:49 <oomichi> gmann: that seems a little strange for me. 00:38:17 <cyeoh> so it does look odd, mainly because v2 vs v3 is more about what code it comes from rather than any relationship to a version 00:38:48 <cyeoh> but we have to do a giant movearama to get rid of the v3 directory at some point. anyway 00:39:24 <cyeoh> I think for now having the samples as just under v3 and the microversions in subdirs under that will work ok 00:40:27 <gmann> cyeoh: yea. we need to define a structure for v2.1 nd microversion stuff (code as well as tests) 00:41:18 <cyeoh> gmann: yep we need to work out a good file structure and I definitely want to eliminate the 1000 identical server-post.req files etc 00:41:35 <cyeoh> at the same time I want microversions out ASAP 00:41:49 <oomichi> so as the first step, how about removing v3 subdir under plugins today? 00:41:58 <gmann> cyeoh: yea that is most painful thing about sample file. 00:41:59 <cyeoh> I'd rather live with a messy api samples hierary for now 00:42:28 <gmann> oomichi: +1 00:42:29 <oomichi> the patch will be big, but it will be easy to be reviewed. 00:42:55 <cyeoh> oomichi, I'm ok with that, but can we delay until after microversions is merged? I don't want to run into a bunch of merge conflicts 00:43:24 <gmann> oomichi: and after that we can define dir structure for tests and sample files too. 00:43:36 <oomichi> cyeoh: yeah, conflicts will happen. but we need them anyway. 00:44:05 <cyeoh> oomichi: yea I just don't want them delaying microversions getting merged... 00:44:30 <oomichi> cyeoh: ah, I see. OK, will defer it. 00:44:32 <cyeoh> after that I'm happy to use up the rest of kilo for cleanup/bug fixing 00:44:54 <cyeoh> I believe jaypipes had a bunch of speedup patches to 00:45:39 <gmann> cyeoh: sounds good. 00:46:50 <cyeoh> alex_xu: did you want to say anything about the api policy group of patches? 00:47:11 <alex_xu> cyeoh: yes 00:47:21 <alex_xu> The nova api policy improvement's current status is all the policy patches already on the gerrit. 00:47:29 <alex_xu> #link https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/v3-api-policy,n,z 00:47:53 <oomichi> alex_xu: good work and a lot of patches! 00:47:53 <alex_xu> it's includes v2.1, db layer cleanup and ec2. totally about 60 paches. 00:48:13 <cyeoh> ok so we have lots of patches to review :-) 00:48:23 <alex_xu> but thinking about ec2 may be removed after few release, I think we can drop the ec2 patches 00:48:53 <alex_xu> Then we left 48 patches. 30 for v2.1 and 12 for db 00:49:00 <cyeoh> alex_xu: yea its not clear what is happening with ec2 :-( 00:49:01 <oomichi> alex_xu: yeah, that is a nice point. let wait for removing ec2 api :) 00:49:24 <alex_xu> cyeoh: oomichi yea, let's focus on v2.1 and db 00:49:35 <oomichi> alex_xu: +1 00:49:38 <alex_xu> v2.1 is on good status, and the review will be very simple 00:49:40 <cyeoh> alex_xu: agreed 00:50:07 <alex_xu> but db layer need to review carefully 00:50:32 <cyeoh> yep, definitely 00:51:11 <cyeoh> alex_xu: anything else on api policy? 00:51:52 <alex_xu> cyeoh: no more, just want to thanks all the people can review them! 00:52:23 <cyeoh> cool - I will do some... 00:52:29 <cyeoh> #topic urgent reviews 00:53:14 <cyeoh> so I think the v2.1/microversions related patches are the highest priority at the moment. Anyone else have urgent patches? 00:54:10 <oomichi> nothing special from me 00:54:38 <cyeoh> #topic open discussion 00:55:08 <cyeoh> Anyone have anything they want to talk about? 00:55:50 <gmann> nothing from my side. 00:56:29 <oomichi> yeah, I also. we will need to review a lot of patches :-) 00:56:32 <cyeoh> ok lets close a bit early today 00:56:34 <alex_xu> Just mention close to China Spring Festival. hope we can merge some patch before vacation. Anyway I will monitor on the patches status on the vacation. And help to update if need. 00:56:36 <cyeoh> thanks all for coming! 00:56:55 <cyeoh> alex_xu: ah yea, thanks for the reminder! 00:57:13 <oomichi> thanks all! 00:57:16 <gmann> Thanks all 00:57:17 <cyeoh> alex_xu: I guess you'll be out for a couple of weeks after this friday? 00:57:20 <alex_xu> thanks all! 00:57:53 <gmann> alex_xu: enjoy the Fest :) 00:57:58 <alex_xu> cyeoh: the vacation begin at next Wed 00:58:21 <alex_xu> cyeoh: but I want to monitor on policy patches, too much patch need to be merged 00:58:25 <alex_xu> gmann: thanks! 00:58:27 <cyeoh> alex_xu: ah ok, hope you have lots of fun and enjoy the break :-) 00:58:38 <alex_xu> cyeoh: thanks! 00:58:42 <cyeoh> #endmeeting