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,24 +100,28 @@
   </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/>
-        <list>
-          <dictionary>
-            <item>
-                <key> <string>action</string> </key>
-                <value> <string>validate</string> </value>
-            </item>
-            <item>
-                <key> <string>validation_state</string> </key>
-                <value> <string>validated</string> </value>
-            </item>
-          </dictionary>
-        </list>
-      </tuple>
+      <dictionary>
+        <item>
+            <key> <string>_log</string> </key>
+            <value>
+              <list>
+                <dictionary>
+                  <item>
+                      <key> <string>action</string> </key>
+                      <value> <string>validate</string> </value>
+                  </item>
+                  <item>
+                      <key> <string>validation_state</string> </key>
+                      <value> <string>validated</string> </value>
+                  </item>
+                </dictionary>
+              </list>
+            </value>
+        </item>
+      </dictionary>
     </pickle>
   </record>
 </ZopeData>