<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">Ok, after all thinking my suggestion is to leave "break-on” but clarify its semantics and behavior a little bit as follow:<div class=""><ul class="MailOutline"><li class="">As Dmitri wrote before “success-on” and “error-on” may be easily confused with “on-success” and “on-error”.</li><li class="">“retry" policy loop may only stop if:</li><ul class=""><li class="">Task action/workflow completed successfully (no need to retry anymore). This case has nothing to do with “break-on”.</li><li class="">Task action/workflow failed and some condition (“break-on”) evaluates to True. So in this case I don’t think we need to give a user opportunity to change semantics of task behavior and be able to say “although task action/workflow failed I want this task to finish with success”. IMO, it may be 1) confusing 2) error-prone 3) complicating our understanding of how workflow works. In other works, I’m now against of this behavior and I think that <b class="">success/error of action/workflow should exactly match to success/error of task</b>.</li></ul><li class="">Based on the previous thoughts evaluation of “break-on” condition should work against task inbound context that doesn’t contain a task result itself (because the action failed) but may include results of other tasks completed in parallel branches. The general use case for this is to “stop waiting for something if we see that fundamental requirements for that are not met”.</li></ul><div class=""><br class=""></div></div><div class="">Thoughts?</div><div class=""><br class=""><div class="">
<div class="">Renat Akhmerov</div><div class="">@ Mirantis Inc.</div><div class=""><br class=""></div><br class="Apple-interchange-newline">
</div>
<br class=""><div><blockquote type="cite" class=""><div class="">On 21 Apr 2015, at 14:11, Nikolay Makhotkin <<a href="mailto:nmakhotkin@mirantis.com" class="">nmakhotkin@mirantis.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" class=""><div class="gmail_extra"><div class="">Any more suggestions/thoughts here? Dmitri? </div><div class=""><br class=""></div><div class="">More words: succeed-on / fail-on, success-expr / error-expr.</div><div class=""><br class=""></div>-- <br class=""><div class="gmail_signature"><div dir="ltr" class=""><div class=""><font size="2" class="">Best Regards,</font></div><div class=""><font size="2" class="">Nikolay</font></div></div></div>
</div></div>
__________________________________________________________________________<br class="">OpenStack Development Mailing List (not for usage questions)<br class="">Unsubscribe: <a href="mailto:OpenStack-dev-request@lists.openstack.org" class="">OpenStack-dev-request@lists.openstack.org</a>?subject:unsubscribe<br class=""><a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" class="">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br class=""></div></blockquote></div><br class=""></div></body></html>