*** rrecio_ has quit IRC | 00:44 | |
*** cheneydc has joined #openstack-mistral | 00:59 | |
*** gyee has quit IRC | 01:03 | |
*** cheneydc has quit IRC | 01:27 | |
*** hparekh has joined #openstack-mistral | 03:42 | |
*** vishwanathj is now known as vishwanathj_zzz | 04:09 | |
*** sharatss has joined #openstack-mistral | 04:11 | |
*** jaosorior has joined #openstack-mistral | 04:44 | |
*** jaosorior has quit IRC | 05:00 | |
*** jaosorior has joined #openstack-mistral | 05:01 | |
openstackgerrit | Asmita singh proposed openstack/mistral: Added pyv8 in requirement.txt in mistral code for bug ID : 1551561 closes-Bug: #1551561 https://review.openstack.org/369830 | 05:42 |
---|---|---|
openstack | bug 1551561 in Mistral "pyv8 is not required by mistral but javascipt action is installed by default" [Undecided,In progress] https://launchpad.net/bugs/1551561 - Assigned to Asmita singh (asmita) | 05:42 |
openstackgerrit | Hardik Parekh proposed openstack/mistral: Fixed task in_bound context when retrying https://review.openstack.org/369834 | 05:58 |
*** jaosorior has quit IRC | 06:29 | |
*** jaosorior has joined #openstack-mistral | 06:30 | |
openstackgerrit | Asmita singh proposed openstack/mistral: Added pyv8 in requirement.txt in mistral code for bug ID : 1551561 closes-Bug: #1551561 https://review.openstack.org/369851 | 06:48 |
openstack | bug 1551561 in Mistral "pyv8 is not required by mistral but javascipt action is installed by default" [Undecided,In progress] https://launchpad.net/bugs/1551561 - Assigned to Asmita singh (asmita) | 06:48 |
*** sharatss is now known as sharat | 06:51 | |
*** FL1SK has quit IRC | 07:02 | |
*** shardy has joined #openstack-mistral | 07:06 | |
*** jistr is now known as jistr|mtgs | 07:09 | |
*** jistr|mtgs is now known as jistr|mtg | 07:09 | |
*** shardy is now known as shardy_mtg | 07:16 | |
*** Ravikiran_K has joined #openstack-mistral | 07:19 | |
*** jaosorior has quit IRC | 07:27 | |
*** jaosorior has joined #openstack-mistral | 07:27 | |
openstackgerrit | Merged openstack/mistral-dashboard: Remove .mailmap and AUTHORS file since it's no longer needed https://review.openstack.org/367323 | 07:28 |
*** jaosorior has quit IRC | 07:29 | |
*** jaosorior has joined #openstack-mistral | 07:30 | |
*** nmakhotkin has joined #openstack-mistral | 07:41 | |
sharat | hi | 08:02 |
sharat | need to discuss about https://bugs.launchpad.net/python-mistralclient/+bug/1581520 this bug | 08:02 |
openstack | Launchpad bug 1581520 in python-mistralclient "Mistral cli needs to be somewhat concise to be usable" [Undecided,New] - Assigned to Sharat Sharma (sharat-sharma) | 08:02 |
*** FL1SK has joined #openstack-mistral | 09:02 | |
*** sharat has quit IRC | 09:04 | |
openstackgerrit | Merged openstack/python-mistralclient: Use oslotest instead of testtools/unittest https://review.openstack.org/365414 | 09:19 |
*** sharatss has joined #openstack-mistral | 09:30 | |
openstackgerrit | Merged openstack/python-mistralclient: Use requests-mock for testing https://review.openstack.org/365415 | 09:35 |
*** jaosorior is now known as jaosorior_lunch | 09:54 | |
openstackgerrit | Dawid Deja proposed openstack/mistral: Delete unnecessary comma https://review.openstack.org/370000 | 10:15 |
*** jaosorior_lunch is now known as jaosorior | 10:27 | |
ddeja | rakhmerov, kong, hparekh: Hi, please take a look on this. Very simple bug, but makes Mistral API do not start on devstack https://review.openstack.org/#/c/370000/ | 10:29 |
hparekh | d0ugal, Ok sure | 10:31 |
hparekh | ddeja, Ok sure | 10:31 |
d0ugal | :) | 10:31 |
rakhmerov | ddeja: hi, ok | 10:46 |
rakhmerov | sharatss: hi, I would say it's not even a bug. It's rather a blueprint to address all those issues | 10:48 |
rakhmerov | we've been planning to work on API and CLI v3 in Newton had to move it to the next cycle | 10:48 |
rakhmerov | it should address these problems | 10:49 |
sharatss | rakhmerov: thanks | 10:49 |
sharatss | rakhmerov: so that needs no work as of now? | 10:50 |
*** shardy_mtg is now known as shardy | 10:51 | |
rakhmerov | yeah, at least my idea is that we probably need to look at all this in complex, because it's not just one place where we need to fix something, we need to evolve the entire approach to how we represent stuff in CLI | 10:51 |
rakhmerov | sharatss: I would say yes, I would postpone it for now till the next cycle | 10:51 |
rakhmerov | and we'll need to start working on it in the beginning of the cycle to have enough time to complete it | 10:52 |
sharatss | rakhmerov: yes. there are a number of locations where this has to be fixed | 10:52 |
sharatss | rakhmerov: a BP must be raised against this issue i guess | 10:53 |
rakhmerov | нуы | 10:54 |
rakhmerov | yes | 10:55 |
rakhmerov | the BP exists already | 10:55 |
rakhmerov | but it doesn't have all the details | 10:55 |
sharatss | rakhmerov: ok. I will search for it and try to add the details | 10:55 |
rakhmerov | this is the API part of it: https://blueprints.launchpad.net/mistral/+spec/mistral-api-v3 | 10:58 |
rakhmerov | but I think we're missing CLI part | 10:58 |
rakhmerov | you can file it at https://blueprints.launchpad.net/python-mistralclient | 10:58 |
rakhmerov | and attach the etherpad https://etherpad.openstack.org/p/mistral-cli-v3 to it | 10:59 |
rakhmerov | I started gathering some ideas in it | 10:59 |
rakhmerov | I have some more in my head but didn't have time yet to put them into it | 11:00 |
sharatss | rakhmerov: thanks for the info | 11:02 |
sharatss | rakhmerov: i will file the Bp ASAP | 11:03 |
rakhmerov | ok | 11:03 |
rakhmerov | sharatss: did you have a chance to look at the bug I gave you recently? | 11:03 |
rakhmerov | launching services | 11:03 |
sharatss | rakhmerov: yes | 11:05 |
sharatss | rakhmerov: i will try to commit it soon | 11:05 |
rakhmerov | looks more or less clear what changes need to be made? | 11:05 |
sharatss | rakhmerov: yes. only thing is I need to map the murano changes with ours | 11:06 |
rakhmerov | ok | 11:06 |
sharatss | rakhmerov: i am working on it | 11:07 |
rakhmerov | good | 11:07 |
*** shardy is now known as shardy_mtg | 11:07 | |
openstackgerrit | Merged openstack/mistral: Delete unnecessary comma https://review.openstack.org/370000 | 11:30 |
*** openstackgerrit has quit IRC | 11:34 | |
*** openstackgerrit has joined #openstack-mistral | 11:34 | |
*** hparekh has quit IRC | 11:43 | |
openstackgerrit | Renat Akhmerov proposed openstack/mistral: Add a smarter delay between workflow completion checks https://review.openstack.org/369449 | 12:17 |
openstackgerrit | Renat Akhmerov proposed openstack/mistral: Optimize the logic that check if 'join' task is allowed to start https://review.openstack.org/369292 | 12:17 |
*** shardy_mtg is now known as shardy | 12:39 | |
*** jaosorior has quit IRC | 13:01 | |
*** dprince has joined #openstack-mistral | 13:01 | |
openstackgerrit | Andras Kovi proposed openstack/python-mistralclient: Send access info to server https://review.openstack.org/354803 | 13:01 |
openstackgerrit | Renat Akhmerov proposed openstack/mistral: Add a smarter delay between workflow completion checks https://review.openstack.org/369449 | 13:01 |
openstackgerrit | Renat Akhmerov proposed openstack/mistral: Optimize the logic that check if 'join' task is allowed to start https://review.openstack.org/369292 | 13:01 |
openstackgerrit | Renat Akhmerov proposed openstack/mistral: WIP: removing redundant stuff from workflow context https://review.openstack.org/370105 | 13:01 |
*** dprince has quit IRC | 13:01 | |
*** jaosorior has joined #openstack-mistral | 13:01 | |
openstackgerrit | Lucky samadhiya proposed openstack/mistral: changes get next execution time from local time zone to UTC timezone Closes-Bug: #1592164 https://review.openstack.org/348146 | 13:06 |
openstack | bug 1592164 in Mistral "cron-trigger shows execution times in local timezone" [Undecided,In progress] https://launchpad.net/bugs/1592164 - Assigned to Lucky samadhiya (lucky-samadhiya) | 13:06 |
openstackgerrit | Lucky samadhiya proposed openstack/mistral: changes get next execution time from local time zone to UTC timezone Closes-Bug: #1592164 https://review.openstack.org/348146 | 13:23 |
openstack | bug 1592164 in Mistral "cron-trigger shows execution times in local timezone" [Undecided,In progress] https://launchpad.net/bugs/1592164 - Assigned to Lucky samadhiya (lucky-samadhiya) | 13:23 |
*** shardy has quit IRC | 13:23 | |
*** jistr|mtg is now known as jistr | 13:24 | |
openstackgerrit | Renat Akhmerov proposed openstack/mistral: Remove workflow spec, input and params from workflow context https://review.openstack.org/370105 | 13:35 |
*** jistr is now known as jistr|mtg | 13:49 | |
*** shardy has joined #openstack-mistral | 13:55 | |
*** bobh has joined #openstack-mistral | 14:01 | |
rakhmerov | hparekh, jaosorior: guys, please take a look at https://review.openstack.org/#/c/355368/ again, Andras responded | 14:01 |
openstackgerrit | Andras Kovi proposed openstack/mistral: Accept service catalog from client side https://review.openstack.org/355368 | 14:05 |
*** bobh has quit IRC | 14:08 | |
*** tonytan4ever has joined #openstack-mistral | 14:24 | |
*** nmakhotkin has quit IRC | 14:27 | |
openstackgerrit | Lucky samadhiya proposed openstack/mistral: changes get next execution time from local time zone to UTC timezone Closes-Bug: #1592164 https://review.openstack.org/348146 | 14:30 |
openstack | bug 1592164 in Mistral "cron-trigger shows execution times in local timezone" [Undecided,In progress] https://launchpad.net/bugs/1592164 - Assigned to Lucky samadhiya (lucky-samadhiya) | 14:30 |
*** jaosorior has quit IRC | 14:31 | |
*** brian_price has joined #openstack-mistral | 14:37 | |
openstackgerrit | Renat Akhmerov proposed openstack/mistral: Add a smarter delay between workflow completion checks https://review.openstack.org/369449 | 14:47 |
openstackgerrit | Renat Akhmerov proposed openstack/mistral: Remove workflow spec, input and params from workflow context https://review.openstack.org/370105 | 14:47 |
openstackgerrit | Renat Akhmerov proposed openstack/mistral: Optimize the logic that check if 'join' task is allowed to start https://review.openstack.org/369292 | 14:47 |
*** rrecio has joined #openstack-mistral | 14:55 | |
*** rrecio_ has joined #openstack-mistral | 14:56 | |
*** rrecio has quit IRC | 14:59 | |
*** jistr|mtg is now known as jistr | 15:01 | |
*** bobh has joined #openstack-mistral | 15:05 | |
*** bobh has quit IRC | 15:10 | |
openstackgerrit | Renat Akhmerov proposed openstack/mistral: Add a smarter delay between workflow completion checks https://review.openstack.org/369449 | 15:12 |
openstackgerrit | Renat Akhmerov proposed openstack/mistral: Remove workflow spec, input and params from workflow context https://review.openstack.org/370105 | 15:12 |
openstackgerrit | Renat Akhmerov proposed openstack/mistral: Optimize the logic that check if 'join' task is allowed to start https://review.openstack.org/369292 | 15:12 |
*** dnalezyty has joined #openstack-mistral | 15:19 | |
*** dnalezyty has quit IRC | 15:19 | |
*** shardy has quit IRC | 15:43 | |
openstackgerrit | Renat Akhmerov proposed openstack/mistral: Add a smarter delay between workflow completion checks https://review.openstack.org/369449 | 15:57 |
openstackgerrit | Renat Akhmerov proposed openstack/mistral: Remove workflow spec, input and params from workflow context https://review.openstack.org/370105 | 15:57 |
openstackgerrit | Renat Akhmerov proposed openstack/mistral: Optimize the logic that check if 'join' task is allowed to start https://review.openstack.org/369292 | 15:57 |
*** bobh has joined #openstack-mistral | 16:06 | |
*** bobh has quit IRC | 16:10 | |
*** Kiall_ is now known as Kiall | 17:03 | |
*** _gryf has quit IRC | 17:06 | |
*** bobh has joined #openstack-mistral | 17:07 | |
*** brian_price has quit IRC | 17:08 | |
*** _gryf has joined #openstack-mistral | 17:10 | |
*** bobh has quit IRC | 17:11 | |
EmilienM | hi ! | 17:27 |
EmilienM | I have a doubt about https://github.com/openstack/mistral/commit/4e1e358c8b595eba42ead91a118fd14b8b5546e7 | 17:27 |
EmilienM | and I think (not sure) it broke tripleo CI | 17:27 |
EmilienM | rakhmerov: fyi ^ | 17:28 |
rakhmerov | EmilienM: hi, what exactly is broken? | 17:29 |
rakhmerov | can you file a bug pls? | 17:29 |
ddeja | EmilienM: hi, maybe this works out your problem? https://review.openstack.org/#/c/370000/ | 17:29 |
EmilienM | sorry I forgot to show bug report | 17:29 |
rakhmerov | it's a new endpoint | 17:29 |
EmilienM | https://bugs.launchpad.net/tripleo/+bug/1623606 | 17:29 |
openstack | Launchpad bug 1623606 in tripleo "CI: jobs failing with mistral MessagingTimeout" [Undecided,New] | 17:29 |
rakhmerov | Oh, yes, that patch that ddeja pointed to may help | 17:30 |
rakhmerov | no? | 17:30 |
ddeja | rakhmerov: I think not by looking on the bug, it was related to API service, not RPC layer | 17:31 |
EmilienM | I think mistral is starting well | 17:31 |
EmilienM | i think we're having a different problem | 17:31 |
rakhmerov | ok | 17:32 |
rakhmerov | hm... I know one issue that may also be related to that (not sure though) | 17:33 |
rakhmerov | https://bugs.launchpad.net/mistral/+bug/1622534 | 17:33 |
openstack | Launchpad bug 1622534 in Mistral "Incorrect launching of Mistral components" [High,New] - Assigned to Renat Akhmerov (rakhmerov) | 17:33 |
rakhmerov | sharatss is working on it | 17:34 |
ddeja | rakhmerov: interesting, but looking into logs http://logs.openstack.org/50/370250/2/check/gate-tripleo-ci-centos-7-nonha-multinode/9e8c125/logs/var/log/mistral/mistral-server.txt.gz | 17:34 |
ddeja | I'm seeing '2016-09-14 16:58:07.466 32010 ERROR wsme.api [-] Server-side error: "MessagingTimeout: Timed out waiting for a reply to message ID 1faf8b08051842478b09af5eee1e2e9c". Detail:' | 17:34 |
ddeja | and a second later 2016-09-14 16:58:07.505 32010 INFO oslo_messaging._drivers.amqpdriver [-] No calling threads waiting for msg_id : 1faf8b08051842478b09af5eee1e2e9c | 17:35 |
ddeja | it looks like timeout is set to 0? | 17:35 |
EmilienM | let me check config | 17:35 |
EmilienM | http://logs.openstack.org/50/370250/2/check/gate-tripleo-ci-centos-7-nonha-multinode/9e8c125/logs/etc/mistral/mistral.conf.txt.gz | 17:35 |
ddeja | it looks like the message is comming, but a little bit too late | 17:35 |
*** sharatss has quit IRC | 17:36 | |
EmilienM | what is the param you're looking for? | 17:36 |
ddeja | EmilienM: if I remember correclty, it is rpc_response_timeout | 17:37 |
ddeja | and it is left as default 60seconds | 17:37 |
*** Alexey_Abashkin has joined #openstack-mistral | 17:38 | |
EmilienM | yeah we don't do anything fancy | 17:39 |
*** AlexeyAbashkin has quit IRC | 17:41 | |
*** brian_price has joined #openstack-mistral | 17:45 | |
ddeja | EmilienM: it occurs every time? From what I see in the logs it looks like the message never left the Engine, it just timedout, so it looks like some connection dropped | 17:56 |
ddeja | and I'm not able to reproduce it using devstack | 17:57 |
EmilienM | ddeja: it looks like it occurs every time | 17:57 |
EmilienM | we promoted tripleo CI to use latest mistral, so we think it might be something merged very recently | 17:58 |
ddeja | EmilienM: can you point me the repo name on gerrit? | 17:58 |
EmilienM | ddeja: which repo? | 17:58 |
ddeja | that has failing CI | 17:59 |
EmilienM | tripleo | 17:59 |
EmilienM | we have a bunch of repos | 17:59 |
EmilienM | running same jobs | 17:59 |
ddeja | oh, so I can choose any of them, OK | 17:59 |
EmilienM | yea | 17:59 |
*** vishwanathj_zzz is now known as vishwanathj | 18:01 | |
EmilienM | ddeja: tripleo.org/cistatus.html | 18:01 |
EmilienM | you can pick gate-tripleo-ci-centos-7-nonha-multinode logs | 18:01 |
EmilienM | and see that it's consistent | 18:01 |
d0ugal | EmilienM: Just catching up - do you have a link to an error? | 18:02 |
d0ugal | EmilienM: got one | 18:02 |
EmilienM | d0ugal: https://bugs.launchpad.net/tripleo/+bug/1623606 | 18:02 |
openstack | Launchpad bug 1623606 in tripleo "CI: jobs failing with mistral MessagingTimeout" [Undecided,New] | 18:02 |
ddeja | EmilienM: ok, I see that it fails everytime it tries to run action tripleo.list_plans, but it is not a standard mistral action, can you point me to its definition please? | 18:03 |
d0ugal | ddeja: https://github.com/openstack/tripleo-common/blob/master/tripleo_common/actions/plan.py#L160 | 18:03 |
ddeja | d0ugal: thanks | 18:03 |
d0ugal | EmilienM: is there a way to easily run a test action? | 18:04 |
d0ugal | EmilienM: i.e. put something like this in CI to check it works: mistral run-action std.echo '{"output":"test"}' | 18:05 |
EmilienM | d0ugal: maybe in tripleo-ci repo? | 18:07 |
*** rbrady has joined #openstack-mistral | 18:07 | |
d0ugal | EmilienM: I'll update my undercloud and see what happens. | 18:07 |
*** bobh has joined #openstack-mistral | 18:07 | |
ddeja | Sorry guys, I need to drop for now (it's 8pm my time) Please confirm if this still happens when using some std action (perfectly both std.echo and std.noop) and ping me tommorow if you need any help | 18:09 |
d0ugal | ddeja: will do, thanks. I wont be around for much longer anyway (7pm here) | 18:10 |
*** brian_price has quit IRC | 18:11 | |
*** bobh has quit IRC | 18:12 | |
*** brian_price has joined #openstack-mistral | 18:13 | |
d0ugal | EmilienM: so, the action that errors runs fine on my new undercloud :/ | 18:22 |
d0ugal | I guess I'm not using the same packages as CI - how do I do that? | 18:23 |
EmilienM | d0ugal: are you running recent packaging? tripleo CI was promoted last night | 18:23 |
d0ugal | EmilienM: openstack-mistral-api-3.0.0-0.20160913191255.94bfbea.el7.centos.noarch | 18:24 |
d0ugal | That looks like last night, if I am understanding it properly | 18:24 |
EmilienM | does it contain https://github.com/openstack/mistral/commit/4e1e358c8b595eba42ead91a118fd14b8b5546e7 ? | 18:24 |
d0ugal | EmilienM: Checking. | 18:25 |
d0ugal | EmilienM: Yes | 18:26 |
rakhmerov | sorry, was distracted by something | 18:29 |
rakhmerov | EmilienM: what version of Mistral are you using? | 18:29 |
EmilienM | rakhmerov: trunk | 18:30 |
EmilienM | let me give you the commit | 18:30 |
EmilienM | 94bfbea | 18:30 |
EmilienM | so the only patch we miss is https://github.com/openstack/mistral/commit/b1937191ee0cf4d525919d2cd9910493babff18f | 18:31 |
EmilienM | could it be causing an RPC timeout? | 18:31 |
* EmilienM brb lunch | 18:31 | |
rakhmerov | it is an RPC timeout but there must be a reason for that somewhere in the log | 18:31 |
rakhmerov | something takes too long or ends up in a deadlock, dunno | 18:32 |
rakhmerov | in the worst case, there's a swallowed exception somewhere | 18:33 |
rakhmerov | ok, sorry, I have to end the day now | 18:34 |
rakhmerov | I'm now working on something that may be related to that so I'll try to have a look at that tomorrow | 18:34 |
*** tonytan4ever has quit IRC | 18:51 | |
*** bobh has joined #openstack-mistral | 19:08 | |
*** bobh has quit IRC | 19:13 | |
openstackgerrit | Emilien Macchi proposed openstack/mistral: Revert "Add event trigger REST API" https://review.openstack.org/370402 | 19:45 |
openstackgerrit | Emilien Macchi proposed openstack/mistral: Revert "Add event trigger REST API" https://review.openstack.org/370402 | 19:48 |
openstackgerrit | Emilien Macchi proposed openstack/mistral: Revert "Add event trigger REST API" https://review.openstack.org/370402 | 19:48 |
*** tonytan4ever has joined #openstack-mistral | 19:52 | |
*** tonytan4ever has quit IRC | 19:53 | |
*** tonytan4ever has joined #openstack-mistral | 19:54 | |
*** bobh has joined #openstack-mistral | 20:04 | |
*** Ravikiran_K has quit IRC | 20:07 | |
*** bobh_ has joined #openstack-mistral | 20:16 | |
*** bobh has quit IRC | 20:19 | |
*** tonytan4ever has quit IRC | 20:22 | |
*** tonytan4ever has joined #openstack-mistral | 21:53 | |
*** tonytan4ever has quit IRC | 21:58 | |
*** rbrady has quit IRC | 22:02 | |
*** bobh_ has quit IRC | 23:01 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!