| *** jamielennox|away is now known as jamielennox | 00:08 | |
| *** harlowja_ has quit IRC | 00:32 | |
| *** harlowja has joined #openstack-mistral | 00:51 | |
| *** jamielennox is now known as jamielennox|away | 01:04 | |
| *** jamielennox|away is now known as jamielennox | 01:20 | |
| *** jrist has quit IRC | 01:21 | |
| *** Jizhaoxuan has joined #openstack-mistral | 01:26 | |
| *** jrist has joined #openstack-mistral | 01:34 | |
| *** bobh has joined #openstack-mistral | 01:58 | |
| *** bobh has quit IRC | 02:06 | |
| openstackgerrit | Jizhaoxuan proposed openstack/mistral: Modify the function "_get_spec_version(spec_dict)" https://review.openstack.org/404510 | 02:07 |
|---|---|---|
| *** bobh has joined #openstack-mistral | 02:18 | |
| openstackgerrit | Lingxian Kong proposed openstack/mistral: Support openstack context for workflow and task https://review.openstack.org/407240 | 02:22 |
| openstackgerrit | fengchaoyang proposed openstack/mistral: Added gnocchi action pack https://review.openstack.org/408005 | 02:37 |
| *** bobh has quit IRC | 02:45 | |
| *** harlowja has quit IRC | 02:55 | |
| openstackgerrit | Jizhaoxuan proposed openstack/mistral: Modify the link in 'README.rst' https://review.openstack.org/408915 | 02:55 |
| *** sharatss has quit IRC | 03:16 | |
| *** Kiall has quit IRC | 03:18 | |
| *** Kiall has joined #openstack-mistral | 03:20 | |
| *** jrist has quit IRC | 03:30 | |
| *** jrist has joined #openstack-mistral | 03:33 | |
| *** sharatss has joined #openstack-mistral | 04:41 | |
| *** ist has joined #openstack-mistral | 05:22 | |
| *** sharatss has quit IRC | 05:31 | |
| *** sharatss has joined #openstack-mistral | 05:32 | |
| *** jaosorior has joined #openstack-mistral | 07:00 | |
| *** ist has quit IRC | 07:07 | |
| *** ist has joined #openstack-mistral | 07:07 | |
| *** janki has joined #openstack-mistral | 07:27 | |
| rakhmerov | ddeja: hi Dawid, did you see this? http://logs.openstack.org/15/408515/3/check/gate-mistral-tox-db-unit-mysql-ubuntu-xenial/d807bd5/testr_results.html.gz | 07:32 |
| *** jtomasek has joined #openstack-mistral | 07:54 | |
| *** jaosorior has quit IRC | 07:55 | |
| *** jaosorior has joined #openstack-mistral | 07:55 | |
| *** jaosorior has quit IRC | 08:00 | |
| *** jaosorior has joined #openstack-mistral | 08:02 | |
| ddeja | rakhmerov: nope | 08:22 |
| ddeja | but that's weird | 08:22 |
| rakhmerov | at first glance it's related to kombu | 08:22 |
| rakhmerov | but I'm not sure | 08:22 |
| rakhmerov | both mysql and postgres gates crash because of this | 08:23 |
| ddeja | but hm, those are unit tests? | 08:24 |
| ddeja | oh, unit tests but using postgres/mysql | 08:24 |
| ddeja | Am I right, rakhmerov? | 08:25 |
| ddeja | in unittests we do not use kombu, so I don't think it is related to it | 08:26 |
| ddeja | it's more likely that it is related to https://review.openstack.org/#/c/406143/ | 08:27 |
| ddeja | but hm, in this change it all passes... | 08:27 |
| ddeja | and it passed in all recent changes | 08:28 |
| ddeja | rakhmerov: Are we sure it is not related to the change itself? | 08:29 |
| rakhmerov | yes | 08:29 |
| rakhmerov | unit test but with a different DB backend | 08:29 |
| ddeja | yes, I see | 08:29 |
| ddeja | hm | 08:29 |
| rakhmerov | I don't think it's related to this time change | 08:30 |
| ddeja | why do you think it is related to the kombu? | 08:30 |
| rakhmerov | well, I guess it's some dependency problem | 08:30 |
| rakhmerov | introduced by someone else | 08:30 |
| ddeja | can we try to recheck it | 08:30 |
| ddeja | I'd like to see if it happens multiple times | 08:30 |
| rakhmerov | many tests failed with this | 08:30 |
| rakhmerov | but yes, we can try | 08:31 |
| * ddeja needs to go for a status meeting, will be back in 15-20 minutes | 08:31 | |
| rakhmerov | File "/home/jenkins/workspace/gate-mistral-tox-db-unit-mysql-ubuntu-xenial/.venv/local/lib/python2.7/site-packages/kombu/five.py", line 6, in <module> | 08:31 |
| rakhmerov | import vine.five | 08:31 |
| rakhmerov | ImportError: No module named vine.five | 08:31 |
| rakhmerov | this is why | 08:32 |
| rakhmerov | what is "vine"? | 08:32 |
| rakhmerov | ddeja: ok, Dawid, I also need to go to a meeting | 08:32 |
| rakhmerov | will be back in an hour | 08:32 |
| rakhmerov | pls take a look if you have time | 08:32 |
| *** Jizhaoxuan has quit IRC | 08:46 | |
| *** janki has quit IRC | 08:47 | |
| *** Jizhaoxuan has joined #openstack-mistral | 08:47 | |
| *** janki has joined #openstack-mistral | 08:53 | |
| openstackgerrit | Merged openstack/mistral: Remove commented-out Apache 2 classifier from setup.cfg https://review.openstack.org/407290 | 08:58 |
| ddeja | rakhmerov: hm, I don't know what "vine" is | 09:00 |
| ddeja | but a bad news is that it is not related to kombu driver itself | 09:00 |
| ddeja | since oslo.messaging also uses kombu | 09:01 |
| openstackgerrit | Dougal Matthews proposed openstack/mistral: Use the with keyword dealing with file objects https://review.openstack.org/408395 | 09:12 |
| openstackgerrit | Jizhaoxuan proposed openstack/mistral: change the cron-trigger execution time from localtime to UTC https://review.openstack.org/406740 | 09:17 |
| rakhmerov | ddeja: I see | 09:31 |
| rakhmerov | maybe it's a known problem | 09:31 |
| ddeja | rakhmerov what is weird | 09:31 |
| ddeja | kombu was installed... twice | 09:31 |
| ddeja | one in version 3.0.37, which not has vine as a dependency | 09:32 |
| ddeja | and one in version 4.0.1, which should be fine | 09:32 |
| ddeja | rakhmerov: we are looking on packages version with _gryf and we are both "WTF O.O" | 09:36 |
| openstackgerrit | Merged openstack/python-mistralclient: Remove commented-out Apache 2 classifier from setup.cfg https://review.openstack.org/408481 | 09:37 |
| openstackgerrit | Merged openstack/python-mistralclient: Move json.loads() method to utils.py, and use "with" to deal with file objects https://review.openstack.org/404336 | 09:38 |
| *** thrash has joined #openstack-mistral | 09:45 | |
| *** thrash has joined #openstack-mistral | 09:45 | |
| ddeja | rakhmerov: ok, we know why gates are failing | 09:45 |
| ddeja | starting today tox started to install kombu in version 4.0.1 | 09:45 |
| ddeja | yesterday it was 3.0.37 | 09:45 |
| rakhmerov | ddeja: I'm still on a meeting, I'll read your messages later | 09:45 |
| ddeja | what is weird, in requirements it still says 3.0.37 | 09:46 |
| *** dtantsur|afk is now known as dtantsur | 09:46 | |
| *** thrash|g0ne has quit IRC | 09:46 | |
| rakhmerov | ddeja: ok, I'm here | 10:04 |
| rakhmerov | I see | 10:05 |
| rakhmerov | ddeja: so is it a bug with oslo.m dependencies? | 10:06 |
| openstackgerrit | Sharat Sharma proposed openstack/python-mistralclient: Initial commit for python-mistralclient document https://review.openstack.org/404736 | 10:07 |
| rakhmerov | in oslo.messaging requirements.txt: kombu!=4.0.0,!=4.0.1,>=3.0.25 # BSD | 10:07 |
| ddeja | rakhmerov: I've just found it | 10:07 |
| ddeja | wait a second | 10:07 |
| rakhmerov | ok | 10:08 |
| ddeja | rakhmerov: https://review.openstack.org/#/c/408643/ | 10:08 |
| ddeja | but it is not in oslo yet | 10:09 |
| ddeja | well, it is, but on master, not in 5.14 | 10:09 |
| ddeja | so we should add it or wait till new version of oslo | 10:10 |
| openstackgerrit | Merged openstack/mistral: Modify the function "_get_spec_version(spec_dict)" https://review.openstack.org/404510 | 10:10 |
| rakhmerov | ddeja: yeah, I see | 10:12 |
| rakhmerov | oslo.m is now 5.2.0 | 10:13 |
| ddeja | it is 5.14 | 10:13 |
| rakhmerov | I see 5.2.0 in our requirements.txt | 10:13 |
| rakhmerov | ooh, it's oslo.messaging>=5.2.0 | 10:13 |
| rakhmerov | yes | 10:13 |
| ddeja | https://review.openstack.org/#/c/408098/ | 10:14 |
| rakhmerov | ok | 10:14 |
| ddeja | it'll be fixed in 5.15 I guess | 10:15 |
| rakhmerov | ok | 10:15 |
| rakhmerov | we can can just wait I guess | 10:15 |
| ddeja | ok | 10:15 |
| rakhmerov | ddeja: I just looked at #openstack-oslo channel | 10:16 |
| openstackgerrit | Merged openstack/mistral: Modify the link in 'README.rst' https://review.openstack.org/408915 | 10:16 |
| rakhmerov | they were discussing this problem | 10:16 |
| rakhmerov | ddeja: https://review.openstack.org/#/c/409047/ | 10:18 |
| ddeja | oh, great | 10:19 |
| rakhmerov | yeah, so just waiting.. | 10:19 |
| ddeja | so it won't be long | 10:19 |
| rakhmerov | today I believe | 10:20 |
| rakhmerov | when people in US wake up ;) | 10:20 |
| ddeja | yup | 10:20 |
| openstackgerrit | Merged openstack/mistral: Use the with keyword dealing with file objects https://review.openstack.org/408395 | 10:46 |
| *** janki has quit IRC | 10:47 | |
| *** janki has joined #openstack-mistral | 10:58 | |
| *** jaosorior is now known as jaosorior_lunch | 11:00 | |
| *** tuan_ has joined #openstack-mistral | 11:12 | |
| tuan_ | Hi guys | 11:15 |
| tuan_ | we have some our use cases related to the timeout in mistral workflow | 11:15 |
| tuan_ | actually, as we know that in action of mistral we have the timeout | 11:16 |
| tuan_ | when the timeout is reached but executor does not recognize it? | 11:16 |
| *** Jizhaoxuan has quit IRC | 11:43 | |
| *** jaosorior_lunch is now known as jaosorior | 11:55 | |
| openstackgerrit | Lingxian Kong proposed openstack/mistral: Support openstack context for workflow and task https://review.openstack.org/407240 | 12:09 |
| *** ist has quit IRC | 12:12 | |
| openstackgerrit | Sharat Sharma proposed openstack/mistral: Added test cases for a few possible scenarios https://review.openstack.org/409117 | 12:22 |
| openstackgerrit | Sharat Sharma proposed openstack/mistral: Added test cases for a few possible scenarios https://review.openstack.org/409117 | 12:25 |
| *** ist has joined #openstack-mistral | 12:26 | |
| *** jamielennox is now known as jamielennox|away | 12:33 | |
| ddeja | Hi tuan_ Could you provide an example workflow and describe which action couses problems? | 12:58 |
| *** pradk has joined #openstack-mistral | 13:01 | |
| *** ist has quit IRC | 13:16 | |
| *** sharatss has quit IRC | 13:19 | |
| openstackgerrit | Pradeep Kilambi proposed openstack/mistral: Add aodh actions to mistral https://review.openstack.org/408684 | 13:22 |
| tuan_ | Hi ddeja | 13:23 |
| tuan_ | it is not the problem of action | 13:23 |
| tuan_ | it is our own scenario | 13:23 |
| *** pradk has left #openstack-mistral | 13:23 | |
| tuan_ | since we want that, if a workflow fails, we would like to do the next workflows | 13:24 |
| tuan_ | we also use on-error condition | 13:24 |
| ddeja | tuan_: "since we want that" -> what do you mean by "that"? | 13:25 |
| tuan_ | it means like: if a workflow fails | 13:25 |
| tuan_ | we would like to trigger some next workflows based on that failure | 13:26 |
| tuan_ | by using on-error? | 13:26 |
| ddeja | yes, it seems OK | 13:26 |
| tuan_ | can we use multiple workflows in a single on-error | 13:26 |
| tuan_ | ? | 13:26 |
| ddeja | just to be clear | 13:27 |
| ddeja | if workflow1 fails, you want to call workflow2 and workflow3? | 13:27 |
| tuan_ | yeap | 13:28 |
| tuan_ | and i would like to put them into one single on-error | 13:28 |
| tuan_ | ? | 13:28 |
| ddeja | yes I think that possible | 13:28 |
| tuan_ | do they run in parallel | 13:28 |
| ddeja | give me a sec, I'll find an example | 13:28 |
| ddeja | well, we can't assure that they would run in parallel | 13:29 |
| tuan_ | what about DirectWorkflowController in mistral | 13:29 |
| tuan_ | it is written by Mirantis and based on doc, it shows that the action will be run in parallel | 13:29 |
| *** ist has joined #openstack-mistral | 13:30 | |
| ddeja | well, they should run in pararell, but we are not assuring that it would happen every time | 13:30 |
| tuan_ | ah ha | 13:30 |
| ddeja | so it will probably work this way, but you should not rely something on it | 13:31 |
| tuan_ | my problem is that: | 13:31 |
| ddeja | you can, for example, have not enough executors, and in such case two workflows started in same time may not work in pararel | 13:31 |
| tuan_ | when we run workflow1, it fails | 13:31 |
| tuan_ | well, so it depends on the pool of thread here | 13:32 |
| ddeja | yes | 13:32 |
| ddeja | tuan_: please see the example http://docs.openstack.org/developer/mistral/dsl/dsl_v2.html#fork | 13:32 |
| ddeja | it's for "on-success", but it should work for "on-error" also | 13:32 |
| tuan_ | perfect | 13:33 |
| tuan_ | thanks alot ddeja | 13:33 |
| tuan_ | let me share our scenario | 13:34 |
| tuan_ | we have something like this | 13:34 |
| tuan_ | if workflow1 fails, previously we implement another action and put it into on-error | 13:35 |
| *** doriftoshoes has quit IRC | 13:36 | |
| tuan_ | but now we have another requirement that, if the workflow1 fails, we would like to gracefully shutdown the instance | 13:36 |
| *** doriftoshoes has joined #openstack-mistral | 13:36 | |
| tuan_ | therefore i would like to implement multiple workflow2, workflow3 to do both of graceful shutdown and previous action | 13:36 |
| ddeja | so yes, In this case you should run workflow1, and on-error run both wokrflow2 and workflow3 | 13:38 |
| tuan_ | okes, let me try to implement this idea and hopefully there still be free threads for both of them:) | 13:40 |
| tuan_ | thanks again ddeja | 13:40 |
| ddeja | tuan_: well, in 99% there would be. I just wanted you to know rely on the fact that they will run in parallel, but for your scenario is totally OK | 13:41 |
| tuan_ | yeap, oh, least but not last | 13:41 |
| tuan_ | what if i would like to check the result of workflow1 and based on that i will implement the workflow2 and 3 | 13:42 |
| tuan_ | ? | 13:42 |
| tuan_ | do we have some ways to implement the condition like "if .. else" here | 13:42 |
| ddeja | tuan_: for sure there is such option | 13:44 |
| ddeja | it's not well document unfortunatelly | 13:46 |
| ddeja | and I'm not an expert on it | 13:46 |
| ddeja | http://docs.openstack.org/developer/mistral/dsl/dsl_v2.html#transitions-with-yaql-expressions | 13:46 |
| ddeja | you can tak a look on this ^ | 13:46 |
| ddeja | and I hope that would fit your needs | 13:47 |
| tuan_ | oh yeah, use yaql for filtering | 13:48 |
| tuan_ | damn stupid for myself | 13:48 |
| tuan_ | thank you | 13:48 |
| tuan_ | oh, may i have a last question | 13:49 |
| tuan_ | :D | 13:49 |
| *** janki has quit IRC | 13:59 | |
| *** hrybacki has quit IRC | 14:08 | |
| *** hrybacki has joined #openstack-mistral | 14:08 | |
| *** cargonza has quit IRC | 14:50 | |
| *** cargonza has joined #openstack-mistral | 14:50 | |
| *** tuan_ has quit IRC | 14:56 | |
| *** jtomasek has quit IRC | 14:57 | |
| *** ist has quit IRC | 15:16 | |
| *** mfisch has joined #openstack-mistral | 15:19 | |
| mfisch | I see from the architecture guide that there's a scheduler, but I dont see a scheduler package. What code provides the scheduler functionality? | 15:19 |
| *** jtomasek has joined #openstack-mistral | 15:27 | |
| *** jaosorior has quit IRC | 15:35 | |
| *** catintheroof has joined #openstack-mistral | 15:54 | |
| *** chlong has joined #openstack-mistral | 16:11 | |
| *** chlong has quit IRC | 16:12 | |
| *** chlong has joined #openstack-mistral | 16:12 | |
| *** FL1SK has quit IRC | 16:13 | |
| *** FL1SK has joined #openstack-mistral | 16:16 | |
| *** bobh has joined #openstack-mistral | 16:20 | |
| *** bobh has quit IRC | 16:20 | |
| *** bobh has joined #openstack-mistral | 16:20 | |
| openstackgerrit | Pradeep Kilambi proposed openstack/mistral: Add aodh actions to mistral https://review.openstack.org/408684 | 16:26 |
| *** Ephur has joined #openstack-mistral | 16:48 | |
| *** clenimar has joined #openstack-mistral | 16:49 | |
| *** bobh has quit IRC | 16:51 | |
| *** dtantsur is now known as dtantsur|afk | 16:52 | |
| *** bobh has joined #openstack-mistral | 16:56 | |
| *** bobh has quit IRC | 17:02 | |
| *** chlong has quit IRC | 17:27 | |
| *** chlong has joined #openstack-mistral | 18:00 | |
| *** harlowja has joined #openstack-mistral | 18:32 | |
| *** harlowja_ has joined #openstack-mistral | 18:34 | |
| *** harlowja has quit IRC | 18:35 | |
| *** openstack has joined #openstack-mistral | 19:45 | |
| *** jamielennox|away is now known as jamielennox | 20:58 | |
| *** bobh has joined #openstack-mistral | 21:23 | |
| *** chlong has quit IRC | 21:29 | |
| *** bobh has quit IRC | 21:40 | |
| *** chlong has joined #openstack-mistral | 22:01 | |
| *** jamielennox is now known as jamielennox|away | 22:04 | |
| *** jamielennox|away is now known as jamielennox | 22:11 | |
| *** bobh has joined #openstack-mistral | 22:15 | |
| *** bobh has quit IRC | 22:40 | |
| *** chlong has quit IRC | 23:28 | |
| *** Ephur has quit IRC | 23:47 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!