*** bobh has joined #openstack-mistral | 00:35 | |
*** bobh has quit IRC | 00:42 | |
*** bobh has joined #openstack-mistral | 00:45 | |
*** bobh has quit IRC | 01:00 | |
*** bobh has joined #openstack-mistral | 01:39 | |
*** bobh has quit IRC | 01:44 | |
*** bobh has joined #openstack-mistral | 01:53 | |
*** bobh has quit IRC | 01:54 | |
*** bobh has joined #openstack-mistral | 02:09 | |
*** apetrich has quit IRC | 02:10 | |
*** bobh has quit IRC | 02:43 | |
*** bobh has joined #openstack-mistral | 02:46 | |
*** bobh has quit IRC | 02:50 | |
*** ricolin has joined #openstack-mistral | 03:49 | |
*** gkadam has joined #openstack-mistral | 03:54 | |
*** gkadam has quit IRC | 03:54 | |
*** jtomasek has joined #openstack-mistral | 04:28 | |
openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Add "published_global" field to the task execution REST resource https://review.opendev.org/678530 | 04:37 |
---|---|---|
*** jtomasek has quit IRC | 04:40 | |
openstackgerrit | Renat Akhmerov proposed openstack/mistral master: moved generic util functions from mistral to mistral-lib https://review.opendev.org/676373 | 04:41 |
*** jtomasek has joined #openstack-mistral | 04:51 | |
*** eyalb has joined #openstack-mistral | 05:21 | |
*** eyalb has quit IRC | 05:23 | |
*** eyalb1 has joined #openstack-mistral | 05:23 | |
*** eyalb has joined #openstack-mistral | 06:11 | |
*** eyalb1 has quit IRC | 06:13 | |
openstackgerrit | Merged openstack/mistral master: Mistral don't work with cron trigger https://review.opendev.org/679533 | 06:25 |
openstackgerrit | Merged openstack/mistral-tempest-plugin master: Add test for OpenStack services dynamic versions https://review.opendev.org/662399 | 06:27 |
*** mmethot_ has joined #openstack-mistral | 06:43 | |
*** mmethot has quit IRC | 06:45 | |
openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Check if workflow execution is empty in integrity checker https://review.opendev.org/680302 | 06:51 |
*** eyalb1 has joined #openstack-mistral | 06:51 | |
*** openstackgerrit has quit IRC | 06:51 | |
*** eyalb has quit IRC | 06:54 | |
*** eyalb1 has quit IRC | 07:01 | |
*** eyalb has joined #openstack-mistral | 07:01 | |
*** abdelal has joined #openstack-mistral | 07:19 | |
*** pgaxatte has joined #openstack-mistral | 07:29 | |
*** abdelal has quit IRC | 08:07 | |
*** openstackgerrit has joined #openstack-mistral | 08:07 | |
openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Add a migration to create the scheduled_jobs table https://review.opendev.org/669558 | 08:07 |
*** vgvoleg has joined #openstack-mistral | 08:14 | |
vgvoleg | Hi everyone! | 08:20 |
vgvoleg | false alarm | 08:34 |
vgvoleg | just found a bug, but it is in our fork =((( | 08:35 |
openstackgerrit | pengyuesheng proposed openstack/mistral master: Blacklist eventlet 0.25.0 https://review.opendev.org/680330 | 08:38 |
rakhmerov | vgvoleg: hi :) | 08:42 |
rakhmerov | ok | 08:42 |
openstackgerrit | dharmendra kushwaha proposed openstack/mistral master: Add pdf build support https://review.opendev.org/680337 | 08:43 |
*** abdelal has joined #openstack-mistral | 08:48 | |
vgvoleg | I have one question about https://blueprints.launchpad.net/mistral/+spec/mistral-task-skipping-feature | 09:07 |
vgvoleg | If the task has on-error, should we have an ability to skip this task? | 09:08 |
vgvoleg | IMO no | 09:39 |
vgvoleg | But one second | 09:39 |
vgvoleg | If the task has on-error, should we have an ability to rerun this task? | 09:39 |
vgvoleg | I think no | 09:40 |
vgvoleg | But there are some cases when we can rerun this task even if on-error branch exist | 09:40 |
vgvoleg | As I understand, transition of wf execution from SUCCESS to RUNNING is incorrect | 09:41 |
vgvoleg | And this is the only rule that forbid rerun of this task | 09:42 |
vgvoleg | but if there is any error in on-error branch, we can rerun the root task | 09:43 |
openstackgerrit | pengyuesheng proposed openstack/mistral master: Blacklist eventlet 0.21.0,0.23.0,0.25.0 https://review.opendev.org/680330 | 09:47 |
rakhmerov | vgvoleg: reading your stuff.. | 10:16 |
rakhmerov | vgvoleg: not sure why you are talking about the transition from SUCCESS to RUNNING | 10:17 |
rakhmerov | it's never supposed to happen | 10:17 |
rakhmerov | now we can only rerun a failed task | 10:17 |
rakhmerov | and even if the failed task has "on-error" it will still be in ERROR state | 10:18 |
rakhmerov | it won't just lead to failing the entire workflow | 10:18 |
rakhmerov | so | 10:18 |
rakhmerov | if a task has "on-error" I think it'll be problematic to rerun this task because we'll have to clean up everything that ran after it (initiated by "on-error") | 10:19 |
rakhmerov | or there will be new copies (instance) of downstream task executions | 10:19 |
rakhmerov | which is bad too | 10:19 |
rakhmerov | so I'd say yes, we shouldn't allow to rerun a task if it has "on-error" | 10:21 |
openstackgerrit | Eyal proposed openstack/mistral master: Add pdf build support https://review.opendev.org/680337 | 10:43 |
vgvoleg | rakhmerov: well, so the rerun behavior is broken too :( | 11:39 |
vgvoleg | I'll try to fix it then, it is needed in skipping feature too | 11:39 |
openstackgerrit | Eyal proposed openstack/mistral master: Add pdf build support https://review.opendev.org/680337 | 11:54 |
openstackgerrit | Eyal proposed openstack/mistral master: Add pdf build support https://review.opendev.org/680337 | 12:14 |
openstackgerrit | Eyal proposed openstack/mistral master: Add pdf build support https://review.opendev.org/680337 | 12:41 |
openstackgerrit | Merged openstack/mistral master: Check if workflow execution is empty in integrity checker https://review.opendev.org/680302 | 13:34 |
*** vgvoleg has quit IRC | 14:03 | |
*** eyalb has quit IRC | 14:32 | |
*** abdelal has quit IRC | 15:27 | |
*** altlogbot_1 has quit IRC | 16:01 | |
*** altlogbot_3 has joined #openstack-mistral | 16:02 | |
*** irclogbot_3 has quit IRC | 16:03 | |
*** irclogbot_2 has joined #openstack-mistral | 16:04 | |
*** irclogbot_2 has quit IRC | 16:07 | |
*** irclogbot_2 has joined #openstack-mistral | 16:08 | |
*** pgaxatte has quit IRC | 16:17 | |
*** zigo has quit IRC | 16:43 | |
*** zigo has joined #openstack-mistral | 17:10 | |
*** ricolin has quit IRC | 17:35 | |
openstackgerrit | Merged openstack/mistral master: Add "published_global" field to the task execution REST resource https://review.opendev.org/678530 | 18:31 |
*** threestrands has joined #openstack-mistral | 23:30 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!