Monday, 2018-11-05

*** slaweq has joined #openstack-oslo00:13
*** slaweq has quit IRC00:45
*** ducnv has joined #openstack-oslo01:00
*** slaweq has joined #openstack-oslo01:11
*** phuongnh has joined #openstack-oslo01:19
*** slaweq has quit IRC01:44
*** slaweq has joined #openstack-oslo02:11
*** slaweq has quit IRC02:45
*** slaweq has joined #openstack-oslo03:13
openstackgerritwangxiyuan proposed openstack/oslo.policy master: Add policy-upgrade tool  https://review.openstack.org/61390603:36
*** slaweq has quit IRC03:44
openstackgerritDuc Nguyen proposed openstack/oslo.config master: update valid value in choice list for the opt  https://review.openstack.org/60306003:55
openstackgerritDuc Nguyen proposed openstack/oslo.config master: update valid value in choice list for the opt  https://review.openstack.org/60306003:57
*** dave-mccowan has quit IRC04:00
*** phuongnh has quit IRC04:07
*** slaweq has joined #openstack-oslo04:16
*** phuongnh has joined #openstack-oslo04:29
*** slaweq has quit IRC04:48
*** slaweq has joined #openstack-oslo05:16
*** pcaruana has joined #openstack-oslo05:23
*** pcaruana has quit IRC05:32
*** slaweq has quit IRC05:44
*** slaweq has joined #openstack-oslo06:11
*** e0ne has joined #openstack-oslo06:17
*** e0ne has quit IRC06:36
*** slaweq has quit IRC06:39
*** jbadiapa has joined #openstack-oslo07:11
*** jaosorior has joined #openstack-oslo07:13
*** jaosorior has quit IRC07:24
*** jaosorior has joined #openstack-oslo07:27
*** pcaruana has joined #openstack-oslo08:06
*** mattgo has joined #openstack-oslo08:46
*** e0ne has joined #openstack-oslo09:10
openstackgerritwangxiyuan proposed openstack/oslo.limit master: Add limit check func  https://review.openstack.org/59652009:31
*** pcaruana has quit IRC09:31
*** pcaruana has joined #openstack-oslo09:32
*** moguimar has joined #openstack-oslo09:39
*** slaweq has joined #openstack-oslo09:41
*** shardy has joined #openstack-oslo10:21
*** cdent has joined #openstack-oslo10:22
*** d0ugal has quit IRC10:36
*** d0ugal has joined #openstack-oslo10:55
openstackgerritHervé Beraud proposed openstack/oslo.config master: Introduce a base yaml parser for all openstack components  https://review.openstack.org/61552911:15
*** sean-k-mooney has joined #openstack-oslo11:27
jaosoriormoguimar: could you check this out if you have some time https://review.openstack.org/#/c/613313/ ?11:36
*** beekneemech has quit IRC11:53
*** bnemec has joined #openstack-oslo11:57
*** dave-mccowan has joined #openstack-oslo12:05
*** raildo has joined #openstack-oslo12:10
*** pbourke has quit IRC12:10
*** pbourke has joined #openstack-oslo12:11
*** phuongnh has quit IRC12:19
openstackgerritJuan Antonio Osorio Robles proposed openstack/oslo.policy master: WIP: Create OPA check  https://review.openstack.org/61422412:21
moguimarsure jaosorior12:21
*** jroll has quit IRC12:32
*** jroll has joined #openstack-oslo12:34
*** ansmith has quit IRC12:37
*** e0ne_ has joined #openstack-oslo12:37
*** e0ne has quit IRC12:40
*** mattgo has quit IRC13:08
*** mattgo has joined #openstack-oslo13:26
openstackgerritDaniel Vincze proposed openstack/oslo.log master: Add Windows Event Log handler  https://review.openstack.org/61455813:30
openstackgerritDaniel Vincze proposed openstack/oslo.log master: Add Windows Event Log handler  https://review.openstack.org/61455813:35
*** ducnv_ has joined #openstack-oslo13:42
openstackgerritHervé Beraud proposed openstack/oslo.config master: Introduce a base yaml parser for all openstack components  https://review.openstack.org/61552913:47
*** ansmith has joined #openstack-oslo13:57
*** kgiusti has joined #openstack-oslo14:04
*** bobh has joined #openstack-oslo14:05
openstackgerritHervé Beraud proposed openstack/oslo.service stable/rocky: Use eventlet Event for loopingcall events  https://review.openstack.org/61448914:10
openstackgerritHervé Beraud proposed openstack/oslo.service stable/rocky: Fix stop of loopingcall  https://review.openstack.org/61526314:11
*** SteelyDan is now known as dansmith14:37
*** phuongnh has joined #openstack-oslo14:54
bnemec#startmeeting oslo15:04
openstackMeeting started Mon Nov  5 15:04:00 2018 UTC and is due to finish in 60 minutes.  The chair is bnemec. Information about MeetBot at http://wiki.debian.org/MeetBot.15:04
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:04
*** openstack changes topic to " (Meeting topic: oslo)"15:04
openstackThe meeting name has been set to 'oslo'15:04
bnemeccourtesy ping for amotoki, amrith, ansmith, bnemec, dansmith, dhellmann, dims15:04
bnemeccourtesy ping for dougwig, e0ne, electrocucaracha, flaper87, garyk, gcb, haypo15:04
bnemeccourtesy ping for hberaud, jd__, johnsom, jungleboyj, kgiusti, kragniz, lhx_15:04
bnemeccourtesy ping for moguimar, njohnston, raildo, redrobot, sileht, sreshetnyak, stephenfin15:04
bnemeccourtesy ping for stevemar, therve, thinrichs, toabctl, zhiyan, zxy, zzzeek15:04
bnemec#link https://wiki.openstack.org/wiki/Meetings/Oslo#Agenda_for_Next_Meeting15:04
moguimaro/15:04
phuongnho/15:04
* stephenfin lurks15:04
hberaudo/15:04
ansmitho/15:04
ducnv_o/15:04
kgiustio/15:04
dhellmanno/15:05
bnemecI think this is the right time for the meeting.15:05
bnemecStupid time changes.15:06
bnemec#topic Red flags for/from liaisons15:06
*** openstack changes topic to "Red flags for/from liaisons (Meeting topic: oslo)"15:06
bnemecWe've had a number of reports of issues with SSL and rabbit in Pike.15:07
kgiusti:(15:07
jungleboyjo/15:08
bnemecObviously that isn't affecting any development though or someone would have reported it before now.15:09
kgiustiI've not been able to reproduce the issue reported about ocata->pike locally15:09
kgiustiI'm not sure if zigo's problem is the same - need to reach out to him today15:10
bnemeckgiusti: Okay, sounds good (or not good ;-). Thanks for looking into this.15:10
kgiustipike's been out for awhile now if this is a hard failure I would've expected to hear about this sooner.15:11
kgiustiSomething is rotten in the state of Oslo  (Messaging)15:11
bnemeckgiusti: Yeah, although you would be surprised how long it takes people to adopt new releases.15:12
kgiustitrue that15:12
bnemecHaving major bugs reported a year and a half after release isn't as uncommon as you might expect.15:12
*** phuongnh has quit IRC15:13
dhellmannthis smells like an openssl (or other library) upgrade problem15:14
bnemecCould be. That would also explain why no one has reported it until now.15:15
kgiustidhellmann: good point, I seem to recall a breaking change at some point in the openssl release history...15:15
zigokgiusti: I'm here !15:15
kgiustiHi zigo!15:15
kgiustiDid you have any luck getting your SSL config working?15:15
zigoIt's working, no problem.15:16
zigoThough my trouble was: no report at all that something was going wrong.15:16
zigoIt took me a real long time to understand.15:16
zigoI would have very much prefer not to loose all of that time with something in my log...15:16
zigoIn my case, there was absolutely nothing in heat-api and heat-engine saying it wouldn't connect to rabbit.15:16
kgiustiYeah I'll work on a fix for that - SSL failures are a pain to debug (even with logs!)15:16
zigoI had to double-guess that it wouldn't connect and that it was the root cause.15:17
kgiustizigo: what release are you using?  We've had a report of problems with SSL going from ocata->pke15:18
kgiustipike, that is15:18
zigokgiusti: Queens, then more recently, Rocky.15:18
zigoBoth had the problem.15:18
kgiustizigo: thanks15:19
zigoMy pleasure.15:19
kgiustiI suspect the ocata->pike failures may be different, since the same SSL config worked on ocata failed when upgrading to pike15:19
kgiustiI'lll continue to work on it15:20
bnemeckgiusti: I think I saw a patch on pike that moved some parameters around too, and I think it may have dropped some deprecated_names in the process.15:20
*** cfriesen has joined #openstack-oslo15:20
bnemecMaybe people were still relying on the deprecated stuff and it broke when that went away?15:21
bnemec(assuming I read the patch correctly, which you shouldn't ;-)15:21
bnemecAnyway, we should probably revisit this after the meeting.15:21
kgiusti+115:22
bnemec#topic Releases15:22
*** openstack changes topic to "Releases (Meeting topic: oslo)"15:22
*** phuongnh has joined #openstack-oslo15:22
bnemecCompletely forgot to do these last week. :-(15:22
*** d34dh0r53 has quit IRC15:22
* bnemec blames a certain TLA for interrupting his weekly routine15:22
bnemecI'll get that done today.15:23
bnemecI anticipate no releases next week since it will be Summit.15:23
bnemec#topic Action items from last meeting15:24
*** openstack changes topic to "Action items from last meeting (Meeting topic: oslo)"15:24
dhellmannbnemec : I feel like we had a couple of things go in late last week that would be good to release15:24
bnemecdhellmann: Yeah, I'll propose releases today.15:25
dhellmannthe oslo.config env stuff from cdent, and something that landed over the weekend as a fix that I can't quite put my finger on15:25
dhellmannok, cool15:25
bnemecIt just slipped my mind last week and by the time I remembered it was late enough in the week that I figured I'd just wait until this week.15:25
bnemec"bnemec send email about project update video"15:26
bnemecAs it turns out, I didn't have to because we got a project update slot.15:26
bnemec\o/15:26
moguimardid the oslo.config env stuff merged?15:26
bnemecmoguimar: I believe so.15:26
moguimaryup15:27
moguimarmerged15:27
bnemecI have some slides made up for the project update and I'll be sending those out today as well for people to review/make additions.15:28
bnemec"bnemec to investigate oslomotw"15:28
bnemecStill on my todo list, but not done. Probably going to have to wait until after Summit.15:29
bnemec"lbragstad to ping jaypipes about oslo.limit API"15:29
bnemecI saw some discussion around this, but I don't recall exactly where it stands.15:29
bnemecWill have to follow up on that later.15:32
bnemec#topic No meeting next week15:32
*** openstack changes topic to "No meeting next week (Meeting topic: oslo)"15:32
bnemecIt's Summit, so I expect a lot of the people attending this will be travelling.15:33
bnemec#action bnemec to send meeting cancellation email to list15:33
bnemec#topic Oslo project update15:34
*** openstack changes topic to "Oslo project update (Meeting topic: oslo)"15:34
bnemec#link https://www.openstack.org/summit/berlin-2018/summit-schedule/events/22952/oslo-project-onboarding15:34
bnemecIt's Thursday at 2:05.15:34
bnemecHope to see everyone there. :-)15:35
bnemec#topic Weekly Wayward Review15:35
*** openstack changes topic to "Weekly Wayward Review (Meeting topic: oslo)"15:35
bnemecLet's see what we need to look at this week...15:35
bnemecThere's a hacking patch for those of you who are hacking cores: https://review.openstack.org/#/c/602464/15:36
bnemecThere's also a kafka messaging patch, but that might be too big to get through right now: https://review.openstack.org/#/c/603190/15:37
bnemecIf someone wanted to write unit tests for https://review.openstack.org/#/c/612376/ we could get that one in.15:39
bnemecShouldn't be too difficult. There's only one branch in the function.15:40
bnemecSo the fact that I'm struggling to find a patch we can resolve during the meeting is kind of a good thing.15:41
bnemecAlmost everything in my top 15 list is related to large feature work that just takes time to review.15:42
bnemecWhich I take to mean that we're doing a reasonably good job of keeping up with reviews overall.15:42
moguimarThere is an oslo.policy patch by ozz: https://review.openstack.org/#/c/61331315:42
bnemecmoguimar: That's approved. :-)15:43
moguimara few minutes ago =D15:43
bnemecOnly 21 patches out of 116 are waiting on reviewer, so high fives all around.15:43
bnemeco/\o15:44
* lbragstad wanders in late15:44
bnemecSo I guess my conclusion for this week is keep up the good review work.15:44
bnemecAnd we'll keep chipping away at the feature reviews.15:45
bnemec#topic Open discussion15:45
*** openstack changes topic to "Open discussion (Meeting topic: oslo)"15:45
lbragstadabout the oslo.limit bits15:45
lbragstadwe have discussion on the API in oslo.limit happening in a few different places15:45
lbragstad#link https://review.openstack.org/#/c/596520/15:45
lbragstad#link https://review.openstack.org/#/c/615180/5/nova/compute/utils.py15:46
lbragstadand15:46
lbragstad#link https://review.openstack.org/#/c/600266/615:46
bnemecAh yes, the first one is where I had seen discussion this week.15:47
lbragstadkinda of a lot to keep a pulse on.. but johnthetubaguy is a little concerned about the initial complexity of the API15:48
lbragstadspecifically with the context manager15:48
lbragstadwhich means it had the opposite effect we were shooting for :)15:48
bnemecYeah, I'm a little surprised to hear that.15:49
bnemecThe context manager was supposed to make it easier to interact with quota code.15:49
lbragstadbut the feedback is good15:49
lbragstadright15:49
bnemecYeah, as long as there is good discussion happening we're making progress.15:49
lbragstadultimately supposed to hide the verify in the __exit__ of the context manager15:49
lbragstadbut it sounds like john want's the verification to be optional - so i'm going to be tinkering with his review today to see if i can experience the pain first hand15:50
lbragstadhopefully, we'll have something more concrete by the EOW is that's alright15:50
bnemecI'm curious what quota is supposed to do if it's optional, but I'll try to go read those reviews and then maybe it will make sense.15:51
lbragstadthe verification on __exit__ is mainly to protect against races conditions between clients15:52
bnemeclbragstad: Thanks for the update.15:52
lbragstadno problem15:52
phuongnhhi bnemec, dhellmann, as we discussed in Denver, we in Fujitsu would like the oslo config migrator feature have more attention. The time does not support us. Please help us. Thanks.15:53
bnemecphuongnh: Yeah, we're getting close. I'm struggling a bit to understand the last patch in the series right now.15:54
bnemecI suppose we could merge the rest of the code as-is, but I assume there would be a bug present if we did.15:54
phuongnhthanks, ducnv has updated source code and answer your comments. Please tell us if you have more question.15:54
phuongnhI know that :)15:55
bnemecThis is the review I'm talking about, for those playing along at home: https://review.openstack.org/#/c/603060/15:55
phuongnhyes, ducnv has updated on this patch15:56
bnemecSo I'll try to take another look at that this week.15:56
phuongnhthank you a lot15:56
ducnv_bnemec, thanks :))15:56
bnemecOkay, we have about three minutes. Anything else?15:57
phuongnhand do you need our support in preparing slide for Oslo project update ?15:57
phuongnhabout oslo config migrator feature?15:57
bnemecphuongnh: I have that in my slides already, but if you had information you specifically want included feel free to send it to me.15:59
phuongnhgeneral information is ok for us16:00
phuongnhthat is all for me, thanks16:00
bnemecAlright, we're out of time anyway.16:00
dhellmannphuongnh : will you or ducnv be able to make it to berlin next week for the summit?16:00
bnemecThanks for joining everyone!16:00
bnemec#endmeeting16:00
*** openstack changes topic to "OpenStack Common Libraries | https://wiki.openstack.org/wiki/Oslo"16:00
openstackMeeting ended Mon Nov  5 16:00:40 2018 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:00
openstackMinutes:        http://eavesdrop.openstack.org/meetings/oslo/2018/oslo.2018-11-05-15.04.html16:00
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/oslo/2018/oslo.2018-11-05-15.04.txt16:00
openstackLog:            http://eavesdrop.openstack.org/meetings/oslo/2018/oslo.2018-11-05-15.04.log.html16:00
bnemec(feel free to continue this discussion, of course)16:00
phuongnhdhellmann: we hope to go there but we cannot :)16:01
phuongnhthank bnemec :)16:01
dhellmannphuongnh : ok, that's no problem; we can continue to work together online16:01
phuongnhyes, I am glad to have met you in Denver, hope that I and ducnv have chance to see you again16:03
*** pcaruana has quit IRC16:06
dhellmann++16:07
*** phuongnh has quit IRC16:09
*** ducnv_ has quit IRC16:19
*** mriedem has joined #openstack-oslo16:28
mriedembnemec: i found out today that some projects are still calling i18n.enable_lazy() and it causes issues with the upgrade checkers, they hit something like this when building the output: http://paste.openstack.org/show/734156/16:29
mriedemremoving the call to enable_lazy() fixes it: https://review.openstack.org/#/c/615594/16:29
mriedemquestion is, can we just torch all of the lazy translation stuff in oslo.i18n so projects aren't trying to use it?16:30
mriedemdhellmann: ^16:30
mriedemas far as i remember, ibm abandoned the lazy translation fad years ago16:30
mriedemedmondsw: ^16:30
mriedemi'd suggest we (1) neuter the enable_lazy function to be a noop and deprecate it so anyone using it can remove their usage, (2) release that and then (3) eventually drop the enable_lazy function16:32
*** e0ne_ has quit IRC16:36
openstackgerritDaniel Vincze proposed openstack/oslo.log master: Add Windows Event Log handler  https://review.openstack.org/61455816:47
*** openstackgerrit has quit IRC16:48
*** zaneb has quit IRC16:51
thervemriedem: I thought by importing "_" we needed lazy function16:53
bnemecmriedem: Hmm, I don't know that we can just noop that. Technically that would be a breaking change.16:53
mriedemhttps://bugs.launchpad.net/oslo.i18n/+bug/180176116:53
openstackLaunchpad bug 1801761 in oslo.i18n "enable_lazy should be deprecated/removed" [Undecided,New]16:53
bnemecCan we just str that text before we pass it to prettytable to force translation of it?16:53
mriedemsix.text_type rather than str16:54
mriedembut maybe16:54
bnemecYeah16:54
* bnemec was using str as lazy shorthand16:54
mriedemeither way that lazy translation stuff should be deprecated16:54
bnemecYeah, I'm fine with that if no one objects.16:54
mriedemto move these upgrade check patches forward in other projects i'm just going to burn it out16:54
mriedemif you want to six.text_type-ify the check results before concatenating them that's fine with me16:55
bnemecYeah, I'll verify that actually fixes the problem and push a patch if it does.16:56
dhellmannmriedem : the lazy stuff is for API responses; IBM was interested in log translation and we dropped that16:56
mriedemthe locale negotiation in api responses isn't supported either as far as i know16:56
dhellmannI don't object to deprecating it, I just want to make sure we're not mixing it up with something else16:56
mriedemin that it was abandoned years ago16:56
dhellmannok, well, then I guess we can drop all of that stuff16:57
mriedemftr, ibm pushed both of these16:57
dhellmannI wonder if we even need oslo.i18n any more16:57
dhellmannok, I the API thing was around before I got involved16:57
dhellmann*I think16:57
bnemecWe probably need to have this discussion on the operators list just to make sure someone didn't start caring about it in the meantime.16:57
bnemecI worked on this and I barely remember what it does. :-)16:58
bnemecI do remember it giving me headaches a lot.16:58
mriedembnemec: wouldn't it be awesome if your new ibm overlords made you care about this again lo these many years later?16:58
mriedemha yeah16:58
mriedemi remember spending a weekend debugging why nova with lazy enabled would randomly brick our CI16:58
mriedemit was great16:59
bnemecmriedem: I'm not dismissing any possibilities. ;-)16:59
bnemecYeah, I recall pushing patches to every project to disable lazy translation right before a release because it catastrophically broke something.16:59
bnemecGood times17:00
*** openstackgerrit has joined #openstack-oslo17:09
openstackgerritMerged openstack/oslo.policy master: Add ability to pass in target data for the oslopolicy-checker  https://review.openstack.org/61331317:09
dhellmannbnemec : I think we have a few folks good at managing those repetitive patches across projects now, so if you want to do a deprecation we can muster the troops after the summit17:13
openstackgerritBen Nemec proposed openstack/oslo.upgradecheck master: Force translation of table header  https://review.openstack.org/61561017:15
bnemecmriedem: ^ fixes it for me in the sample main.py17:15
bnemecdhellmann: Yep, sounds good.17:15
mriedembnemec: +2, thanks17:17
mriedemi'll send an email to the ops list17:40
bnemecmriedem: Thanks17:41
*** dave-mccowan has quit IRC17:49
*** zaneb has joined #openstack-oslo17:56
*** e0ne has joined #openstack-oslo17:57
*** phuongnh has joined #openstack-oslo18:16
*** phuongnh has quit IRC18:21
openstackgerritBen Nemec proposed openstack/oslo.upgradecheck master: Force translation of table header  https://review.openstack.org/61561018:38
bnemecNow with pep8 compatibility!18:38
bnemecttx: I'm thinking https://review.openstack.org/#/c/607951/ probably calls for a major release of rootwrap.18:46
bnemecIt is technically incompatible if anyone was relying on the fd limit inheritance behavior.18:47
bnemecAny objections?18:47
*** mriedem has left #openstack-oslo19:08
openstackgerritMerged openstack/oslo.upgradecheck master: Force translation of table header  https://review.openstack.org/61561019:30
*** mattgo has quit IRC19:40
*** shardy has quit IRC19:52
*** shardy has joined #openstack-oslo19:52
openstackgerritLance Bragstad proposed openstack/oslo.limit master: WIP: Expose enforcement API outside of ctx manager  https://review.openstack.org/61564320:27
cfriesenbnemec: dhellmann: I respun the fair locks oslo.concurrency change.20:33
bnemeccfriesen: Thanks, will take a look.20:43
openstackgerritMerged openstack/devstack-plugin-kafka master: Update to switch to confluent-kafka library for driver  https://review.openstack.org/60313420:43
*** e0ne has quit IRC20:45
*** cdent has left #openstack-oslo20:47
openstackgerritKen Giusti proposed openstack/oslo.messaging master: Use ensure_connection to prevent loss of connection error logs  https://review.openstack.org/61564921:07
*** raildo has quit IRC21:15
openstackgerritGage Hugo proposed openstack/oslo.cache master: WIP - memcache encryption  https://review.openstack.org/61566321:31
*** dhellmann has quit IRC21:44
cfriesenbnemec: whoops on spelling mistake.  should I fix it since it's got a +W?21:44
bnemeccfriesen: Nah, it's just a typo in a comment.21:45
bnemecCould push a followup, but I'm not even too concerned about doing that.21:45
bnemecI don't think anyone's going to misunderstand it because of one missing letter.21:45
*** mattgo has joined #openstack-oslo21:46
cfriesenworks.  someone can fix it when they touch the file next. :)   thanks21:46
bnemecYeah, it'll give someone a chance to bump their commit count somewhere down the line. :-)21:46
*** dhellman_ has joined #openstack-oslo21:49
*** dhellman_ is now known as dhellmann21:49
*** ansmith has quit IRC21:54
*** e0ne has joined #openstack-oslo22:02
*** e0ne has quit IRC22:05
*** bobh has quit IRC22:12
*** dhellmann_ has joined #openstack-oslo22:18
*** dhellmann has quit IRC22:18
*** bobh has joined #openstack-oslo22:19
*** dhellmann_ is now known as dhellmann22:20
*** bobh has quit IRC22:24
*** bobh has joined #openstack-oslo22:25
*** bobh has quit IRC22:29
sorrisonkgiusti: Are you around? Can talk about https://bugs.launchpad.net/oslo.messaging/+bug/180095722:57
openstackLaunchpad bug 1800957 in oslo.messaging "Upgrading to pike version causes rabbit timeouts with ssl" [Undecided,New] - Assigned to Ken Giusti (kgiusti)22:57
*** lbragstad has quit IRC23:09
*** lbragstad has joined #openstack-oslo23:10
openstackgerritMerged openstack/oslo.concurrency master: Add support for fair locks  https://review.openstack.org/47397323:22
*** mattgo has quit IRC23:30
*** kgiusti has left #openstack-oslo23:43
*** zaneb has quit IRC23:43

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