14:00:51 #startmeeting glance 14:00:52 Meeting started Thu May 7 14:00:51 2020 UTC and is due to finish in 60 minutes. The chair is abhishekk. Information about MeetBot at http://wiki.debian.org/MeetBot. 14:00:53 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 14:00:55 The meeting name has been set to 'glance' 14:00:57 o/ 14:01:06 #topic roll call 14:01:12 #link https://etherpad.openstack.org/p/glance-team-meeting-agenda 14:01:15 o/ 14:01:16 o/ 14:01:32 Brian is facing some issues with freenode 14:01:48 so most likely he will not be here today 14:01:55 lets start 14:02:09 #topic Updates 14:02:21 seems like they have been bit unstable ... my client had dc'd/deauthed as well 14:02:31 ohh 14:03:05 I have started drafting topics for our 1st (hopefully last) virtual PTG 14:03:19 #link https://etherpad.opendev.org/p/Glance-Victoria-PTG-planning 14:03:23 nice 14:03:35 Will identify more topics and add those as well 14:03:45 Good list! 14:03:52 kindly have a look and fill free to add your topics as well 14:04:09 smcginnis, thank you :D 14:04:41 moving ahead 14:04:56 #topic release/periodic update 14:05:19 We already tagged rc2 last week, which will be our final release candidate for Ussuri 14:05:58 nice! 14:06:04 getting close 14:06:09 I ran a check yesteday and it didn't look like anything else merged. We should be all set. 14:06:16 I am still finding the better way to move specs into implemented directory 14:06:28 smcginnis, ack 14:06:53 there is some problem with symlinks I will discuss with Brian for the same 14:07:25 Regarding periodic jobs we have 2 failures for openstack-tox-py36-with-oslo-master job 14:07:47 I am not able to reproduce the failure on local so it's little bit tricky to understand 14:07:58 test_best_match_language_expected is failing (testtools.matchers._impl.MismatchError: 'it' != None) 14:08:11 is that consistent failing now due something in oslo master breaking us or are those sporadic failures? Did you have a chance to look? 14:08:37 these two jobs are consistent failure 14:08:51 since last week 14:09:17 kk 14:09:29 apart form those everything is green 14:09:49 moving ahead 14:10:01 #topic All queries to glance API fails with 503 error if rbd store configured incorrectly 14:10:23 So this is little bit tricky to reproduce but needs to be handled 14:10:35 #link https://bugs.launchpad.net/glance-store/+bug/1875281 14:10:35 Launchpad bug 1875281 in glance_store "[MultipleStores] All queries to glance API fails with 503 error if rbd store configured incorrectly" [Undecided,In progress] - Assigned to Abhishek Kekane (abhishek-kekane) 14:10:51 I have added steps to reproduce this issue and submitted a fix as well 14:11:03 #link https://review.opendev.org/#/c/722915 14:11:03 patch 722915 - glance_store - Fix: API returns 503 if one of the store is mis-co... - 2 patch sets 14:11:17 abhishekk: just a sec back to the failures 14:11:24 smcginnis, thank you for quick review 14:11:26 There were some changes in oslo.i18n recently that probably affect that test. 14:11:59 bnemec, ack 14:12:05 https://opendev.org/openstack/oslo.i18n/commit/4e907ed2f39329eaa12d1712d49ca8903db15124 14:12:24 by the quick look of it, looks like oslo might have lost their Italian translations which means that we cannot provide it anymore and the test fails as we are not returning "it" when it's requested 14:13:13 jokke_, ack 14:13:16 or i18n does not provide best match anymore 14:13:35 that sounds like more in line what bnemec said there 14:13:55 Ok, I will take a look again 14:14:07 That might be a legitimate bug on oslo. 14:14:23 The change was not intended to remove any languages. 14:14:44 good to know 14:14:51 bnemec, could you please report this bug? 14:15:05 (if necessary) 14:15:15 lets look into that ... if it indeed is bug in oslo, that means that our testing against master is indeed useful ;) 14:15:42 It may have changed names. I see it_IT in the stdlib locale list. 14:15:45 jokke_, correct 14:15:54 https://zuul.openstack.org/build/5afdd36ab4e04476bf0f2870f51d6daf 14:16:00 Yeah, I'll open a bug so we can figure out what to do with this. 14:16:02 this is the tips job build 14:16:16 bnemec, thank you 14:16:25 bnemec: gr8, thanks! 14:16:30 jokke_, thank you for pointers :D 14:16:55 I really didn't have clue what was going wrong 14:17:21 Coming back to our topic, 14:17:50 #link https://review.opendev.org/#/c/722915/ 14:17:50 patch 722915 - glance_store - Fix: API returns 503 if one of the store is mis-co... - 2 patch sets 14:17:57 jokke_, kindly have a look 14:18:18 and also should we backport it to stable/ussuri (I guess we need to) 14:18:31 yes, so the 503, did I understand correctly from our discussion yesterday, that it does not happen if the store becomes unavailable suddenly, but only if the service starts that store being faulty? 14:18:56 jokke_, yes 14:19:32 and that's indeed only RBD, not all store drivers? 14:20:06 I have checked file, cinder and swift those were ok 14:20:15 ok, that's a good start 14:20:30 now one concern I have with the proposed fix 14:20:54 IIUC the change will prevent glance-api starting if the configured store is not available 14:21:07 whic is really bad 14:21:28 jokke_, it will not prevail 14:21:46 glance api will start and it will disable add/delete method for that store 14:22:01 oh, ok ... that's great 14:22:21 it will log those messages as well 14:22:27 I thought raising BadStoreConfiguration pevented us from starting 14:22:49 which kind of made sense before multi-store but great if it doesn't anymore 14:22:50 it will only prevent when there is only one store in enabled_backends 14:22:57 goodie 14:23:14 cool 14:23:16 kk, will review that properly 14:23:21 thank you 14:23:48 let me know if you need to test it 14:24:05 sure, thanks for diggin into it 14:24:47 no problem 14:24:50 Moving ahead 14:25:05 https://bugs.launchpad.net/oslo.i18n/+bug/1877386 14:25:05 Launchpad bug 1877386 in oslo.i18n "Dropping babel broke Glance unit tests" [High,Triaged] 14:25:16 If someone could drop a link to the failing job there it would be helpful. 14:25:26 bnemec, thank you, I will do it 14:25:36 Thanks! 14:25:41 #topic Upstream backport of glance-store 14:25:55 jokke_, need your review on this 14:26:05 #link https://review.opendev.org/#/c/709048/ 14:26:06 patch 709048 - glance_store (stable/train) - Image upload fails if cinder multipath is enabled - 1 patch set 14:26:18 We have added two new config options in this patch 14:26:43 Kindly review this backport as well 14:27:44 I think the main question is will it work with those default values? 14:28:22 or will it still stay broken unless the multipathing support is configured 14:30:01 I guess it will work with default values 14:31:11 that's the normal case but will need to dig into that bit more carefully 14:31:26 adding to my review list 14:31:35 thank you 14:31:50 moving ahead 14:32:09 #topic Open discussion 14:32:43 I guess we don't have any update on tools which will be used for virtual PTG 14:33:04 smcginnis, am I right? 14:33:55 I guess the most important part of the virtual ptg atm. is that the registration is open so we should register 14:34:04 saw some mail about it 14:34:17 jokke_, yes, forgot to mention it 14:34:36 Nothing I have heard yet. 14:35:04 There is an infra run jitsi server, but I don't think that has been tested enough to use for the PTG. 14:35:39 smcginnis, ack, thank you 14:36:33 Do we have anything else for discussion 14:37:33 I don't have anything ... catching up after long weekend anyways 14:39:11 yes, looking all mails, comments is really time consuming after long break 14:39:50 Cool, lets wrap up and use remaining time for some productive work 14:39:55 thank you all 14:39:55 ++ 14:39:58 Thanks! 14:39:59 Thanks! 14:40:06 #endmeeting