Thursday, 2020-03-12

*** andrebeltrami has quit IRC00:54
*** hongbin has joined #openstack-meeting-401:12
*** yamamoto has joined #openstack-meeting-401:35
*** yamamoto has quit IRC01:36
*** yamamoto has joined #openstack-meeting-401:36
*** vishalmanchanda has joined #openstack-meeting-401:51
*** diablo_rojo has quit IRC02:13
*** yamamoto has quit IRC02:20
*** rosmaita has left #openstack-meeting-402:28
*** yamamoto has joined #openstack-meeting-402:35
*** hongbin has quit IRC03:03
*** yamamoto has quit IRC03:19
*** Xuchu_ has joined #openstack-meeting-403:20
*** Xuchu has quit IRC03:22
*** Xuchu_ is now known as Xuchu03:22
*** yamamoto has joined #openstack-meeting-403:24
*** ricolin_ has joined #openstack-meeting-403:31
*** ricolin_ has quit IRC03:33
*** ricolin_ has joined #openstack-meeting-403:34
*** psachin has joined #openstack-meeting-403:37
*** yamamoto has quit IRC03:44
*** yamamoto has joined #openstack-meeting-403:45
*** evrardjp has quit IRC05:35
*** evrardjp has joined #openstack-meeting-405:36
*** links has joined #openstack-meeting-405:58
*** Luzi has joined #openstack-meeting-406:02
*** gcheresh has joined #openstack-meeting-406:22
*** e0ne has joined #openstack-meeting-406:28
*** yamamoto has quit IRC06:40
*** yamamoto has joined #openstack-meeting-406:42
*** yamamoto has quit IRC06:50
*** yamamoto has joined #openstack-meeting-406:55
*** LiangFang has quit IRC07:02
*** smcginnis has quit IRC07:03
*** smcginnis has joined #openstack-meeting-407:04
*** Liang__ has joined #openstack-meeting-407:04
*** e0ne has quit IRC07:10
*** roman_g has quit IRC07:11
*** e0ne has joined #openstack-meeting-407:19
*** e0ne has quit IRC07:23
*** slaweq has joined #openstack-meeting-408:02
*** e0ne has joined #openstack-meeting-408:34
*** k_mouza has joined #openstack-meeting-408:44
*** k_mouza has quit IRC08:49
*** ralonsoh has joined #openstack-meeting-408:58
*** ktibi has joined #openstack-meeting-409:11
*** k_mouza has joined #openstack-meeting-409:34
*** Xuchu_ has joined #openstack-meeting-409:49
*** Xuchu has quit IRC09:50
*** Xuchu_ is now known as Xuchu09:50
*** e0ne_ has joined #openstack-meeting-410:50
*** e0ne has quit IRC10:51
*** vishalmanchanda has quit IRC11:00
*** andrebeltrami has joined #openstack-meeting-411:08
*** ktibi has quit IRC11:10
*** ktibi has joined #openstack-meeting-411:11
*** yamamoto has quit IRC11:22
*** yamamoto has joined #openstack-meeting-411:24
*** yamamoto has quit IRC11:27
*** eharney has joined #openstack-meeting-411:52
*** yamamoto has joined #openstack-meeting-411:59
*** yamamoto has quit IRC12:05
*** enriquetaso has joined #openstack-meeting-412:09
*** Luzi has quit IRC12:10
*** yamamoto has joined #openstack-meeting-412:14
*** yamamoto has quit IRC12:15
*** psachin has quit IRC12:47
*** yamamoto has joined #openstack-meeting-412:52
*** eharney has quit IRC12:53
*** yamamoto has quit IRC12:57
*** psachin has joined #openstack-meeting-413:12
*** Xuchu has quit IRC13:16
*** Xuchu has joined #openstack-meeting-413:17
*** k_mouza has quit IRC13:21
*** yamamoto has joined #openstack-meeting-413:24
*** Xuchu has quit IRC13:24
*** gcheresh has quit IRC13:29
*** eharney has joined #openstack-meeting-413:38
*** yamamoto has quit IRC13:39
*** gcheresh has joined #openstack-meeting-413:39
abhishekk#startmeeting glance14:00
abhishekk#topic roll call14:00
openstackMeeting started Thu Mar 12 14:00: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:00
abhishekk#link https://etherpad.openstack.org/p/glance-team-meeting-agenda14:00
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.14:00
*** openstack changes topic to " (Meeting topic: glance)"14:00
openstackThe meeting name has been set to 'glance'14:00
*** openstack changes topic to "roll call (Meeting topic: glance)"14:00
*** nao-shark has joined #openstack-meeting-414:00
abhishekk#link https://etherpad.openstack.org/p/glance-team-meeting-agenda14:00
abhishekko/14:00
jokke_o/14:00
abhishekklets wait for 2-3 minutes14:01
smcginniso/14:01
*** jraju__ has joined #openstack-meeting-414:01
nao-sharko/14:02
abhishekklets start14:02
*** links has quit IRC14:02
abhishekk#topic release/periodic jobs update14:02
*** openstack changes topic to "release/periodic jobs update (Meeting topic: glance)"14:02
*** dave-mccowan has joined #openstack-meeting-414:02
abhishekkWe are in last month of Ussuri cycle, some of the important dates are;14:03
abhishekkFinal release for non-client libraries - 2 weeks14:03
abhishekkMilestone 3 - 3 weeks14:03
abhishekkSo guys please review all remaining and important patches14:03
abhishekkSurprisingly, Periodic jobs all green14:03
*** yamamoto has joined #openstack-meeting-414:03
abhishekkfor last couple of days14:03
*** rosmaita has joined #openstack-meeting-414:04
smcginnisMakes me wonder if it's actually working. :D14:04
rosmaitasorry, lost track of time14:04
abhishekkI have local patch do suppress  some deprecation warning messages14:04
abhishekkwill submit it tomorrow14:04
abhishekksmcginnis, likewise :D14:04
abhishekkrosmaita, NP14:04
abhishekkmoving ahead;14:05
abhishekk#topic S3 support for glance_store14:05
*** openstack changes topic to "S3 support for glance_store (Meeting topic: glance)"14:05
abhishekkThis one is important change and we agree to get it in at the start of the cycle14:05
abhishekkspecs and patch are both in good shape, also we have etherpad which tells us how to test this change14:06
abhishekkspec, #link https://review.opendev.org/68739014:06
abhishekkImplementation, #link https://review.opendev.org/69584414:06
abhishekknao-shark, do you want to share anything on this?14:07
nao-sharkyes14:07
abhishekkplease carry on14:07
nao-sharki also summarized how to configure s3 driver on etherpad14:08
nao-sharkand i know deadline of glance_store's final release is approaching14:08
jokke_I'll try to find time to review that14:08
nao-sharkso kindly review above patches14:08
nao-sharkthanks.14:09
jokke_nao-shark: thanks for the very good documentation how to utilize this and sorry for neglegting it for this long14:09
abhishekkjokke_, thank you14:09
abhishekkmoving ahead14:10
abhishekk#topic Delete image from single store14:10
*** openstack changes topic to "Delete image from single store (Meeting topic: glance)"14:10
abhishekk#link https://review.opendev.org/#/c/69804914:10
abhishekkNeed review on this as it is a important change for us14:11
abhishekkrosmaita, jokke_ smcginnis kindly reveiw14:11
abhishekkI have added my findings on the patch and made changes accordingly14:11
rosmaitaok14:11
abhishekkI hope we will find middle way here (fingers crossed))14:11
abhishekkrosmaita, thank you14:12
abhishekkmoving ahead14:12
abhishekk#topic Transition Rocky to EM14:12
*** openstack changes topic to "Transition Rocky to EM (Meeting topic: glance)"14:12
abhishekk#link https://review.opendev.org/#/c/709888/214:12
abhishekk#link14:12
abhishekkhttps://etherpad.openstack.org/p/glance-stable-rocky-em14:12
abhishekkthis etherpad tells about some patches are pending to be merged against stable/rocky14:13
abhishekkbut IMO those are not important and not as per the backport standards14:13
*** eharney has quit IRC14:13
abhishekkI have given my approval for it, kindly suggest if you think otherwise14:13
smcginnisYeah, looks like maybe some useful stuff in openstack/glance, but nothing too critical.14:15
jokke_abhishekk: if those backports are not approriate by the standards, are they blocked?14:15
abhishekkjokke_, no as of now14:15
smcginnisAnd we can still continue to merge fixes after we transition, just can't do an official community release.14:15
jokke_as in -2'd or -W'd14:15
jokke_ok, we might want to do that and abandon after14:15
abhishekkjokke_,  not ye14:15
abhishekkjokke_, ack14:15
jokke_kk14:15
abhishekkwill do this later tonight14:15
abhishekkalso if you have time, kindly have a look14:16
*** thgcorrea has joined #openstack-meeting-414:16
abhishekksmcginnis, makes sense14:16
abhishekkCool, moving ahead;14:17
abhishekk#topic Open discussion14:17
*** openstack changes topic to "Open discussion (Meeting topic: glance)"14:17
abhishekkSo as per community goal we were missing some pieces in contributors guide14:17
abhishekkI have added the same14:17
abhishekkkindly have a look;14:17
abhishekk#link https://review.opendev.org/71223614:18
abhishekkthank you to rosmaita for this :P14:18
rosmaitawhat did i do?14:18
abhishekkyou submitted same for cinder, I took reference from there14:19
rosmaitaok, ok, as long as it was something good14:19
rosmaitathought maybe you were being sarcastic14:19
abhishekkhaha14:19
abhishekkReload is broken under py314:20
abhishekk#link https://bugs.launchpad.net/glance/+bug/185570814:20
openstackLaunchpad bug 1855708 in Glance "Reload tests broken in Py3" [Critical,Triaged] - Assigned to Khuong Luu (organic-doge)14:20
abhishekkWe are in serious trouble14:20
abhishekkjokke_, stage is yours14:20
jokke_ok, so this turned from "flaky test" to actually quite nasty bug14:21
jokke_This was originally bundled to same pile ith our broken ssl test and at least I thought it was test issue with python 3. feshfood took a proper spin on it and we looked his findings yesterday as he couldn't figure out why the parent process disappeared under him in the middle of the test14:22
jokke_well looks like py3 multithreading changes actually properly broke our reload. So current situation is that if you try to HUP the glance-api service to pick up config changes on flight, you end up loosing the parent process and leaking orphaned child processes14:24
jokke_So big hand to Khuong who spent time on this and provided me enough details to draw initial conclusions, but this is nasty and we really need to figure it out14:24
*** dave-mccowan has quit IRC14:25
abhishekkyes, this is very critical or else we will be missing out on big feature of reload14:25
jokke_and very easy to reproduce, just run glance-ap with bunch of workers, sighup ip (even without config changes) and you see the outcome14:25
jokke_Yes so one thing which makes this super critical and nasty is that we have been marketing the multistore feature being very easy to add new stores without downtime as you can just add the store in configs and reload and api should pick them up. Well, not exactly14:27
rosmaitai think our code was done before oslo.config supported reloadable config14:27
rosmaitabut now it does14:27
abhishekkrosmaita, yes14:27
rosmaitabut i think you have to mark the reloadable items explicitly or something14:27
abhishekknot able to recollect it, but there was some limitations on some config parameters which cannot be reloaded14:29
*** yamamoto has quit IRC14:29
jokke_we do allow only subset of configs too, what looks like the problem is, that iiuc py3 multithreading works in a way where signal handlers are persisten and only processed in the main thread, and we overload different handlers to child and parent which might actually cause us to gracefully die on parent instead of doing any of the reload logic14:29
smcginnisThe only issue is where we store off the config item rather than reading it from oslo.config. So "self.value = CONF.value" during init, then just always referencing "self.value".14:30
jokke_as the child hup handler is just "finish requests in flight and die"14:30
rosmaitasmcginnis: thanks for that clarification14:31
jokke_I don't know how the oslo_config reload works but we actually most of the cases maintain the socket and just respun all the workers with the new config14:31
jokke_or that's how it worked in py2714:31
bnemecYou do also have to explicitly mark opts as mutable or they won't change on a config reload.14:32
*** dave-mccowan has joined #openstack-meeting-414:32
bnemecIf there's code you need to run on a reload there's a hook for that too.14:32
abhishekkrosmaita, we don't use oslo.services so oslo.config reload will not work for us14:32
jokke_glance used to support even changing the bind on the flight where we released the socket and rebind if that was changed, but indeed that's fully separate topic as abhishekk said ^^14:33
abhishekkbnemec, AFAIK, this will work if and only if you are using oslo.services, right?14:33
jokke_that's what I thought as well14:34
bnemecYou get it for essentially free in oslo.service, but it should be possible to mutate config without it.14:35
rosmaitai guess the question is how much re-architecting we will have to do, and if so, what direction it should take14:35
jokke_yeah, I do not know yet. I'm still trying to find the actual rot cause, I know the symptoms and likely origin for them14:36
jokke_root14:36
smcginnisRot may be applicable too. :)14:36
abhishekkI will also spend some time around this14:36
abhishekkhaha14:36
jokke_mhm14:37
jokke_I'm just happy that we found out now, not when someone had production failing under them due to this14:37
rosmaitayes indeed14:37
abhishekk+114:37
jokke_but this needs some cycles sooner than later14:37
rosmaitayeah, because with this release its no longer possible to run under py2714:38
jokke_indeed14:38
abhishekkyes14:38
jokke_and we claim py3 support14:38
bnemecIf you're using cotyledon it looks like that supports mutable config too, FYI.14:38
jokke_cotyledon?14:39
abhishekkneed to roll our sleeves for this14:39
rosmaitaso, may need to send something to the ML that it's not a good idea to run train, etc under py3 if you intend to use the reload config via sighup feature14:39
jokke_rosmaita: yeah, will do that14:39
bnemecIt was a non-eventlet replacement for oslo.service.14:39
jokke_bnemec: ah, well we're running eventlet+oslo_config :P14:39
jokke_I think we're on some modified incubator stage of oslo.service before it became oslo.service14:40
abhishekkyes14:40
jokke_like we've been on lots of those incubator thingies14:40
bnemecAh, yuck. :-/14:40
*** ricolin_ has quit IRC14:41
abhishekkI remember I have migrated glance to oslo.service but that was rejected at that time :P14:41
*** ricolin_ has joined #openstack-meeting-414:41
jokke_it was craploads of refactoring and broke about dozen core things :P14:41
jokke_I remember you looking into it at the time14:42
abhishekkyeah14:42
jokke_but that's it from me, at this point just awareness and shout out to be careful and have a look if you have spare cycles14:43
abhishekkI think jokke_ and I should co-ordinate with each other to share the findings14:44
jokke_I'll keep my focus on those couple of critical reviews, this and the uncompress plugin for now14:44
abhishekkack, I will try to spend some time on it as don't have anything big atm14:44
abhishekkapart from delete from store thing, which is already in good shape14:45
jokke_it was in good shape 3 months ago :P14:45
abhishekkhaha14:46
rosmaitacyril ran into something yesterday ... looks like that gate job cinderclient was using to make sure it still worked with v1 API wasn't actually testing patches until *after* they were merged to master already14:46
rosmaitawe didn't notice because we never broke v1 compatability, apparently14:47
jokke_nice14:47
abhishekkrosmaita, yes, he has backported some patches to stable/train yesterday14:47
*** ricolin_ has quit IRC14:48
rosmaitayeah, so my point is, don't use that zuul config as a model if you need to do something like that in the future!14:48
jokke_;)14:49
jokke_good point, thnx14:49
abhishekklast 10 minutes14:50
jokke_I'm good14:52
*** psachin has quit IRC14:52
abhishekkrosmaita, smcginnis ?14:52
rosmaitanothing from me14:53
abhishekkcool, wrapping it up for today14:54
abhishekkguys keep reviewing important patches14:54
abhishekkThank you all14:54
jokke_Thanks all!14:54
abhishekk#endmeeting14:55
*** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings/"14:55
openstackMeeting ended Thu Mar 12 14:55:00 2020 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)14:55
openstackMinutes:        http://eavesdrop.openstack.org/meetings/glance/2020/glance.2020-03-12-14.00.html14:55
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/glance/2020/glance.2020-03-12-14.00.txt14:55
openstackLog:            http://eavesdrop.openstack.org/meetings/glance/2020/glance.2020-03-12-14.00.log.html14:55
*** nao-shark has quit IRC14:55
*** rosmaita has left #openstack-meeting-414:55
*** Xuchu has joined #openstack-meeting-415:01
*** danielarthurt has joined #openstack-meeting-415:01
*** yamamoto has joined #openstack-meeting-415:02
*** enriquetaso has quit IRC15:02
*** enriquetaso has joined #openstack-meeting-415:02
*** yamamoto has quit IRC15:07
*** Xuchu has left #openstack-meeting-415:16
*** gcheresh has quit IRC15:43
*** eharney has joined #openstack-meeting-415:49
*** gcheresh has joined #openstack-meeting-415:49
*** diablo_rojo has joined #openstack-meeting-415:58
*** diablo_rojo has quit IRC16:06
*** roman_g has joined #openstack-meeting-416:07
*** diablo_rojo has joined #openstack-meeting-416:10
*** yamamoto has joined #openstack-meeting-416:20
*** gcheresh has quit IRC16:30
*** Xuchu has joined #openstack-meeting-416:34
*** gcheresh has joined #openstack-meeting-416:37
*** enriquetaso has quit IRC16:38
*** gcheresh has quit IRC16:50
*** e0ne_ has quit IRC17:06
*** jraju__ has quit IRC17:16
*** evrardjp has quit IRC17:35
*** evrardjp has joined #openstack-meeting-417:36
*** kozhukalov has joined #openstack-meeting-417:46
*** andrebeltrami has quit IRC18:03
*** ralonsoh has quit IRC18:29
*** e0ne has joined #openstack-meeting-418:48
*** e0ne has quit IRC18:52
*** gmann is now known as gmann_lunch19:00
*** danielarthurt has quit IRC19:01
*** kozhukalov has quit IRC19:13
*** gcheresh has joined #openstack-meeting-419:23
*** e0ne has joined #openstack-meeting-419:36
*** trident has quit IRC19:51
*** trident has joined #openstack-meeting-419:59
*** gmann_lunch is now known as gmann20:01
*** gcheresh has quit IRC20:11
*** gcheresh has joined #openstack-meeting-420:18
*** ktibi has quit IRC20:45
*** gcheresh has quit IRC21:09
*** e0ne has quit IRC21:19
*** enriquetaso has joined #openstack-meeting-421:28
*** enriquetaso has quit IRC21:32
*** senrique_ has joined #openstack-meeting-421:32
*** senrique_ has quit IRC21:38
*** slaweq has quit IRC22:28
*** dave-mccowan has quit IRC22:50
*** dave-mccowan has joined #openstack-meeting-423:10
*** liuyulong has quit IRC23:47

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!