15:00:06 #startmeeting defcore 15:00:07 Meeting started Wed Oct 7 15:00:06 2015 UTC and is due to finish in 60 minutes. The chair is eglute. Information about MeetBot at http://wiki.debian.org/MeetBot. 15:00:08 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 15:00:11 The meeting name has been set to 'defcore' 15:00:15 #chair hogepodge 15:00:19 Current chairs: eglute hogepodge 15:00:22 o/ 15:00:25 #topic roll call 15:00:25 o/ 15:00:33 morning everyone! 15:00:42 Good morning! 15:00:55 raise your hand if you are here for defcore meeting 15:01:04 #topic agenda 15:01:05 Good morning (more morningish than usual for me since I'm on the west coast today) 15:01:23 please review today's agenda: #link https://etherpad.openstack.org/p/DefCoreFlag.18 15:01:36 Good morning .o/ 15:01:42 sorry markvoelker, then this is really early for you, just like for hogepodge! 15:02:00 o/ 15:02:03 eglute: nah, not really....I'm an early riser, particularly when my internal clock is still on EDT. =) 15:02:41 markvoelker you are lucky, i adjust to PST super fast, then hard to switch back 15:03:03 anyways, last time we ran out of time, so will start with items we didnt get to 15:03:17 o/ 15:03:18 #topic tests that no longer exist in tempest 15:03:23 #link https://review.openstack.org/#/c/229177/ 15:03:44 o/ 15:03:50 catherineD brought up that one test was renamed 15:04:00 and we currently do not have a good way to handle it 15:04:09 So on this one I think tactically we just need to adjust our documentation, strategically we need to think about how to handle renames. 15:04:30 a script to generate the required list is the way to go. 15:04:57 hogepodge: one that runs against whatever version of tempest you have, you mean? 15:04:59 If possible on the refstack side checking should be done against just idempotent id 15:05:04 markvoelker: yes 15:05:28 If a test is showing as failed because of a rename in refstack, we would account for that on the foundation side. 15:05:33 For that I am evaluating the SHA that Mark suggesting .. at RefStack 15:06:08 hogepodge: so idempotent ID's aren't unique to tests though, so would have to be some conglomeration of name + id 15:06:15 idempotent ids only collide when the tests have the same functionality, so there's no loss of generality 15:06:46 for each guideline we also have a "procedure" doc. We could specify version there as well. https://github.com/openstack/defcore/blob/master/2015.05/procedure.rst 15:06:53 checking on ID is not possible ... especially with Neutron tests .. 15:07:02 * markvoelker notes that we still don't have one of those for the 2015.07 guideline 15:07:22 markvoelker: there's a patch up for that 15:07:32 yup, there is a patch 15:07:55 #link https://review.openstack.org/#/c/227645/ < patch to add 2015.07 docs 15:08:22 if everyone is ok with it, i will merge it after the meeting 15:08:38 What would be better is a script. I was working on one in Vancouver and never finished it. 15:08:52 That patch doesn't have guidance about the SHA though. Should we just land it and add SHA guidance in follow-up once Catherine verifies it? Or someone writes the script Chris is suggesting? 15:08:54 hogepodge what kind of script do you have in mind 15:09:49 eglute: something that runs against tempest to produce the required and flagged test lists 15:10:09 hogepodge i think that would be great 15:10:10 so it's up to date against both the tempest tree and the defcore standard 15:11:01 Who's volunteering to write it? =) 15:11:02 I am not sure how that would work ... in the list of tests do we include the new or old name? 15:11:05 so everyone is ok with having a script as well as listing version in the procedure doc? 15:11:09 me 15:11:29 #action hogepodge finish work on required/flagged test script 15:11:36 #action hogepodge will write a very nice script, something that runs against tempest to produce the required and flagged test lists 15:11:39 hah 15:11:52 ok, how about update procedure doc with version? 15:11:56 eglute: your action item sounds much nicer. :-D 15:12:00 :D 15:12:01 that too 15:12:16 anyone is interested in taking that action? 15:12:21 Like catherine I'm a little curious how this will work, but I'm content to look at the code once proposed to get the big picture. 15:12:23 eglute: I can do that too 15:12:44 #action hogepodge will update procedure doc to include test versions 15:12:54 Also a little curious how to handle this given that refstack.net will need to show the results properly and that the test lists in the Guideline doc are immutable.... 15:13:06 But it seems like a problem that could be solved 15:13:25 markvoelker i think you have valid concerns, lets see what hogepodge comes up with! 15:13:29 next item? 15:13:30 Let's give it a trial ... 15:13:32 Another proposal to address Catherine's concern is to add an "aka": entry to the guideline 15:13:42 to capture name changes 15:13:55 that might be good as well 15:13:56 Sure. One more thing: since there are a couple of issues to resolve there it seems like it may take some time for all that to get in place 15:14:04 markvoelker: catherineD: something like that would resolve the problem, no? 15:14:16 In the interim I'd suggest Catherine continue testing the SHA to make sure it works, and we update docs to point to it 15:14:23 +1 15:14:37 hogepodge: sounds like a good proposal for schema 1.4 15:14:39 also, updating procedure doc will help, until we have other things in place 15:14:49 talking of schema! next item 15:14:55 markvoelker: +1 for testing SHA .. 15:15:08 #topic cutoff score in schema 15:15:21 markvoelker can you give a bried overview 15:15:24 brief 15:15:32 hogepodge: aka addition may work .. 15:15:33 #link https://review.openstack.org/#/c/224868/ Add field to record cutoff score 15:15:50 Basically this adds a field in the schema to record the cutoff score we used when deciding what to make required in a Guideline 15:16:01 (note: only the cutoff for required, not for advisory) 15:16:43 The general idea is to help with transparency and to help us score fairly 15:16:55 +1 15:17:13 no objections 15:18:12 so my only concern is that the cutoff score has been more as a recommendation, not a requirement. ie, if score is high enough, does not mean capability would make it as required. thoughts? 15:19:16 We are required to select a cutoff score (http://git.openstack.org/cgit/openstack/defcore/tree/doc/source/process/2015A.rst#n109) so seems like making that visible is good for transparency 15:19:32 o/ 15:20:12 markvoelker good point. 15:20:58 #action everyone review #https://review.openstack.org/#/c/224868/ by end of week and comment 15:21:25 #topic tokyo working sessions 15:21:49 we have 2 working sessions, we started preliminary agenda #link https://etherpad.openstack.org/p/DefCoreFlagTokyo 15:22:03 please review agenda and add/update 15:22:35 if we have too many items on the agenda, we can hold a separate brief meeting to review the agenda and prioritize 15:22:55 eglute: where should we add/update? Is there an etherpad or something? 15:23:06 oh, duh...first link 15:23:09 heh 15:23:11 looked right past it. =) 15:23:32 sorry dont mean to rush through things, but still a lot to cover today 15:23:34 #link https://etherpad.openstack.org/p/DefCoreFlagTokyo Tokyo summit working session agenda 15:23:47 No worries, let's move on 15:24:22 also, if you know of other tokyo defcore/refstack sessions, update today's etherpad or add it to the agenda for tokyo 15:24:40 #topic review spec for refstack registration 15:25:04 #link https://review.openstack.org/#/c/226902/ 15:25:08 there's one proposed for cross project but won't be scheduled for a bit yet 15:25:24 rockyg cool, update when you know the time 15:25:41 markvoelker can you give a brief summary of the proposed spec? 15:25:49 So a while back Catherine asked us to write a draft proposal for how to handle vendor registration/affiliation in refstack. 15:26:05 She was extremely patient while I was busy with other things, and eventually this came out of my keyboard. =) 15:26:36 There's some good discussion in the comments about how to deal with user affiliation data/verification, and whether vendor *products* need to be "first class citizens" 15:26:47 And also some questions that probably need answers from the Foundation 15:27:04 Thank you for the spec ... I feel like the vendor registration topic is a good one for face-2-face at the summit ... 15:27:41 catherine_: +1. Chris, if you haven't already it would be good to peruse some of the comments there if you can speak to the Foundation's capabilities/feelings about a couple of things 15:27:58 +1 for the summit discussion 15:28:06 (or point us to someone who can) 15:28:53 ok 15:28:58 ok, so we will move this topic to F2F 15:29:15 I will add the topic to the summit etherpad .. 15:29:23 thanks catherine_ 15:29:38 now we can go back to scoring, unless there is something you want to move ahead on the agenda 15:29:52 #topic scoring 15:30:23 cinder did not receive a ton of comments, #link https://review.openstack.org/#/c/221631/ 15:30:25 eglute: thanks for allowing time to discuss the RefStack related topics ... :-) 15:30:38 catherine_ this seems very related! 15:31:15 for cinder, i think there was one comment about extra characters left in, 15:31:21 is it ready otherwise/ 15:31:23 ? 15:31:39 eglute: I just updated that patch before the meeting, so good to go 15:31:57 eglute: it also needs the achievements added to teh json and a rebase 15:31:59 cinder? i dont see an update 15:32:08 #link https://review.openstack.org/#/c/221631/ 15:32:46 ithink it happend 9/28 15:33:02 I see a few other issues in the json too....commenting now 15:33:18 But I think we've had sufficient soak time to generally agree on what should go into it. 15:33:30 oh yeah, nevermind, rebase failed. 15:33:45 ok, then will wait for the cleanup, and then merge at the end of the week, or as soon as all the issues are fixed 15:34:29 The hour after the meeting is devoted to cleaning up the cinder, glance, and keystone patches for final review. 15:34:36 :) 15:34:54 talking of glance! 15:34:58 #link https://review.openstack.org/#/c/213353/ 15:35:28 rockyg i know you raised some concerns that markvoelker responded. 15:35:54 Question: so if the vendors have underlying code that allow v2 api to work, does that make it acceptable for inclusion? 15:36:25 the discussion on the mailing list has both cinder and nova saying their code base doesn't work with v2 15:36:25 rockyg: not sure what you mean.....v2 does work with upstream code 15:36:49 YEs, they use the v1 API. That doesn't mean v1 has to be exposed to end users or that other projects have to use v2 though. 15:37:05 Nope. v2 tests work, but the capability has not been added to work with either project. Neither uses v2 api 15:37:40 rockyg: I'm not seeing why that would prevent v2 from being required to be exposed to end users. 15:37:40 which pretty much mean that the only people who *do* have it working are outside of the dev community 15:37:51 rockyg: huh? 15:38:01 the nova client and the cinder client use v1 15:38:11 because v2 breaks 15:38:28 no v2 is called from either code base at the moment 15:38:40 it's an api sole restricted to glance itself 15:39:24 So, I'm still not seeing why it matters that nova and cinder talk to glance over v1. Maybe that impacts the clients score, which has been discussed already in the review. 15:39:29 That's what dhellmann's email thread was about. 15:39:33 But v2 works for end users and has for some time 15:39:40 And in fact most products out there support it 15:39:44 And don't need special code to do so 15:39:50 Also noted in the review 15:40:04 DefCore specifies what end users can rely on being present 15:40:10 well, if our own projects are not using it yet, how much in use is it really? 15:40:16 Quite a lot 15:40:40 which means the code supporting it is all vendors, not dev community. 15:40:50 Monty's notes for example: on 11 public clouds it is the only way for you to upload an image 15:41:04 That's why I asked if it's ok to use out of tree code, vendor code, to make it work 15:41:17 it's not vendor code making that work 15:41:24 rockyg: There is no out-of-tree code involved in v2 working 15:41:28 According to the dev thread, it is 15:41:34 it's configuration to expose v2, which is enabled by default 15:42:00 I don't think v2 is enabled by default. 15:42:04 rockyg: I'm not reading that from the dev thread at all. 15:42:09 If it is, it just happended in the gate 15:42:24 . I'm getting that from a glance core and others 15:42:30 rockyg: if someone were saying they had nova and cinder talking to glance via v2, that would be out of tree code 15:42:59 create image needs to talk to nova. 15:42:59 But the concern here is what is exposed to end users, not to nova/cinder. If you want to enable the v2 API for tenants, that works. Out of the box. Has for some time. 15:43:16 No it doesn't. Other way around. 15:43:28 If you want to use the nova create image API, that talks to glance (via v1) 15:43:43 ok, so then nova calls image-create? It is using v1 15:43:49 If you want to upload an image to glance directly (e.g. not create one from a running instance) you can do that with v2 directly to cinder 15:43:51 err, glance 15:44:02 our goal should be to eliminate nova create image api, and other proxy apis. Way too much pain there 15:44:12 but to create an image, noav and glance have to talk. the only way for that to happen is v1 15:44:35 None of these capabilities are concerned with creating an image from an existing instance. 15:45:29 We should also foster interoperability through stable apis that will have future support. In 9 months, the earliest that these can be required, glance v1 will be entirely done. Cinder v1 also. This is in part because of the problems we as a committee have exposed. 15:46:05 rockyg: If you want to create an image, nova and glance basically don't have to talk. If you want to create an image *from and existing instance* then they do. 15:46:30 hogepodge: is glance actually going to deprecate v1? I haven't actually heard that.... 15:46:37 I'll remove my -1. for advisory, but don't be surprised if it stays advisory for more than one release. There are few to any client tests and the code doesn't exist in the other projects which mean the tempest tests don't exercise the broken areas. 15:46:44 making glance v2 and cinder v2 advisory (ane keystonve v3) starting in January only helps our interoperability efforts by aligning us with the community and tc, sending a clear message as to our intentions 15:46:52 But, it may get the devs to get it working. 15:47:11 i agree with hogepodge. I think as is, is ok for advisory. remember that just because something is advisory, does not mean it will be required next cycle. if it is still an issue in 6 months, we will revisit. 15:47:12 rockyg: which of the tests that are listed in the json are not used by devs? 15:47:15 Just don't expect a 6 month turnaround. 15:47:56 I've seen some pretty amazing 6 month turnarounds in the last year. More than one. 15:48:04 It's not that the tests aren't used, it's that they don't exercise what's not there 15:48:36 for the sake of time, lets take this discussion to defcore irc channel 15:48:51 i would like to go over identity again 15:48:54 #link https://review.openstack.org/#/c/213330/ 15:49:17 this also a bit controversial issue, 15:49:24 and has a lot of - 15:49:26 -1 15:49:27 I really think the api discussion needs to happen f2f, which is why I'm taking my -1 away and letting the advisory go through 15:49:46 thanks rockyg 15:50:00 can you update the agenda for tokyo? 15:50:21 for identity patch, same thing, versions. 15:50:45 to move forward, i suggest updating the patch to drop v2 15:50:52 Yeah, I noticed the incoming PTL reversed his vote on this one. I'm ok with removing v2, though I would really, really like to see the project deprecate it if they're saying nobody should use it anymore. Sounds like Steve is going to try to make that happen, which is good. 15:51:01 I think we need a *serious* discussion about versions with dev at Tokyo. 15:51:33 i agree with you markvoelker they should work on deprecating it 15:52:00 I think that's what most of the issues are over and the dev community is just realizing the implications with the gate and codebase of the various projects going to new api/s 15:52:34 i am sure we will have more versions discussions in the future, so we will need to find a good way to work with these issues... or with the devs, or both 15:53:07 i am very happy to see PTLs commenting on these patches, hopefully this trend continues 15:53:15 we need to think about existing v2/v3 tests for tokens too 15:53:35 I can deprecate them in next 15:53:37 hogepodge for this guideline or the next 15:53:43 added to agenda as general versions handling discussion 15:53:53 thanks rockyg 15:53:53 hogepodge: If we're not going to include v2 even as advisory, then we should deprecate the existing v2 stuff in this patch too. 15:53:59 It's all of one test I think 15:54:06 if a vendor is running into a v2/v3 testing issue I'd recommend they propose a flag 15:54:17 markvoelker: +1 15:54:36 markvoelker: +1 on revmoving v2 token test from requirement ... 15:55:10 On that note, maybe this is a good time to mention an item that's actually later on the agenda (it's quick, promise) 15:55:25 Is anyone tuning into API WG meetings at all? 15:55:40 no 15:55:46 The TC has some guidance about feature deprecation that's useful for API transitions, and the API WG I think has been planning to come up with a plan 15:55:52 just following the patches 15:55:59 I think our experience here would be useful in that conversation 15:56:02 #topic apis 15:56:16 If nobody else is, I'm happy to liaise with them... 15:56:25 thank you markvoelker that would be great 15:56:49 #action markvoelker will act as a liaison between defcore and api wg 15:56:57 Ok, sounds like a plan. See, told you that was a quick one. =) 15:56:58 please. I'm mostly focused on where it intersects logging 15:57:15 ok, so back to identity... ok to drop v2 and move forward? 15:57:34 yah. 15:57:55 cool! i think hogepodge already volunteered to make that change 15:57:59 I'm fine with that. I'd also like to be present when keystone discusses deprecating v2. =) 15:58:04 2 min left, any last words? 15:58:09 +2 15:58:19 Two more meeting before Tokyo. 15:58:28 s/meeting/meetings/ 15:58:35 i will not be able to attend either, 15:58:45 I'll be here for both. 15:58:48 because of the conference/other meetings 15:58:49 we might consider a defcore/crossproject dinner for a longer discussion on versions 15:58:50 thanks hogepodge 15:59:14 if we can pull it off, we might get better info for moving forward 15:59:29 rockyg for moving forward on apis? 15:59:33 or breakfast 16:00:07 the versioning stuff. Get a senior dev from each project with versions to talk about process to make it work for all of us 16:00:17 rockyg lets plan on that. but not for breakfast. 16:00:25 great discussion today, thanks everyone! 16:00:26 agreed ;-) 16:00:28 #endmeeting