openstack/senlin
Code Issues Proposed changes

Added two comments as TODOs

Browse Source 
It can be safely ignored if the logic is implemented somewhere else.
This commit is contained in:
tengqm 2014-12-31 17:51:56 +08:00
parent 7047ca0bf0
commit c05c2fcefd
1 changed files with 2 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
senlin/db/sqlalchemy/api.py
View File

@ -713,6 +713,7 @@ def action_add_depends_on(context, action_id, *actions):
_('Action with id "%s" not found') % action_id) _('Action with id "%s" not found') % action_id)
action.depends_on = list(set(actions).union(set(action.depends_on))) action.depends_on = list(set(actions).union(set(action.depends_on)))
# TODO(liuh): Set status to WAITING if 'depends_on' is not empty
action.save(_session(context)) action.save(_session(context))
return action return action
@ -724,6 +725,7 @@ def action_del_depends_on(context, action_id, *actions):
_('Action with id "%s" not found') % action_id) _('Action with id "%s" not found') % action_id)
action.depends_on = list(set(action.depends_on).different(set(actions))) action.depends_on = list(set(action.depends_on).different(set(actions)))
# TODO(liuh): Set status to READY if 'depends_on' is empty
action.save(_session(context)) action.save(_session(context))
return action return action