*** gildub has quit IRC | 00:11 | |
*** gildub has joined #openstack-sdks | 00:23 | |
*** adriant has quit IRC | 02:05 | |
*** adriant has joined #openstack-sdks | 02:14 | |
*** adriant has quit IRC | 02:15 | |
*** adriant has joined #openstack-sdks | 02:17 | |
*** adriant has quit IRC | 02:31 | |
*** gildub_ has joined #openstack-sdks | 02:38 | |
*** gildub has quit IRC | 02:41 | |
*** imacdonn has quit IRC | 02:50 | |
*** imacdonn has joined #openstack-sdks | 02:50 | |
*** gildub_ has quit IRC | 02:56 | |
*** adriant has joined #openstack-sdks | 03:07 | |
*** dave-mccowan has quit IRC | 03:44 | |
*** dave-mccowan has joined #openstack-sdks | 03:46 | |
openstackgerrit | LIU Yulong proposed openstack/openstacksdk master: Support firewall service for SDK https://review.openstack.org/592303 | 03:47 |
---|---|---|
*** gildub_ has joined #openstack-sdks | 04:05 | |
*** gildub_ has quit IRC | 04:14 | |
*** gildub has joined #openstack-sdks | 04:26 | |
*** e0ne has joined #openstack-sdks | 05:08 | |
*** dave-mccowan has quit IRC | 05:39 | |
*** e0ne has quit IRC | 05:49 | |
*** gildub_ has joined #openstack-sdks | 06:08 | |
*** gildub has quit IRC | 06:12 | |
*** olivierb has joined #openstack-sdks | 07:13 | |
*** olivierb has quit IRC | 07:17 | |
*** olivierb has joined #openstack-sdks | 07:18 | |
*** jpich has joined #openstack-sdks | 07:48 | |
openstackgerrit | Matthias Lisin proposed openstack/openstacksdk master: openstackcloud.py: Implement FWaaS wrapper methods. https://review.openstack.org/604811 | 07:56 |
*** gildub_ has quit IRC | 08:05 | |
*** e0ne has joined #openstack-sdks | 08:10 | |
*** gkadam has joined #openstack-sdks | 08:16 | |
*** e0ne has quit IRC | 08:20 | |
*** noama has joined #openstack-sdks | 08:25 | |
*** gildub_ has joined #openstack-sdks | 08:29 | |
*** gildub_ has quit IRC | 08:35 | |
*** olivier__ has joined #openstack-sdks | 08:46 | |
*** olivierb has quit IRC | 08:46 | |
*** tosky has joined #openstack-sdks | 08:48 | |
*** e0ne has joined #openstack-sdks | 09:04 | |
*** dtantsur|afk is now known as dtantsur | 09:10 | |
*** gildub_ has joined #openstack-sdks | 09:22 | |
*** gildub_ has quit IRC | 09:36 | |
*** a-pugachev has joined #openstack-sdks | 09:42 | |
*** ttsiouts has joined #openstack-sdks | 10:02 | |
*** ttsiouts has quit IRC | 10:24 | |
*** ttsiouts has joined #openstack-sdks | 10:25 | |
*** e0ne has quit IRC | 10:27 | |
*** ttsiouts has quit IRC | 10:29 | |
*** ttsiouts has joined #openstack-sdks | 10:30 | |
*** Luzi has joined #openstack-sdks | 10:42 | |
*** ttsiouts has quit IRC | 11:02 | |
openstackgerrit | Chen proposed openstack/python-openstackclient master: Use 2.latest compute microversion as default https://review.openstack.org/605398 | 11:13 |
*** olivier__ has quit IRC | 11:28 | |
*** olivierb has joined #openstack-sdks | 11:28 | |
*** pooja-jadhav is now known as pooja_jadhav | 11:33 | |
*** ttsiouts has joined #openstack-sdks | 11:49 | |
*** e0ne has joined #openstack-sdks | 11:53 | |
*** dtantsur is now known as dtantsur|brb | 12:18 | |
*** gildub_ has joined #openstack-sdks | 12:47 | |
*** gildub_ has quit IRC | 12:51 | |
*** ttsiouts has quit IRC | 12:55 | |
*** ttsiouts has joined #openstack-sdks | 13:03 | |
*** mriedem has joined #openstack-sdks | 13:03 | |
*** ttsiouts has quit IRC | 13:19 | |
*** dtantsur|brb is now known as dtantsur | 13:33 | |
openstackgerrit | Matthias Lisin proposed openstack/openstacksdk master: openstackcloud.py: Implement FWaaS wrapper methods. https://review.openstack.org/604811 | 13:34 |
*** lbragstad has quit IRC | 13:45 | |
*** slaweq has quit IRC | 13:45 | |
*** lbragstad has joined #openstack-sdks | 13:50 | |
*** ttsiouts has joined #openstack-sdks | 14:03 | |
*** bobh has joined #openstack-sdks | 14:08 | |
*** slaweq has joined #openstack-sdks | 14:11 | |
*** bobh has quit IRC | 14:13 | |
*** ttsiouts has quit IRC | 14:14 | |
*** slaweq has quit IRC | 14:15 | |
*** olivierb has quit IRC | 14:18 | |
*** ttsiouts has joined #openstack-sdks | 14:21 | |
*** olivierb has joined #openstack-sdks | 14:23 | |
*** Luzi has quit IRC | 14:27 | |
*** bobh has joined #openstack-sdks | 14:35 | |
*** bobh has quit IRC | 15:03 | |
openstackgerrit | Merged openstack-infra/shade master: Trim away the cover and py35 jobs https://review.openstack.org/605168 | 15:07 |
openstackgerrit | Merged openstack-infra/shade master: Remove the task manager https://review.openstack.org/605052 | 15:07 |
*** slaweq has joined #openstack-sdks | 15:11 | |
*** slaweq has quit IRC | 15:15 | |
*** bobh has joined #openstack-sdks | 15:17 | |
openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Clean up python3 test and remove duplicate jobs https://review.openstack.org/604517 | 15:50 |
*** dave-mccowan has joined #openstack-sdks | 15:54 | |
*** dave-mccowan has quit IRC | 15:57 | |
*** dtantsur is now known as dtantsur|afk | 16:08 | |
*** Leo_m has joined #openstack-sdks | 16:10 | |
*** noama has quit IRC | 16:10 | |
*** jpich has quit IRC | 16:22 | |
*** a-pugachev has quit IRC | 16:23 | |
*** gkadam has quit IRC | 16:25 | |
*** ttsiouts has quit IRC | 16:35 | |
*** e0ne has quit IRC | 16:43 | |
*** olivierb has quit IRC | 16:54 | |
*** slaweq has joined #openstack-sdks | 17:11 | |
*** slaweq has quit IRC | 17:15 | |
openstackgerrit | Merged openstack/openstacksdk master: Restore timeout_scaling_factor https://review.openstack.org/604628 | 17:18 |
*** slaweq has joined #openstack-sdks | 18:12 | |
*** slaweq has quit IRC | 18:22 | |
*** lbragstad has quit IRC | 18:24 | |
*** lbragstad has joined #openstack-sdks | 18:25 | |
openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Remove profile https://review.openstack.org/604414 | 18:47 |
openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Set endpoint_override from endpoint with noauth https://review.openstack.org/604415 | 18:47 |
openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Use discovery instead of config to create proxies https://review.openstack.org/600292 | 18:47 |
openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Remove api version default values https://review.openstack.org/600330 | 18:47 |
openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Special case network and use it in openstack.cloud https://review.openstack.org/604645 | 18:47 |
openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Remove all the deprecated stuff https://review.openstack.org/605508 | 18:47 |
openstackgerrit | Merged openstack/openstacksdk master: Update vendor support info for catalyst https://review.openstack.org/605245 | 18:56 |
*** dave-mccowan has joined #openstack-sdks | 19:05 | |
openstackgerrit | Merged openstack/openstacksdk master: Update vendor support info for ecs https://review.openstack.org/605248 | 19:05 |
openstackgerrit | Merged openstack/openstacksdk master: Update vendor support info for switchengines https://review.openstack.org/605249 | 19:05 |
openstackgerrit | Merged openstack/openstacksdk master: Update vendor support info for vexxhost https://review.openstack.org/605252 | 19:05 |
openstackgerrit | Merged openstack/openstacksdk master: Add compute API info and fix provider names https://review.openstack.org/605253 | 19:05 |
openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Replace TaskManager with a semaphore and lock https://review.openstack.org/604926 | 19:08 |
mordred | timburke: 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 tests | 19:08 |
timburke | no 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 |
timburke | tests 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 order | 19:10 |
timburke | (er, not "out of order" exactly... "interleaved") | 19:10 |
*** slaweq has joined #openstack-sdks | 19:11 | |
mordred | the most fun code are the classes where there are more comments and test than code :) | 19:11 |
mordred | ++ yeah - interleaved, blocked, etc | 19:11 |
timburke | fwiw, 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 |
mordred | all the eventlet/gevent stuff totally blows my mind | 19:13 |
mordred | timburke: 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 IRC | 19:15 | |
*** e0ne has joined #openstack-sdks | 19:15 | |
timburke | probably? 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, though | 19:16 |
mordred | that's a good point | 19: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 |
mordred | https://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 not | 19:19 |
timburke | oh... 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 |
openstack | Launchpad 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 |
timburke | i 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 themselves | 19:21 |
mordred | yes. I agree with both of those | 19:21 |
Shrews | hrm, this entire conversation leads to wonder if we should leave well enough alone with the TaskManager :) | 19:21 |
mordred | BUT - that bug does indicate that eventlet patches out threading.RLock | 19:22 |
mordred | so as long as the calling program is all threads or all mutexes life should be fine | 19:22 |
mordred | Shrews: 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 ourselves | 19:23 |
mordred | Shrews: OR - we could just write the semaphore class in C++ | 19:23 |
timburke | yep, 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#L535 | 19:24 |
*** slaweq has joined #openstack-sdks | 19:24 | |
mordred | cool. I'll probably need that to do an eventlet test case | 19:29 |
*** e0ne has quit IRC | 19:30 | |
*** gildub_ has joined #openstack-sdks | 20:49 | |
*** bobh has quit IRC | 20:59 | |
*** gildub_ has quit IRC | 20:59 | |
openstackgerrit | Samuel de Medeiros Queiroz proposed openstack/openstacksdk master: Format URL when updating image props in Glance v1 https://review.openstack.org/605534 | 21:06 |
*** slaweq has quit IRC | 21:06 | |
*** slaweq has joined #openstack-sdks | 21:11 | |
*** slaweq has quit IRC | 21:15 | |
openstackgerrit | Samuel de Medeiros Queiroz proposed openstack/openstacksdk master: Send image fields in headers for PUT in Glance v1 https://review.openstack.org/605535 | 21:20 |
*** bobh has joined #openstack-sdks | 21:35 | |
*** bobh has quit IRC | 21:40 | |
*** slaweq has joined #openstack-sdks | 22:11 | |
*** slaweq has quit IRC | 22:16 | |
openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Clarify error message is from nova https://review.openstack.org/605544 | 22:28 |
*** tosky has quit IRC | 22:37 | |
*** Leo_m has quit IRC | 22:47 | |
*** mriedem is now known as mriedem_away | 23:30 | |
*** openstackgerrit has quit IRC | 23:49 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!