14:00:25 <jokke_> #startmeeting glance
14:00:26 <openstack> Meeting started Thu Nov 30 14:00:25 2017 UTC and is due to finish in 60 minutes.  The chair is jokke_. Information about MeetBot at http://wiki.debian.org/MeetBot.
14:00:27 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
14:00:30 <openstack> The meeting name has been set to 'glance'
14:00:33 <abhishekk> o/
14:00:34 <smcginnis> o/
14:00:42 <jokke_> #topic roll call
14:00:53 <jokke_> #link https://etherpad.openstack.org/p/glance-team-meeting-agenda
14:01:08 <jokke_> Brian will be bit late today so I'll be kicking the meeting off
14:01:17 <abhishekk> great
14:01:30 <jokke_> so hi all
14:01:39 <smcginnis> Hi jokke_ :)
14:01:40 <jokke_> lets give minute or so
14:02:03 <smcginnis> abhishekk: Thanks for the ping too.
14:02:43 <abhishekk> smcginnis: i thought jokke_ was not around as well :D
14:02:43 <jokke_> So we have short agenda today, lets see if we are done by the time rosmaita arrives ;)
14:02:58 <abhishekk> jokke_ :P
14:03:02 <jokke_> #topic Updates
14:03:26 <smcginnis> Shhh, he's here.
14:03:29 <rosmaita> o/
14:03:33 <rosmaita> :)
14:03:43 <rosmaita> guess i need to read through the log
14:03:44 <jokke_> As you might have seen the discussion on our last meeting and the corresponding mail to the mailing list, nikhil is back to the core group! \\o \o/ o// o/7
14:03:56 <jokke_> #chair rosmaita
14:03:56 <openstack> Current chairs: jokke_ rosmaita
14:04:00 <abhishekk> \o/
14:04:01 <jokke_> rosmaita: welcome :)
14:04:08 <jokke_> way less late than you expected
14:04:24 <jokke_> darn we did not have time to go through the agenda before you got back ;)
14:04:34 <rosmaita> not much on it though
14:04:38 <jokke_> rosmaita: so wanna take over
14:04:44 <rosmaita> ok, thanks
14:04:44 <jokke_> indeed
14:04:47 <rosmaita> hello everyone
14:04:56 <abhishekk> o/
14:05:06 <rosmaita> as Erno mentioned, nikhil is back as a glance core
14:05:24 <rosmaita> i think he's travelling this week, so may not see much activity from him until next week
14:06:03 <rosmaita> as you all know, he's got a really deep knowledge of glance, so feel free to reach out to him with questions
14:06:24 <abhishekk> yup
14:06:26 <rosmaita> you may need to do it via email, not sure what his connectivity situation will be
14:06:47 <rosmaita> but he is always happy to help, so we should all take advantage of that!
14:07:03 <smcginnis> Awesome.
14:07:18 <rosmaita> ok, that's all i have for updates ... anyone else have any news we should be aware of?
14:07:57 <smcginnis> PTG registration is open. But that can wait until open discussion I think.
14:08:18 <rosmaita> ok, thanks ... also, nominations for the OpenStack board are open
14:08:29 <rosmaita> actually, let
14:08:37 <rosmaita> s  talk PTG for a minute
14:08:57 <rosmaita> i put in that Glance would want 2.5 days with approximately 8-10 people
14:09:20 <jokke_> sounds good
14:09:30 <rosmaita> i think they plan to keep the current structure of 2 days of cross-project work, 3 days individual project focus
14:09:36 <jokke_> I just registed about 20min ago
14:09:48 <smcginnis> jokke_: Sure you can travel all that way?
14:09:52 <rosmaita> so we'll plan for wed-thurs-and-friday-morning
14:10:34 <jokke_> smcginnis: we are getting better and better on planning. Got the travel approval already :P
14:10:53 <rosmaita> nice ... i don't know if i can get approval until 2018
14:10:59 <abhishekk> :D
14:11:05 <rosmaita> which is not that far away
14:11:17 <rosmaita> which brings us to the next topic ...
14:11:36 <rosmaita> #topic Q-2 milestone
14:11:52 <rosmaita> the Q-2 milestone release is next week
14:12:07 <jokke_> yes, I seriously need to start pushing patches up for review before thisl laptop blows and I loose all my work :P
14:12:08 <rosmaita> jokke_ : when does all stuff need to be merged by?
14:12:32 <rosmaita> push! push!
14:12:56 <jokke_> rosmaita: Wed ... we want to have the release patch ready for review by Thu morn
14:13:05 <jokke_> so we can get the release out in Thu
14:13:24 <smcginnis> That would be appreciated on my end. :)
14:13:30 <rosmaita> ok, so we should shoot for approvals by noon UTC on wednesday?
14:13:43 <jokke_> yeah
14:13:45 <rosmaita> hopefully that will leave time to get through the gate
14:14:04 <rosmaita> ok, as far as what i'd like to see get in
14:14:14 <rosmaita> 1. scrubber refactor
14:14:17 <jokke_> so I will look the last things waiting for +A Wed morn and we can coordinate that with abhishekk
14:14:31 <abhishekk> sure
14:14:37 <rosmaita> 2. abhishekk 's parade of interoperable-import bugs
14:14:50 <jokke_> and first thing Thu morn when I wake up I take the last hash from the branch and submit the release patch
14:15:05 <rosmaita> sounds good
14:15:09 <jokke_> abhishekk: thanks for all those catches!
14:15:21 <abhishekk> if some of the approved patches goes in merge conflict then after refactor single +2 +A will be enough?
14:15:22 <rosmaita> there are a few other bugs i have marked for Q-2
14:15:33 <abhishekk> jokke_: my pleasure
14:15:48 <jokke_> I knew there was lots what was initially missed, but didn't realize that we had pretty much none of the weird conditions looked after ;)
14:16:00 <abhishekk> s/refatcor/ resolving conflicts
14:16:08 <rosmaita> abhishekk that sounds ok
14:16:24 <abhishekk> rosmaita: ack
14:16:32 <rosmaita> i guess use your judgement, if there's some kind of crazy change, hold off
14:16:43 <abhishekk> make sense
14:16:54 <rosmaita> i'll write that down in the weekly email
14:17:02 <rosmaita> thanks for bringing it up
14:17:08 <jokke_> what rosmaita just said ... straight forward rebase I'm at least ok to +2A it after the check passes
14:17:44 <abhishekk> jokke_, rosmaita that will save lot of time
14:17:51 <rosmaita> i agree
14:18:03 <rosmaita> ok, about the scrubber
14:18:14 <rosmaita> i'd like as many cores as possible to look it over
14:18:17 <rosmaita> here's my concern
14:18:41 <rosmaita> when it had to use the registry, we had an extra layer of auth that had to happen
14:18:45 <rosmaita> and now we don't
14:19:17 <rosmaita> i just want to be sure that someone without sufficient credentials can't run the scrubber
14:19:46 <rosmaita> although, i guess that's not such a big deal, since it will only delete pending_delete images anyway
14:19:53 <jokke_> rosmaita: that still needs the DB credentials, right?
14:19:58 <rosmaita> so maybe i'm being overly paranoid
14:20:26 <abhishekk> rosmaita: that's why it will not hurt anywhere else
14:20:53 <jokke_> so if the user has those credentials or access to the glance-*.conf having those credentials there's noting much we can do
14:21:29 <rosmaita> that sounds right
14:21:38 <jokke_> also the user needs to have the admin credentials still to glance to be actually able to issue the deletes to the backend iiuc
14:22:27 <rosmaita> this is a side point, but we need to reorganize the glance.common.config options ... i think that the generated files have a lot of stuff in them that strictly speaking they don't need
14:22:36 <jokke_> but will put my security hat on when reading through that patch
14:22:44 <rosmaita> ok, great
14:23:17 <rosmaita> i had a question about how metadata encryption was being used, but xyz did some research and was able to answer it
14:23:44 <jokke_> rosmaita: the caviat of using automatic config generation ... lots of clutter just pulled in because they happen to be available
14:23:55 <rosmaita> jokke_ exactly
14:24:25 <rosmaita> also off topic, smcginnis has been putting up patches to reduce test output clutter
14:24:30 <rosmaita> thank you very much!
14:24:31 <jokke_> rosmaita: but we never have cared about clutering our config files :P
14:24:54 <jokke_> smcginnis: indeed and thanks a million for the argparse patch as well
14:25:07 <smcginnis> No problem!
14:25:17 <rosmaita> i missed that one, will have to look
14:25:20 <smcginnis> I hate seeing all that crud. :)
14:25:49 <rosmaita> i think everyone does, but you get kudos for doing something about it!
14:26:09 <rosmaita> ok, anyone else have anything to mention about Q-2?
14:26:47 <abhishekk> No
14:26:55 <jokke_> it came way faster than I expected :(
14:27:21 <rosmaita> yes, it did seem to happen pretty fast
14:27:23 * jokke_ is getting old ... still trashed after SYD
14:28:15 <rosmaita> ok, moving on
14:28:21 <rosmaita> #topic open discussion
14:28:28 <abhishekk> Do we need to backport IIR bugs to stable/pike?
14:28:37 <jokke_> no
14:28:47 <abhishekk> sounds good
14:28:48 <jokke_> experimental feature does not warrant backports
14:28:54 <bhagyashris> Request all to review patch https://review.openstack.org/#/c/524060/ it's regarding Add 'import_image' policy for import image API. Bug reference: https://bugs.launchpad.net/glance/+bug/1732141 . abhishekk has reviewed the same and given few comments that I have fixed and resubmitted for review.
14:28:55 <openstack> Launchpad bug 1732141 in Glance "'upload-image' policy blocks the image-import API" [High,In progress] - Assigned to Bhagyashri Shewale (bhagyashri-shewale)
14:29:28 <jokke_> bhagyashris: oh cool! That has been on our TODO list :)
14:29:38 <rosmaita> bhagyashris thanks for the reminder, you may have to be patient as the Q-2 bugs & scrubber have priority this week
14:29:56 <rosmaita> but we are anxious to get your change in
14:30:01 <jokke_> I would really like to see the scrubber making it's way to Q2
14:30:13 <rosmaita> ++ on the scrubber
14:30:26 <rosmaita> and actually, i think it looks pretty good
14:30:34 <bhagyashris> jokke_, rosmaita: Thank you in advanced :)
14:30:51 <jokke_> I've also got quite a bit of feedback about III and happy to see people picking up the bits that needs work done ... makes me happy
14:31:11 <rosmaita> as long as we're talking patches, i have some glance-specs patches that could use some love
14:31:23 * jokke_ is confident that it will be amazing once it becomes CURRENT
14:31:43 <abhishekk> IIR unit tests coverage patches are up as well
14:32:00 <jokke_> rosmaita: I +A'd one of them yesterday but by the looks of zuul did not like to pick it up
14:32:09 <rosmaita> darn
14:32:23 <jokke_> rechecked few min ago
14:33:11 <rosmaita> jokke_ you mean https://review.openstack.org/#/c/523472/ ?
14:33:29 <rosmaita> it's built off a patch that hasn't been approved yet
14:33:31 <jokke_> yup
14:33:36 <jokke_> looked neat
14:33:42 <rosmaita> https://review.openstack.org/#/c/523233/
14:33:54 <rosmaita> ^^ that has to go in before doug's patch will merge
14:34:12 <jokke_> ohhh
14:34:23 <jokke_> crap that's why it didn't get picked up
14:34:41 <rosmaita> yeah, that's the problem with in-repo dependencies, they aren't as obvious as a depends-on tag
14:34:51 <jokke_> I hate how unclear it is nowadays in gerrit if patch is depending on another
14:34:59 <rosmaita> ++
14:35:29 <rosmaita> anything else for open discussion?
14:36:39 <jokke_> abhishekk: mind to review that Brian's patch?
14:36:45 <jokke_> I just +2'd it
14:36:56 <abhishekk> jokke_: will have a look
14:37:04 <jokke_> abhishekk: obviously when you have time ... I know it's getting late for you
14:37:25 <abhishekk> jokke_: :) no issues
14:37:34 <jokke_> one more thing
14:37:42 <rosmaita> abhishekk i should know this, but what city do you work from?
14:37:55 <abhishekk> Pune, same as Pranali
14:38:08 <rosmaita> is that same TZ as mumbai?
14:38:20 <jokke_> It's been long while since we've released anything stable ... I'd like some help looking through bugs we've fixed and propose appropriate backports
14:38:21 <abhishekk> rosmaita: correct
14:38:33 <jokke_> I'd like to get stables release week after q2
14:38:56 <smcginnis> Good idea.
14:39:05 <rosmaita> jokke_ also on that theme, are we due for a python-glanceclient release?
14:39:17 <jokke_> rosmaita: have we merged anything there?
14:39:21 <jokke_> like we are ;)
14:39:34 <jokke_> could aim for that as well for the week after Q2
14:39:35 <rosmaita> well, we had a bunch of fixes at the end of pike
14:39:42 <rosmaita> don't think they were ever released
14:39:46 <bhagyashris> rosmaita: just checked Bug: https://bugs.launchpad.net/glance/+bug/1732141 marked as Q-2 bug
14:39:46 <openstack> Launchpad bug 1732141 in Glance "'upload-image' policy blocks the image-import API" [High,In progress] - Assigned to Bhagyashri Shewale (bhagyashri-shewale)
14:39:46 <jokke_> hmm-m
14:39:49 <jokke_> crap you might be right
14:40:10 <rosmaita> bhagyashris thanks! ok, everyone, make sure you review bhagyashris 's patch!
14:40:37 <bhagyashris> rosmaita: thank you :)
14:41:49 <jokke_> rosmaita: smcginnis: if the client is in release shape, can we try to get it released today?
14:42:11 <smcginnis> jokke_: Yep, we should be able to get that processed quickly.
14:42:22 <smcginnis> Can always do another one later in the cycle too.
14:42:22 <rosmaita> sounds good
14:42:26 <jokke_> I'd like to have that in not competing with Q2 but not waiting for two weeks still if we really didn't release the pike fixes
14:42:38 <rosmaita> that makes sense
14:42:51 <smcginnis> Good to get changes out soon to give time in case there is anything causing problems.
14:43:11 <rosmaita> sounds like we're out of open discussion topics ... let's end the meeting and discuss glanceclient release in the glance channel
14:43:13 <jokke_> rosmaita: I think biggest is to check that we have all the needed renos in place
14:43:27 <jokke_> yeah, thanks all!
14:44:00 <rosmaita> thanks everyone, enjoy your 17 free minutes!
14:44:07 <abhishekk> thank you
14:44:11 <rosmaita> #endmeeting