*** bobh has joined #openstack-mistral | 00:41 | |
*** bobh has quit IRC | 01:19 | |
*** apetrich_things_ has joined #openstack-mistral | 02:55 | |
*** apetrich_ has quit IRC | 02:58 | |
*** icatchnullptrs has quit IRC | 03:18 | |
*** icatchnullptrs1 has joined #openstack-mistral | 03:19 | |
openstackgerrit | Hardik Jasani proposed openstack/mistral master: Fix SSHAction under python3 https://review.openstack.org/599347 | 05:01 |
---|---|---|
*** apetrich_things_ has quit IRC | 05:30 | |
*** threestrands has joined #openstack-mistral | 06:13 | |
*** gkadam has joined #openstack-mistral | 06:25 | |
*** hardikjasani has joined #openstack-mistral | 06:38 | |
*** xavierhardy has joined #openstack-mistral | 06:40 | |
*** xavierhardy1 has joined #openstack-mistral | 06:41 | |
*** xavierhardy has quit IRC | 06:45 | |
*** AlexeyAbashkin has joined #openstack-mistral | 07:15 | |
*** apetrich has joined #openstack-mistral | 07:54 | |
*** jistr|off is now known as jistr | 07:55 | |
*** xavierhardy1 has quit IRC | 07:59 | |
*** xavierhardy has joined #openstack-mistral | 08:00 | |
*** apetrich has quit IRC | 08:10 | |
*** apetrich has joined #openstack-mistral | 08:11 | |
openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Fix how Mistral calculates workflow output https://review.openstack.org/601949 | 08:22 |
*** apetrich has quit IRC | 08:24 | |
*** apetrich has joined #openstack-mistral | 08:25 | |
*** apetrich has quit IRC | 08:25 | |
*** apetrich has joined #openstack-mistral | 08:26 | |
*** apetrich has quit IRC | 08:27 | |
openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Fix how Mistral calculates workflow output https://review.openstack.org/601949 | 08:28 |
*** apetrich has joined #openstack-mistral | 08:28 | |
*** apetrich has quit IRC | 08:29 | |
*** apetrich has joined #openstack-mistral | 08:29 | |
*** apetrich has quit IRC | 08:31 | |
*** apetrich has joined #openstack-mistral | 08:32 | |
pgaxatte | hello | 08:32 |
pgaxatte | I have a use case where a workflow would be run as remediation of an alarm but several identical alarm could trigger the same workflow | 08:33 |
pgaxatte | and I'd like that workflow to not run more than once at the same time across all workers | 08:34 |
pgaxatte | is there a way to do that natively? | 08:34 |
pgaxatte | I could create some sort of lock on an outside web service but I'd rather not have to manage an external locking system | 08:35 |
*** jlejeune has joined #openstack-mistral | 08:40 | |
rakhmerov | pgaxatte: hi | 08:45 |
rakhmerov | no, we don't have such feature | 08:45 |
*** apetrich has quit IRC | 08:45 | |
rakhmerov | although we discussed it several times at summits/PTGs | 08:45 |
pgaxatte | rakhmerov: hello | 08:46 |
*** apetrich has joined #openstack-mistral | 08:46 | |
pgaxatte | oh and what did come out of those discussion? is this something that might land in mistral one day or is it outside the scope? | 08:47 |
rakhmerov | d0ugal: hi, do you see any issues with backporting https://review.openstack.org/601949 to Queens and Pike? | 08:47 |
rakhmerov | pgaxatte: https://blueprints.launchpad.net/mistral/+spec/mistral-workflow-concurrency-property | 08:47 |
d0ugal | rakhmerov: do you mean Queens and Rocky? | 08:48 |
pgaxatte | rakhmerov: thanks :) | 08:48 |
rakhmerov | pgaxatte: someone even started working on this BP some time ago but then he left the project (and OpenStack as far as I know) | 08:48 |
rakhmerov | d0ugal: well, I need it in all three: Rocky, Queens and Pike | 08:48 |
pgaxatte | rakhmerov: ok, maybe we could pick it up | 08:49 |
*** apetrich has quit IRC | 08:49 | |
d0ugal | rakhmerov: I think it should be okay, but a release note would be a good idea | 08:49 |
rakhmerov | d0ugal: it fixes a very important issue for some of our customers. Some of them are on Pike and Queens still | 08:49 |
rakhmerov | d0ugal: absolutely, yes | 08:49 |
rakhmerov | d0ugal: would a separate patch be ok for that? | 08:50 |
d0ugal | rakhmerov: if you like, it is easier to backport as one patch | 08:50 |
d0ugal | but up to you | 08:50 |
rakhmerov | hm.. ok, np, I'll do it within the same patch | 08:50 |
d0ugal | since you want to backport 3 versions that is either 3 or 6 CI runs you need to pass :) | 08:50 |
d0ugal | Although, I think most CI jobs skip release note only patches | 08:51 |
openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Fix how Mistral calculates workflow output https://review.openstack.org/601949 | 08:54 |
*** josecastroleon has quit IRC | 09:01 | |
*** josecastroleon has joined #openstack-mistral | 09:02 | |
pgaxatte | rakhmerov: I can't any change related to the blueprint so I guess the work was never submitted | 09:12 |
rakhmerov | d0ugal: done | 09:17 |
rakhmerov | pgaxatte: sorry, I didn't get your last sentence :) | 09:18 |
rakhmerov | ooh.. you don't see a related patch? Yes | 09:18 |
rakhmerov | maybe, I'm not even sure he sent anything on review | 09:19 |
pgaxatte | yes :) | 09:19 |
pgaxatte | I did not find anything so i guess nothing was sent | 09:19 |
rakhmerov | pgaxatte: you can try to search in the mistral-specs repo | 09:20 |
rakhmerov | we might have a spec or a hanging review for the related spec | 09:21 |
pgaxatte | rakhmerov: alright i'm checking | 09:21 |
pgaxatte | rakhmerov: i don't see anything there either | 09:23 |
rakhmerov | sorry ) | 09:25 |
rakhmerov | it was never really a big priority | 09:25 |
rakhmerov | may be because there's always a way how to control a number of running workflows of some type from the outside | 09:26 |
rakhmerov | from the 3rd party | 09:27 |
pgaxatte | yes I understand | 09:31 |
rakhmerov | mcdoker18: hi, I replied to your comment in https://review.openstack.org/#/c/601949 | 09:41 |
mcdoker18 | rakhmerov: Hi, i mean why we expired entities if there are incomplete tasks? | 09:46 |
rakhmerov | mcdoker18: oooh, yes!! | 09:47 |
rakhmerov | sorry... | 09:47 |
rakhmerov | my slopiness | 09:47 |
rakhmerov | you're absolutely right | 09:47 |
rakhmerov | I'll fix it now | 09:48 |
mcdoker18 | And why we expire all entities, maybe the best solution is to refresh the workflow execution? | 09:48 |
mcdoker18 | db_api.refresh | 09:49 |
openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Fix how Mistral calculates workflow output https://review.openstack.org/601949 | 09:49 |
rakhmerov | mcdoker18: no, refreshing workflow execution is not enough | 09:51 |
rakhmerov | because parallel transactions may also change task executions | 09:51 |
rakhmerov | based on which we calculate something too | 09:51 |
rakhmerov | I first did "refresh" yes, but it's not right | 09:52 |
*** josecastroleon has quit IRC | 09:52 | |
*** d0ugal has quit IRC | 09:52 | |
rakhmerov | mcdoker18: using "expire_all" is fairly safe and even better than refreshing because it postpones loading from DB till the corresponding data is needed (read by the thread) | 09:53 |
*** josecastroleon has joined #openstack-mistral | 09:54 | |
mcdoker18 | rakhmerov: Yes, you're right. Got it | 09:56 |
rakhmerov | mcdoker18: thanks a lot for pointing to this | 09:57 |
*** d0ugal has joined #openstack-mistral | 10:41 | |
*** d0ugal has quit IRC | 10:46 | |
*** d0ugal has joined #openstack-mistral | 10:47 | |
*** apetrich has joined #openstack-mistral | 11:28 | |
*** xavierhardy has quit IRC | 11:38 | |
*** d0ugal has quit IRC | 11:47 | |
*** xavierhardy has joined #openstack-mistral | 11:51 | |
*** jtomasek has joined #openstack-mistral | 12:00 | |
*** jtomasek has quit IRC | 12:01 | |
*** zigo has joined #openstack-mistral | 12:28 | |
*** jtomasek has joined #openstack-mistral | 12:30 | |
*** jtomasek has quit IRC | 12:35 | |
*** gkadam has quit IRC | 12:49 | |
openstackgerrit | Merged openstack/mistral master: Fix SSHAction under python3 https://review.openstack.org/599347 | 12:56 |
*** d0ugal has joined #openstack-mistral | 13:10 | |
*** jtomasek has joined #openstack-mistral | 13:13 | |
*** hardikjasani has quit IRC | 13:18 | |
*** jtomasek has quit IRC | 13:22 | |
*** threestrands has quit IRC | 13:24 | |
*** xavierhardy has quit IRC | 13:34 | |
*** xavierhardy has joined #openstack-mistral | 13:34 | |
*** bobh has joined #openstack-mistral | 14:13 | |
*** jtomasek has joined #openstack-mistral | 14:13 | |
*** jtomasek has quit IRC | 14:45 | |
*** jtomasek has joined #openstack-mistral | 14:53 | |
*** bobh has quit IRC | 14:53 | |
*** jaosorior has joined #openstack-mistral | 15:05 | |
*** jtomasek has quit IRC | 15:19 | |
*** jtomasek has joined #openstack-mistral | 15:19 | |
*** jrist has joined #openstack-mistral | 15:23 | |
*** bobh has joined #openstack-mistral | 15:25 | |
*** d0ugal has quit IRC | 15:28 | |
*** xavierhardy has quit IRC | 15:41 | |
*** d0ugal has joined #openstack-mistral | 16:01 | |
*** xavierhardy has joined #openstack-mistral | 16:03 | |
*** bobh has quit IRC | 16:06 | |
*** bobh has joined #openstack-mistral | 16:06 | |
*** AlexeyAbashkin has quit IRC | 16:20 | |
*** jaosorior has quit IRC | 16:28 | |
*** xavierhardy has quit IRC | 16:31 | |
*** jaosorior has joined #openstack-mistral | 16:36 | |
*** jtomasek has quit IRC | 16:39 | |
*** jtomasek has joined #openstack-mistral | 16:41 | |
*** bobh has quit IRC | 17:00 | |
*** bobh has joined #openstack-mistral | 17:13 | |
*** csatari has joined #openstack-mistral | 17:18 | |
csatari | d0ugal: Do you have time to check quickly what is needed to merge https://review.openstack.org/#/c/476499/ ? | 17:18 |
d0ugal | csatari: It's my end of day now, I'll add it to my list to look tomorrow. | 17:21 |
*** d0ugal has quit IRC | 17:28 | |
csatari | Okay | 17:29 |
csatari | Thanks. | 17:29 |
*** rakhmerov has quit IRC | 17:36 | |
*** rakhmerov has joined #openstack-mistral | 17:36 | |
*** jtomasek has quit IRC | 17:52 | |
*** jrist has quit IRC | 17:57 | |
*** bobh has quit IRC | 18:39 | |
*** bobh has joined #openstack-mistral | 18:40 | |
*** xavierhardy has joined #openstack-mistral | 18:45 | |
*** xavierhardy has quit IRC | 19:00 | |
*** jrist has joined #openstack-mistral | 19:39 | |
*** jtomasek has joined #openstack-mistral | 19:51 | |
*** bobh has quit IRC | 19:58 | |
*** bobh has joined #openstack-mistral | 19:58 | |
*** jrist has quit IRC | 20:46 | |
*** jtomasek has quit IRC | 20:48 | |
*** jrist has joined #openstack-mistral | 20:52 | |
*** bobh has quit IRC | 21:01 | |
*** bobh has joined #openstack-mistral | 21:02 | |
*** jtomasek has joined #openstack-mistral | 21:04 | |
*** jtomasek has quit IRC | 21:04 | |
*** bobh has quit IRC | 21:07 | |
*** d0ugal has joined #openstack-mistral | 21:19 | |
*** jtomasek has joined #openstack-mistral | 21:35 | |
*** bobh has joined #openstack-mistral | 21:37 | |
*** bobh has quit IRC | 21:41 | |
*** d0ugal has quit IRC | 21:43 | |
*** bobh has joined #openstack-mistral | 21:47 | |
*** rakhmerov has quit IRC | 21:57 | |
*** rakhmerov has joined #openstack-mistral | 22:01 | |
*** bobh has quit IRC | 22:01 | |
*** rakhmerov has quit IRC | 22:08 | |
*** rakhmerov has joined #openstack-mistral | 22:12 | |
*** jtomasek has quit IRC | 22:33 | |
*** jaosorior has quit IRC | 22:40 | |
*** jrist has quit IRC | 23:03 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!