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