*** bobh_ has joined #openstack-mistral | 00:19 | |
*** achanda has quit IRC | 00:40 | |
*** harlowja has quit IRC | 01:00 | |
*** harlowja has joined #openstack-mistral | 01:03 | |
*** cheneydc has joined #openstack-mistral | 01:08 | |
*** achanda has joined #openstack-mistral | 01:18 | |
*** dprince has joined #openstack-mistral | 01:24 | |
*** toddjohn has joined #openstack-mistral | 01:50 | |
*** toddjohn has quit IRC | 01:53 | |
*** toddjohn has joined #openstack-mistral | 01:53 | |
*** toddjohn has quit IRC | 01:58 | |
*** dprince has quit IRC | 02:19 | |
*** bobh_ has quit IRC | 02:54 | |
*** achanda has quit IRC | 03:09 | |
*** toddjohn has joined #openstack-mistral | 03:22 | |
*** toddjohn has quit IRC | 03:27 | |
*** achanda has joined #openstack-mistral | 04:22 | |
*** toddjohn has joined #openstack-mistral | 04:24 | |
*** toddjohn has quit IRC | 04:28 | |
rakhmerov | ooh, shoot.. | 04:33 |
---|---|---|
rakhmerov | derekh: yes, I see | 04:34 |
rakhmerov | nmakhotkin: can you please look at how it can be fixed so it works on 2.7 as well? | 04:34 |
rakhmerov | kong: here? | 04:34 |
rakhmerov | one more problem | 04:34 |
rakhmerov | in the conversation above | 04:34 |
*** achanda has quit IRC | 04:37 | |
*** achanda has joined #openstack-mistral | 05:03 | |
*** achanda has quit IRC | 05:33 | |
*** achanda has joined #openstack-mistral | 05:59 | |
*** cheneydc has quit IRC | 06:22 | |
*** achanda has quit IRC | 06:22 | |
*** toddjohn has joined #openstack-mistral | 06:25 | |
*** toddjohn has quit IRC | 06:30 | |
*** cheneydc has joined #openstack-mistral | 06:31 | |
*** cheneydc has quit IRC | 06:39 | |
*** cheneydc has joined #openstack-mistral | 06:40 | |
*** cheneydc has quit IRC | 07:05 | |
nmakhotkin | rakhmerov, hi! | 07:39 |
rakhmerov | nmakhotkin: hi | 07:39 |
nmakhotkin | yep, I tested it before I sent the commit | 07:40 |
nmakhotkin | and it works fine on 2.7 | 07:40 |
nmakhotkin | an on 3.4 as well | 07:42 |
*** pbelamge has joined #openstack-mistral | 07:51 | |
*** pbelamge has quit IRC | 07:57 | |
openstackgerrit | Merged openstack/mistral: Unblock skipped tests in test_action_defaults.py https://review.openstack.org/302004 | 08:08 |
*** toddjohn has joined #openstack-mistral | 08:26 | |
*** toddjohn has quit IRC | 08:31 | |
openstackgerrit | hardik proposed openstack/mistral: Filtering support for actions. https://review.openstack.org/269971 | 08:42 |
openstackgerrit | hardik proposed openstack/mistral: Filtering support for actions. https://review.openstack.org/269971 | 08:50 |
*** pbelamge has joined #openstack-mistral | 09:12 | |
*** pbelamge has quit IRC | 09:29 | |
openstackgerrit | Renat Akhmerov proposed openstack/mistral: Replace self._wait(lamdba: ..) constructs with more readable calls https://review.openstack.org/302633 | 09:49 |
*** achanda has joined #openstack-mistral | 09:56 | |
*** achanda has quit IRC | 10:02 | |
openstackgerrit | Renat Akhmerov proposed openstack/mistral: Add utility methods to test action executions more conveniently https://review.openstack.org/302651 | 10:26 |
*** toddjohn has joined #openstack-mistral | 10:28 | |
openstackgerrit | Renat Akhmerov proposed openstack/mistral: Unblock skipped engine race condition test https://review.openstack.org/302656 | 10:32 |
*** toddjohn has quit IRC | 10:32 | |
*** achanda has joined #openstack-mistral | 10:58 | |
*** achanda has quit IRC | 11:03 | |
openstackgerrit | Renat Akhmerov proposed openstack/mistral: Leave more relevant comment in engine race condition test https://review.openstack.org/302656 | 11:09 |
*** bobh_ has joined #openstack-mistral | 11:31 | |
openstackgerrit | Renat Akhmerov proposed openstack/mistral: Leave more relevant comment in engine race condition test https://review.openstack.org/302656 | 11:34 |
*** derekh has joined #openstack-mistral | 12:00 | |
derekh | ddeja: about that problem I was havin with the call to inspect.getargspec | 12:04 |
ddeja | derekh: hm? | 12:05 |
derekh | ddeja: I'm using trunk of oslo.messaging, looks like its a problem introduced in a recent patch | 12:05 |
derekh | https://review.openstack.org/#/c/297988/9/oslo_messaging/rpc/dispatcher.py | 12:05 |
derekh | the __call__ method has been removed | 12:05 |
derekh | RE: https://review.openstack.org/#/c/302354/1 | 12:05 |
ddeja | nmakhotkin, rakhmerov you should look at this ^^ | 12:07 |
*** achanda has joined #openstack-mistral | 12:07 | |
nmakhotkin | ddeja, derekh: oh... | 12:08 |
ddeja | derekh: thanks for letting us know | 12:08 |
ddeja | nmakhotkin: the logic we want to overwrite is now here https://review.openstack.org/#/c/297988/9/oslo_messaging/rpc/server.py | 12:09 |
rakhmerov | yeah... | 12:09 |
rakhmerov | derekh: thanks a lot | 12:09 |
rakhmerov | this is a good example that shouldn't be using such tricks ) | 12:10 |
rakhmerov | I guess the official Mitaka release is already made | 12:10 |
derekh | rakhmerov: ddeja: no prob, I got other problems deploying trunk at the moment with other projects so I'll circle back to mistral once I've figured them out ;-) | 12:10 |
rakhmerov | 2.0.0.0 | 12:10 |
rakhmerov | or it's in progress at least | 12:11 |
rakhmerov | nmakhotkin: I guess we need to fix it, backport to stable/mitaka and request for a point release | 12:11 |
ddeja | rakhmerov: This change is not present on oslo.messaging mitaka branch | 12:13 |
rakhmerov | you mean in their stable branch? | 12:14 |
ddeja | yup | 12:14 |
rakhmerov | hm... | 12:14 |
ddeja | https://github.com/openstack/oslo.messaging/blob/stable/mitaka/oslo_messaging/rpc/dispatcher.py | 12:14 |
ddeja | the __call__ method is still there | 12:14 |
rakhmerov | I see | 12:14 |
ddeja | nevertheless, it need to be changed in Netwon dev cycle, to suit master of oslo | 12:15 |
rakhmerov | hm... then seems like it's not a problem for Mitaka release | 12:15 |
ddeja | I hope so | 12:15 |
rakhmerov | but it became a problem for master | 12:15 |
*** achanda has quit IRC | 12:15 | |
rakhmerov | ddeja: I think it's not a good solution in general so for now, yes, we have to be in sync with oslo but get rid of it on first opportunity | 12:16 |
ddeja | rakhmerov: agree. Syncinc with oslo may become a neverending story | 12:16 |
nmakhotkin | rakhmerov, ddeja: yes, it can be fixed in master | 12:16 |
rakhmerov | ok, we're on the same page | 12:18 |
rakhmerov | I'll proceed with that post processing ACKs story as soon as I can | 12:18 |
ddeja | rakhmerov: will you be in Austin? | 12:18 |
rakhmerov | once I sort out my current tasks | 12:18 |
rakhmerov | ddeja: working on it ) | 12:18 |
rakhmerov | don't know yet | 12:19 |
*** dprince has joined #openstack-mistral | 12:19 | |
rakhmerov | will know on Apr 13 | 12:19 |
ddeja | rakhmerov: ok | 12:19 |
rakhmerov | trying to get US visa | 12:19 |
ddeja | rakhmerov: oh, I see. I got mine yesterday | 12:19 |
rakhmerov | ok | 12:19 |
ddeja | rakhmerov: I think maybe we don't need to use post-ACK | 12:20 |
rakhmerov | why? | 12:20 |
rakhmerov | you mean in your particular case? | 12:20 |
ddeja | no no, I mean in general | 12:20 |
ddeja | since post-ACK is resolving the story 'I want to have my task completed even if executor dies while executing it' | 12:21 |
ddeja | maybe there is another way of addresing this story? | 12:21 |
ddeja | before I found out about Mistral I was working on small PoC that worked similar to how mistral Executor works | 12:22 |
ddeja | and I had same problem with 'what happen if executor dies in the middle of execution' | 12:22 |
ddeja | i didn't resolve it, but I had an idea of storing in the db who is working on which tasks | 12:23 |
ddeja | and periodically check if they are still alive working on it; if not, i can take apropierate action | 12:23 |
ddeja | maybe something like this may work for Mistral? | 12:24 |
ddeja | rakhmerov: what do you think? ^^ | 12:25 |
rakhmerov | well, we've had a huge discussion devoted to this | 12:26 |
rakhmerov | there are different considerations here that should be taken into account | 12:27 |
rakhmerov | like "executor must not have access to DB" and others | 12:27 |
rakhmerov | for various reasons | 12:27 |
rakhmerov | unfortunately, I can't be discussing it right now. Sorry | 12:27 |
rakhmerov | let me ping you tomorrow | 12:28 |
*** bobh_ has quit IRC | 12:28 | |
ddeja | rakhmerov: No problem - to be honest, I wish if we could discuss it in Austin | 12:28 |
rakhmerov | yes | 12:28 |
rakhmerov | I hope we'll do | 12:28 |
ddeja | but tommorow is a good start also | 12:28 |
ddeja | I'll be available from around 7 a.m. UTC, if you have time tommorow | 12:29 |
*** toddjohn has joined #openstack-mistral | 12:29 | |
*** toddjohn has quit IRC | 12:34 | |
*** achanda has joined #openstack-mistral | 12:47 | |
*** toddjohn has joined #openstack-mistral | 12:49 | |
*** achanda has quit IRC | 12:58 | |
*** toddjohn has quit IRC | 12:59 | |
*** achanda has joined #openstack-mistral | 13:08 | |
*** achanda has quit IRC | 13:13 | |
*** bobh_ has joined #openstack-mistral | 13:23 | |
*** toddjohn has joined #openstack-mistral | 13:31 | |
*** toddjohn has quit IRC | 14:17 | |
*** toddjohn has joined #openstack-mistral | 14:35 | |
*** achanda has joined #openstack-mistral | 15:24 | |
*** achanda has quit IRC | 15:37 | |
*** derekh has quit IRC | 15:53 | |
*** FL1SK has quit IRC | 16:06 | |
*** toddjohn has quit IRC | 16:07 | |
*** toddjohn has joined #openstack-mistral | 16:07 | |
*** bobh_ has quit IRC | 16:11 | |
*** toddjohn has quit IRC | 16:12 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/mistral: Updated from global requirements https://review.openstack.org/302858 | 16:19 |
*** toddjohn has joined #openstack-mistral | 16:30 | |
*** toddjohn has quit IRC | 16:39 | |
*** toddjohn has joined #openstack-mistral | 16:40 | |
*** toddjohn has quit IRC | 16:44 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/mistral: Updated from global requirements https://review.openstack.org/302858 | 16:50 |
*** dprince has quit IRC | 16:58 | |
*** dprince has joined #openstack-mistral | 16:59 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/mistral: Updated from global requirements https://review.openstack.org/302858 | 17:00 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/mistral: Updated from global requirements https://review.openstack.org/302858 | 17:09 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/mistral-extra: Updated from global requirements https://review.openstack.org/302972 | 17:09 |
*** toddjohn has joined #openstack-mistral | 17:42 | |
*** toddjohn has quit IRC | 17:47 | |
*** toddjohn has joined #openstack-mistral | 17:52 | |
*** dprince has quit IRC | 18:00 | |
*** dprince has joined #openstack-mistral | 18:00 | |
*** dprince has quit IRC | 18:13 | |
*** dprince has joined #openstack-mistral | 18:15 | |
openstackgerrit | Merged openstack/mistral-extra: Updated from global requirements https://review.openstack.org/302972 | 18:33 |
*** dprince has quit IRC | 18:42 | |
*** dprince has joined #openstack-mistral | 18:43 | |
*** bobh_ has joined #openstack-mistral | 18:46 | |
*** bobh_ has quit IRC | 18:50 | |
*** bobh_ has joined #openstack-mistral | 18:50 | |
*** bobh_ has quit IRC | 18:52 | |
*** bobh_ has joined #openstack-mistral | 18:54 | |
*** bobh_ has quit IRC | 19:00 | |
*** bobh_ has joined #openstack-mistral | 19:16 | |
*** bobh_ has quit IRC | 19:17 | |
*** dprince has quit IRC | 19:42 | |
*** jtomasek has quit IRC | 20:03 | |
*** jtomasek has joined #openstack-mistral | 20:05 | |
openstackgerrit | Merged openstack/mistral: Updated from global requirements https://review.openstack.org/302858 | 20:50 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!