slapos_cloud: force open order to be created to allocate

master/bt5/slapos_cloud/SkinTemplateItem/portal_skins/slapos_cloud/SoftwareInstance_tryToAllocatePartition.py

@@ -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())

master/bt5/slapos_cloud/TestTemplateItem/portal_components/test.erp5.SlapOSTestCaseMixin.py

@@ -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)

master/bt5/slapos_cloud/TestTemplateItem/portal_components/test.erp5.testSlapOSCloudAllocationAlarm.xml

@@ -100,11 +100,13 @@
   </record>
   <record id="4" aka="AAAAAAAAAAQ=">
     <pickle>
-      <global name="WorkflowHistoryList" module="Products.ERP5Type.patches.WorkflowTool"/>
+      <global name="WorkflowHistoryList" module="Products.ERP5Type.Workflow"/>
     </pickle>
     <pickle>
-      <tuple>
-        <none/>
+      <dictionary>
+        <item>
+            <key> <string>_log</string> </key>
+            <value>
               <list>
                 <dictionary>
                   <item>
@@ -117,7 +119,9 @@
                   </item>
                 </dictionary>
               </list>
-      </tuple>
+            </value>
+        </item>
+      </dictionary>
     </pickle>
   </record>
 </ZopeData>