*** mgershen has quit IRC | 00:42 | |
*** mgershen has joined #openstack-mistral | 00:43 | |
*** dmellado has quit IRC | 01:00 | |
*** dmellado has joined #openstack-mistral | 01:02 | |
*** thrash is now known as thrash|g0ne | 01:33 | |
tung_doan | thrash|g0ne: d0ugal kong: that is a very good news! I will firgure out how Mistral + Zaquar work together. Hopw to talk with you later. thanks | 01:43 |
---|---|---|
*** bobh has quit IRC | 01:59 | |
*** zhurong has joined #openstack-mistral | 02:33 | |
*** bobh has joined #openstack-mistral | 02:45 | |
*** bobh has quit IRC | 02:49 | |
*** zhurong has quit IRC | 03:15 | |
*** gkadam has joined #openstack-mistral | 03:19 | |
*** gongysh has joined #openstack-mistral | 04:47 | |
*** gongysh has quit IRC | 04:49 | |
*** sharatss has joined #openstack-mistral | 05:04 | |
*** zhurong has joined #openstack-mistral | 05:07 | |
*** zhurong has quit IRC | 05:52 | |
*** gongysh has joined #openstack-mistral | 05:53 | |
gongysh | d0ugal, kong | 05:53 |
gongysh | hi | 05:53 |
d0ugal | gongysh: hi | 06:21 |
gongysh | hi | 06:21 |
gongysh | I want mistral not to save task execution, how to do it? | 06:24 |
gongysh | d0ugal | 06:24 |
d0ugal | gongysh: Why don't you want to save it? | 06:25 |
gongysh | I have a loop workflow, which will run tasks for a long time. task1 -> task2 -> task1 | 06:26 |
*** zhurong has joined #openstack-mistral | 06:26 | |
gongysh | with this loop flow, the mistral db will have thousands of task1 and task2 executions. | 06:26 |
d0ugal | gongysh: there is no way, that I know of, to stop Mistral storing the action executions | 06:27 |
d0ugal | However, you can use the execution expiration policy to clean them up | 06:27 |
d0ugal | https://docs.openstack.org/developer/mistral/main_features.html?highlight=execution_expiration_policy#execution-expiration-policy | 06:27 |
gongysh | d0ugal https://github.com/openstack/mistral/blob/master/mistral/engine/actions.py#L246 | 06:31 |
openstackgerrit | luong tuan proposed openstack/mistral master: Refactor mistral context using oslo_context https://review.openstack.org/455407 | 06:31 |
gongysh | the code has the save=True parameter, but it seems other codes are not using it. | 06:32 |
d0ugal | gongysh: that is used with `mistral run-action` only, which is a way of running an action outside of workflows - for testing etc. | 06:33 |
*** zhurong has quit IRC | 06:33 | |
d0ugal | I wrote that :) | 06:33 |
gongysh | d0ugal, then how do pass in save=False? | 06:34 |
d0ugal | gongysh: You can't from workflows. | 06:34 |
gongysh | d0ugal, you expected there is 'gongysh' who will use it | 06:35 |
d0ugal | False is the default btw | 06:35 |
gongysh | :) | 06:35 |
gongysh | d0ugal, https://github.com/openstack/mistral/blob/master/mistral/engine/actions.py#L231 | 06:35 |
gongysh | the save=True is default. | 06:35 |
*** d0ugal has quit IRC | 06:39 | |
*** d0ugal has joined #openstack-mistral | 06:41 | |
d0ugal | gongysh: sorry, I had a connection issue. | 06:42 |
d0ugal | gongysh: I didn't see anything after I said "False is the default btw" | 06:42 |
gongysh | d0ugal, https://github.com/openstack/mistral/blob/master/mistral/engine/actions.py#L231 | 06:42 |
gongysh | [14:35:51] <gongysh>the save=True is default. | 06:42 |
d0ugal | gongysh: right, but I mean for `mistral run-action` it defaults to false | 06:43 |
d0ugal | for everything else it defaults to True and there is no way to change it | 06:43 |
*** mgershen1 has joined #openstack-mistral | 06:48 | |
*** mgershen has quit IRC | 06:49 | |
*** mgershen has joined #openstack-mistral | 07:01 | |
*** mgershen1 has quit IRC | 07:02 | |
*** shardy has joined #openstack-mistral | 07:05 | |
*** Kevin_Zheng has joined #openstack-mistral | 07:21 | |
*** zhurong has joined #openstack-mistral | 07:27 | |
*** jpich has joined #openstack-mistral | 07:42 | |
*** warface has joined #openstack-mistral | 07:44 | |
d0ugal | rakhmerov: hey, are you around? | 07:59 |
*** zhurong has quit IRC | 08:08 | |
*** gongysh has quit IRC | 08:16 | |
*** jaosorior_away is now known as jaosorior | 08:28 | |
*** gkadam is now known as gkadam-afk | 08:33 | |
rakhmerov | d0ugal: yes | 09:16 |
rakhmerov | hi | 09:16 |
*** zhurong has joined #openstack-mistral | 09:22 | |
*** zhurong has quit IRC | 09:37 | |
d0ugal | rakhmerov: to trigger the rpm package build, I need something to merge into mistral-lib :) | 09:49 |
d0ugal | https://review.openstack.org/#/q/status:open+project:openstack/mistral-lib,n,z - could we land one of these? | 09:50 |
rakhmerov | yes sir | 09:50 |
d0ugal | Thanks, they are both pretty small | 09:50 |
openstackgerrit | Merged openstack/mistral-lib master: Fix documentation for custom actions https://review.openstack.org/459695 | 09:58 |
openstackgerrit | Merged openstack/mistral-lib master: Remove the unused ApplicationContextNotFoundException https://review.openstack.org/459698 | 09:58 |
d0ugal | \o/ | 09:58 |
*** gkadam-afk is now known as gkadam | 10:30 | |
*** gkadam has quit IRC | 10:31 | |
*** gkadam has joined #openstack-mistral | 10:31 | |
*** gkadam is now known as gkadam-afk | 10:31 | |
*** jkilpatr has quit IRC | 10:45 | |
*** jkilpatr has joined #openstack-mistral | 11:05 | |
*** shardy is now known as shardy_lunch | 11:09 | |
*** gkadam-afk is now known as gkadam | 11:23 | |
jaosorior | sharatss: seems to me that it won't work, there is something wrong with the cahnges to the functional tests. Did you check the logs? I haven't had time to work on this. If you have time to dig deeper into the current issue it would be appreciated. | 11:31 |
sharatss | jaosorior, i didn't check the logs yet. i will do it today and let you know | 11:32 |
rakhmerov | d0ugal: I'd appreciate if you could chime in to https://review.openstack.org/#/c/455083 | 11:36 |
rakhmerov | another architectural discussion | 11:36 |
d0ugal | rakhmerov: we just use zaqar for that... it works great. | 12:18 |
d0ugal | but we don't have all the events, seems like it could be useful | 12:18 |
*** mgershen1 has joined #openstack-mistral | 12:20 | |
*** mgershen has quit IRC | 12:22 | |
rakhmerov | d0ugal: yeah, also keep in mind that lots of people have nothing to do with OpenStack when they use Mistral, not sure if Zaqar would be ok for that | 12:23 |
*** thrash|g0ne is now known as thrash | 12:25 | |
d0ugal | yeah, maybe not, but I guess we at least need to support zaqar | 12:28 |
*** jpich has quit IRC | 12:39 | |
*** shardy_lunch is now known as shardy | 12:39 | |
*** jpich has joined #openstack-mistral | 12:40 | |
openstackgerrit | Sharat Sharma proposed openstack/mistral master: Minor nits to README https://review.openstack.org/460100 | 13:04 |
*** gkadam has quit IRC | 13:12 | |
*** dprince has joined #openstack-mistral | 13:20 | |
*** sharatss has quit IRC | 13:30 | |
*** mgershen has joined #openstack-mistral | 13:36 | |
*** jaosorior is now known as jaosorior_away | 13:36 | |
*** mgershen1 has quit IRC | 13:38 | |
*** bobh has joined #openstack-mistral | 13:42 | |
*** catintheroof has joined #openstack-mistral | 13:56 | |
*** chlong has joined #openstack-mistral | 13:58 | |
*** bobh has quit IRC | 14:01 | |
*** livelace has joined #openstack-mistral | 14:08 | |
*** bobh has joined #openstack-mistral | 14:27 | |
flaper87 | d0ugal: u around? any thoughts on this https://github.com/d0ugal/mistral-ansible-actions/pull/9 ? | 14:39 |
*** catintheroof has quit IRC | 15:10 | |
*** catintheroof has joined #openstack-mistral | 15:11 | |
*** catintheroof has quit IRC | 15:11 | |
d0ugal | flaper87: oh, sorry, I forgot to look yesterday. On it now. | 15:24 |
d0ugal | flaper87: LGTM, is there something you want to change on it? | 15:38 |
flaper87 | d0ugal: having a prob with the inventory property | 15:38 |
flaper87 | d0ugal: registering the action seems to fail if there are properties | 15:39 |
d0ugal | oh really, that is interesting. | 15:39 |
d0ugal | flaper87: any errors> | 15:39 |
d0ugal | ?* | 15:39 |
flaper87 | d0ugal: yes, need to replicate. 2s, in a call | 15:40 |
flaper87 | :D | 15:40 |
d0ugal | k | 15:40 |
*** warface has quit IRC | 15:41 | |
openstackgerrit | Dougal Matthews proposed openstack/mistral master: Add support for mistral-lib to Mistral https://review.openstack.org/453517 | 15:45 |
flaper87 | d0ugal: put the error on the PR | 15:57 |
flaper87 | d0ugal: btw, I'd like to do something similar w/ playbooks. Allow to pass a file or a template | 15:57 |
d0ugal | flaper87: cool | 15:57 |
d0ugal | woah | 15:57 |
d0ugal | I didn't know we stored action attributes! | 15:58 |
flaper87 | d0ugal: :) | 16:00 |
d0ugal | flaper87: working on a fix | 16:00 |
flaper87 | d0ugal: awesome, looking forward to it. gtg now | 16:00 |
flaper87 | <3 | 16:00 |
d0ugal | sure, I might not have one until tomorrow morning :) | 16:01 |
*** jpich has quit IRC | 16:08 | |
*** mgershen1 has joined #openstack-mistral | 16:09 | |
*** mgershen has quit IRC | 16:10 | |
*** jpich has joined #openstack-mistral | 16:12 | |
openstackgerrit | Dougal Matthews proposed openstack/mistral master: Don't save @property methods with other attributes https://review.openstack.org/460206 | 16:13 |
d0ugal | flaper87: ^ | 16:14 |
*** jaosorior_away is now known as jaosorior | 16:22 | |
openstackgerrit | Dougal Matthews proposed openstack/mistral master: Don't save @property methods with other attributes https://review.openstack.org/460206 | 16:24 |
*** livelace has quit IRC | 16:24 | |
*** jaosorior has quit IRC | 16:26 | |
*** jaosorior has joined #openstack-mistral | 16:27 | |
*** jaosorior is now known as jaosorior_away | 16:27 | |
openstackgerrit | Dougal Matthews proposed openstack/mistral master: Force Python 2 for pep8 linting https://review.openstack.org/460212 | 16:27 |
*** jamielennox is now known as jamielennox|away | 16:31 | |
*** jpich has quit IRC | 16:37 | |
*** dprince has quit IRC | 16:46 | |
*** dprince has joined #openstack-mistral | 16:46 | |
*** mgershen has joined #openstack-mistral | 16:51 | |
*** mgershen1 has quit IRC | 16:52 | |
*** warface has joined #openstack-mistral | 16:53 | |
*** Qiming has quit IRC | 17:12 | |
*** Qiming has joined #openstack-mistral | 17:14 | |
*** warface has quit IRC | 17:36 | |
*** jkilpatr_ has joined #openstack-mistral | 17:44 | |
*** jkilpatr has quit IRC | 17:46 | |
*** shardy has quit IRC | 17:50 | |
*** thrash is now known as thrash|biab | 18:25 | |
*** jkilpatr_ has quit IRC | 18:37 | |
*** jkilpatr has joined #openstack-mistral | 18:39 | |
*** thrash|biab is now known as thrash | 19:12 | |
*** Kevin_Zheng has quit IRC | 19:17 | |
*** Qiming has quit IRC | 19:39 | |
*** Qiming has joined #openstack-mistral | 19:42 | |
*** shardy has joined #openstack-mistral | 19:48 | |
*** jamielennox|away is now known as jamielennox | 19:52 | |
*** dprince has quit IRC | 20:00 | |
*** shardy has quit IRC | 20:07 | |
*** zhenguo has quit IRC | 21:22 | |
*** thrash is now known as thrash|g0ne | 21:44 | |
*** chlong has quit IRC | 21:48 | |
*** jkilpatr has quit IRC | 21:51 | |
*** rbrady has quit IRC | 22:18 | |
*** jkilpatr has joined #openstack-mistral | 22:32 | |
*** bobh has quit IRC | 22:38 | |
openstackgerrit | Lingxian Kong proposed openstack/mistral master: Add "running_region" param for OpenStack actions https://review.openstack.org/460351 | 23:43 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!