Wednesday, 2018-09-26

*** gildub has quit IRC00:11
*** gildub has joined #openstack-sdks00:23
*** adriant has quit IRC02:05
*** adriant has joined #openstack-sdks02:14
*** adriant has quit IRC02:15
*** adriant has joined #openstack-sdks02:17
*** adriant has quit IRC02:31
*** gildub_ has joined #openstack-sdks02:38
*** gildub has quit IRC02:41
*** imacdonn has quit IRC02:50
*** imacdonn has joined #openstack-sdks02:50
*** gildub_ has quit IRC02:56
*** adriant has joined #openstack-sdks03:07
*** dave-mccowan has quit IRC03:44
*** dave-mccowan has joined #openstack-sdks03:46
openstackgerritLIU Yulong proposed openstack/openstacksdk master: Support firewall service for SDK  https://review.openstack.org/59230303:47
*** gildub_ has joined #openstack-sdks04:05
*** gildub_ has quit IRC04:14
*** gildub has joined #openstack-sdks04:26
*** e0ne has joined #openstack-sdks05:08
*** dave-mccowan has quit IRC05:39
*** e0ne has quit IRC05:49
*** gildub_ has joined #openstack-sdks06:08
*** gildub has quit IRC06:12
*** olivierb has joined #openstack-sdks07:13
*** olivierb has quit IRC07:17
*** olivierb has joined #openstack-sdks07:18
*** jpich has joined #openstack-sdks07:48
openstackgerritMatthias Lisin proposed openstack/openstacksdk master: openstackcloud.py: Implement FWaaS wrapper methods.  https://review.openstack.org/60481107:56
*** gildub_ has quit IRC08:05
*** e0ne has joined #openstack-sdks08:10
*** gkadam has joined #openstack-sdks08:16
*** e0ne has quit IRC08:20
*** noama has joined #openstack-sdks08:25
*** gildub_ has joined #openstack-sdks08:29
*** gildub_ has quit IRC08:35
*** olivier__ has joined #openstack-sdks08:46
*** olivierb has quit IRC08:46
*** tosky has joined #openstack-sdks08:48
*** e0ne has joined #openstack-sdks09:04
*** dtantsur|afk is now known as dtantsur09:10
*** gildub_ has joined #openstack-sdks09:22
*** gildub_ has quit IRC09:36
*** a-pugachev has joined #openstack-sdks09:42
*** ttsiouts has joined #openstack-sdks10:02
*** ttsiouts has quit IRC10:24
*** ttsiouts has joined #openstack-sdks10:25
*** e0ne has quit IRC10:27
*** ttsiouts has quit IRC10:29
*** ttsiouts has joined #openstack-sdks10:30
*** Luzi has joined #openstack-sdks10:42
*** ttsiouts has quit IRC11:02
openstackgerritChen proposed openstack/python-openstackclient master: Use 2.latest compute microversion as default  https://review.openstack.org/60539811:13
*** olivier__ has quit IRC11:28
*** olivierb has joined #openstack-sdks11:28
*** pooja-jadhav is now known as pooja_jadhav11:33
*** ttsiouts has joined #openstack-sdks11:49
*** e0ne has joined #openstack-sdks11:53
*** dtantsur is now known as dtantsur|brb12:18
*** gildub_ has joined #openstack-sdks12:47
*** gildub_ has quit IRC12:51
*** ttsiouts has quit IRC12:55
*** ttsiouts has joined #openstack-sdks13:03
*** mriedem has joined #openstack-sdks13:03
*** ttsiouts has quit IRC13:19
*** dtantsur|brb is now known as dtantsur13:33
openstackgerritMatthias Lisin proposed openstack/openstacksdk master: openstackcloud.py: Implement FWaaS wrapper methods.  https://review.openstack.org/60481113:34
*** lbragstad has quit IRC13:45
*** slaweq has quit IRC13:45
*** lbragstad has joined #openstack-sdks13:50
*** ttsiouts has joined #openstack-sdks14:03
*** bobh has joined #openstack-sdks14:08
*** slaweq has joined #openstack-sdks14:11
*** bobh has quit IRC14:13
*** ttsiouts has quit IRC14:14
*** slaweq has quit IRC14:15
*** olivierb has quit IRC14:18
*** ttsiouts has joined #openstack-sdks14:21
*** olivierb has joined #openstack-sdks14:23
*** Luzi has quit IRC14:27
*** bobh has joined #openstack-sdks14:35
*** bobh has quit IRC15:03
openstackgerritMerged openstack-infra/shade master: Trim away the cover and py35 jobs  https://review.openstack.org/60516815:07
openstackgerritMerged openstack-infra/shade master: Remove the task manager  https://review.openstack.org/60505215:07
*** slaweq has joined #openstack-sdks15:11
*** slaweq has quit IRC15:15
*** bobh has joined #openstack-sdks15:17
openstackgerritMonty Taylor proposed openstack/openstacksdk master: Clean up python3 test and remove duplicate jobs  https://review.openstack.org/60451715:50
*** dave-mccowan has joined #openstack-sdks15:54
*** dave-mccowan has quit IRC15:57
*** dtantsur is now known as dtantsur|afk16:08
*** Leo_m has joined #openstack-sdks16:10
*** noama has quit IRC16:10
*** jpich has quit IRC16:22
*** a-pugachev has quit IRC16:23
*** gkadam has quit IRC16:25
*** ttsiouts has quit IRC16:35
*** e0ne has quit IRC16:43
*** olivierb has quit IRC16:54
*** slaweq has joined #openstack-sdks17:11
*** slaweq has quit IRC17:15
openstackgerritMerged openstack/openstacksdk master: Restore timeout_scaling_factor  https://review.openstack.org/60462817:18
*** slaweq has joined #openstack-sdks18:12
*** slaweq has quit IRC18:22
*** lbragstad has quit IRC18:24
*** lbragstad has joined #openstack-sdks18:25
openstackgerritMonty Taylor proposed openstack/openstacksdk master: Remove profile  https://review.openstack.org/60441418:47
openstackgerritMonty Taylor proposed openstack/openstacksdk master: Set endpoint_override from endpoint with noauth  https://review.openstack.org/60441518:47
openstackgerritMonty Taylor proposed openstack/openstacksdk master: Use discovery instead of config to create proxies  https://review.openstack.org/60029218:47
openstackgerritMonty Taylor proposed openstack/openstacksdk master: Remove api version default values  https://review.openstack.org/60033018:47
openstackgerritMonty Taylor proposed openstack/openstacksdk master: Special case network and use it in openstack.cloud  https://review.openstack.org/60464518:47
openstackgerritMonty Taylor proposed openstack/openstacksdk master: Remove all the deprecated stuff  https://review.openstack.org/60550818:47
openstackgerritMerged openstack/openstacksdk master: Update vendor support info for catalyst  https://review.openstack.org/60524518:56
*** dave-mccowan has joined #openstack-sdks19:05
openstackgerritMerged openstack/openstacksdk master: Update vendor support info for ecs  https://review.openstack.org/60524819:05
openstackgerritMerged openstack/openstacksdk master: Update vendor support info for switchengines  https://review.openstack.org/60524919:05
openstackgerritMerged openstack/openstacksdk master: Update vendor support info for vexxhost  https://review.openstack.org/60525219:05
openstackgerritMerged openstack/openstacksdk master: Add compute API info and fix provider names  https://review.openstack.org/60525319:05
openstackgerritMonty Taylor proposed openstack/openstacksdk master: Replace TaskManager with a semaphore and lock  https://review.openstack.org/60492619:08
mordredtimburke: thanks for the review on that ^^ I think that should fix the logic error - but I agree, I need to write a big pile of tests19:08
timburkeno problem. i remember an awful lot of staring and thinking while reviewing when we added a concurrency primitive to Swift (https://review.openstack.org/#/c/493636/)19:09
timburketests are going to be invaluable, but we'll still need to think a whole lot about all the possible ways things could get out of order19:10
timburke(er, not "out of order" exactly... "interleaved")19:10
*** slaweq has joined #openstack-sdks19:11
mordredthe most fun code are the classes where there are more comments and test than code :)19:11
mordred++ yeah - interleaved, blocked, etc19:11
timburkefwiw, a co-worker of mine wrote https://github.com/swiftstack/python-stack-xray/blob/master/python-stack-xray that's proven invaluable when i suspect that i've hit a deadlock (particularly if eventlet is involved, too)19:12
mordredall the eventlet/gevent stuff totally blows my mind19:13
mordredtimburke: seeing as how you have more eventlet experience than I do - my assumption with the semaphore/mutex code above is that it should work fine for eventlet callers too, right?19:15
mordred(should probaby make an eventlet test case to test that out)19:15
*** slaweq has quit IRC19:15
*** e0ne has joined #openstack-sdks19:15
timburkeprobably? i'd need to think on it a bit more. i suspect it'll hurt if eventlet hasn't patched out threading and queue yet, though19:16
mordredthat's a good point19:17
timburke(ie, one greenthread would wait on the semaphore and block the hub, so no other greenthreads in that os thread would be able to execute)19:17
mordredhttps://eventlet.net/doc/modules/queue.html and https://eventlet.net/doc/modules/semaphore.html exist at least - but I don't know if that means they monkeypatch those or not19:19
timburkeoh... but if you've got a mix of greenthreads and os threads... you might get into a situation like https://bugs.launchpad.net/swift/+bug/1710328 ...19:19
openstackLaunchpad bug 1710328 in OpenStack Object Storage (swift) "object server deadlocks when a worker thread logs something" [High,Fix released] - Assigned to Samuel Merritt (torgomatic)19:19
timburkei think the logic generally goes like this though: (1) don't mix greenthreads and os threads unless you know what you're doing. (2) no one knows what they're doing, and those that think they do are fooling themselves19:21
mordredyes. I agree with both of those19:21
Shrewshrm, this entire conversation leads to wonder if we should leave well enough alone with the TaskManager  :)19:21
mordredBUT - that bug does indicate that eventlet patches out threading.RLock19:22
mordredso as long as the calling program is all threads or all mutexes life should be fine19:22
mordredShrews: I think we're fine - we're not spawning threads or eventlets ourselves - calling code _should_ be fine from either calling convention, and we're not adding threads to the mix ourselves19:23
mordredShrews: OR - we could just write the semaphore class in C++19:23
timburkeyep, for sure eventlet can patch out threading. just gotta do something like https://github.com/openstack/swift/blob/2.19.0/swift/common/utils.py#L53519:24
*** slaweq has joined #openstack-sdks19:24
mordredcool. I'll probably need that to do an eventlet test case19:29
*** e0ne has quit IRC19:30
*** gildub_ has joined #openstack-sdks20:49
*** bobh has quit IRC20:59
*** gildub_ has quit IRC20:59
openstackgerritSamuel de Medeiros Queiroz proposed openstack/openstacksdk master: Format URL when updating image props in Glance v1  https://review.openstack.org/60553421:06
*** slaweq has quit IRC21:06
*** slaweq has joined #openstack-sdks21:11
*** slaweq has quit IRC21:15
openstackgerritSamuel de Medeiros Queiroz proposed openstack/openstacksdk master: Send image fields in headers for PUT in Glance v1  https://review.openstack.org/60553521:20
*** bobh has joined #openstack-sdks21:35
*** bobh has quit IRC21:40
*** slaweq has joined #openstack-sdks22:11
*** slaweq has quit IRC22:16
openstackgerritMonty Taylor proposed openstack/openstacksdk master: Clarify error message is from nova  https://review.openstack.org/60554422:28
*** tosky has quit IRC22:37
*** Leo_m has quit IRC22:47
*** mriedem is now known as mriedem_away23:30
*** openstackgerrit has quit IRC23:49

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