15:00:02 <sjmc7> #startmeeting openstack search
15:00:08 <openstack> Meeting started Thu Jan 26 15:00:02 2017 UTC and is due to finish in 60 minutes.  The chair is sjmc7. Information about MeetBot at http://wiki.debian.org/MeetBot.
15:00:09 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
15:00:12 <openstack> The meeting name has been set to 'openstack_search'
15:00:16 <sjmc7> You’re always so prompt, rosmaita
15:00:25 <TravT> o/
15:00:42 <lei-zh> o/
15:00:45 <rosmaita> i try
15:00:59 <rosmaita> o/
15:01:07 <sjmc7> it is appreciated
15:01:47 <sjmc7> rosmaita: i do have a question for you if you can stick around a bit
15:01:55 <rosmaita> consider me stuck
15:02:17 <sjmc7> i will rearrange the agenda
15:02:36 <sjmc7> #topic Glance image visibility
15:02:56 <sjmc7> The change related to https://etherpad.openstack.org/p/glance-ocata-sharing-release-note-draft
15:02:57 <rosmaita> ready for questions
15:03:04 <sjmc7> the question was about ‘community’ images
15:03:13 <sjmc7> Glance doesn’t return them from list_images
15:03:30 <rosmaita> correct, we were worried about spam
15:03:31 <sjmc7> but that’s not an access level thing; seems to be because there may be many of them
15:03:39 <rosmaita> well, not from the default list images
15:03:56 <sjmc7> Searchlight doesn’t generally impose additional filters on stuff, and if we did, it would be hard to determine when to turn them off
15:04:07 <rosmaita> btw, here's the patch for the release note, has more details, might be useful:
15:04:09 <sjmc7> So i’m not sure whether we should include them or not in searches
15:04:10 <rosmaita> https://review.openstack.org/#/c/422897/
15:04:59 <sjmc7> Yeah, I was going off the spec. The access level change is pretty simple; we just need to check that an image is ‘shared’ as well as including the tenant in the members list
15:05:00 <rosmaita> well, communitizing an image is controlled by a policy
15:05:16 <rosmaita> so it can be restricted to "responsible" users
15:05:23 <sjmc7> Right.. but viewing them?
15:05:35 <rosmaita> so i'd think including them in results would be ok
15:05:43 <sjmc7> “ community images are not included in the image-list response unless
15:05:43 <sjmc7> specifically requested by a user.”
15:05:59 <rosmaita> right, for our primitive glance api
15:06:24 <rosmaita> but with searchlight, a user could say give me no community images in this response
15:06:35 <sjmc7> They could, yeah
15:06:41 <rosmaita> if i'm at a cloud where there's a lot of spam
15:06:47 <sjmc7> But that will mean the default is different from Glance's
15:06:49 <sjmc7> maybe that’s ok
15:07:08 <sjmc7> If that is the case, then I think the Searchlight patch will be very simple, just checking the two additional visibility values
15:07:23 <sjmc7> Is it safe to assume we can treat the ‘members’ list as empty unless an image is ‘shared'?
15:07:29 <rosmaita> no
15:07:40 <rosmaita> the member list persists across visibility changes
15:07:41 <sjmc7> Ok - that list is preserved for ‘private’ images?
15:07:50 <rosmaita> yes, and community, and even public
15:07:58 <rosmaita> you can't create members on those images
15:08:08 <rosmaita> but you could have added members and then changed the visibility
15:08:24 <rosmaita> we decided not to purge the member list on visibilty changes
15:08:27 <sjmc7> But you could have already-created ones. Ok. I think right now we may ignore the members list for public ones so i’ll maybe remove the exception
15:08:30 <sjmc7> Ok, thanks!
15:08:53 <sjmc7> #topic Ocata-3
15:08:57 <rosmaita> cool
15:09:05 <sjmc7> I’m going to be tagging o-3 today
15:09:27 <sjmc7> The flavor notifications review (https://review.openstack.org/#/c/420609) I’m going to test this morning and merge that first
15:09:44 <sjmc7> I was intending to get the Ironic plugin merged but I think it’ll have to slip to RC1
15:10:09 <sjmc7> There are a number of outstanding smaller reviews for bugs but I think at this stage most of them will have to wait for RC1
15:10:45 <TravT> i believe you'll need to also tag searchlight ui
15:10:52 <sjmc7> Yeah. Does anyone have any questions about any of the outstanding reviews?
15:11:25 * TravT looking over list now
15:11:49 <TravT> should we get this one in as well?
15:11:49 <TravT> https://review.openstack.org/#/c/409374/
15:12:20 <sjmc7> ideally we’d get everything in
15:12:45 <TravT> okay
15:13:03 <sjmc7> yeah, that’s one could probably go in, although it’s not life-changing
15:13:10 <sjmc7> especially since ES5 isn’t properly supported yet
15:13:17 <sjmc7> and by ‘properly’ i mean ‘at all'
15:13:36 <sjmc7> i DO want to get https://review.openstack.org/#/c/424275/ merged but that seems unlikely to happen today
15:13:52 <sjmc7> that fixes the problems under ES5
15:14:18 <TravT> we don't have a ES5 zuul test, yet, right?
15:14:38 <sjmc7> no :(  again, something i keep meaning to do but never get round to
15:14:58 <TravT> i remember talking about maybe making python 27 be ES2 and python 35 be es5 as a kind of a hack to test both
15:15:06 <sjmc7> yeah
15:15:39 <sjmc7> it’s not totally straightforward to do properly, but i’ll see if i can fudge it without changes to infra
15:16:20 <sjmc7> i don’t really want to do it that way because it overloads the test
15:16:21 <TravT> if we don't get both versions tested, we are guaranteed to see the untested one get broken at some point
15:16:32 <TravT> yeah, doing a second test would be better.
15:17:08 <sjmc7> maybe it’s ok for now, and fix it next release; it should be immediately apparent if it’s the python version breaking things
15:17:53 <sjmc7> well, if anyone’s able to do reviews today i’ll put up the o-3 tag review this afternoon my time
15:18:02 <sjmc7> rc-1 is next week i think
15:18:21 <sjmc7> TravT has just reminded me it’s chinese new year so we’re short a bunch of people
15:18:28 <lei-zh> I'll see if I can have a review before going to bed : )
15:18:30 <sjmc7> thankyou lei-zh for showing up :)
15:18:40 <lei-zh> sjmc7: :)
15:18:50 <lei-zh> yeah, the holiday begins from tomorrow
15:19:09 <sjmc7> oh. then where is everyone? :)
15:19:32 <lei-zh> people maybe on their way home
15:19:41 <lei-zh> by train or flight
15:19:48 <sjmc7> I could do with another vacation. Ok, last topic I had
15:19:50 <TravT> Yeah, day before holiday is typically already holiday
15:19:55 <sjmc7> #topic ptl nominations
15:20:07 <sjmc7> I think the deadline is January 29th for nominations
15:20:20 <sjmc7> details at https://governance.openstack.org/election/
15:20:48 <sjmc7> I am happy to keep on doing it for Pike and I’ll put forward a self nomination today
15:21:06 <rosmaita> ++
15:21:14 <lei-zh> +1
15:21:14 <sjmc7> If anyone else would like to run, I’m equally happy to step down and let them have a turn, so if you are interested, feel free to nominate yourself
15:22:04 <sjmc7> I’ve not had as much time this cycle as I’d like for Openstack but I’m happy to keep things ticking over. So if you want to run in a hotly contested election, decide by January 29th
15:22:15 <TravT> i'm happy to support any of our cores as PTL.
15:22:21 <sjmc7> And if you have questions about what’s involved, feel free to ask me or TravT
15:22:44 <sjmc7> #topic Open discussion
15:22:58 <sjmc7> Since we are short today, i’ll keep the meeting short so lei-zh can start his vacation
15:23:03 <sjmc7> Anything else anyone wanted to discuss?
15:23:09 <TravT> i don't today
15:23:19 <TravT> thanks lei-zh for helping with reviews
15:23:41 <lei-zh> you're welcome : )
15:24:00 <TravT> did it look like yuriy did what you wanted on the ironic plugin?
15:24:07 <TravT> for pipeline support?
15:24:19 <sjmc7> i think there’s one outstanding change
15:24:33 <sjmc7> i may make it if he isn’t able to get round to it by next week
15:24:48 <lei-zh> yeah, code looks good, it's very close
15:25:06 <TravT> okay, yes, i'd hate to see it miss the release cycle
15:25:10 <sjmc7> i did want to test the notifications. hoping ot do that today or tomorrow
15:25:13 <sjmc7> yeah, we can get it in
15:26:15 <TravT> okay, thanks everybody!
15:26:19 <sjmc7> that’s my line!
15:26:37 <sjmc7> thanks all. have a great holiday, those that are going on one
15:26:56 <sjmc7> and a good weekend everyone else
15:26:59 <TravT> i just wanted to encourage you...
15:27:00 <TravT> ;)
15:27:06 <lei-zh> thanks
15:27:21 <sjmc7> #endmeeting