14:00:23 #startmeeting Glance Drivers 14:00:24 Meeting started Tue Dec 15 14:00:23 2015 UTC and is due to finish in 60 minutes. The chair is flaper87. Information about MeetBot at http://wiki.debian.org/MeetBot. 14:00:26 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 14:00:29 The meeting name has been set to 'glance_drivers' 14:00:34 Hello folks! 14:00:37 who's around? 14:00:44 o/ 14:01:33 o/ 14:02:30 o/ 14:02:41 ativelkov: hey there :D 14:02:42 let's give 2 more mins for ppl to join, otherwise we'll skip the meeting this week 14:02:59 sigmavirus24: mfedosin hey :) 14:03:24 o/ 14:03:33 as of now, I'm the only one from the drivers team here :/ 14:03:37 wow, rosmaita HEY ! 14:03:39 :D 14:03:41 now we have 2 14:03:42 hey 14:03:43 :D 14:03:44 o/ 14:03:45 w000h0000 14:03:48 and three 14:03:52 yeay, let's start 14:04:01 2 is HA. Well, almost :) 14:04:03 #topic changes in nova around glance configuration 14:04:08 no idea who added that 14:04:14 me 14:04:14 but I guess it was a heads up? 14:04:19 yes, exactly 14:04:21 #link https://review.openstack.org/#/q/topic:glance_image_config,n,z 14:04:27 rosmaita: coolio 14:04:41 just want to make sure it doesn't affect v1->v2 work 14:04:49 yeah, I just talked with johnthetubaguy to schedule reviews for that and try to avoid conflicts w/ mfedosin's work 14:04:55 great 14:05:13 it'd be also good for us to provide some reviews on Sean's work 14:05:19 let's see what can be done 14:05:32 ok, moving on! Thanks rosmaita 14:05:34 #topic Switch VMware Driver to use requests (sabari) 14:05:36 np 14:05:40 sabari: I guess you're sleeping 14:05:47 this looks like a good change to me 14:05:56 is the question just about whether it needs a spec? 14:05:57 but after looking at the patch/bug, I'd recommend to tag it as spec-lite and get the patch merged 14:06:02 rosmaita: yeah 14:06:05 flaper87: +1 14:06:14 nikhil: objections ? 14:06:15 sounds good to me 14:06:21 yay! 14:09:16 it is awful quiet in here ... hope i didn't get netsplit 14:10:35 don't think so 14:10:44 may be flaper87's conn dropped 14:10:55 flaper87 nikhil rosmaita: thanks guys :) 14:11:02 :) 14:11:09 sabari: np 14:11:37 we can start next topic, the "in" operator spec 14:11:51 rosmaita: sure 14:11:56 #link https://review.openstack.org/#/c/246971/ 14:12:04 info: topic change won't work 14:12:09 ops, sorry 14:12:12 back now 14:12:13 #chair rosmaita 14:12:13 Current chairs: flaper87 rosmaita 14:12:15 #topic Add 'in' operartor (flaper87) 14:12:30 I think I'm back 14:12:37 you are 14:12:42 my conn is slow today, too 14:12:49 w00h000 14:12:58 ok, that one needs a couple of more +2's 14:13:05 I'd like to get at least 3 since it impacts the API 14:13:12 quick question 14:13:19 about erno's comment 14:13:34 i sort of agree that we want to push off fancy search stuff to searchlight 14:13:37 but 14:13:47 we don't have any kind of "or" filtering ATM 14:13:58 so i think maybe this is OK 14:14:07 just wondering what others think 14:14:38 rosmaita: you mean, it'd be fine to have `in` and `or` but not more than that ? 14:14:54 I like the 'in' operator 14:15:02 flaper87: well, the "in" is kind of an "or", the way i understand it 14:15:03 it's harmless for the most part 14:15:14 rosmaita: ah sure, yeah! 14:15:23 we'll have some "in" searches in glare as well - so this one looks consistent 14:15:38 officially it's called "in" :) 14:15:42 I think this is fine. I'd be hesitant to add more filters, tbh. But I can see how this can be useful for folks talking to Glance 14:15:46 so while i think this change is good, i don't think we want to encourage this too much 14:15:51 flaper87: +1 14:15:56 ativelkov: that is a good data point 14:15:58 agreed rosmaita 14:16:04 consistency with glare is good 14:16:19 also, Searchlight, to me, is more for UI's and other use-cases. When it comes to service-to-service (undercloud) comms, I think we should have support for a minimum set of filters 14:16:31 flaper87: that makes sense 14:16:40 flaper87: ++ 14:16:52 just wanted to make sure we discussed it 14:16:57 ok, just +2'd that spec 14:17:08 i am good with this spec 14:17:17 rosmaita: yup, makes sense. Also, we should probably reply to Erno, I don't want his comment to be left unanswered 14:17:20 I can do that 14:17:54 ok, next topic, unless there's more to discuss in this one 14:18:10 flaper87: i'll reply to erno, since i share his concern 14:18:15 nothing from me 14:18:16 will summarize the above 14:18:20 rosmaita: awesome, thanks! 14:18:25 #topic Deprecate V3 API (flaper87) 14:18:32 #link https://review.openstack.org/#/c/254163/ 14:19:07 That one seems complete to me. rosmaita had +2'd it already but I asked mfedosin to move the spec under the `approved` dir 14:19:17 it'll need to be +2'd again 14:19:22 looking 14:19:47 and I changed "testing" section 14:19:59 mfedosin: right, thanks for that :) 14:21:05 ok, let's move on if there's nothing more here. We have some lite-specs to review 14:21:27 #topic Lite Specs 14:21:37 #link https://bugs.launchpad.net/python-glanceclient/+bug/1487086 14:21:37 Launchpad bug 1487086 in python-glanceclient "Pagination doesn't work with v2 API image-list" [Wishlist,New] - Assigned to Kairat Kushaev (kkushaev) 14:21:46 That one proposes adding pagination to the CLI 14:22:11 I think it's useful but, now that the OSC cross-spec is landing, I'm wondering whether we should have it in glanceclient 14:22:14 thoughts ? 14:22:20 hmm, is the title a bit misleading? 14:22:43 we currently have sorting available 14:22:52 so pagination doesn't seem to hurt 14:22:57 If I do a vanilla v2 image list I get pages of 25 back right? 14:23:14 mclaren: it didn't start as a spec-lite. WE can fix the title later, lets focus on the content 14:23:48 plus, the OSC will take time 14:23:51 though I don't get the problem behind it 14:24:05 I agree it doesn't hurt and it shouldn't require lot of work 14:24:26 ok, is this the problem statement: v2 client listing doesn't have support for markers? 14:24:28 hmm, looks a small change 14:24:56 mostly to proxy on glance-api pagination and nothing new to be added within the client itself so should be relatively small 14:25:11 nikhil: right 14:25:34 mclaren: rosmaita any objections to having this in glanceclient ? 14:26:13 we've like 4 mins left 14:26:19 3 14:26:24 was looking at the bug 14:26:33 flaper87: this is just allowing the user to specify a marker when listing? 14:26:49 I updated the title based on that 14:26:53 if so seems fine to me 14:26:59 mclaren: yes, and limit I believe 14:27:02 umm 14:27:04 yeah 14:27:08 I think we need both 14:27:14 mclaren: that's my understanding 14:27:16 nikhil: +1 14:27:17 yup, ok! 14:27:26 I'll approve that lite spec 14:27:33 that's it 14:27:36 yeah it's just a small client change 14:27:37 #topic Open Discussion 14:27:48 #info We're 2 weeks away from Glance's spec freeze 14:28:07 please, lets focus on specs, I'm keeping a list of spec prios in the meeting ehterpad 14:28:11 etherpad* 14:28:15 guess i need to update import again 14:28:28 We need to finalize the import spec too 14:28:29 i will do it before thurs meeting 14:28:34 there's another lite spec here https://bugs.launchpad.net/glance/+bug/1525259 14:28:34 Launchpad bug 1525259 in Glance "Add request_ids attribute to v2 schemas" [Wishlist,New] - Assigned to Abhishek Kekane (abhishek-kekane) 14:28:38 mclaren has to stop asking tough questions 14:28:44 Also, there are some work items on the import spec we can start working on 14:28:55 mclaren: will look offline, time's up 14:29:04 Thanks folks :) 14:29:10 #endmeeting