14:02:05 <abhishekk> #startmeeting glance
14:02:05 <openstack> Meeting started Thu Jul 16 14:02:05 2020 UTC and is due to finish in 60 minutes.  The chair is abhishekk. Information about MeetBot at http://wiki.debian.org/MeetBot.
14:02:07 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
14:02:10 <openstack> The meeting name has been set to 'glance'
14:02:11 <abhishekk> #topic roll call
14:02:18 <abhishekk> #link https://etherpad.openstack.org/p/glance-team-meeting-agenda
14:02:18 <rosmaita> o/
14:02:21 <abhishekk> o/
14:02:35 <abhishekk> lets wait couple of minutes more
14:02:55 <jokke> o/
14:03:09 <rosmaita> ok, hope you & your family are doing well in these pandemic times
14:03:20 <abhishekk> looks like 3 of us
14:03:22 <rosmaita> you too jokke
14:03:35 <abhishekk> rosmaita, we are in lockdown again :(
14:03:45 <rosmaita> bummer
14:04:02 <rosmaita> just you or your whole area?
14:04:17 <abhishekk> i hope its well enough for you
14:04:17 <abhishekk> rosmaita, entire city
14:04:19 <rosmaita> wow
14:04:27 <rosmaita> my state is OK for now
14:04:37 <abhishekk> cool
14:04:46 <rosmaita> but school is supposed to open in a few weeks, not sure what will actually happen
14:05:01 <abhishekk> :O
14:05:11 <smcginnis> o/
14:05:19 <abhishekk> cool
14:05:22 <abhishekk> lets start
14:05:39 <abhishekk> #topic release/periodic jobs update
14:05:51 <jokke> rosmaita: likewise.
14:05:58 <abhishekk> Victoria milestone 2 weeks away
14:06:09 <abhishekk> we managed to get Dan's policy work in
14:06:24 <rosmaita> nice work, too
14:06:26 <abhishekk> his last patch is in the gate at the moment
14:06:53 <abhishekk> thank you dansmith for working on this on priority
14:07:16 <abhishekk> also we managed to get rid of registry and v1 api
14:07:25 <abhishekk> jokke, thank you for your efforts
14:07:28 <rosmaita> \o/
14:07:29 <smcginnis> \o/
14:07:35 <smcginnis> :)
14:07:58 <abhishekk> we are doing well on reviews as well
14:08:25 <jokke> dansmith: thanks for catching the copy_from policy. Something I missed clearly
14:09:03 <abhishekk> Regarding periodic job it was green for last 2/3 days, 1 job related to functional-py36 timed out recently
14:09:07 <jokke> I'm over the moon happy of getting rid of the v1 code ... thanks all for reviews and helping to get that through smoothly
14:09:13 <abhishekk> other than that every thing is good
14:09:42 <abhishekk> Moving ahead
14:10:03 <abhishekk> #topic Specs status
14:10:33 <abhishekk> We managed to get image encryption and policy for copy-image specs merged in last week
14:10:42 <abhishekk> this week we need to review below specs
14:10:59 <abhishekk> Make cinder driver compatible with multiple stores - https://review.opendev.org/695152 - Need reviewes
14:11:00 <abhishekk> Handle sparse images in glance_store - https://review.opendev.org/733157 -Need reviews
14:11:00 <abhishekk> Update proposal for duplication image download - https://review.opendev.org/734683 - Need reviews
14:11:00 <abhishekk> Optimize Ceph store network usage - https://review.opendev.org/#/c/740980/ - Need reviews
14:11:00 <abhishekk> Introspect import plugin to calculate virtual size of image - https://review.opendev.org/741121 - Need reviews
14:11:03 <abhishekk> Cache API - https://review.opendev.org/#/c/665258/ - Need reviews
14:11:28 <abhishekk> Sparse image is just rebase, if rosmaita is good with it we can get that in today
14:11:58 <abhishekk> Introspect plugin and cache API are simple one, so any two approvals will be Ok for those specs
14:12:00 <jokke> #link https://review.opendev.org/#/c/740980/ as well please
14:12:13 <rosmaita> will look at sparse images today
14:12:26 <abhishekk> jokke, its there in the list
14:12:29 <jokke> the litespec for ceph optimizations beyond the sparse image
14:12:42 <alistarle> I am interested at "Optimize Ceph store network usage" and "Introspect import plugin to calculate virtual size of image", I can have a look if you want
14:12:54 <jokke> oh yeah, sorry, my bad
14:13:10 <abhishekk> alistarle, always welcome
14:13:29 <abhishekk> I am ready with Poc for Introspect as well
14:13:38 <abhishekk> will upload it for reference
14:14:21 <abhishekk> Please have a look at those specs
14:14:27 <abhishekk> moving ahead
14:14:33 <abhishekk> #Open discussion
14:14:41 <abhishekk> #topic Open discussion
14:15:17 <abhishekk> I guess cyril has added two points but he is not around
14:15:28 <jokke> Quick update. I've taken over to look into that race condition on copy-image form abhishekk to free up some time for him.
14:15:48 <abhishekk> jokke, thank you \0/
14:15:54 <jokke> I think I'll have solution that would address everybody's concerns about that locking this week
14:16:05 <abhishekk> great
14:16:17 <abhishekk> cyril needs reviews on  below patches
14:16:18 <alistarle> For the sparse upload, we nearly finish the code with gregoire and I think we will submit it soon, maybe beginning of next week without the tests
14:16:18 <jokke> just trying to figure out few lasts bits of logic in that and clean up the mess I've caused :P
14:16:21 <abhishekk> Remove the six library: https://review.opendev.org/#/c/735658/ and https://review.opendev.org/#/c/735670/
14:16:30 <abhishekk> :D
14:16:45 <smcginnis> Ooh, code removal. I'll make time for that. ;)
14:16:49 <jokke> alistarle: great, please ping me for review if I miss it
14:16:53 <abhishekk> :D
14:17:36 <abhishekk> He also wants to address, Fix glance-replicator, which has been broken for a while (see https://bugs.launchpad.net/glance/+bug/1830026 and https://review.opendev.org/#/c/661304/)
14:17:36 <openstack> Launchpad bug 1830026 in Glance "glance-replicator is deprecated since glance v2" [Undecided,In progress] - Assigned to Cyril Roelandt (cyril-roelandt)
14:18:01 <jokke> more code deletion! :P
14:18:13 <abhishekk> I am not sure anyone is using it
14:18:14 <smcginnis> I put up a patch to remove deprecated options from there yesterday, but if we can just drop the whole thing, that's even better.
14:18:43 <abhishekk> I will talk with cyril when he is available about this
14:19:38 <abhishekk> As suggested by smcginnis I have created ethercalc for Glance list of ToDo's
14:19:38 <abhishekk> #link https://ethercalc.openstack.org/kq4gwn9gqzck
14:19:57 <abhishekk> smcginnis, already submitted patch to clear his ToDo's :D
14:20:07 <smcginnis> Some of those look fairly straight forward. Some look a little more complicated to address.
14:20:18 <jokke> Also feel free to whack out any tooling for v1 that might have slipped me. I just realized that I did not look into those when I was deleting the v1 code so there might be still stuff lingering that has no chanse to work since v1 is gone
14:20:28 <abhishekk> Next week I will try to fix straight forward ToDo's
14:20:43 <abhishekk> jokke, ack
14:21:22 <jokke> cache should be fineish as of now, but yeah forgot replicator etc.
14:22:19 <jokke> there shouldn't be anything relying on registry as I was able to throw the registry client away without pep8 blowing up
14:22:45 <abhishekk> I guess there is nothing
14:23:17 <abhishekk> I just need to check whether there is any skipped test which were depending on registry
14:24:01 <jokke> #link https://review.opendev.org/#/c/740821/ also pretty please. Single line fix in client side for the global request id _with_ tests!!! :P
14:24:08 <smcginnis> Not too bad just grep'ing for registry.
14:26:16 <abhishekk> that's it from me today
14:26:32 <abhishekk> we have 30 minutes left
14:26:39 <smcginnis> Nothing fom me.
14:26:41 <smcginnis> *8om
14:26:47 <smcginnis> from    gah!!
14:27:05 <abhishekk> Unless anyone wants to discuss anything we can use those for reviews of specs :D
14:27:15 <smcginnis> +1
14:27:18 <jokke> Oh I have one more thing!
14:27:25 <abhishekk> go ahead
14:28:11 <jokke> Our stable branch could od with bit of love like https://review.opendev.org/#/c/740821/ would be great to merge so we can release that fix
14:28:33 <jokke> there is couple of other things as well that are pending the second review
14:29:08 <smcginnis> Side note: If you didn't see it, OpenStack 10 year celebration starts in ~30 minutes.
14:29:22 <jokke> that's the one I wanted to figure out what we need to do about it
14:29:31 <abhishekk> \o/
14:30:14 <jokke> smcginnis: you had concern that it's not cherry-pick as it's doing different thing to solve the same problem between brnaches
14:30:16 <abhishekk> jokke, smcginnis https://review.opendev.org/741086
14:31:17 <abhishekk> Congratulations all for 10 years of OpenStack, cheers :D
14:31:56 <jokke> as train with py27 we still support sl termination to glance, so can't just whack the things out there that are breaking the py3 behaviour
14:32:26 <smcginnis> jokke: ack
14:32:35 <jokke> Indeed great 10 years of OS, happy that I've had opportunity to be part of most of it
14:32:54 <abhishekk> Yes
14:33:20 <abhishekk> lets conclude the meeting
14:33:24 <abhishekk> thank you all
14:33:31 <abhishekk> keep reviewing :D
14:33:32 <jokke> so unless we're in rush to call this meeting done, can we please have a look on that and get to an agreement how we get that fix merged?
14:33:57 <abhishekk> ack
14:34:56 <jokke> I'd love to be able to get train release out where our reload actually works and not just blow up :P
14:35:18 <smcginnis> ++
14:35:23 <jokke> Kind of slipped through the cracks of other things and have been stale for months
14:36:57 <abhishekk> I guess as it is directly proposed against train branch it doesn't fall under cherry pick category
14:37:57 <abhishekk> or, should it be cherry-picked manually and then do the changes related to py27?
14:38:38 <abhishekk> #link https://review.opendev.org/#/c/713720/
14:39:27 <jokke> so I understood smcginnis was feeling that we should do rather cherry-pick and then call that the fix breaks py27 and fix that, which kind of feels counter intuitive for me. But I rather get the red tape sorted than keep that lingering forever
14:41:06 <abhishekk> I also think he is talking about same
14:43:03 <abhishekk> looks like we lost him
14:43:19 <abhishekk> lets take this offline when he is back
14:43:25 <jokke> sounds good
14:43:28 <jokke> indeed
14:43:30 <abhishekk> cool
14:43:49 <jokke> I guess he ran as soon as you said to conclude the meeting :P
14:43:56 <abhishekk> :D
14:43:57 <jokke> Thanks all!
14:44:04 <abhishekk> thank you all
14:44:13 <abhishekk> see you next week
14:44:23 <abhishekk> #endmeeting