openstack/deb-python-taskflow
Code Issues Proposed changes

Merge "Show the failure discarded (and the future intention)"

Browse Source
This commit is contained in:
Jenkins
2015-01-23 22:35:25 +00:00
committed by Gerrit Code Review
parent e9c250af57 93d73b85c6
commit cfe478a2bc
1 changed files with 18 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
taskflow/engines/action_engine/runner.py
View File

@@ -143,8 +143,24 @@ class _MachineBuilder(object):
try:
event, result = fut.result()
retain = self._completer.complete(node, event, result)
if retain and isinstance(result, failure.Failure):
memory.failures.append(result)
if isinstance(result, failure.Failure):
if retain:
memory.failures.append(result)
else:
# NOTE(harlowja): avoid making any
# intention request to storage unless we are
# sure we are in DEBUG enabled logging (otherwise
# we will call this all the time even when DEBUG
# is not enabled, which would suck...)
if LOG.isEnabledFor(logging.DEBUG):
intention = self._storage.get_atom_intention(
node.name)
LOG.debug("Discarding failure '%s' (in"
" response to event '%s') under"
" completion units request during"
" completion of node '%s' (intention"
" is to %s)", result, event,
node, intention)
except Exception:
memory.failures.append(failure.Failure())
else: