
<html><head><meta http-equiv="Content-Type" content="text/html charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">Got some questions while fixing a bug <a href="https://review.openstack.org/#/c/102391/">https://review.openstack.org/#/c/102391/</a>: <div><br></div><div><div>* We must convey the action ERROR details back to the engine, and to the end user. Log is not sufficient. How exactly? Via context? Via extra parameters to convey_execution_results? Need a field in the model.</div><div><a href="https://github.com/stackforge/mistral/blob/master/mistral/engine/drivers/default/executor.py#L46-L59">https://github.com/stackforge/mistral/blob/master/mistral/engine/drivers/default/executor.py#L46-L59</a></div><div><br></div><div><br></div><div>* What is the reason to update status on task failure in handle_task_error via direct DB access, not via convey_task_results? </div><div><a href="https://github.com/stackforge/mistral/blob/master/mistral/engine/drivers/default/executor.py#L61">https://github.com/stackforge/mistral/blob/master/mistral/engine/drivers/default/executor.py#L61</a> Bypassing convey_task_results can cause grief from missing TRACE statements to more serious stuff… And looks like we are failing the whole execution there? Just because one action had failed? Please clarify the intend here. Note: details may all go away while doing Refine Engine <-> Executor protocol blueprint <a href="https://blueprints.launchpad.net/mistral/+spec/mistral-engine-executor-protocol">https://blueprints.launchpad.net/mistral/+spec/mistral-engine-executor-protocol</a>, we just need to clarify the intent</div></div><div><br></div><div>DZ> </div></body></html>