jamielennox | lhcheng the problem is knowing everywhere its in use and knowing the code base week enough to fix it. My current plan is to have a review that deletes the old stuff and see what fails | 00:00 |
---|---|---|
harlowja | then clients can verify the signature themselves, and check the expiry and reuse the tokens ... | 00:00 |
*** stevemar_ has joined #openstack-oslo | 00:00 | |
harlowja | doesn't seem so complicated :-P | 00:00 |
lhcheng | harlowja: we need only the expiry, keep the service token in cache for some time (expiry - seconds buffer). | 00:03 |
harlowja | right | 00:04 |
lhcheng | then make the client smart enough to refetch the token if its expired | 00:04 |
*** stevemar_ has quit IRC | 00:04 | |
harlowja | yup | 00:04 |
harlowja | shitty part is u need to not have clients pass around tokens in there apis | 00:05 |
harlowja | u need clients to say do this operation, no token provided, and the internals of the library/client do the token fetching/refetching... | 00:05 |
lhcheng | harlowja: as jamielennox mentioned, the challenge is finding all the code doing this and applying the logic. | 00:05 |
harlowja | yup | 00:05 |
harlowja | its a shitty one to apply post library/client creation | 00:05 |
lhcheng | jamielennox: I wonder if that logic can be built-in into the session or keystoneclient object? | 00:06 |
jamielennox | lhcheng it is :) that's largely the point of sessions | 00:06 |
lhcheng | session only caches the request connection right, not the token? | 00:07 |
jamielennox | Thats essentially the reason we need a session and not just plug the auth plugins directly into the clients so we can do things like smart reauthentication | 00:08 |
jamielennox | Well the plugin caches the token and handles expiry | 00:08 |
jamielennox | The session let's is so things like reauth on 401 | 00:08 |
jamielennox | Us do | 00:08 |
jamielennox | And we are trying to make it so that services only ever have to deal with plugins, never touching the token directly | 00:10 |
*** krotscheck has quit IRC | 00:11 | |
lhcheng | ah aside from auth_plugin managing the credential it does the token caching! | 00:12 |
lhcheng | okay, now it clearer now (the relationship between session and auth_plugin) | 00:12 |
lhcheng | jamielennox: theorically, I can follow the pattern in : https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L211-L253 | 00:13 |
lhcheng | and apply it for ironic call | 00:13 |
jamielennox | lhcheng you shouldn't need to | 00:14 |
jamielennox | One of the last things in liberty ksc was locking around the token fetch | 00:15 |
harlowja | people pass around a session into every client api call? | 00:15 |
jamielennox | So you should be able to just store the service token globally and reuse across threads | 00:15 |
jamielennox | Don't need to do the token_endpoint hack any more | 00:16 |
jamielennox | harlowja people create a client with a session, every api call uses that session | 00:16 |
jamielennox | Passing for every call would work just fine but it's a much bigger change from existing clients than people would accept | 00:17 |
harlowja | right, i guess there is the auth plugin to clients that should know how to use that session and update it as needed | 00:17 |
jamielennox | But on the plus side, because the clients no longer do their own auth creating client objects is cheap | 00:18 |
harlowja | cool | 00:19 |
jamielennox | Close enough, session uses the plugin - but Same end result | 00:19 |
*** krotscheck has joined #openstack-oslo | 00:19 | |
*** achanda has quit IRC | 00:23 | |
lhcheng | jamielennox: ah right, the locking thing on the nova-neutron code can be removed | 00:24 |
*** achanda has joined #openstack-oslo | 00:26 | |
openstackgerrit | Joshua Harlow proposed openstack/taskflow: Add a executor backed conductor and have existing impl. use it https://review.openstack.org/174643 | 00:33 |
*** jecarey has joined #openstack-oslo | 00:42 | |
openstackgerrit | Joshua Harlow proposed openstack/taskflow: Add a executor backed conductor and have existing impl. use it https://review.openstack.org/174643 | 00:44 |
*** jecarey has quit IRC | 00:46 | |
openstackgerrit | Joshua Harlow proposed openstack/taskflow: Use conductor entity class constant instead of raw string https://review.openstack.org/237831 | 00:48 |
openstackgerrit | Joshua Harlow proposed openstack/taskflow: Add a executor backed conductor and have existing impl. use it https://review.openstack.org/174643 | 00:49 |
openstackgerrit | Joshua Harlow proposed openstack/taskflow: Add a executor backed conductor and have existing impl. use it https://review.openstack.org/174643 | 00:51 |
openstackgerrit | Joshua Harlow proposed openstack/taskflow: Use conductor entity class constant instead of raw string https://review.openstack.org/237831 | 00:52 |
*** boris-42 has quit IRC | 00:58 | |
*** kbyrne has quit IRC | 01:14 | |
*** kbyrne has joined #openstack-oslo | 01:15 | |
*** itisha has quit IRC | 01:31 | |
*** salv-orlando has joined #openstack-oslo | 01:32 | |
*** Fdaisuke has joined #openstack-oslo | 01:32 | |
*** dims has quit IRC | 01:34 | |
*** salv-orlando has quit IRC | 01:46 | |
*** lhcheng has quit IRC | 01:59 | |
*** stevemar_ has joined #openstack-oslo | 02:02 | |
*** browne has quit IRC | 02:14 | |
*** browne has joined #openstack-oslo | 02:15 | |
*** browne has quit IRC | 02:18 | |
*** jecarey has joined #openstack-oslo | 02:19 | |
*** boris-42 has joined #openstack-oslo | 02:31 | |
*** yamahata has quit IRC | 02:32 | |
*** achanda has quit IRC | 02:53 | |
*** achanda has joined #openstack-oslo | 02:56 | |
*** achanda has quit IRC | 02:57 | |
*** morgan has quit IRC | 03:00 | |
*** waynr has quit IRC | 03:13 | |
*** salv-orlando has joined #openstack-oslo | 03:45 | |
openstackgerrit | Lin Hua Cheng proposed openstack/taskflow: Move dispatch and consume/abandon to helper function https://review.openstack.org/235735 | 03:49 |
openstackgerrit | Lin Hua Cheng proposed openstack/taskflow: Move dispatch and consume/abandon to helper function https://review.openstack.org/235735 | 03:50 |
*** salv-orlando has quit IRC | 03:55 | |
*** rpodolyaka1 has joined #openstack-oslo | 04:04 | |
*** links has joined #openstack-oslo | 04:11 | |
*** rpodolyaka1 has quit IRC | 04:15 | |
*** rpodolyaka1 has joined #openstack-oslo | 04:18 | |
*** achanda has joined #openstack-oslo | 04:27 | |
*** rpodolyaka1 has quit IRC | 04:32 | |
*** jecarey_ has joined #openstack-oslo | 04:32 | |
*** achanda has quit IRC | 04:32 | |
*** jecarey has quit IRC | 04:34 | |
*** achanda has joined #openstack-oslo | 04:35 | |
*** rpodolyaka1 has joined #openstack-oslo | 04:36 | |
*** salv-orlando has joined #openstack-oslo | 05:13 | |
*** openstackgerrit has quit IRC | 05:16 | |
*** morgan has joined #openstack-oslo | 05:16 | |
*** openstackgerrit has joined #openstack-oslo | 05:16 | |
*** yamahata has joined #openstack-oslo | 05:16 | |
*** salv-orlando has quit IRC | 05:19 | |
*** stevemar_ has quit IRC | 05:19 | |
*** stevemar_ has joined #openstack-oslo | 05:19 | |
*** stevemar_ has quit IRC | 05:22 | |
*** vks has joined #openstack-oslo | 05:26 | |
vks | how can we declare multilevel dictionary in config file. Is DictopT support this? | 05:27 |
*** rpodolyaka1 has quit IRC | 05:33 | |
*** rpodolyaka1 has joined #openstack-oslo | 05:34 | |
vks | dhellmann, around? | 05:34 |
*** morgan has quit IRC | 05:45 | |
*** yamahata has quit IRC | 05:46 | |
*** morgan has joined #openstack-oslo | 06:01 | |
*** nkrinner has joined #openstack-oslo | 06:06 | |
*** zqfan_afk has joined #openstack-oslo | 06:19 | |
*** rpodolyaka1 has quit IRC | 06:24 | |
*** browne has joined #openstack-oslo | 06:25 | |
*** e0ne has joined #openstack-oslo | 06:32 | |
*** jamielennox is now known as jamielennox|away | 06:37 | |
*** e0ne has quit IRC | 06:42 | |
*** rpodolyaka1 has joined #openstack-oslo | 06:45 | |
*** salv-orlando has joined #openstack-oslo | 06:46 | |
*** e0ne has joined #openstack-oslo | 06:47 | |
*** vks has quit IRC | 07:07 | |
*** bnemec has quit IRC | 07:19 | |
*** bnemec has joined #openstack-oslo | 07:25 | |
*** e0ne has quit IRC | 07:27 | |
*** jongchoi has joined #openstack-oslo | 07:31 | |
*** e0ne has joined #openstack-oslo | 07:31 | |
*** markus_z has joined #openstack-oslo | 07:36 | |
*** rpodolyaka1 has quit IRC | 07:43 | |
*** salv-orlando has quit IRC | 07:46 | |
*** browne has quit IRC | 07:51 | |
*** browne has joined #openstack-oslo | 07:58 | |
*** dims has joined #openstack-oslo | 07:58 | |
*** browne has quit IRC | 08:02 | |
*** dims has quit IRC | 08:05 | |
*** achanda has quit IRC | 08:10 | |
*** yassine has joined #openstack-oslo | 08:12 | |
*** achanda has joined #openstack-oslo | 08:13 | |
*** achanda has quit IRC | 08:15 | |
*** rpodolyaka1 has joined #openstack-oslo | 08:17 | |
*** achanda has joined #openstack-oslo | 08:20 | |
*** ihrachys has joined #openstack-oslo | 08:26 | |
*** ihrachys has quit IRC | 08:33 | |
*** e0ne has quit IRC | 08:37 | |
*** zqfan_afk is now known as zqfan | 08:37 | |
*** ihrachys has joined #openstack-oslo | 08:42 | |
openstackgerrit | Matthew Booth proposed openstack/oslo.messaging: Fix a race calling blocking MessageHandlingServer.start() https://review.openstack.org/237038 | 08:45 |
openstackgerrit | Matthew Booth proposed openstack/oslo.messaging: Fix assumptions in test_server_wait_method https://review.openstack.org/237037 | 08:45 |
openstackgerrit | Matthew Booth proposed openstack/oslo.messaging: Rename MessageHandlingServer._executor for readability https://review.openstack.org/237036 | 08:45 |
*** jongchoi has quit IRC | 08:48 | |
*** achanda has quit IRC | 08:48 | |
*** links has quit IRC | 09:00 | |
*** openstackgerrit has quit IRC | 09:01 | |
*** openstackgerrit has joined #openstack-oslo | 09:01 | |
*** amotoki has joined #openstack-oslo | 09:01 | |
mdbooth | "By default, the most simple executor is used - the blocking executor." | 09:02 |
mdbooth | 'most simple': heh | 09:03 |
*** ihrachys has quit IRC | 09:03 | |
*** ihrachys has joined #openstack-oslo | 09:08 | |
*** links has joined #openstack-oslo | 09:15 | |
*** e0ne has joined #openstack-oslo | 09:17 | |
openstackgerrit | Komei Shimamura proposed openstack/oslo.messaging: Add devstack plugin for kafka https://review.openstack.org/227522 | 09:18 |
*** openstackgerrit has quit IRC | 09:31 | |
*** openstackgerrit has joined #openstack-oslo | 09:31 | |
mdbooth | sileht: Restarting a MessagingHandlingServer | 09:32 |
mdbooth | Looking at _setup_listener, if we're not going to support restarting a MessageHandlingServer, I don't think anything is reusable there | 09:34 |
mdbooth | So it's really a question of whether we want to support this. I'm going to need direction on this, as it's a question of design rather than correctness. | 09:35 |
mdbooth | sileht: What's the best way to ensure I do this properly? | 09:36 |
*** ihrachys has quit IRC | 09:36 | |
*** salv-orlando has joined #openstack-oslo | 09:36 | |
*** exploreshaifali has joined #openstack-oslo | 09:43 | |
*** ihrachys has joined #openstack-oslo | 09:44 | |
openstackgerrit | Oleksii Zamiatin proposed openstack/oslo.messaging: Fix target resolution mismatch in neutron, nova, heat https://review.openstack.org/232720 | 09:54 |
*** salv-orlando has quit IRC | 09:59 | |
*** ihrachys has quit IRC | 10:04 | |
*** ihrachys has joined #openstack-oslo | 10:05 | |
*** ozamiatin has joined #openstack-oslo | 10:09 | |
*** jaypipes has joined #openstack-oslo | 10:26 | |
*** salv-orlando has joined #openstack-oslo | 10:27 | |
*** flaper87 has quit IRC | 10:29 | |
openstackgerrit | Matthew Booth proposed openstack/oslo.messaging: Robustify locking in MessageHandlingServer https://review.openstack.org/235347 | 10:29 |
openstackgerrit | Matthew Booth proposed openstack/oslo.messaging: Fix test_two_pools_three_listener to not restart servers https://review.openstack.org/236862 | 10:29 |
*** rpodolyaka1 has quit IRC | 10:31 | |
*** dims has joined #openstack-oslo | 10:34 | |
*** flaper87 has joined #openstack-oslo | 10:35 | |
*** flaper87 has quit IRC | 10:35 | |
*** flaper87 has joined #openstack-oslo | 10:35 | |
*** stevemar_ has joined #openstack-oslo | 10:35 | |
*** achanda has joined #openstack-oslo | 10:36 | |
openstackgerrit | Matthew Booth proposed openstack/oslo.messaging: Remove unused event in ServerThreadHelper https://review.openstack.org/237977 | 10:36 |
*** achanda has quit IRC | 10:37 | |
*** stevemar_ has quit IRC | 10:38 | |
openstackgerrit | Matthew Booth proposed openstack/oslo.messaging: Trivial locking cleanup in test_listener https://review.openstack.org/236859 | 10:40 |
*** salv-orlando has quit IRC | 10:40 | |
*** links has quit IRC | 10:41 | |
flaper87 | dims: hey there! | 10:42 |
flaper87 | dims: https://review.openstack.org/#/c/231039/ | 10:42 |
flaper87 | All green now! It'd be great to get those 2 in | 10:42 |
flaper87 | I don't like having kilo things pending | 10:43 |
flaper87 | if you're ok, I can merge them since I'm part of stable-maint core too | 10:43 |
dims | flaper87: give me a few mins | 10:43 |
flaper87 | dims: sure! (fwiw, you had reviewed them already) | 10:43 |
dims | yep. done | 10:44 |
flaper87 | dims: ++ | 10:44 |
flaper87 | thanks | 10:44 |
*** links has joined #openstack-oslo | 10:59 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/oslo.context: Ensure to_dict() supports non-ascii characters https://review.openstack.org/237767 | 11:01 |
kashyap | dims: Hi | 11:14 |
kashyap | When you're about (not sure you're following the discussion on #openstack-nova). To not break the conv. I'll ask my question there -) | 11:14 |
*** jecarey_ has quit IRC | 11:29 | |
*** salv-orlando has joined #openstack-oslo | 11:30 | |
*** e0ne has quit IRC | 11:37 | |
dims | ack kashyap i'll look there | 11:37 |
*** exploreshaifali has quit IRC | 11:38 | |
*** gordc has joined #openstack-oslo | 11:49 | |
*** rpodolyaka1 has joined #openstack-oslo | 11:50 | |
*** e0ne has joined #openstack-oslo | 11:54 | |
*** thumpba has joined #openstack-oslo | 11:55 | |
*** binarin has quit IRC | 11:57 | |
*** _amrith_ is now known as amrith | 12:03 | |
*** binarin has joined #openstack-oslo | 12:06 | |
*** openstackgerrit has quit IRC | 12:16 | |
*** openstackgerrit has joined #openstack-oslo | 12:16 | |
*** jecarey has joined #openstack-oslo | 12:17 | |
*** itisha has joined #openstack-oslo | 12:22 | |
*** thumpba has quit IRC | 12:30 | |
*** ihrachys has quit IRC | 12:31 | |
*** ihrachys has joined #openstack-oslo | 12:32 | |
*** ihrachys has quit IRC | 12:33 | |
*** jecarey has quit IRC | 12:33 | |
*** jecarey has joined #openstack-oslo | 12:34 | |
dims | jecarey: ping | 12:35 |
*** edmondsw has joined #openstack-oslo | 12:35 | |
dims | jecarey: would you have a few cycles to help with this? https://review.openstack.org/#/c/237767/ | 12:35 |
*** ihrachys has joined #openstack-oslo | 12:37 | |
*** ihrachys has quit IRC | 12:37 | |
*** ihrachys has joined #openstack-oslo | 12:37 | |
*** boris-42 has quit IRC | 12:38 | |
*** jecarey has quit IRC | 12:38 | |
*** fultonj has joined #openstack-oslo | 12:42 | |
*** ihrachys has quit IRC | 12:42 | |
*** kgiusti has joined #openstack-oslo | 12:46 | |
*** thumpba has joined #openstack-oslo | 12:47 | |
*** ndipanov has joined #openstack-oslo | 12:47 | |
*** salv-orlando has quit IRC | 12:53 | |
*** jecarey has joined #openstack-oslo | 12:55 | |
*** ihrachys has joined #openstack-oslo | 13:00 | |
*** jecarey has quit IRC | 13:02 | |
*** jecarey has joined #openstack-oslo | 13:03 | |
*** stevemar_ has joined #openstack-oslo | 13:03 | |
*** stevemar_ has quit IRC | 13:07 | |
*** salv-orlando has joined #openstack-oslo | 13:07 | |
*** jecarey has quit IRC | 13:07 | |
*** stevemar_ has joined #openstack-oslo | 13:09 | |
*** rlrossit has joined #openstack-oslo | 13:11 | |
*** stevemar_ has quit IRC | 13:12 | |
*** ihrachys has quit IRC | 13:16 | |
*** stevemar_ has joined #openstack-oslo | 13:17 | |
*** jecarey has joined #openstack-oslo | 13:21 | |
*** rpodolyaka1 has quit IRC | 13:24 | |
*** rpodolyaka1 has joined #openstack-oslo | 13:31 | |
*** dims has quit IRC | 13:45 | |
*** dims has joined #openstack-oslo | 13:46 | |
mdbooth | kgiusti: I have a pyngus question if you're about | 13:52 |
kgiusti | I am about - and I have a pyngus answer (probably). | 13:53 |
mdbooth | kgiusti: I've run the oslo.messaging tests a ton of times recently, and I always get 2 local failures related to authentication | 13:53 |
mdbooth | The first part of that is the authentication failure itself | 13:53 |
mdbooth | I haven't looked into that at all, but I expect it's some underlying library change outside python | 13:54 |
mdbooth | Or something | 13:54 |
*** stevemar_ has quit IRC | 13:54 | |
mdbooth | Anyway, another day | 13:54 |
mdbooth | The issue I'm looking at right now is why this results in a timeout, not in a failure | 13:54 |
mdbooth | I see in your patch which added pyngus 2.0 support, you've put the following in Connection.sasl_done: | 13:55 |
mdbooth | # connection failure will be handled later | 13:55 |
mdbooth | This doesn't appear to be handled later at all | 13:55 |
mdbooth | In fact, I can't see anything which would handle an authentication failure | 13:55 |
* kgiusti looking... | 13:56 | |
mdbooth | Unfortunately something from earlier have fallen out of my feeble brain | 13:57 |
mdbooth | However, what actually happens is it continues merrily and ends up blocking waiting on input which will obviously never arrive | 13:57 |
mdbooth | Until the testrunner kills it | 13:57 |
kgiusti | mdbooth: ah - I see. | 13:57 |
mdbooth | So, sasl authentication may succeed on your setup | 13:58 |
mdbooth | In order to reproduce, you may have to sabotage it :) | 13:58 |
mdbooth | I always get 2 test failures | 13:58 |
mdbooth | The one I'm currently looking at is oslo_messaging.tests.test_amqp_driver.TestCyrusAuthentication.test_authentication_default_username | 13:58 |
kgiusti | mdbooth: what the failure message/traceback for that? | 13:59 |
*** pradk has joined #openstack-oslo | 13:59 | |
mdbooth | kgiusti: It hangs | 13:59 |
mdbooth | I can get a traceback with a ctrl-c | 13:59 |
mdbooth | ...sec | 13:59 |
kgiusti | mdbooth: ugh | 14:00 |
mdbooth | http://paste.fedoraproject.org/281919/45436035 | 14:01 |
mdbooth | However, there are multiple threads in play | 14:01 |
mdbooth | Let me see if I can get you a better one from the debugger | 14:01 |
*** zqfan is now known as zqfan_afk | 14:02 | |
mdbooth | kgiusti: Ok, so it actually hangs on the select in eventloop.run | 14:02 |
mdbooth | So the issue seems to be that the connection failed unrecoverably, but nothing signaled this to the eventloop | 14:03 |
*** e0ne has quit IRC | 14:03 | |
kgiusti | mdbooth: interesting, so it's waiting for data that will not arrive. The failure detection should've kicked in when the eventloop processed the last bit of I/O | 14:04 |
mdbooth | Which failure detection? | 14:04 |
kgiusti | mdbooth: can you try something for me? | 14:04 |
mdbooth | I couldn't find any failure detection | 14:04 |
kgiusti | mdbooth: I would expect a reply to arrive on that wait with an error result | 14:05 |
*** achanda has joined #openstack-oslo | 14:05 | |
mdbooth | sasl_done has been called, btw | 14:05 |
kgiusti | mdbooth: with a non-zero outcome? | 14:06 |
mdbooth | Yes | 14:06 |
*** sigmavirus24_awa is now known as sigmavirus24 | 14:06 | |
mdbooth | The log message is output | 14:06 |
*** ozamiatin has quit IRC | 14:07 | |
*** tristanC has quit IRC | 14:07 | |
kgiusti | mdbooth: yeah... hrm. the negative tests all have timeouts in the send request. | 14:07 |
mdbooth | That's probably not valid... | 14:08 |
kgiusti | mdbooth: can you re-run that cyrus test with the env var PN_TRACE_FRM=1 ? | 14:08 |
kgiusti | mdbooth: yes I agree | 14:08 |
mdbooth | Yup, sec | 14:08 |
*** jongchoi has joined #openstack-oslo | 14:08 | |
*** achanda has quit IRC | 14:09 | |
mdbooth | http://paste.fedoraproject.org/281925/14454365 | 14:09 |
kgiusti | mdbooth: ha - no mech sent. Ok I can repro that easily. | 14:10 |
mdbooth | Could that also relate to why it fails in the first place, btw? | 14:10 |
kgiusti | mdbooth: but I'm surprised pyngus signals EOS, yet the app never sees the close | 14:11 |
kgiusti | mdbooth: yes definitely | 14:11 |
kgiusti | mdbooth: and I've seen that error before | 14:11 |
mdbooth | kgiusti: Is this related to negotiation? Perhaps no common mech? | 14:11 |
kgiusti | mdbooth: sort of - the client isn't sending _any_ mechs. | 14:11 |
* mdbooth is utterly unfamiliar with this code, btw | 14:11 | |
kgiusti | mdbooth: it's like the sasl lib doesn't have any configured mechs (?) | 14:12 |
mdbooth | kgiusti: Would that be an OS thing? | 14:13 |
mdbooth | Or configured by the calling app? | 14:13 |
mdbooth | Did I mention I'm utterly unfamiliar with this code? ;) | 14:13 |
*** ozamiatin has joined #openstack-oslo | 14:13 | |
mdbooth | Just wondering if I should be looking for libraries which aren't even installed, or something in /etc | 14:13 |
kgiusti | mdbooth: the test should create a proper config. What kind of system are you on? fedoraish or debian-ish? | 14:14 |
mdbooth | Fedora actual | 14:14 |
mdbooth | F23 | 14:14 |
mdbooth | It writes a config file at one point iirc | 14:15 |
mdbooth | Yeah, it creates openstack.conf which contains: | 14:16 |
mdbooth | mech_list: DIGEST-MD5 SCRAM-SHA-1 CRAM-MD5 PLAIN | 14:16 |
mdbooth | I don't know if anything ever reads that, though | 14:16 |
mdbooth | I wonder... | 14:16 |
mdbooth | Does it have any weird permissions? | 14:16 |
kgiusti | mdbooth: fyi: http://paste.fedoraproject.org/281928/54369691/ | 14:16 |
kgiusti | mdbooth: that's my local system config | 14:17 |
mdbooth | Looks similar to mine | 14:17 |
mdbooth | Are you a Red Hatter by any chance, btw? | 14:17 |
kgiusti | mdbooth: re: permissions - the config files should be generated using the default mode of the process running the test | 14:18 |
*** rpodolyaka1 has quit IRC | 14:18 | |
kgiusti | mdbooth: yep. | 14:18 |
mdbooth | So... straced it | 14:18 |
kgiusti | mdbooth: kewl | 14:18 |
mdbooth | openstack.conf is written, but never read | 14:18 |
*** ozamiatin has quit IRC | 14:19 | |
* mdbooth wonders what would read it | 14:19 | |
kgiusti | mdbooth: really? Hmmm... should be read by the _broker_ side, I don't think the client reads that, however | 14:20 |
mdbooth | I ran the test with strace -f -eopen | 14:20 |
mdbooth | It's only opened once by any process, and that's to write it | 14:20 |
kgiusti | mdbooth: the test broker thread should read that config - it needs the sasldb at least. | 14:20 |
kgiusti | mdbooth: k - I'll try that locally too - see if there is a difference. Also, can you hack the test setup in test_amqp_driver.py? | 14:21 |
mdbooth | Nothing seems to read sasldb either | 14:21 |
mdbooth | Sure | 14:21 |
kgiusti | mdbooth: in TestCyrusAuthentication.setUp() - add ANONYMOUS to the default list of mechs (line ~ 384). Then rerun with that PN_TRACE_FRM=1 evn again. | 14:22 |
mdbooth | kgiusti: Ok, but that list is written to openstack.conf | 14:23 |
kgiusti | mdbooth: btw - are you on latest master? | 14:23 |
mdbooth | And I think the underlying issue is that nothing is reading that file | 14:23 |
mdbooth | Yeah, latest master | 14:23 |
mdbooth | Ah, nm | 14:24 |
kgiusti | mdbooth: yeah, good point. So what cyrus rpms do you have installed? Just to be clear. | 14:24 |
mdbooth | It's also passed as an arg to FakeBroken | 14:24 |
kgiusti | mdbooth: what? | 14:24 |
mdbooth | Running | 14:24 |
mdbooth | kgiusti: With ANONYMOUS that runs successfully | 14:24 |
mdbooth | openstack.conf might be a red herring | 14:24 |
kgiusti | mdbooth: does it now read that file (or sasldb?) | 14:25 |
mdbooth | Sec, will strace it again | 14:25 |
mdbooth | openstack.conf still not read | 14:25 |
mdbooth | Same for sasldb | 14:25 |
kgiusti | mdbooth: hrm. Can you post the PN_TRACE_FRM=1 from the ANONYMOUS run? | 14:26 |
mdbooth | http://paste.fedoraproject.org/281936/43762714 | 14:27 |
kgiusti | mdbooth: btw - I haven't tested this stuff on F23 - only up to F22. I should try F23 locally - you may be seeing something related to the new system. | 14:27 |
mdbooth | cyrus-sasl-scram-2.1.26-25.2.fc23.x86_64 | 14:27 |
mdbooth | cyrus-sasl-2.1.26-25.2.fc23.x86_64 | 14:28 |
mdbooth | cyrus-sasl-plain-2.1.26-25.2.fc23.x86_64 | 14:28 |
mdbooth | cyrus-sasl-lib-2.1.26-25.2.fc23.x86_64 | 14:28 |
mdbooth | cyrus-sasl-gssapi-2.1.26-25.2.fc23.x86_64 | 14:28 |
mdbooth | cyrus-sasl-md5-2.1.26-25.2.fc23.x86_64 | 14:28 |
mdbooth | python-saslwrapper-0.16-11.fc23.x86_64 | 14:28 |
mdbooth | saslwrapper-0.16-11.fc23.x86_64 | 14:28 |
*** jongchoi has quit IRC | 14:28 | |
mdbooth | kgiusti: Do you want me to raise a bug for this, btw? | 14:29 |
kgiusti | mdbooth: yes, let's do that. | 14:29 |
mdbooth | I'll raise 2 | 14:29 |
mdbooth | But I think the hang is more serious | 14:29 |
kgiusti | mdbooth: yes definitely. I need to address that ASAP | 14:30 |
kgiusti | mdbooth: hey can you "rpm -qa | grep "*proton*" for me? Curious. | 14:30 |
mdbooth | Nothing | 14:31 |
mdbooth | However, proton seemed to be available in tox's venv | 14:31 |
mdbooth | Version 0.10 iirc | 14:31 |
kgiusti | mdbooth: bingo. Try installing cyrus-sasl-devel, then blow away your tox env and rebuild it. | 14:31 |
mdbooth | I'll also have to delete the pyngus wheel from my pip cache :) | 14:32 |
mdbooth | sec | 14:32 |
mdbooth | Ok, deleted pyngus and python_qpid_proton wheels | 14:33 |
kgiusti | mdbooth: I think tox is pulling down the proton bits via pypi. | 14:33 |
mdbooth | But it compiles locally, no? | 14:34 |
kgiusti | mdbooth: if the proton c libs are not installed, the python proton setup.py will pull down the proton/c sources from apache and build them locally. | 14:34 |
kgiusti | mdbooth: but if the cyrus dev libs are _not_ available, proton won't be able to use the cyrus system. | 14:35 |
kgiusti | mdbooth: that's another bug in the test if that is the case. | 14:35 |
kgiusti | mdbooth: the test should skip cyrus tests if proton hasn't been built with the proper libs - I can fix that. | 14:36 |
*** zz_dimtruck is now known as dimtruck | 14:38 | |
*** ihrachys has joined #openstack-oslo | 14:40 | |
mdbooth | kgiusti: So, I just blew away all my local pip state (I use devpi-server and wheelhouse) and rebuilt | 14:44 |
*** rpodolyaka1 has joined #openstack-oslo | 14:44 | |
mdbooth | python_qpid_proton comes back, but I didn't notice it being compiled | 14:45 |
mdbooth | Does it download binaries? | 14:45 |
* mdbooth would find that odd | 14:45 | |
* kgiusti would too | 14:45 | |
mdbooth | Anyway, it contains a single .so: _cproton.so | 14:45 |
kgiusti | mdbooth: no - it should pull down a tarball from apache | 14:45 |
mdbooth | And that is *not* linked directly to a sasl library | 14:46 |
mdbooth | libssl, libcrypto, libgssapi_krb5, libkrb5, libk5crypto and libkrb5support | 14:46 |
mdbooth | But no sasl | 14:46 |
kgiusti | mdbooth: no libsasl2.so? | 14:47 |
mdbooth | No | 14:47 |
kgiusti | mdbooth: and the cyrus sasl dev libs were installed prior to the download? | 14:47 |
mdbooth | Yup | 14:48 |
mdbooth | I might try to compile it manually, see what's going on | 14:48 |
kgiusti | mdbooth: pop into a venv and pip install pyngus | 14:48 |
mdbooth | Yeah, it just installs the version from the local wheel | 14:49 |
mdbooth | which was presumably just built locally without my noticing | 14:50 |
*** links has quit IRC | 14:50 | |
krotscheck | dims, dhellmann: Question - I'm running into a lot of requests from the various services team to see functional tests for the oslo cors middleware. sdague proposed that it might be easiest if we add a generic devstack run to oslo_middleware to do that. There already seems to be a neutron run - is this something we can reuse and generify? | 14:50 |
*** stevemar_ has joined #openstack-oslo | 14:55 | |
*** stevemar_ has quit IRC | 14:56 | |
*** e0ne has joined #openstack-oslo | 14:56 | |
*** nkrinner has quit IRC | 14:57 | |
kgiusti | mdbooth: if you pull down the proton src http://qpid.apache.org/releases/qpid-proton-0.10/ goto the proton-c/bindings/python dir and try python setup.py build | 14:58 |
mdbooth | kgiusti: Just doing a little git archaeology. Looks like that timeout has been there in the failure case since your first patch in change Ib9279ed40b0f4cff62e1c742069c8f49f5625659 | 14:58 |
*** stevemar_ has joined #openstack-oslo | 14:59 | |
kgiusti | mdbooth: I'm a sad panda | 14:59 |
mdbooth | So I guess if the hang is real, nobody's hitting it hard enough to complain about it :) | 14:59 |
kgiusti | mdbooth: welcome to my nightmare. | 14:59 |
kgiusti | mdbooth: yeah, so here's the problem: a send (or notify) is queued up to pyngus for sending and the connection establishment is kicked off... | 15:00 |
*** ihrachys has quit IRC | 15:00 | |
*** rpodolyaka1 has quit IRC | 15:01 | |
* mdbooth is anticipating a scheduling-related deadlock... | 15:01 | |
*** alejandrito has joined #openstack-oslo | 15:01 | |
kgiusti | mdbooth: the connection fails to connect due to auth error. At that point, the o.m. driver attempt to failover/reconnect in a (misguided) attempt to connection to other hosts which may have proper auth. | 15:01 |
*** alejandrito has quit IRC | 15:02 | |
kgiusti | mdbooth: the pending sends are held on the queue until the connection is established or they time out. | 15:02 |
*** alejandrito has joined #openstack-oslo | 15:02 | |
kgiusti | mdbooth: which is very, very bad. | 15:02 |
kgiusti | mdbooth: deadlock? uh oh. Where? | 15:02 |
mdbooth | kgiusti: Nah, just guessing. I've been having fun with deadlocks involving manual scheduling over the last few days | 15:03 |
kgiusti | mdbooth: cripes don't do that! You've scared me enough for one day :) | 15:03 |
mdbooth | Sorry :) | 15:03 |
*** f13o has joined #openstack-oslo | 15:03 | |
kgiusti | mdbooth: np. So what the driver _should_ do is cycle through all configured hosts and if none pass auth abort all pending requests. | 15:04 |
kgiusti | mdbooth: operator intervention is needed at that point. probably. | 15:04 |
mdbooth | Well, I'm happy that I correctly called it when I decided I was out of my depth :) | 15:05 |
kgiusti | mdbooth: might me out of mine, too :) | 15:05 |
kgiusti | mdbooth: this was a big hole that the unit tests should've caught. bad kenny. | 15:05 |
kgiusti | mdbooth: no soup for me. | 15:05 |
kgiusti | mdbooth: but the other problem - no cyrus lib linked in - that's something entirely different. | 15:06 |
kgiusti | mdbooth: hopefully easier to solve. | 15:07 |
mdbooth | Yeah. Just writing up the hang bug now I've decided how to describe it. | 15:08 |
kgiusti | mdbooth: when you setup.py build proton-c/bindings/python - do you see the file "cyrus_sasl.c" included in the build, or "none_sasl.c"? | 15:10 |
mdbooth | Not got there, yet | 15:10 |
*** regXboi has joined #openstack-oslo | 15:12 | |
*** ihrachys has joined #openstack-oslo | 15:12 | |
mdbooth | https://bugs.launchpad.net/oslo.messaging/+bug/1508512 | 15:12 |
openstack | Launchpad bug 1508512 in oslo.messaging "oslo.messaging hangs if SASL authentication fails" [Undecided,New] | 15:12 |
*** stevema__ has joined #openstack-oslo | 15:13 | |
*** mhorban_ has joined #openstack-oslo | 15:14 | |
mdbooth | dims: If you've got a sec, I've got a patch with +2s blocked waiting on 2 trivial patches with only a single +2 | 15:15 |
dims | mdbooth: which one? | 15:15 |
mdbooth | Trivial patches are: https://review.openstack.org/#/c/237036/ and https://review.openstack.org/#/c/237037/ | 15:15 |
mdbooth | This one has 2 +2s: https://review.openstack.org/#/c/237038/ | 15:16 |
dims | mdbooth: ack will be a few mins | 15:16 |
mdbooth | (Thanks sileht and harlowja) | 15:16 |
*** stevemar_ has quit IRC | 15:16 | |
*** Kiall has quit IRC | 15:19 | |
*** ihrachys has quit IRC | 15:20 | |
*** ihrachys_ has joined #openstack-oslo | 15:20 | |
*** Kiall has joined #openstack-oslo | 15:20 | |
openstackgerrit | Tom Cammann proposed openstack/oslo.config: Allow naming of custom types https://review.openstack.org/221394 | 15:21 |
openstackgerrit | Cyril Roelandt proposed openstack/oslo.messaging: Use yaml.safe_load instead of yaml.load https://review.openstack.org/238135 | 15:22 |
*** ihrachys_ has quit IRC | 15:23 | |
*** ihrachys has joined #openstack-oslo | 15:23 | |
*** ihrachys has quit IRC | 15:25 | |
mdbooth | kgiusti: So, I *think* the issue is that somebody has uploaded a wheel for python-qpid-proton which doesn't contain the compiled bits | 15:28 |
mdbooth | Because I have wheels configured | 15:28 |
mdbooth | I *think* I'm getting the pure python version by default | 15:28 |
mdbooth | I haven't confirmed this, but when I explicitly tell it not to use an existing wheel it links with sasl | 15:29 |
*** ihrachys has joined #openstack-oslo | 15:31 | |
kgiusti | mdbooth: interesting - where is that wheel coming from, pypi? | 15:31 |
*** dims_ has joined #openstack-oslo | 15:31 | |
mdbooth | kgiusti: I have read allusions to the existence of a suppository of wheels | 15:31 |
mdbooth | However, I have not confirmed its existence | 15:32 |
mdbooth | It's mysterious, dark, and vaguely smelly | 15:33 |
kgiusti | mdbooth: trust no one... | 15:33 |
mdbooth | The local thing is completely awesome, though | 15:33 |
* mdbooth caches everything | 15:33 | |
kgiusti | mdbooth: I'm going to open another bug against the tests to skip the cyrus tests if the underlying proton lib hasn't been linked with cyrus. | 15:34 |
mdbooth | kgiusti: I'm literally just doing that | 15:35 |
kgiusti | mdbooth: thanks! | 15:35 |
*** dims has quit IRC | 15:35 | |
mdbooth | I can confirm that, with the correctly linked python_qpid_proton, the tests all run successfully :) | 15:35 |
mdbooth | kgiusti: Thank you! | 15:35 |
kgiusti | mdbooth: np - I'm just glad other folks are finally banging on this driver. | 15:37 |
mdbooth | kgiusti: https://bugs.launchpad.net/oslo.messaging/+bug/1508523 | 15:38 |
openstack | Launchpad bug 1508523 in oslo.messaging "TestCyrusAuthentication runs even when python_qpid_proton is not linked against SASL libraries" [Undecided,New] | 15:38 |
mdbooth | dims_: Incidentally, sileht is concerned about removing the ability to restart servers, which he raises in this review: https://review.openstack.org/#/c/236862/2 | 15:39 |
mdbooth | This is a reasonable concern, but I don't know how to proceed | 15:40 |
mdbooth | The only thing I can think of it raising a spec and having the discussion in that review | 15:40 |
mdbooth | From my pov, I can restart or not | 15:41 |
mdbooth | My first patch supported restarts, but I removed it after discussion | 15:41 |
dims_ | mdbooth: i'd go with sileht's view as he has been taking care of this stuff for a while now | 15:48 |
*** ndipanov has quit IRC | 15:52 | |
*** markus_z has quit IRC | 15:52 | |
*** ihrachys has quit IRC | 15:53 | |
*** ihrachys has joined #openstack-oslo | 15:53 | |
mdbooth | dims_: Ok, I'll put it back in. | 15:53 |
*** ozamiatin has joined #openstack-oslo | 15:54 | |
*** yamahata has joined #openstack-oslo | 15:55 | |
*** binarin has quit IRC | 15:56 | |
*** sputnik13 has joined #openstack-oslo | 15:57 | |
*** rlrossit has quit IRC | 15:59 | |
*** ozamiatin has quit IRC | 15:59 | |
*** stevema__ has quit IRC | 16:00 | |
*** jecarey_ has joined #openstack-oslo | 16:00 | |
*** jecarey_ has quit IRC | 16:02 | |
*** jecarey_ has joined #openstack-oslo | 16:03 | |
*** jecarey has quit IRC | 16:03 | |
*** ihrachys has quit IRC | 16:04 | |
*** exploreshaifali has joined #openstack-oslo | 16:05 | |
*** rlrossit has joined #openstack-oslo | 16:10 | |
*** bnemec has quit IRC | 16:10 | |
*** yamahata has quit IRC | 16:11 | |
*** yamahata has joined #openstack-oslo | 16:11 | |
*** ihrachys has joined #openstack-oslo | 16:16 | |
dims_ | folks, i've intiailized the etherpads, please take a look and add/update as needed - https://wiki.openstack.org/wiki/Design_Summit/Mitaka/Etherpads#Oslo | 16:16 |
*** e0ne has quit IRC | 16:16 | |
*** e0ne has joined #openstack-oslo | 16:17 | |
*** bnemec has joined #openstack-oslo | 16:17 | |
*** achanda has joined #openstack-oslo | 16:28 | |
*** jecarey_ has quit IRC | 16:31 | |
*** jecarey_ has joined #openstack-oslo | 16:34 | |
openstackgerrit | Merged openstack/oslo.messaging: Rename MessageHandlingServer._executor for readability https://review.openstack.org/237036 | 16:36 |
*** dims_ has quit IRC | 16:36 | |
openstackgerrit | Merged openstack/oslo.messaging: Fix assumptions in test_server_wait_method https://review.openstack.org/237037 | 16:36 |
openstackgerrit | Merged openstack/oslo.messaging: Fix a race calling blocking MessageHandlingServer.start() https://review.openstack.org/237038 | 16:36 |
*** stevemar_ has joined #openstack-oslo | 16:36 | |
*** dims has joined #openstack-oslo | 16:36 | |
*** exploreshaifali has quit IRC | 16:38 | |
*** stevemar_ has quit IRC | 16:41 | |
*** amotoki has quit IRC | 16:42 | |
*** ihrachys has quit IRC | 16:47 | |
*** ihrachys has joined #openstack-oslo | 16:47 | |
*** zigo_ has quit IRC | 16:59 | |
*** zigo has joined #openstack-oslo | 17:00 | |
*** sputnik13 has quit IRC | 17:11 | |
*** sputnik13 has joined #openstack-oslo | 17:12 | |
*** nikhil_k has joined #openstack-oslo | 17:15 | |
*** nikhil has quit IRC | 17:15 | |
*** fultonj has left #openstack-oslo | 17:16 | |
*** salv-orl_ has joined #openstack-oslo | 17:18 | |
*** ihrachys has quit IRC | 17:19 | |
*** salv-orlando has quit IRC | 17:21 | |
*** pasquier-s has quit IRC | 17:25 | |
*** pasquier-s has joined #openstack-oslo | 17:27 | |
*** sputnik13 has quit IRC | 17:27 | |
*** tchaypo has quit IRC | 17:29 | |
*** tchaypo has joined #openstack-oslo | 17:31 | |
*** stevemar_ has joined #openstack-oslo | 17:32 | |
*** ihrachys has joined #openstack-oslo | 17:33 | |
*** sputnik13 has joined #openstack-oslo | 17:34 | |
*** exploreshaifali has joined #openstack-oslo | 17:40 | |
*** zqfan_afk is now known as zqfan | 17:42 | |
*** amotoki has joined #openstack-oslo | 17:43 | |
*** salv-orl_ has quit IRC | 17:45 | |
*** rlrossit has quit IRC | 17:46 | |
*** amotoki has quit IRC | 17:48 | |
*** e0ne has quit IRC | 17:49 | |
*** rlrossit has joined #openstack-oslo | 17:50 | |
*** exploreshaifali has quit IRC | 17:53 | |
*** harlowja has quit IRC | 17:57 | |
*** stevemar_ has quit IRC | 17:58 | |
*** harlowja has joined #openstack-oslo | 18:02 | |
*** shardy is now known as shardy_afk | 18:02 | |
*** dims has quit IRC | 18:03 | |
*** dims has joined #openstack-oslo | 18:04 | |
*** yamahata has quit IRC | 18:08 | |
*** jecarey_ has quit IRC | 18:19 | |
*** ihrachys has quit IRC | 18:19 | |
*** jecarey has joined #openstack-oslo | 18:20 | |
*** ihrachys has joined #openstack-oslo | 18:21 | |
*** jecarey_ has joined #openstack-oslo | 18:23 | |
*** jecarey has quit IRC | 18:24 | |
*** salv-orlando has joined #openstack-oslo | 18:25 | |
*** exploreshaifali has joined #openstack-oslo | 18:31 | |
*** zqfan is now known as zqfan_afk | 18:31 | |
*** thumpba has quit IRC | 18:34 | |
*** amotoki has joined #openstack-oslo | 18:44 | |
*** ihrachys has quit IRC | 18:47 | |
*** amotoki has quit IRC | 18:49 | |
*** ihrachys has joined #openstack-oslo | 18:52 | |
*** sputnik13 has quit IRC | 18:53 | |
*** sputnik13 has joined #openstack-oslo | 18:55 | |
*** yamahata has joined #openstack-oslo | 18:57 | |
*** gordc has quit IRC | 18:57 | |
*** stevemar_ has joined #openstack-oslo | 19:07 | |
*** yassine has quit IRC | 19:12 | |
*** stevemar_ has quit IRC | 19:15 | |
*** stevemar_ has joined #openstack-oslo | 19:16 | |
*** achanda has quit IRC | 19:16 | |
*** ihrachys has quit IRC | 19:18 | |
*** stevemar_ has quit IRC | 19:20 | |
*** ihrachys has joined #openstack-oslo | 19:25 | |
*** shardy_afk is now known as shardy | 19:32 | |
*** thumpba has joined #openstack-oslo | 19:35 | |
*** thumpba has quit IRC | 19:41 | |
*** ihrachys has quit IRC | 19:42 | |
*** harlowja has quit IRC | 19:50 | |
*** openstackstatus has joined #openstack-oslo | 20:07 | |
*** ChanServ sets mode: +v openstackstatus | 20:07 | |
*** dimtruck is now known as zz_dimtruck | 20:11 | |
*** browne has joined #openstack-oslo | 20:15 | |
*** jecarey_ has quit IRC | 20:15 | |
*** jecarey has joined #openstack-oslo | 20:16 | |
*** achanda has joined #openstack-oslo | 20:16 | |
*** dims has quit IRC | 20:18 | |
*** jaypipes has quit IRC | 20:18 | |
*** dims has joined #openstack-oslo | 20:18 | |
*** zz_dimtruck is now known as dimtruck | 20:19 | |
*** jecarey has quit IRC | 20:20 | |
*** achanda has quit IRC | 20:22 | |
*** salv-orlando has quit IRC | 20:22 | |
*** agentrooker has joined #openstack-oslo | 20:26 | |
*** agentrooker has left #openstack-oslo | 20:26 | |
*** jecarey has joined #openstack-oslo | 20:31 | |
*** harlowja has joined #openstack-oslo | 20:39 | |
*** harlowja_ has joined #openstack-oslo | 20:40 | |
*** alejandrito has quit IRC | 20:42 | |
*** achanda has joined #openstack-oslo | 20:42 | |
*** browne has quit IRC | 20:44 | |
*** harlowja has quit IRC | 20:44 | |
*** amotoki has joined #openstack-oslo | 20:45 | |
*** achanda has quit IRC | 20:46 | |
*** salv-orlando has joined #openstack-oslo | 20:50 | |
*** shardy has quit IRC | 20:54 | |
*** salv-orlando has quit IRC | 20:55 | |
*** achanda has joined #openstack-oslo | 20:57 | |
*** rlrossit1 has joined #openstack-oslo | 21:05 | |
*** rlrossit has quit IRC | 21:06 | |
*** achanda has quit IRC | 21:10 | |
*** stevemar_ has joined #openstack-oslo | 21:11 | |
*** jongchoi_ has joined #openstack-oslo | 21:11 | |
*** regXboi has quit IRC | 21:20 | |
*** rlrossit1 has left #openstack-oslo | 21:22 | |
*** kgiusti has quit IRC | 21:28 | |
*** amotoki has quit IRC | 21:31 | |
*** pballand has quit IRC | 21:44 | |
*** abitha has joined #openstack-oslo | 21:45 | |
*** avarner has quit IRC | 21:51 | |
*** jongchoi_ has quit IRC | 22:02 | |
*** ozamiatin has joined #openstack-oslo | 22:02 | |
*** jongchoi_ has joined #openstack-oslo | 22:05 | |
*** jamielennox|away is now known as jamielennox | 22:07 | |
*** thumpba has joined #openstack-oslo | 22:07 | |
*** jongchoi_ has quit IRC | 22:10 | |
*** pradk has quit IRC | 22:10 | |
*** salv-orlando has joined #openstack-oslo | 22:22 | |
*** dimtruck is now known as zz_dimtruck | 22:35 | |
*** sigmavirus24 is now known as sigmavirus24_awa | 22:35 | |
*** jecarey has quit IRC | 22:36 | |
*** thumpba has quit IRC | 22:38 | |
*** salv-orlando has quit IRC | 22:40 | |
*** thumpba_ has joined #openstack-oslo | 22:42 | |
*** thumpba_ has quit IRC | 22:44 | |
*** thumpba has joined #openstack-oslo | 22:48 | |
*** otherwiseguy has quit IRC | 22:53 | |
*** thumpba has quit IRC | 22:55 | |
*** thumpba has joined #openstack-oslo | 22:58 | |
openstackgerrit | Joshua Harlow proposed openstack/taskflow: Add a executor backed conductor and have existing impl. use it https://review.openstack.org/174643 | 23:02 |
openstackgerrit | Joshua Harlow proposed openstack/taskflow: Add a executor backed conductor and have existing impl. use it https://review.openstack.org/174643 | 23:04 |
*** thumpba has quit IRC | 23:04 | |
*** pballand has joined #openstack-oslo | 23:05 | |
*** pballand has quit IRC | 23:09 | |
*** pballand has joined #openstack-oslo | 23:11 | |
*** sheeprine has quit IRC | 23:11 | |
*** otherwiseguy has joined #openstack-oslo | 23:11 | |
*** sheeprine has joined #openstack-oslo | 23:13 | |
openstackgerrit | Joshua Harlow proposed openstack/futurist: Allow for providing different run work sync functions https://review.openstack.org/238272 | 23:18 |
harlowja_ | mdbooth ^ might be helpful also | 23:19 |
*** sheeprine has quit IRC | 23:26 | |
*** amotoki has joined #openstack-oslo | 23:28 | |
*** dims_ has joined #openstack-oslo | 23:29 | |
*** sheeprine has joined #openstack-oslo | 23:31 | |
*** dims__ has joined #openstack-oslo | 23:31 | |
*** dims has quit IRC | 23:31 | |
*** amotoki has quit IRC | 23:33 | |
*** harlowja_ has quit IRC | 23:33 | |
*** dims_ has quit IRC | 23:34 | |
*** harlowja has joined #openstack-oslo | 23:37 | |
*** salv-orlando has joined #openstack-oslo | 23:37 | |
openstackgerrit | Joshua Harlow proposed openstack/futurist: Allow lazy sync futures https://review.openstack.org/238274 | 23:38 |
harlowja | mdbooth other idea is ^ | 23:38 |
*** exploreshaifali has quit IRC | 23:38 | |
*** harlowja_ has joined #openstack-oslo | 23:45 | |
*** harlowja has quit IRC | 23:49 | |
openstackgerrit | Joshua Harlow proposed openstack/futurist: Allow lazy sync futures https://review.openstack.org/238274 | 23:52 |
harlowja_ | mdbooth ok ^ should work (vs not, ha) | 23:52 |
*** sheeprine has quit IRC | 23:54 | |
harlowja_ | actually that still has some issues, hmmm | 23:56 |
*** sheeprine has joined #openstack-oslo | 23:57 | |
*** salv-orlando has quit IRC | 23:57 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!