Commit ffc31666 authored by Romain Courteaud's avatar Romain Courteaud 🐙

slapos_pdm: no need to check all rejected decisions

parent 2ebab63d
...@@ -79,13 +79,13 @@ if len(allocation_cell_list) == 1: ...@@ -79,13 +79,13 @@ if len(allocation_cell_list) == 1:
if compute_node is not None: if compute_node is not None:
assert compute_node.getRelativeUrl() in allocation_cell_list[0].getParentValue().getParentValue().getAggregateList() assert compute_node.getRelativeUrl() in allocation_cell_list[0].getParentValue().getParentValue().getAggregateList()
for previous_upgrade_decision in portal.portal_catalog( if portal.portal_catalog.getResultValue(
portal_type='Upgrade Decision', portal_type='Upgrade Decision',
aggregate__uid=instance_tree.getUid(), aggregate__uid=instance_tree.getUid(),
simulation_state=['rejected'] simulation_state=['rejected'],
software_release__uid=allocation_cell_list[0].getSoftwareReleaseUid()
): ):
# If same upgrade decision has been rejected, do nothing # If same upgrade decision has been rejected, do nothing
if previous_upgrade_decision.getSoftwareReleaseValue().getUrlString() == allocation_cell_list[0].getSoftwareReleaseValue().getUrlString():
return return
decision_title = 'A new upgrade is available for %s' % instance_tree.getTitle() decision_title = 'A new upgrade is available for %s' % instance_tree.getTitle()
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment