| *** Piet has joined #openstack-mistral | 00:07 | |
| *** achanda has quit IRC | 00:23 | |
| *** bobh has quit IRC | 00:25 | |
| *** achanda has joined #openstack-mistral | 00:26 | |
| *** gyee has quit IRC | 01:15 | |
| *** dmowrer has joined #openstack-mistral | 01:33 | |
| *** dmowrer has quit IRC | 01:38 | |
| *** bobh has joined #openstack-mistral | 02:52 | |
| *** achanda has quit IRC | 02:53 | |
| *** achanda has joined #openstack-mistral | 02:56 | |
| *** achanda has quit IRC | 02:57 | |
| *** bobh has quit IRC | 03:16 | |
| *** bobh has joined #openstack-mistral | 03:26 | |
| *** bobh has quit IRC | 03:28 | |
| xylan_kong | ping rakhmerov | 03:50 |
|---|---|---|
| *** achanda has joined #openstack-mistral | 04:27 | |
| openstackgerrit | hardik proposed openstack/mistral: Refactored filter implementation. https://review.openstack.org/234625 | 04:27 |
| *** achanda has quit IRC | 04:32 | |
| *** achanda has joined #openstack-mistral | 04:35 | |
| *** openstackgerrit has quit IRC | 05:16 | |
| *** openstackgerrit has joined #openstack-mistral | 05:17 | |
| *** dmowrer has joined #openstack-mistral | 05:29 | |
| *** dmowrer has quit IRC | 05:33 | |
| rakhmerov | xylan_kong: here | 06:43 |
| *** sergmelikyan has joined #openstack-mistral | 06:56 | |
| *** sergmelikyan has quit IRC | 07:06 | |
| *** jongchoi has joined #openstack-mistral | 07:30 | |
| *** dmowrer has joined #openstack-mistral | 07:40 | |
| *** dmowrer has quit IRC | 07:45 | |
| *** melisha has joined #openstack-mistral | 07:50 | |
| melisha | Hi, all. We were occasionally encountering a "Failed to find action" error on various system actions like "std.echo" and "std.http" and we were trying to track the reason for that. | 07:52 |
| melisha | We were reviewing the fix for https://review.openstack.org/#/c/223536 and we think we understand the issue | 07:52 |
| melisha | Our setups are running puppet and every X minutes they run "puppet agent -t" to make sure the VM is up to date | 07:53 |
| melisha | This causes the mistral-db-manager to run again and reinstall all system actions / workflows | 07:53 |
| melisha | During that time - running workflows are failing | 07:53 |
| *** achanda has quit IRC | 08:10 | |
| *** achanda has joined #openstack-mistral | 08:13 | |
| rakhmerov | melisha: hi | 08:13 |
| rakhmerov | you mean this happens because mistral-db-manage deletes standard actions before recreating them? | 08:14 |
| melisha | rakhmerov: Hi. Yes | 08:14 |
| *** achanda has quit IRC | 08:15 | |
| rakhmerov | hm.. thinking | 08:15 |
| rakhmerov | so the better way would be not to delete them | 08:15 |
| melisha | Yes. We could do an update instead of delete / create | 08:16 |
| melisha | Or we can put the while thing in a transaction and commit only after both delete and create are done | 08:16 |
| rakhmerov | yes | 08:16 |
| melisha | I think transaction is simpler, right? | 08:17 |
| rakhmerov | it will only work thought for REPEATABLE READ transactions | 08:17 |
| rakhmerov | right, transaction approach looks simpler | 08:17 |
| melisha | READ COMMITTED | 08:17 |
| melisha | ? | 08:17 |
| rakhmerov | I thought REPEATABLE READ because we need consistency on a DB table rows level | 08:18 |
| rakhmerov | say during update we understood that part of actions are gone (unlikely) | 08:18 |
| rakhmerov | so that leads to the point that we shouldn't see an inconsistent state of the whole table | 08:19 |
| rakhmerov | when part of actions are up to date already and part are not | 08:19 |
| *** achanda has joined #openstack-mistral | 08:20 | |
| rakhmerov | if we need that level of consistency then we need REPEATABLE READ | 08:20 |
| rakhmerov | if we're ok that Mistral sees at some point partially obsolete actions and partially actual actions then READ COMMITTED is enough | 08:21 |
| melisha | OK. I agree. What is the default isolation level of Mistral? | 08:21 |
| rakhmerov | makes sense? | 08:21 |
| rakhmerov | there's only a recommendation somewhere in the docs to use either READ_COMMITTED or REPEATABLE_READ | 08:22 |
| rakhmerov | so in fact by default if Mistral config doesn't contain that the default DB settings are used | 08:22 |
| rakhmerov | for every database it may vary but in most cases (at least for mysql and postgres) it is at least READ_COMMITTED | 08:23 |
| rakhmerov | which is enough | 08:23 |
| melisha | REPEATABLE_READ will theoretically guarantee that a workflow that does "std.echo" as first task and uses "std.echo" again as last task will use the exact same "std.echo" even if the action definition was updated, right? But I say theoretically because every task is running in a different transaction anyway so it will read the new definition because | 08:25 |
| melisha | that is the first time it reads that definition in that transaction | 08:25 |
| melisha | Correct? | 08:26 |
| rakhmerov | melisha: not, currently it may be different | 08:28 |
| rakhmerov | because, as you said, it gets fetched from DB within different transactions | 08:28 |
| rakhmerov | the only way to guarantee that is to make snapshots of all actions (as we do for wf itself) at the beginning of workflow | 08:29 |
| melisha | So I am saying that REPEATABLE_READ cannot help anyway so READ_COMMITTED is good for now | 08:30 |
| melisha | Until we will do snapshot | 08:30 |
| xylan_kong | rakhmerov: still here? | 08:35 |
| rakhmerov | melisha: yes | 08:35 |
| xylan_kong | rakhmerov: please see http://lists.openstack.org/pipermail/openstack-dev/2015-October/077033.html, do you need help for mistral release process? I'm willing to be release liaison in M. | 08:35 |
| rakhmerov | melisha: at least for now, I would suggest we fix the current update algorithm | 08:36 |
| melisha | rakhmerov: To be in transaction like we said? | 08:37 |
| rakhmerov | yes, it seems to be the best way for now | 08:37 |
| rakhmerov | xylan_kong: you won't believe :) | 08:37 |
| melisha | OK. Thanks. I will open a bug and will work on a fix. Thanks. | 08:37 |
| rakhmerov | xylan_kong: I wanted to suggest you do this ) | 08:38 |
| rakhmerov | melisha: ok, thanks. That's a good finding actually | 08:38 |
| rakhmerov | xylan_kong: so go ahead and do this, it could be very helpful | 08:38 |
| xylan_kong | rakhmerov: BTW, you can add Mistral liaison information on this page: https://wiki.openstack.org/wiki/CrossProjectLiaisons, we should take participate in cross project affairs, maybe this is a good chance to start with. | 08:38 |
| rakhmerov | xylan_kong: yes, agree. Honestly, I've wanted to do this for months but didn't have a capacity so far ) | 08:41 |
| xylan_kong | rakhmerov: ok, I have already repied to Doug Hellmann about that :-) | 08:43 |
| xylan_kong | rakhmerov: by the way, I've completed my slides for mistral session, Need I send it to you before I get there? | 08:44 |
| rakhmerov | yes, please do | 08:45 |
| rakhmerov | asap ) | 08:45 |
| xylan_kong | rakhmerov: sure | 08:45 |
| *** jongchoi has quit IRC | 08:48 | |
| *** achanda has quit IRC | 08:48 | |
| xylan_kong | rakhmerov: please check you email | 09:00 |
| rakhmerov | yeah, I see | 09:00 |
| *** openstackgerrit has quit IRC | 09:01 | |
| openstackgerrit | Merged openstack/mistral: Refactored filter implementation. https://review.openstack.org/234625 | 09:02 |
| *** openstackgerrit has joined #openstack-mistral | 09:02 | |
| rakhmerov | xylan_kong: I like you slides | 09:09 |
| rakhmerov | looks good | 09:09 |
| openstackgerrit | hardik proposed openstack/mistral: Fixed some unit test issue. https://review.openstack.org/237524 | 09:16 |
| *** ^Gal^ has joined #openstack-mistral | 09:20 | |
| ^Gal^ | Hi, please review me :) https://review.openstack.org/#/c/234800/ | 09:24 |
| xylan_kong | ^Gal^: it looks good to me, thanks for the effort :-) | 09:26 |
| *** openstackgerrit has quit IRC | 09:31 | |
| *** openstackgerrit has joined #openstack-mistral | 09:32 | |
| rakhmerov | ^Gal^: ok, I will | 09:33 |
| *** sergmelikyan has joined #openstack-mistral | 09:34 | |
| ^Gal^ | thanks :) np | 09:35 |
| * xylan_kong LEAVE | 09:42 | |
| openstackgerrit | Merged openstack/mistral: mistral-documentation: dashboard documentation regarding debug known issue https://review.openstack.org/234800 | 09:44 |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral: Fixed some unit test issue. https://review.openstack.org/237524 | 09:48 |
| *** sergmelikyan has quit IRC | 09:53 | |
| openstackgerrit | Nikolay Mahotkin proposed openstack/mistral-extra: Fixing upgrade kernel example https://review.openstack.org/237968 | 10:06 |
| *** zhenguo has quit IRC | 10:10 | |
| *** hparekh2 has joined #openstack-mistral | 10:15 | |
| *** hparekh has quit IRC | 10:16 | |
| *** dmowrer has joined #openstack-mistral | 10:23 | |
| *** sergmelikyan has joined #openstack-mistral | 10:26 | |
| *** dmowrer has quit IRC | 10:27 | |
| openstackgerrit | Merged openstack/mistral-extra: Fixing upgrade kernel example https://review.openstack.org/237968 | 10:32 |
| *** achanda has joined #openstack-mistral | 10:36 | |
| *** achanda has quit IRC | 10:37 | |
| *** melisha has left #openstack-mistral | 10:51 | |
| *** sergmelikyan has quit IRC | 10:55 | |
| openstackgerrit | Nikolay Mahotkin proposed openstack/mistral: Fixing SSH actions to use names of private keys https://review.openstack.org/236992 | 10:57 |
| *** dmowrer has joined #openstack-mistral | 11:06 | |
| *** ^Gal^ has quit IRC | 11:20 | |
| *** ^Gal^__ has joined #openstack-mistral | 11:20 | |
| *** ^Gal^__ has quit IRC | 11:20 | |
| *** ^Gal^ has joined #openstack-mistral | 11:22 | |
| openstackgerrit | hardik proposed openstack/mistral: Add decode() function for string comparison. https://review.openstack.org/234587 | 11:38 |
| *** Ephur has joined #openstack-mistral | 12:01 | |
| *** Ephur has quit IRC | 12:06 | |
| *** ^Gal^__ has joined #openstack-mistral | 12:16 | |
| *** ^Gal^ is now known as Guest23580 | 12:16 | |
| *** ^Gal^__ is now known as ^Gal^ | 12:16 | |
| *** openstackgerrit has quit IRC | 12:16 | |
| *** openstackgerrit has joined #openstack-mistral | 12:17 | |
| *** ^Gal^ has quit IRC | 12:17 | |
| *** Guest23580 has quit IRC | 12:17 | |
| *** ^Gal^ has joined #openstack-mistral | 12:18 | |
| openstackgerrit | Nikolay Mahotkin proposed openstack/mistral: Fixing SSH actions to use names of private keys https://review.openstack.org/236992 | 12:24 |
| *** shangxdy has joined #openstack-mistral | 12:44 | |
| *** dmowrer has quit IRC | 12:47 | |
| *** dmowrer has joined #openstack-mistral | 13:09 | |
| *** bobh has joined #openstack-mistral | 13:17 | |
| *** tej has joined #openstack-mistral | 13:43 | |
| openstackgerrit | Nikolay Mahotkin proposed openstack/mistral: Fixing SSH actions to use names of private keys https://review.openstack.org/236992 | 14:04 |
| *** achanda has joined #openstack-mistral | 14:05 | |
| *** tristanC has quit IRC | 14:07 | |
| *** jongchoi has joined #openstack-mistral | 14:08 | |
| *** achanda has quit IRC | 14:09 | |
| *** tonytan4ever has joined #openstack-mistral | 14:27 | |
| *** jongchoi has quit IRC | 14:28 | |
| *** dmowrer has quit IRC | 14:44 | |
| *** dmowrer has joined #openstack-mistral | 14:45 | |
| *** dmowrer has quit IRC | 14:49 | |
| *** Piet has quit IRC | 14:51 | |
| *** dmowrer has joined #openstack-mistral | 15:00 | |
| *** dmowrer has quit IRC | 15:04 | |
| *** dmowrer has joined #openstack-mistral | 15:17 | |
| *** Ephur has joined #openstack-mistral | 15:17 | |
| *** Piet has joined #openstack-mistral | 16:11 | |
| *** bobh has quit IRC | 16:25 | |
| *** achanda has joined #openstack-mistral | 16:28 | |
| *** gyee has joined #openstack-mistral | 16:45 | |
| *** shangxdy has quit IRC | 16:56 | |
| *** shangxdy has joined #openstack-mistral | 17:05 | |
| *** dmowrer has quit IRC | 17:12 | |
| *** dmowrer has joined #openstack-mistral | 17:13 | |
| *** shangxdy has quit IRC | 17:16 | |
| *** dmowrer has quit IRC | 17:17 | |
| *** dmowrer has joined #openstack-mistral | 17:31 | |
| *** harlowja has quit IRC | 17:57 | |
| *** harlowja has joined #openstack-mistral | 18:02 | |
| *** Piet has quit IRC | 18:17 | |
| *** dmowrer has quit IRC | 18:32 | |
| *** tonytan4ever has quit IRC | 18:32 | |
| *** dmowrer has joined #openstack-mistral | 18:33 | |
| *** tonytan4ever has joined #openstack-mistral | 18:33 | |
| *** dmowrer has quit IRC | 18:37 | |
| *** dmowrer has joined #openstack-mistral | 18:51 | |
| *** dmowrer has quit IRC | 18:51 | |
| *** dmowrer has joined #openstack-mistral | 18:51 | |
| *** dmowrer has quit IRC | 18:56 | |
| *** dmowrer has joined #openstack-mistral | 19:09 | |
| *** tej has quit IRC | 19:10 | |
| *** dmowrer has quit IRC | 19:12 | |
| *** dmowrer has joined #openstack-mistral | 19:13 | |
| *** achanda has quit IRC | 19:16 | |
| *** bobh has joined #openstack-mistral | 19:22 | |
| *** Piet has joined #openstack-mistral | 19:43 | |
| *** harlowja has quit IRC | 19:50 | |
| *** tej has joined #openstack-mistral | 19:52 | |
| *** tej has quit IRC | 20:07 | |
| *** achanda has joined #openstack-mistral | 20:16 | |
| *** achanda has quit IRC | 20:22 | |
| *** tej has joined #openstack-mistral | 20:33 | |
| *** harlowja has joined #openstack-mistral | 20:39 | |
| *** harlowja_ has joined #openstack-mistral | 20:40 | |
| *** achanda has joined #openstack-mistral | 20:42 | |
| *** harlowja has quit IRC | 20:44 | |
| *** achanda has quit IRC | 20:46 | |
| *** bobh has quit IRC | 20:47 | |
| *** FL1SK has quit IRC | 20:55 | |
| *** FL1SK has joined #openstack-mistral | 20:57 | |
| *** achanda has joined #openstack-mistral | 20:57 | |
| *** achanda has quit IRC | 21:10 | |
| *** jongchoi_ has joined #openstack-mistral | 21:11 | |
| *** tej has quit IRC | 21:30 | |
| *** jongchoi_ has quit IRC | 22:02 | |
| *** jongchoi_ has joined #openstack-mistral | 22:05 | |
| *** jongchoi_ has quit IRC | 22:10 | |
| *** tej has joined #openstack-mistral | 22:19 | |
| *** tej has quit IRC | 22:31 | |
| *** dmowrer has quit IRC | 22:33 | |
| *** dmowrer has joined #openstack-mistral | 22:34 | |
| *** dmowrer has quit IRC | 22:39 | |
| *** harlowja_ has quit IRC | 23:33 | |
| *** tej has joined #openstack-mistral | 23:35 | |
| *** harlowja has joined #openstack-mistral | 23:37 | |
| *** harlowja_ has joined #openstack-mistral | 23:45 | |
| *** harlowja has quit IRC | 23:49 | |
| *** tej has quit IRC | 23:55 | |
| *** tonytan4ever has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!