13:59:58 <jokke_> #startmeeting glance
13:59:59 <openstack> Meeting started Thu Jul  5 13:59:58 2018 UTC and is due to finish in 60 minutes.  The chair is jokke_. Information about MeetBot at http://wiki.debian.org/MeetBot.
14:00:00 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
14:00:02 <openstack> The meeting name has been set to 'glance'
14:00:04 <jokke_> #topic roll-call
14:00:06 <jokke_> o/
14:00:06 <smcginnis> o/
14:00:09 <abhishekk> o/
14:00:13 <rosmaita> o/
14:00:14 <jokke_> #link https://etherpad.openstack.org/p/glance-team-meeting-agenda
14:00:14 <McClymontS> o/
14:00:49 <jokke_> I think we're good to go, everybody onboard :D
14:00:56 <jokke_> #topic updates
14:01:18 <jokke_> As expected nothing major has happened this week.
14:02:05 <jokke_> Couple of things to point out. Abhishek has the multi back-end patches up, lets get the spec amendment and the implementation reviewed and merged
14:03:06 <jokke_> Then he also got the migitation for OSSN-0075 up which will need reviews as well and the hidden images
14:03:19 <abhishekk> About that tomoki has tested multiple backend with cinder and it is working, he has replied to my mail as well
14:03:32 <smcginnis> ++
14:03:54 <jokke_> I have the v1 endpoint removals close to done and it's currently passing all but functional tests but more about that later
14:04:12 <rosmaita> i'll test out ossn-0075 after the meeting
14:04:35 <jokke_> #topic release updates
14:04:56 <rosmaita> periodic test report
14:05:06 <rosmaita> client, store are all green
14:05:18 <rosmaita> the cursive test that was failing last week for glance is now passing
14:05:43 <rosmaita> there's an intermittent timeout failure in glance-tox-functional-py35-keystone-tips
14:06:00 <rosmaita> it has passed, failed, passed, failed, passed over last few days
14:06:20 <rosmaita> #link http://zuul.openstack.org/builds.html?pipeline=periodic&project=openstack%2Fglance&job_name=glance-tox-functional-py35-keystone-tips
14:06:24 <rosmaita> i will keep an eye on it
14:06:30 <rosmaita> doesn't look like anything to worry about yet
14:06:46 <rosmaita> upcoming release deadlines
14:06:50 <jokke_> k
14:06:58 <rosmaita> glance_store Rocky release 18 July (1 day less than 2 weeks away)
14:07:05 <rosmaita> so that is coming fast
14:07:08 <jokke_> did you check yet what is timing out there?
14:07:24 <rosmaita> looks like a timeout during the tests
14:07:39 <rosmaita> couldn't tell exactly which one
14:07:45 <rosmaita> without further looking
14:08:06 <jokke_> ok, so the store release, like said I'd like to get the release out rather next week than the following
14:08:09 <rosmaita> actually, it could be something else entirely
14:08:33 <jokke_> yeah, lets keep eye on that and if it keeps going dig in bit deeper
14:08:45 <rosmaita> right
14:09:13 <rosmaita> ok, and in other release news, the glanceclient final release is the week after glance_store deadline
14:09:24 <rosmaita> and that is also the week for Rocky-3 milestone
14:09:53 <rosmaita> that's about it
14:10:04 <jokke_> thanks Brian
14:10:10 <rosmaita> #link https://launchpad.net/glance-store/+milestone/0.25.0
14:10:48 <jokke_> so Lets try to get the multi back-end store parts in, the we need to make the decision in which order we merge the glance stuff and what API versions we'll set
14:11:23 <rosmaita> ok
14:11:27 <abhishekk> ok
14:12:08 <abhishekk> rosmaita, similar kind of blueprint need to be created in glanceclient?
14:12:20 <rosmaita> possibly
14:12:35 <abhishekk> ok
14:12:38 <rosmaita> we're traditionally pretty cavalier with glanceclient
14:12:39 <rosmaita> :)
14:12:48 <abhishekk> :D
14:13:01 <jokke_> oh and please lets try to have the release notes ready around when the changes themselves are good to go so we don't miss any
14:13:28 <rosmaita> i promised abhishekk that i will write up relnotes for this stuff
14:13:29 <jokke_> as usual we seem to be merging a lot at the last minute
14:13:39 <abhishekk> rosmaita, +2
14:13:43 <rosmaita> that's the glance way
14:13:45 <rosmaita> !
14:13:50 <jokke_> ;)
14:14:39 <abhishekk> between we need to modify hide old images specs to reflect 400 to 409 if hidden column is not present
14:14:49 <abhishekk> I will do that tomorrow
14:14:53 <rosmaita> i'll do the relnote and admin doc for ossn-0075 while i;m testing later
14:15:16 <jokke_> sounds good to me
14:15:20 <rosmaita> abhishekk thanks for remembering that one
14:15:29 <abhishekk> Also I have added a check to restrict queued images to be updated with hidden (which is not present in the spec as well)
14:16:05 <rosmaita> how do you mean?
14:16:25 <jokke_> abhishekk: good point
14:16:50 <abhishekk> means when image is created and before uploading data to it if someone tries to update image with hidden = True then it should be rejected
14:17:26 <rosmaita> makes sense
14:18:53 <jokke_> ok anything else?
14:19:11 <abhishekk> no
14:19:33 <jokke_> #topic open discussion
14:19:41 <jokke_> I have couple of things here
14:20:21 <jokke_> 1) Once I finally got rid of the obvious test failures I stumbled upon something and got abhishekk double check it for me today as well
14:20:40 <jokke_> so we have this cache management command for glance
14:21:20 <jokke_> which is still using v1 api so obviously no-one has reported using it as part of the v1 removal feedback what prevents them moving to v2
14:21:55 <jokke_> I think, for that reason, we should get rid of it as part of the v1 removal
14:22:10 <abhishekk> it is broken for v2 as well, we need to get it functional
14:22:37 <jokke_> abhishekk: did you find actual v2 implementation for it?
14:23:25 <smcginnis> Lack of feedback makes me wonder if anyone is actually using that.
14:23:26 <abhishekk> jokke_, i went through code and found something related to v2 in auth.py
14:24:10 <jokke_> yeah, what I mean is that the command itself is v1 only implementation
14:25:41 <jokke_> so taken into account we have had v1 api disabled by default for quite a while and no-one has even mentioned it not working indicates to me that we should just remove it with the API and if there is use case for it we can do it for v2 right at that point
14:26:39 <abhishekk> ok
14:26:54 <jokke_> smcginnis: my take on this as well. Honestly I did not even remember that the command exists
14:27:21 <smcginnis> It sounds useful, but maybe it's not actually needed?
14:28:02 <jokke_> yeah, I'm not exactly sure how useful/meaningful manual cache management per api node is to anyone
14:28:26 <jokke_> kind of one of those things where you likely could use your time for something much more productive ;)
14:29:47 <jokke_> rosmaita: what do you think?
14:30:10 <rosmaita> i don't have an opinion
14:30:41 <rosmaita> i think it's a LRU ejection strategy
14:30:59 <jokke_> LRU?
14:31:12 <rosmaita> kicks out least recently used
14:31:46 <rosmaita> what did the cache manager do, exactly?
14:31:59 <abhishekk> we have cache-cleaner, cache-prefetcher and cache-pruner as well
14:32:16 <abhishekk> it just utility tool to show cached images or delete cached images
14:33:12 <jokke_> it allows you to list cached and queued images, delete images from cache and queue and add images to queue
14:34:08 <jokke_> so literally just micromanage your cache on that node
14:34:11 <rosmaita> the cache-prefetcher was going to be useful for edge computing (or am i mis-remembering)
14:35:31 <jokke_> rosmaita: yeah, but it was never gonna be utilizing the cache_manage cli command (like said I did not even remember we had it) as it would be also touching only the local node glance service
14:36:08 <rosmaita> well, there does not seem to be an outpouring of support for the cache manager
14:36:08 <jokke_> you don't want to be sshing in to every glance node you want precache images :P
14:36:19 <rosmaita> that's what parallel ssh is for!
14:36:27 <jokke_> LOL
14:36:38 <rosmaita> yeah, i didn't really mean that
14:37:06 <rosmaita> so let me summarize what i think the situation is
14:38:05 <rosmaita> the glance cache manager tool has a strong dependency on v1, which is being removed in rocky.  we don't have bandwidth to mess with it and we also believe that it's not much used
14:38:24 <rosmaita> so the current situation is that we will include a release note saying the tool no longer exists in rocky
14:39:10 <jokke_> that was my idea, but instead of just doing it I wanted to hear what you guys think first ;)
14:39:12 <rosmaita> maybe the thing to do is to put up a spec-lite proposing removal, and send a link to the operator's list for comments
14:39:32 <rosmaita> we could aim for removal in Rocky-3 milestone
14:40:09 <smcginnis> I would just lump it in as part of the v1 API deprecation and removal and just drop it.
14:40:38 <rosmaita> my concern is that it's not obviously a v1 thing
14:40:38 <rosmaita> it's independent functionality
14:40:50 <smcginnis> Although a release note would be good to have to make sure if someone gets surprised it is gone they can have a chance to find out what happened to it.
14:41:09 <rosmaita> yeah, i agree that a release note is necessary
14:41:11 <jokke_> rosmaita: If you want note of it into the spec I can put amendment for it into the v1 removal spec on top of the reno
14:41:21 <smcginnis> Yeah, it's independent, but just looking at an excuse not to have to deprecate it in rocky and remove in stein when that wouldn't make sense.
14:41:33 <jokke_> no-one reads either anyways but at least we could point that it's mentioned in multiple places :P
14:41:38 <smcginnis> jokke_: That sounds like a good plan to me.
14:41:47 <rosmaita> agreed
14:41:49 <abhishekk> me too
14:42:25 <jokke_> gr8!
14:42:30 <jokke_> then the next thing
14:43:28 <jokke_> this has been hanging for long time, but I finally got abhishekk's test fixes encorporated to the patch that prevents us to launch the taskflow that we know will fail 557712
14:44:03 <jokke_> pretty well contained patch so would greatly appreciate your review efforts on it as well. abhishekk already went through it
14:44:16 <abhishekk> yes
14:44:50 <smcginnis> Do you have a link handy?
14:44:58 <rosmaita> https://review.openstack.org/#/c/557712/
14:46:31 <rosmaita> that looks like a good change
14:47:31 <rosmaita> looks like it's up to me and sean since you and abhishekk are co-authors
14:47:49 <jokke_> yes, that's why I brought it up
14:48:20 <abhishekk> yes
14:48:27 <rosmaita> if we end the meeting early, i will review it right now
14:48:30 <rosmaita> :)
14:48:33 <abhishekk> you guys have lots of stuff to review :D
14:48:53 <jokke_> well that's all from me
14:48:57 <jokke_> anyone else?
14:49:19 <abhishekk> nothing from me
14:50:00 <abhishekk> I need to put another patch to deprecate stores and default_store options
14:50:14 <jokke_> ok, going first
14:50:35 <smcginnis> Nothing from me.
14:50:36 <jokke_> going second
14:51:07 <jokke_> sold! Thanks all!
14:51:12 <jokke_> #endmeeting