Wednesday, 2018-11-07

*** slaweq has quit IRC00:12
*** bobh has joined #openstack-oslo00:12
*** slaweq has joined #openstack-oslo00:13
*** bobh has quit IRC00:17
*** phuongnh has joined #openstack-oslo00:28
*** irclogbot_2 has quit IRC00:39
*** cfriesen has quit IRC02:06
*** threestrands has joined #openstack-oslo02:30
*** dave-mccowan has joined #openstack-oslo03:24
*** dave-mccowan has quit IRC03:36
*** phuongnh has quit IRC05:00
*** phuongnh has joined #openstack-oslo05:00
*** d0ugal has quit IRC06:05
*** d0ugal has joined #openstack-oslo06:21
*** threestrands has quit IRC06:23
*** phuongnh has quit IRC06:32
*** phuongnh has joined #openstack-oslo06:33
*** andreaf has quit IRC06:53
*** andreaf has joined #openstack-oslo06:55
*** njohnston_ has quit IRC07:01
*** njohnston has joined #openstack-oslo07:02
*** njohnston has quit IRC07:14
*** njohnston has joined #openstack-oslo07:26
*** pcaruana has joined #openstack-oslo07:36
*** e0ne has joined #openstack-oslo07:49
*** lpetrut has joined #openstack-oslo08:20
*** dims has quit IRC08:52
*** dims has joined #openstack-oslo08:53
openstackgerritwangqi proposed openstack/oslo.vmware master: delete useless indent  https://review.openstack.org/61611008:53
openstackgerritwangqi proposed openstack/oslosphinx master: Title underline too short  https://review.openstack.org/61611108:55
openstackgerritwangqi proposed openstack/oslotest master: Title underline too long  https://review.openstack.org/61611208:56
*** dims has quit IRC08:58
*** dims has joined #openstack-oslo08:59
*** shardy has joined #openstack-oslo09:19
*** ttx has quit IRC09:37
*** ttx has joined #openstack-oslo09:50
*** cdent has joined #openstack-oslo09:58
*** e0ne has quit IRC10:02
*** e0ne has joined #openstack-oslo10:04
*** phuongnh has quit IRC10:10
*** e0ne has quit IRC10:45
*** e0ne has joined #openstack-oslo10:54
*** gokhan has joined #openstack-oslo11:02
*** cdent has quit IRC11:40
*** e0ne has quit IRC12:04
*** e0ne has joined #openstack-oslo12:08
*** e0ne has quit IRC12:09
*** pbourke has quit IRC12:11
*** pbourke has joined #openstack-oslo12:11
openstackgerritMerged openstack/oslotest master: Title underline too long  https://review.openstack.org/61611212:20
*** njohnston has quit IRC12:21
openstackgerritMerged openstack/oslosphinx master: Title underline too short  https://review.openstack.org/61611112:21
*** njohnston has joined #openstack-oslo12:21
*** snapiri has joined #openstack-oslo12:22
*** raildo has joined #openstack-oslo12:24
*** e0ne has joined #openstack-oslo12:26
*** cdent has joined #openstack-oslo12:51
*** ansmith has joined #openstack-oslo13:30
*** cdent has quit IRC13:37
*** e0ne has quit IRC13:41
*** bobh has joined #openstack-oslo13:42
*** pcaruana has quit IRC13:45
*** e0ne has joined #openstack-oslo13:45
*** kgiusti has joined #openstack-oslo13:50
*** pcaruana has joined #openstack-oslo14:00
*** e0ne has quit IRC14:13
*** e0ne has joined #openstack-oslo14:14
*** pcaruana has quit IRC14:33
*** pcaruana has joined #openstack-oslo14:34
*** cdent has joined #openstack-oslo14:36
*** bobh has quit IRC14:42
*** bobh has joined #openstack-oslo14:50
*** cfriesen has joined #openstack-oslo15:02
openstackgerritEric Fried proposed openstack/oslo.service master: Limit monotonic to py2  https://review.openstack.org/61601315:34
openstackgerritEric Fried proposed openstack/oslo.service master: Fixture to mock loopingcall wait()  https://review.openstack.org/61597815:34
efriedzaneb, bnemec, mordred: ^^ Still kind of fish-out-of-water here, but this passes locally...15:38
zanebefried: that lgtm15:40
efriedcool, thx15:40
mordredefried: that looks great! (I don't have +2 in that repo, but +1 from me)16:05
efriedmordred: Thanks.16:05
bnemecefried: Thanks for jumping on this!16:13
efriedyahyoubetcha16:13
*** lpetrut has quit IRC16:14
efriedbnemec, dhellmann: I noticed there's tests in oslo.service that incur real sleeps. That may be necessary/expected/intentional - I didn't look into it too far - but could also be places where we would want to use the SleepFixture to cut the test wallclock time down.16:20
bnemecefried: Yeah, I suspect some of that may be needed because we're waiting for test services to start, but it's certainly something to look into.16:22
efriedbnemec: I'll noodle around with it a little bit here16:24
openstackgerritMerged openstack/oslo.vmware master: delete useless indent  https://review.openstack.org/61611016:24
bnemecThanks16:24
*** pcaruana has quit IRC16:38
*** ducnv_ has joined #openstack-oslo16:40
*** moguimar has quit IRC16:41
*** ducnv has quit IRC16:42
*** e0ne has quit IRC16:44
*** e0ne has joined #openstack-oslo16:44
*** e0ne has quit IRC16:44
*** shardy has quit IRC16:46
*** efried is now known as efried_rollin17:08
*** bobh has quit IRC17:32
*** e0ne has joined #openstack-oslo18:36
*** e0ne has quit IRC18:38
*** irclogbot_2 has joined #openstack-oslo18:45
*** kmalloc is now known as needscoffee19:05
*** bobh has joined #openstack-oslo19:06
*** bobh has quit IRC19:07
*** e0ne has joined #openstack-oslo19:24
*** e0ne has quit IRC19:31
openstackgerritMerged openstack/oslo.service stable/rocky: Use eventlet Event for loopingcall events  https://review.openstack.org/61448920:17
*** efried_rollin is now known as efried20:18
*** needscoffee is now known as kmalloc20:19
*** jungleboyj has quit IRC20:47
*** hogepodge has quit IRC20:47
*** hogepodge has joined #openstack-oslo20:48
*** kgiusti has left #openstack-oslo20:50
*** andreaf has quit IRC20:51
*** andreaf has joined #openstack-oslo20:52
openstackgerritMorgan Fainberg proposed openstack/oslo.cache master: Add pymemcache backend  https://review.openstack.org/61630420:59
kmallocbnemec, zzzeek: ^20:59
kmallocpymemcache base implementation21:00
kmalloci'll work to submit the changes to dogpile.cache as well directly21:00
cdent++21:00
kmallocthe intention is to replace python-memcached with pymemcache21:00
kmallocbut i figure we might want to have this in oslo_cache quickly-ish21:00
kmalloccdent: best part is... it has native memcache_pooling built-in21:00
kmalloccdent: i really want to expose .add, .cas, .gets, etc as well, but that is  hard to do via dogpile since it has to cater to lowest-common support.21:01
cdentyeah. bit of a bummer that but good reasons21:01
kmallocbut holy crap, pymemcache is a dream to read/work with compared to python-memcached21:02
kmallocit's like... the pinterest folks are maintaining it or something21:02
kmalloc:P21:02
*** raildo has quit IRC21:02
kmallocand the BEST part is we can drop the thread.local bits if pooling is used21:03
kmallocjust baked in21:03
cdentmaintained code? NO WAY21:03
kmalloc:)21:03
kmallocikr?!21:03
bnemecPfft, sounds boring. :-P21:03
kmallocthere is no dirty hacking needed to support pooled connections *omg*21:03
kmallocanyway, please comment away, i'll take any/all comments and submit for zzzeek to dogpile directly21:04
kmallocand hopefully we can just deprecate memcache_pool in oslo_cache :)21:04
cdentI've released Paste 5 times in the two weeks since taking over. 1 to get it out there. Again to fix a mistake and then 3 more times because people came along with a backlog of fixes.21:04
kmalloccdent: i'm so sorry i owe you my paste-code soon21:04
kmallocthis took some priority because i broke people trying to fix oslo_cache :(21:04
kmalloci have my oslo_middleware loader bits started locally. should be able to hack it together by the end of the summit21:05
kmalloccdent: i am somewhat sad you've had to take paste over fwiw.21:05
kmallocs/somewhat/very21:05
cdenti don't reckon that paste-code thing is urgent, but will be very useful21:06
kmallocit's super small. i just need to work out how ordering will work21:07
kmallocso we are flexible enough if folks want to opt in to "external loading"21:07
cdentsomebody had to take over Paste, and I've become something of a home for old tired web-related things (wsgi-intercept is the other biggie)21:07
kmallocthe hard part is people shove config into paste-ini :(21:07
kmalloc... fwiw, i tried to take over python-memcached21:07
kmallocthe owner just stopped talking to me after i offered and he agreed21:07
bnemecShut up and take my money^W project!21:08
kmalloci didn't say it at the time, but i was just going to make it a wrapper for pymemcache so folks could convert21:08
cdentwoot21:08
cdentopen source is rather challenging21:08
* kmalloc needs to spend some serious time helping clean up dogpile.cache21:08
kmallocit needs some love to become a little easier to use21:09
kmallocthe interfaces are still super rough21:09
cdentI think there will be histories that look back on some of this stuff as exploitation21:09
kmalloclike... oslo_cache fixes SO much stuff (except it also has gross things in it too)21:09
kmallocthe best part of pymemcache, i can migrate to msgpack and away from pickle21:09
kmallocso we can stop ... being well... dangerously insecure in our memcache object loading21:10
kmalloci cringe if someone gets access to a memcache server in openstack. I can add HMAC checking and such for the objects, but pickle is so very dangerous21:10
bnemecI've actually wondered if we should treat that driver as a security vulnerability.21:11
kmallocbnemec: responded to your comment, but i want eyes on the general design before making moves to replace dogpile python-memcached one with this21:12
kmalloci am hoping to just forklift the work over there and drop python-memcached as a backend21:13
kmallocbut we have significantly more eyes here than there.21:13
bnemeckmalloc: Ah, gotcha. I thought we were going to merge it to oslo.cache first, but that works for me.21:13
kmallocnah, thinking about it, nope21:14
kmallocand i think it really should be a hard dep at this point21:14
kmallocregardless.21:14
kmalloci keep fielding trhe bug "why is this in extras" :P21:14
bnemecYeah, but before that we kept fielding the bug "why do I have to install qpid when I use rabbit". ;-)21:15
kmallocyeah. this case it's a pure python module21:15
kmallocbut i also don't feel bad about making pymemcache a hard dep of keystone21:15
kmallocand ksm21:15
kmalloc:P21:15
kmalloci feel bad about making python-memcache a hard dep21:16
cdentthat's like the 97th time you've said "I feel bad" in the last few minutes. Have you considered a new career?21:18
kmallocHAhahaha21:18
kmallocmotivational support for folks? :P21:18
cdent:)21:19
* cdent packs for berlin21:19
cdentgoodnight all21:19
*** cdent has quit IRC21:19
bnemecMorgan the motivational speaker: "Your code is bad and you should feel bad". :-)21:19
kmallochehehe21:20
kmallocbnemec: no no, it's "Your code is fine, but mine is bad, and I feel bad about it. Therefore you should be happier with your code"21:21
kmallocsee motivational!21:21
* bnemec is motivated!21:21
* kmalloc forgot to get flu shot.21:21
kmallocI guess 4 days before flight is better than day-before21:22
*** ansmith has quit IRC21:37
openstackgerritMerged openstack/oslo.service master: Limit monotonic to py2  https://review.openstack.org/61601321:47
openstackgerritMerged openstack/oslo.service master: Fixture to mock loopingcall wait()  https://review.openstack.org/61597821:47
*** bobh has joined #openstack-oslo21:48
*** bobh has quit IRC21:51
*** jungleboyj has joined #openstack-oslo21:59
efriedbnemec: Thanks for releasing oslo.service, that was fast. Does a bot still do the requirements u-c bump or do I get to do it by hand?22:48
*** achentir has joined #openstack-oslo22:49
efrieddhellmann: --^22:49
dhellmannefried : the automation will take care of that22:51
efrieddhellmann: cool, thx. I now see https://review.openstack.org/#/c/615676/ which is alllmost merged.22:51
efried(for the previous release)22:52
efriedmelwitt: FYI ^22:52
*** achentir has quit IRC22:55
*** slaweq has quit IRC23:01
openstackgerritEric Fried proposed openstack/oslo.service master: Use SleepFixture in looping call test suite  https://review.openstack.org/61637323:23
efriedbnemec: There ya go ^23:23
efriedpared off 12s wallclock time, 14s execute time.23:24
efriedDidn't work too hard on that test_looping_call_timed_out one, but it didn't respond to the fixture.23:25
*** dhellmann has quit IRC23:45

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