Wednesday, 2018-09-12

*** bobh has joined #openstack-mistral00:41
*** bobh has quit IRC01:19
*** apetrich_things_ has joined #openstack-mistral02:55
*** apetrich_ has quit IRC02:58
*** icatchnullptrs has quit IRC03:18
*** icatchnullptrs1 has joined #openstack-mistral03:19
openstackgerritHardik Jasani proposed openstack/mistral master: Fix SSHAction under python3  https://review.openstack.org/59934705:01
*** apetrich_things_ has quit IRC05:30
*** threestrands has joined #openstack-mistral06:13
*** gkadam has joined #openstack-mistral06:25
*** hardikjasani has joined #openstack-mistral06:38
*** xavierhardy has joined #openstack-mistral06:40
*** xavierhardy1 has joined #openstack-mistral06:41
*** xavierhardy has quit IRC06:45
*** AlexeyAbashkin has joined #openstack-mistral07:15
*** apetrich has joined #openstack-mistral07:54
*** jistr|off is now known as jistr07:55
*** xavierhardy1 has quit IRC07:59
*** xavierhardy has joined #openstack-mistral08:00
*** apetrich has quit IRC08:10
*** apetrich has joined #openstack-mistral08:11
openstackgerritRenat Akhmerov proposed openstack/mistral master: Fix how Mistral calculates workflow output  https://review.openstack.org/60194908:22
*** apetrich has quit IRC08:24
*** apetrich has joined #openstack-mistral08:25
*** apetrich has quit IRC08:25
*** apetrich has joined #openstack-mistral08:26
*** apetrich has quit IRC08:27
openstackgerritRenat Akhmerov proposed openstack/mistral master: Fix how Mistral calculates workflow output  https://review.openstack.org/60194908:28
*** apetrich has joined #openstack-mistral08:28
*** apetrich has quit IRC08:29
*** apetrich has joined #openstack-mistral08:29
*** apetrich has quit IRC08:31
*** apetrich has joined #openstack-mistral08:32
pgaxattehello08:32
pgaxatteI have a use case where a workflow would be run as remediation of an alarm but several identical alarm could trigger the same workflow08:33
pgaxatteand I'd like that workflow to not run more than once at the same time across all workers08:34
pgaxatteis there a way to do that natively?08:34
pgaxatteI could create some sort of lock on an outside web service but I'd rather not have to manage an external locking system08:35
*** jlejeune has joined #openstack-mistral08:40
rakhmerovpgaxatte: hi08:45
rakhmerovno, we don't have such feature08:45
*** apetrich has quit IRC08:45
rakhmerovalthough we discussed it several times at summits/PTGs08:45
pgaxatterakhmerov: hello08:46
*** apetrich has joined #openstack-mistral08:46
pgaxatteoh 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
rakhmerovd0ugal: hi, do you see any issues with backporting https://review.openstack.org/601949 to Queens and Pike?08:47
rakhmerovpgaxatte: https://blueprints.launchpad.net/mistral/+spec/mistral-workflow-concurrency-property08:47
d0ugalrakhmerov: do you mean Queens and Rocky?08:48
pgaxatterakhmerov: thanks :)08:48
rakhmerovpgaxatte: 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
rakhmerovd0ugal: well, I need it in all three: Rocky, Queens and Pike08:48
pgaxatterakhmerov: ok, maybe we could pick it up08:49
*** apetrich has quit IRC08:49
d0ugalrakhmerov: I think it should be okay, but a release note would be a good idea08:49
rakhmerovd0ugal: it fixes a very important issue for some of our customers. Some of them are on Pike and Queens still08:49
rakhmerovd0ugal: absolutely, yes08:49
rakhmerovd0ugal: would a separate patch be ok for that?08:50
d0ugalrakhmerov: if you like, it is easier to backport as one patch08:50
d0ugalbut up to you08:50
rakhmerovhm.. ok, np, I'll do it within the same patch08:50
d0ugalsince you want to backport 3 versions that is either 3 or 6 CI runs you need to pass :)08:50
d0ugalAlthough, I think most CI jobs skip release note only patches08:51
openstackgerritRenat Akhmerov proposed openstack/mistral master: Fix how Mistral calculates workflow output  https://review.openstack.org/60194908:54
*** josecastroleon has quit IRC09:01
*** josecastroleon has joined #openstack-mistral09:02
pgaxatterakhmerov: I can't any change related to the blueprint so I guess the work was never submitted09:12
rakhmerovd0ugal: done09:17
rakhmerovpgaxatte: sorry, I didn't get your last sentence :)09:18
rakhmerovooh.. you don't see a related patch? Yes09:18
rakhmerovmaybe, I'm not even sure he sent anything on review09:19
pgaxatteyes :)09:19
pgaxatteI did not find anything so i guess nothing was sent09:19
rakhmerovpgaxatte: you can try to search in the mistral-specs repo09:20
rakhmerovwe might have a spec or a hanging review for the related spec09:21
pgaxatterakhmerov: alright i'm checking09:21
pgaxatterakhmerov: i don't see anything there either09:23
rakhmerovsorry )09:25
rakhmerovit was never really a big priority09:25
rakhmerovmay be because there's always a way how to control a number of running workflows of some type from the outside09:26
rakhmerovfrom the 3rd party09:27
pgaxatteyes I understand09:31
rakhmerovmcdoker18: hi, I replied to your comment in https://review.openstack.org/#/c/60194909:41
mcdoker18rakhmerov: Hi, i mean why we expired entities if there are incomplete tasks?09:46
rakhmerovmcdoker18: oooh, yes!!09:47
rakhmerovsorry...09:47
rakhmerovmy slopiness09:47
rakhmerovyou're absolutely right09:47
rakhmerovI'll fix it now09:48
mcdoker18And why we expire all entities, maybe the best solution is to refresh the workflow execution?09:48
mcdoker18db_api.refresh09:49
openstackgerritRenat Akhmerov proposed openstack/mistral master: Fix how Mistral calculates workflow output  https://review.openstack.org/60194909:49
rakhmerovmcdoker18: no, refreshing workflow execution is not enough09:51
rakhmerovbecause parallel transactions may also change task executions09:51
rakhmerovbased on which we calculate something too09:51
rakhmerovI first did "refresh" yes, but it's not right09:52
*** josecastroleon has quit IRC09:52
*** d0ugal has quit IRC09:52
rakhmerovmcdoker18: 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-mistral09:54
mcdoker18rakhmerov: Yes, you're right. Got it09:56
rakhmerovmcdoker18: thanks a lot for pointing to this09:57
*** d0ugal has joined #openstack-mistral10:41
*** d0ugal has quit IRC10:46
*** d0ugal has joined #openstack-mistral10:47
*** apetrich has joined #openstack-mistral11:28
*** xavierhardy has quit IRC11:38
*** d0ugal has quit IRC11:47
*** xavierhardy has joined #openstack-mistral11:51
*** jtomasek has joined #openstack-mistral12:00
*** jtomasek has quit IRC12:01
*** zigo has joined #openstack-mistral12:28
*** jtomasek has joined #openstack-mistral12:30
*** jtomasek has quit IRC12:35
*** gkadam has quit IRC12:49
openstackgerritMerged openstack/mistral master: Fix SSHAction under python3  https://review.openstack.org/59934712:56
*** d0ugal has joined #openstack-mistral13:10
*** jtomasek has joined #openstack-mistral13:13
*** hardikjasani has quit IRC13:18
*** jtomasek has quit IRC13:22
*** threestrands has quit IRC13:24
*** xavierhardy has quit IRC13:34
*** xavierhardy has joined #openstack-mistral13:34
*** bobh has joined #openstack-mistral14:13
*** jtomasek has joined #openstack-mistral14:13
*** jtomasek has quit IRC14:45
*** jtomasek has joined #openstack-mistral14:53
*** bobh has quit IRC14:53
*** jaosorior has joined #openstack-mistral15:05
*** jtomasek has quit IRC15:19
*** jtomasek has joined #openstack-mistral15:19
*** jrist has joined #openstack-mistral15:23
*** bobh has joined #openstack-mistral15:25
*** d0ugal has quit IRC15:28
*** xavierhardy has quit IRC15:41
*** d0ugal has joined #openstack-mistral16:01
*** xavierhardy has joined #openstack-mistral16:03
*** bobh has quit IRC16:06
*** bobh has joined #openstack-mistral16:06
*** AlexeyAbashkin has quit IRC16:20
*** jaosorior has quit IRC16:28
*** xavierhardy has quit IRC16:31
*** jaosorior has joined #openstack-mistral16:36
*** jtomasek has quit IRC16:39
*** jtomasek has joined #openstack-mistral16:41
*** bobh has quit IRC17:00
*** bobh has joined #openstack-mistral17:13
*** csatari has joined #openstack-mistral17:18
csatarid0ugal: Do you have time to check quickly what is needed to merge https://review.openstack.org/#/c/476499/ ?17:18
d0ugalcsatari: It's my end of day now, I'll add it to my list to look tomorrow.17:21
*** d0ugal has quit IRC17:28
csatariOkay17:29
csatariThanks.17:29
*** rakhmerov has quit IRC17:36
*** rakhmerov has joined #openstack-mistral17:36
*** jtomasek has quit IRC17:52
*** jrist has quit IRC17:57
*** bobh has quit IRC18:39
*** bobh has joined #openstack-mistral18:40
*** xavierhardy has joined #openstack-mistral18:45
*** xavierhardy has quit IRC19:00
*** jrist has joined #openstack-mistral19:39
*** jtomasek has joined #openstack-mistral19:51
*** bobh has quit IRC19:58
*** bobh has joined #openstack-mistral19:58
*** jrist has quit IRC20:46
*** jtomasek has quit IRC20:48
*** jrist has joined #openstack-mistral20:52
*** bobh has quit IRC21:01
*** bobh has joined #openstack-mistral21:02
*** jtomasek has joined #openstack-mistral21:04
*** jtomasek has quit IRC21:04
*** bobh has quit IRC21:07
*** d0ugal has joined #openstack-mistral21:19
*** jtomasek has joined #openstack-mistral21:35
*** bobh has joined #openstack-mistral21:37
*** bobh has quit IRC21:41
*** d0ugal has quit IRC21:43
*** bobh has joined #openstack-mistral21:47
*** rakhmerov has quit IRC21:57
*** rakhmerov has joined #openstack-mistral22:01
*** bobh has quit IRC22:01
*** rakhmerov has quit IRC22:08
*** rakhmerov has joined #openstack-mistral22:12
*** jtomasek has quit IRC22:33
*** jaosorior has quit IRC22:40
*** jrist has quit IRC23:03

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