14:00:13 <rosmaita> #startmeeting glance
14:00:14 <openstack> Meeting started Thu Jun 29 14:00:13 2017 UTC and is due to finish in 60 minutes.  The chair is rosmaita. Information about MeetBot at http://wiki.debian.org/MeetBot.
14:00:15 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
14:00:17 <openstack> The meeting name has been set to 'glance'
14:00:30 <rosmaita> #topic roll call
14:00:37 <rosmaita> hello everyone
14:00:49 <abhishekk> hi
14:00:52 <abhishekk> o/
14:01:17 <rosmaita> hi abhishekk, nice work on that download bug
14:01:45 <rosmaita> pretty sure i saw mfedosin and jokke_ online
14:01:56 <mfedosin> I'm here
14:01:58 <abhishekk> rosmaita: thank you, I have couple of questions about it
14:01:59 <mfedosin> hi!
14:02:10 <rosmaita> \o
14:02:30 <mfedosin> what bug?
14:02:38 <rosmaita> item 2 on the agenda
14:02:50 <rosmaita> #link https://etherpad.openstack.org/p/glance-team-meeting-agenda
14:03:02 <rosmaita> ok, guess we should get started
14:03:06 <rosmaita> #topic updates
14:03:33 <rosmaita> life stuff forced me to take a week off, so i have been off irc and not read the ML ... anything i need to know about?
14:04:27 <abhishekk> no update other than nikhil has decided to hang is boots
14:04:51 <rosmaita> bummer
14:05:03 <jokke_> o/
14:05:04 <abhishekk> and regarding tempest/glance store issue ghansham has submitted one patch
14:05:13 <jokke_> sorry for being bit late
14:05:18 <rosmaita> jokke_: np
14:05:31 <mfedosin> jokke_: hi :)
14:05:34 <rosmaita> abhishekk was just getting me caught up on the last week's events
14:06:10 <rosmaita> ok, i will adjust the core list later this afternoon to reflect nikhil's departure
14:06:44 <rosmaita> and i will let jokke_ decide about whether ghansham's patch is sufficient for the tempest/glance_store issue
14:06:55 <rosmaita> ok, let's move to item 2
14:07:13 <rosmaita> #topic glance image download error
14:07:15 <rosmaita> #link
14:07:17 <rosmaita> oops
14:07:32 <abhishekk> regarding this bug, if Image size is 0 then we are getting it from cache
14:07:34 <rosmaita> #link https://bugs.launchpad.net/glance/+bug/1686488
14:07:35 <openstack> Launchpad bug 1686488 in Glance "glance image-download error" [High,In progress] - Assigned to jiaopengju (pj-jiao)
14:07:45 <abhishekk> but why we are not setting it to image?
14:08:27 <abhishekk> if someone tries to download that image again then every time it will be fetched from cache
14:09:46 <rosmaita> this is when the image corresponds to a volume?
14:10:06 <abhishekk> rosmaita: yes
14:10:41 <abhishekk> instance is booted from volume and someone takes snapshot then in glance image-size is set as 0
14:11:09 <jokke_> so quick question. Is this cache enabled only issue or where the actual issue is coming from if not?
14:12:12 <abhishekk> jokke_: cache enabled issue I guess
14:13:48 <jokke_> that would be great. I just want to make sure we don't fix something in the cache that stays broken if the cache is disabled
14:14:19 <abhishekk> jokke_:right
14:15:21 <abhishekk> is it possible to create 0 size image?
14:16:04 <abhishekk> I remember some discussions with nova team when v2 api was enabled
14:17:03 <rosmaita> sorry, i was trying to track down something i remembered seeing, but this seems to be a different issue
14:17:10 <rosmaita> but here it is for reference:
14:17:18 <rosmaita> #link https://answers.launchpad.net/glance/+question/632486
14:18:01 <rosmaita> different case, the question is about an image that shouldn't be zero but is, not an image that should be zero
14:19:42 <abhishekk> #link https://github.com/openstack/nova/blob/master/nova/image/glance.py#L382
14:19:44 <rosmaita> abhishekk: do you have any concerns about the patch?
14:19:53 <abhishekk> no
14:20:43 <abhishekk> only thing is should we set that size to image permanently to avoid to fetch again?
14:21:56 <jokke_> no
14:22:10 <rosmaita> jokke_: tell us more
14:23:03 <jokke_> one should never set the original record based on cached data
14:23:08 <abhishekk> then its good to merge
14:24:42 <jokke_> lets say we get caching error that puts only half image into the cache and we set the image size to that based on invalid cache on one node. Now we suddenly have actual error in the records rather than this special case 0 size we know we need to deal with
14:26:28 <rosmaita> jokke_: good point, you have convinced me
14:26:45 <rosmaita> ok, let's move on
14:26:47 <abhishekk> we can move ahead
14:27:18 <rosmaita> cool, i have +2d the patch ... abhishekk is co-author, so someone else needs to do the +2A honors
14:27:38 <rosmaita> #topic priorities for the week
14:28:09 <rosmaita> on the agenda is dhellman's doc migration patches, but i think flavio approved them while we were talking
14:28:21 <rosmaita> #link https://review.openstack.org/#/q/topic:doc-migration+is:open+(+project:openstack/glance+OR+project:openstack/python-glanceclient+OR+project:openstack/glance_store+)
14:28:28 <abhishekk> yes
14:28:31 <rosmaita> so that's no longer a priority
14:28:39 <rosmaita> any candidates?
14:28:51 <abhishekk> protected filter patch
14:29:10 <rosmaita> that would be good to get done
14:29:31 <abhishekk> I have seen another strange issue on that patch for which I have earlier voted -2 but later changed it to -1
14:29:42 <abhishekk> #link https://review.openstack.org/#/c/449108/
14:30:43 <rosmaita> that is a strange issue indeed
14:30:45 <abhishekk> please see my -2 comment, I am not able to guess why is that behaviour
14:30:53 <rosmaita> i might go so far as to say wtf?
14:31:23 <abhishekk> exactly
14:32:56 <rosmaita> ok, let's put it on the list and hopefully our combined brain power will come up with an answer
14:33:10 <rosmaita> anything else?
14:33:31 <jokke_> I think bug scrub would be great
14:33:56 <rosmaita> yes, it would
14:33:57 <jokke_> at least I have been ignoring our open bugs for way too long time
14:34:04 <abhishekk> yes
14:34:14 <jokke_> so if we don't have lots for priorities, lets make that priority
14:34:31 <rosmaita> ok, i will put it on the list
14:34:47 <rosmaita> there is also a security bug that needs some follow up
14:35:27 <jokke_> i it the one Fei Long was working on?
14:35:30 <rosmaita> i think i will add abhishekk to that bug so we can get his opinion ... a lot of the glance-cores who are on that bug are no longer working on glance
14:35:34 <rosmaita> jokke_: the same
14:35:42 <rosmaita> and mfedosin is familiar with it as well
14:35:47 <rosmaita> :)
14:36:38 <rosmaita> ok, let's move on
14:36:43 <rosmaita> #topic open discussion
14:38:11 <rosmaita> mfedosin: you teased us about some news 2 weeks ago, and i forgot to remind you last week
14:38:15 <rosmaita> anything to share?
14:38:21 <mfedosin> I run the risk of incurring the wrath of Erno, but want to say that we are applying under Big Tent
14:38:32 <mfedosin> today
14:38:51 <rosmaita> "we" == ?
14:38:59 <mfedosin> Glare team :)
14:39:06 <rosmaita> ahh, thought so
14:39:20 <jokke_> wrath from me for that :o
14:39:21 <mfedosin> Today we got a new guy from India
14:39:24 <mfedosin> Devi
14:39:30 <jokke_> I thought Glare was part of big tent
14:39:35 <rosmaita> me too
14:39:38 <mfedosin> no, it wasn't
14:39:54 <mfedosin> so, there are 3 full time developers now
14:39:57 <mfedosin> and Geetika
14:40:02 <rosmaita> nice
14:40:30 <mfedosin> there were several production deployments of Glare
14:40:37 <mfedosin> 5 actually
14:40:59 <mfedosin> so, we got and fixed several bugs :)
14:41:03 <rosmaita> cool ... i assume they use a glare database independent of the glance db?
14:41:15 <rosmaita> (just thinking about the glare-ectomy in Pike)
14:41:25 <rosmaita> (we didn't include any kind of migrations)
14:41:34 <jokke_> mfedosin: as you should know I'm not against Glare ... I'm against any short term plans/claims Glare replacing Glance ;)
14:41:36 <mfedosin> It's based on Ocata
14:41:46 <rosmaita> excellent
14:42:14 <mfedosin> so, we defined a list of tasks for next couple of months
14:42:28 <mfedosin> https://docs.google.com/spreadsheets/d/1H2R2UhkwHkNEFjtLYmXZsPXWIJ148SCP-WjMplnytD8/edit#gid=0
14:42:47 <mfedosin> So far, everything is preliminary
14:43:01 <mfedosin> Devi is not included there
14:43:12 <mfedosin> jokke_: I'm against it too!
14:43:30 <mfedosin> No plans at all to replace glance in next couple of years
14:43:45 <rosmaita> i'd suggest adding zero-downtime db migrations
14:43:56 <rosmaita> (will be needed to replace glance :) )
14:44:25 <rosmaita> ok, cool ... well, best wishes to the Glare team from the Glance team
14:44:25 <mfedosin> I thought we already have them...
14:44:40 <mfedosin> we use alembic from the very beginning
14:44:40 <jokke_> have more :)
14:44:52 <mfedosin> jokke_: lol, okay
14:45:06 <mfedosin> so, I'll send an update today
14:45:14 <rosmaita> gosh, you kids with your modern software development techniques
14:45:37 <rosmaita> alright, thanks for the news
14:45:47 <mfedosin> my pleasure :)
14:45:54 <rosmaita> anyone have anything else for open discussion?
14:45:54 <mfedosin> thanks for listening
14:46:00 <mfedosin> I have
14:46:09 <abhishekk> all the best mfedosin
14:46:15 <mfedosin> it's related my patch for trust support
14:46:27 <mfedosin> wait a sec...
14:46:51 <mfedosin> #link https://github.com/openstack/glance/blob/master/glance/common/trust_auth.py
14:47:35 <mfedosin> ppl from keystone told me that it looks ugly and many things were deprecated since it had been merged
14:48:03 <mfedosin> so, I refactor this code a little bit, to follow the recommendations
14:48:30 <mfedosin> Tests are fine, I just need to write a commit message and push it on review
14:49:30 <mfedosin> *I've refactored
14:50:14 <jokke_> looking forward for it
14:50:16 <rosmaita> that sounds like a good change
14:51:08 <rosmaita> anything else?
14:51:39 <jokke_> just fyi
14:52:01 <jokke_> I merged the patch that was transations sync removing LC translations
14:52:28 <jokke_> didn't wait for second review ... so revert and blame me if you don't agree :P
14:56:32 <rosmaita> jokke_: you don't happen to have a link?
14:57:16 <jokke_> https://review.openstack.org/#/c/477085/
14:58:02 <rosmaita> ok, thanks
14:58:42 <jokke_> was one of those automated bot syncs
14:59:15 <rosmaita> that seems fine
14:59:24 <rosmaita> ok, just about out of time ... thanks everyone!
14:59:30 <abhishekk> thank you
14:59:30 <jokke_> thanks
14:59:37 <rosmaita> #endmeeting