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