Wednesday, 2016-09-14

*** rrecio_ has quit IRC00:44
*** cheneydc has joined #openstack-mistral00:59
*** gyee has quit IRC01:03
*** cheneydc has quit IRC01:27
*** hparekh has joined #openstack-mistral03:42
*** vishwanathj is now known as vishwanathj_zzz04:09
*** sharatss has joined #openstack-mistral04:11
*** jaosorior has joined #openstack-mistral04:44
*** jaosorior has quit IRC05:00
*** jaosorior has joined #openstack-mistral05:01
openstackgerritAsmita singh proposed openstack/mistral: Added pyv8 in requirement.txt in mistral code for bug ID : 1551561 closes-Bug: #1551561  https://review.openstack.org/36983005:42
openstackbug 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
openstackgerritHardik Parekh proposed openstack/mistral: Fixed task in_bound context when retrying  https://review.openstack.org/36983405:58
*** jaosorior has quit IRC06:29
*** jaosorior has joined #openstack-mistral06:30
openstackgerritAsmita singh proposed openstack/mistral: Added pyv8 in requirement.txt in mistral code for bug ID : 1551561 closes-Bug: #1551561  https://review.openstack.org/36985106:48
openstackbug 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 sharat06:51
*** FL1SK has quit IRC07:02
*** shardy has joined #openstack-mistral07:06
*** jistr is now known as jistr|mtgs07:09
*** jistr|mtgs is now known as jistr|mtg07:09
*** shardy is now known as shardy_mtg07:16
*** Ravikiran_K has joined #openstack-mistral07:19
*** jaosorior has quit IRC07:27
*** jaosorior has joined #openstack-mistral07:27
openstackgerritMerged openstack/mistral-dashboard: Remove .mailmap and AUTHORS file since it's no longer needed  https://review.openstack.org/36732307:28
*** jaosorior has quit IRC07:29
*** jaosorior has joined #openstack-mistral07:30
*** nmakhotkin has joined #openstack-mistral07:41
sharathi08:02
sharatneed to discuss about https://bugs.launchpad.net/python-mistralclient/+bug/1581520 this bug08:02
openstackLaunchpad 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-mistral09:02
*** sharat has quit IRC09:04
openstackgerritMerged openstack/python-mistralclient: Use oslotest instead of testtools/unittest  https://review.openstack.org/36541409:19
*** sharatss has joined #openstack-mistral09:30
openstackgerritMerged openstack/python-mistralclient: Use requests-mock for testing  https://review.openstack.org/36541509:35
*** jaosorior is now known as jaosorior_lunch09:54
openstackgerritDawid Deja proposed openstack/mistral: Delete unnecessary comma  https://review.openstack.org/37000010:15
*** jaosorior_lunch is now known as jaosorior10:27
ddejarakhmerov, 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
hparekhd0ugal, Ok sure10:31
hparekhddeja, Ok sure10:31
d0ugal:)10:31
rakhmerovddeja: hi, ok10:46
rakhmerovsharatss: hi, I would say it's not even a bug. It's rather a blueprint to address all those issues10:48
rakhmerovwe've been planning to work on API and CLI v3 in Newton had to move it to the next cycle10:48
rakhmerovit should address these problems10:49
sharatssrakhmerov: thanks10:49
sharatssrakhmerov: so that needs no work as of now?10:50
*** shardy_mtg is now known as shardy10:51
rakhmerovyeah, 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 CLI10:51
rakhmerovsharatss: I would say yes, I would postpone it for now till the next cycle10:51
rakhmerovand we'll need to start working on it in the beginning of the cycle to have enough time to complete it10:52
sharatssrakhmerov: yes. there are a number of locations where this has to be fixed10:52
sharatssrakhmerov: a BP must be raised against this issue i guess10:53
rakhmerovнуы10:54
rakhmerovyes10:55
rakhmerovthe BP exists already10:55
rakhmerovbut it doesn't have all the details10:55
sharatssrakhmerov: ok. I will search for it and try to add the details10:55
rakhmerovthis is the API part of it: https://blueprints.launchpad.net/mistral/+spec/mistral-api-v310:58
rakhmerovbut I think we're missing CLI part10:58
rakhmerovyou can file it at https://blueprints.launchpad.net/python-mistralclient10:58
rakhmerovand attach the etherpad https://etherpad.openstack.org/p/mistral-cli-v3 to it10:59
rakhmerovI started gathering some ideas in it10:59
rakhmerovI have some more in my head but didn't have time yet to put them into it11:00
sharatssrakhmerov: thanks for the info11:02
sharatssrakhmerov: i will file the Bp ASAP11:03
rakhmerovok11:03
rakhmerovsharatss: did you have a chance to look at the bug I gave you recently?11:03
rakhmerovlaunching services11:03
sharatssrakhmerov: yes11:05
sharatssrakhmerov: i will try to commit it soon11:05
rakhmerovlooks more or less clear what changes need to be made?11:05
sharatssrakhmerov: yes. only thing is I need to map the murano changes with ours11:06
rakhmerovok11:06
sharatssrakhmerov: i am working on it11:07
rakhmerovgood11:07
*** shardy is now known as shardy_mtg11:07
openstackgerritMerged openstack/mistral: Delete unnecessary comma  https://review.openstack.org/37000011:30
*** openstackgerrit has quit IRC11:34
*** openstackgerrit has joined #openstack-mistral11:34
*** hparekh has quit IRC11:43
openstackgerritRenat Akhmerov proposed openstack/mistral: Add a smarter delay between workflow completion checks  https://review.openstack.org/36944912:17
openstackgerritRenat Akhmerov proposed openstack/mistral: Optimize the logic that check if 'join' task is allowed to start  https://review.openstack.org/36929212:17
*** shardy_mtg is now known as shardy12:39
*** jaosorior has quit IRC13:01
*** dprince has joined #openstack-mistral13:01
openstackgerritAndras Kovi proposed openstack/python-mistralclient: Send access info to server  https://review.openstack.org/35480313:01
openstackgerritRenat Akhmerov proposed openstack/mistral: Add a smarter delay between workflow completion checks  https://review.openstack.org/36944913:01
openstackgerritRenat Akhmerov proposed openstack/mistral: Optimize the logic that check if 'join' task is allowed to start  https://review.openstack.org/36929213:01
openstackgerritRenat Akhmerov proposed openstack/mistral: WIP: removing redundant stuff from workflow context  https://review.openstack.org/37010513:01
*** dprince has quit IRC13:01
*** jaosorior has joined #openstack-mistral13:01
openstackgerritLucky samadhiya proposed openstack/mistral: changes get next execution time from local time zone to UTC timezone Closes-Bug: #1592164  https://review.openstack.org/34814613:06
openstackbug 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
openstackgerritLucky samadhiya proposed openstack/mistral: changes get next execution time from local time zone to UTC timezone Closes-Bug: #1592164  https://review.openstack.org/34814613:23
openstackbug 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 IRC13:23
*** jistr|mtg is now known as jistr13:24
openstackgerritRenat Akhmerov proposed openstack/mistral: Remove workflow spec, input and params from workflow context  https://review.openstack.org/37010513:35
*** jistr is now known as jistr|mtg13:49
*** shardy has joined #openstack-mistral13:55
*** bobh has joined #openstack-mistral14:01
rakhmerovhparekh, jaosorior: guys, please take a look at https://review.openstack.org/#/c/355368/ again, Andras responded14:01
openstackgerritAndras Kovi proposed openstack/mistral: Accept service catalog from client side  https://review.openstack.org/35536814:05
*** bobh has quit IRC14:08
*** tonytan4ever has joined #openstack-mistral14:24
*** nmakhotkin has quit IRC14:27
openstackgerritLucky samadhiya proposed openstack/mistral: changes get next execution time from local time zone to UTC timezone Closes-Bug: #1592164  https://review.openstack.org/34814614:30
openstackbug 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 IRC14:31
*** brian_price has joined #openstack-mistral14:37
openstackgerritRenat Akhmerov proposed openstack/mistral: Add a smarter delay between workflow completion checks  https://review.openstack.org/36944914:47
openstackgerritRenat Akhmerov proposed openstack/mistral: Remove workflow spec, input and params from workflow context  https://review.openstack.org/37010514:47
openstackgerritRenat Akhmerov proposed openstack/mistral: Optimize the logic that check if 'join' task is allowed to start  https://review.openstack.org/36929214:47
*** rrecio has joined #openstack-mistral14:55
*** rrecio_ has joined #openstack-mistral14:56
*** rrecio has quit IRC14:59
*** jistr|mtg is now known as jistr15:01
*** bobh has joined #openstack-mistral15:05
*** bobh has quit IRC15:10
openstackgerritRenat Akhmerov proposed openstack/mistral: Add a smarter delay between workflow completion checks  https://review.openstack.org/36944915:12
openstackgerritRenat Akhmerov proposed openstack/mistral: Remove workflow spec, input and params from workflow context  https://review.openstack.org/37010515:12
openstackgerritRenat Akhmerov proposed openstack/mistral: Optimize the logic that check if 'join' task is allowed to start  https://review.openstack.org/36929215:12
*** dnalezyty has joined #openstack-mistral15:19
*** dnalezyty has quit IRC15:19
*** shardy has quit IRC15:43
openstackgerritRenat Akhmerov proposed openstack/mistral: Add a smarter delay between workflow completion checks  https://review.openstack.org/36944915:57
openstackgerritRenat Akhmerov proposed openstack/mistral: Remove workflow spec, input and params from workflow context  https://review.openstack.org/37010515:57
openstackgerritRenat Akhmerov proposed openstack/mistral: Optimize the logic that check if 'join' task is allowed to start  https://review.openstack.org/36929215:57
*** bobh has joined #openstack-mistral16:06
*** bobh has quit IRC16:10
*** Kiall_ is now known as Kiall17:03
*** _gryf has quit IRC17:06
*** bobh has joined #openstack-mistral17:07
*** brian_price has quit IRC17:08
*** _gryf has joined #openstack-mistral17:10
*** bobh has quit IRC17:11
EmilienMhi !17:27
EmilienMI have a doubt about https://github.com/openstack/mistral/commit/4e1e358c8b595eba42ead91a118fd14b8b5546e717:27
EmilienMand I think (not sure) it broke tripleo CI17:27
EmilienMrakhmerov: fyi ^17:28
rakhmerovEmilienM: hi, what exactly is broken?17:29
rakhmerovcan you file a bug pls?17:29
ddejaEmilienM: hi, maybe this works out your problem? https://review.openstack.org/#/c/370000/17:29
EmilienMsorry I forgot to show bug report17:29
rakhmerovit's a new endpoint17:29
EmilienMhttps://bugs.launchpad.net/tripleo/+bug/162360617:29
openstackLaunchpad bug 1623606 in tripleo "CI: jobs failing with mistral MessagingTimeout" [Undecided,New]17:29
rakhmerovOh, yes, that patch that ddeja pointed to may help17:30
rakhmerovno?17:30
ddejarakhmerov: I think not by looking on the bug, it was related to API service, not RPC layer17:31
EmilienMI think mistral is starting well17:31
EmilienMi think we're having a different problem17:31
rakhmerovok17:32
rakhmerovhm... I know one issue that may also be related to that (not sure though)17:33
rakhmerovhttps://bugs.launchpad.net/mistral/+bug/162253417:33
openstackLaunchpad bug 1622534 in Mistral "Incorrect launching of Mistral components" [High,New] - Assigned to Renat Akhmerov (rakhmerov)17:33
rakhmerovsharatss is working on it17:34
ddejarakhmerov: 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.gz17:34
ddejaI'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
ddejaand a second later 2016-09-14 16:58:07.505 32010 INFO oslo_messaging._drivers.amqpdriver [-] No calling threads waiting for msg_id : 1faf8b08051842478b09af5eee1e2e9c17:35
ddejait looks like timeout is set to 0?17:35
EmilienMlet me check config17:35
EmilienMhttp://logs.openstack.org/50/370250/2/check/gate-tripleo-ci-centos-7-nonha-multinode/9e8c125/logs/etc/mistral/mistral.conf.txt.gz17:35
ddejait looks like the message is comming, but a little bit too late17:35
*** sharatss has quit IRC17:36
EmilienMwhat is the param you're looking for?17:36
ddejaEmilienM: if I remember correclty, it is rpc_response_timeout17:37
ddejaand it is left as default 60seconds17:37
*** Alexey_Abashkin has joined #openstack-mistral17:38
EmilienMyeah we don't do anything fancy17:39
*** AlexeyAbashkin has quit IRC17:41
*** brian_price has joined #openstack-mistral17:45
ddejaEmilienM: 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 dropped17:56
ddejaand I'm not able to reproduce it using devstack17:57
EmilienMddeja: it looks like it occurs every time17:57
EmilienMwe promoted tripleo CI to use latest mistral, so we think it might be something merged very recently17:58
ddejaEmilienM: can you point me the repo name on gerrit?17:58
EmilienMddeja: which repo?17:58
ddejathat has failing CI17:59
EmilienMtripleo17:59
EmilienMwe have a bunch of repos17:59
EmilienMrunning same jobs17:59
ddejaoh, so I can choose any of them, OK17:59
EmilienMyea17:59
*** vishwanathj_zzz is now known as vishwanathj18:01
EmilienMddeja: tripleo.org/cistatus.html18:01
EmilienMyou can pick gate-tripleo-ci-centos-7-nonha-multinode logs18:01
EmilienMand see that it's consistent18:01
d0ugalEmilienM: Just catching up - do you have a link to an error?18:02
d0ugalEmilienM: got one18:02
EmilienMd0ugal: https://bugs.launchpad.net/tripleo/+bug/162360618:02
openstackLaunchpad bug 1623606 in tripleo "CI: jobs failing with mistral MessagingTimeout" [Undecided,New]18:02
ddejaEmilienM: 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
d0ugalddeja: https://github.com/openstack/tripleo-common/blob/master/tripleo_common/actions/plan.py#L16018:03
ddejad0ugal: thanks18:03
d0ugalEmilienM: is there a way to easily run a test action?18:04
d0ugalEmilienM: i.e. put something like this in CI to check it works: mistral run-action std.echo '{"output":"test"}'18:05
EmilienMd0ugal: maybe in tripleo-ci repo?18:07
*** rbrady has joined #openstack-mistral18:07
d0ugalEmilienM: I'll update my undercloud and see what happens.18:07
*** bobh has joined #openstack-mistral18:07
ddejaSorry 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 help18:09
d0ugalddeja: will do, thanks. I wont be around for much longer anyway (7pm here)18:10
*** brian_price has quit IRC18:11
*** bobh has quit IRC18:12
*** brian_price has joined #openstack-mistral18:13
d0ugalEmilienM: so, the action that errors runs fine on my new undercloud :/18:22
d0ugalI guess I'm not using the same packages as CI - how do I do that?18:23
EmilienMd0ugal: are you running recent packaging? tripleo CI was promoted last night18:23
d0ugalEmilienM: openstack-mistral-api-3.0.0-0.20160913191255.94bfbea.el7.centos.noarch18:24
d0ugalThat looks like last night, if I am understanding it properly18:24
EmilienMdoes it contain https://github.com/openstack/mistral/commit/4e1e358c8b595eba42ead91a118fd14b8b5546e7 ?18:24
d0ugalEmilienM: Checking.18:25
d0ugalEmilienM: Yes18:26
rakhmerovsorry, was distracted by something18:29
rakhmerovEmilienM: what version of Mistral are you using?18:29
EmilienMrakhmerov: trunk18:30
EmilienMlet me give you the commit18:30
EmilienM94bfbea18:30
EmilienMso the only patch we miss is https://github.com/openstack/mistral/commit/b1937191ee0cf4d525919d2cd9910493babff18f18:31
EmilienMcould it be causing an RPC timeout?18:31
* EmilienM brb lunch18:31
rakhmerovit is an RPC timeout but there must be a reason for that somewhere in the log18:31
rakhmerovsomething takes too long or ends up in a deadlock, dunno18:32
rakhmerovin the worst case, there's a swallowed exception somewhere18:33
rakhmerovok, sorry, I have to end the day now18:34
rakhmerovI'm now working on something that may be related to that so I'll try to have a look at that tomorrow18:34
*** tonytan4ever has quit IRC18:51
*** bobh has joined #openstack-mistral19:08
*** bobh has quit IRC19:13
openstackgerritEmilien Macchi proposed openstack/mistral: Revert "Add event trigger REST API"  https://review.openstack.org/37040219:45
openstackgerritEmilien Macchi proposed openstack/mistral: Revert "Add event trigger REST API"  https://review.openstack.org/37040219:48
openstackgerritEmilien Macchi proposed openstack/mistral: Revert "Add event trigger REST API"  https://review.openstack.org/37040219:48
*** tonytan4ever has joined #openstack-mistral19:52
*** tonytan4ever has quit IRC19:53
*** tonytan4ever has joined #openstack-mistral19:54
*** bobh has joined #openstack-mistral20:04
*** Ravikiran_K has quit IRC20:07
*** bobh_ has joined #openstack-mistral20:16
*** bobh has quit IRC20:19
*** tonytan4ever has quit IRC20:22
*** tonytan4ever has joined #openstack-mistral21:53
*** tonytan4ever has quit IRC21:58
*** rbrady has quit IRC22:02
*** bobh_ has quit IRC23:01

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