Commit 5dcf6088 authored by Romain Courteaud's avatar Romain Courteaud

slapos_cloud: force open order to be created to allocate

parent 3e89dea0
......@@ -22,6 +22,14 @@ def assignComputePartition(software_instance, instance_tree):
if instance_tree is None:
raise ValueError('%s does not have related instance tree' % software_instance.getRelativeUrl())
# allocation must be based on the Open Order related to the Instance Tree
# it is unrelated to the Person document
# Migrating all the code is needed
# Step1: force open order to be created to allocated
# Step2: XXX
if instance_tree.getAggregateRelatedValue(portal_type="Open Sale Order Line") is None:
raise Unauthorized("No Open Order")
person = instance_tree.getDestinationSectionValue(portal_type='Person')
if person is None:
raise ValueError('%s does not have person related' % instance_tree.getRelativeUrl())
......
......@@ -260,6 +260,9 @@ class SlapOSTestCaseMixin(testSlapOSMixin):
self.portal.portal_workflow._jumpToStateFor(self.requested_software_instance, 'start_requested')
self.requested_software_instance.validate()
self.tic()
# XXX Not part of slapos cloud
self.portal.portal_alarms.slapos_request_update_instance_tree_open_sale_order.activeSense()
self.tic()
def _makeSlaveTree(self, requested_template_id='template_slave_instance'):
return self._makeTree(requested_template_id=requested_template_id)
......
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