The trade condition is only mandatory for automated invoices, if the accountant does things manually, it is
  not required to create/use one.

   Be consistent and also search for a Remote Node, since it can also contain Compute Partitions.

   Otherwise, whenever you run the tests multiple times, it ends up failing by selecting the wrong one on the listbox.

  Accountant should create the proper document rather them use this workarround action.

  This helps to emulate payments, so we can test outcome before/after payment to wechat

  Only launch caucase when needed.  Not all tests need
  caucase, this saves a bit the overhead to run it

   We do expect that all trade conditions be attached to a business process, so for manual accounting
   the condition changed.

    Se would like to skip consistency check if the invoice isn't automated (ledger) and has no lines,
   otherwise this consistency check should still apply as usual.

   This patch would replace guard for deliver_action to not call isDivergent that would make slow when open a Transaction (I don't remember the reason exactly, probably due large simulation trees).

   This breaks guards when simulation is not used (so it will never be solved) since the transition would never be allowed to the accountant.

   So if isDivergent proves itself slow, we should patch differently, for now keep the original implementation, since it is proven to work.

   Only require project when ledger is automated
   Accept no ledger (1 or 0) on accounting transactions in case manual accounting is used.

   Let the site administrator configure proper accounting instead.

   Business Process are mandatory for Trade Condition, so this process covers
   manual accounting.

See merge request nexedi/slapos.core!677

   Shadow user have no access to Allocation Supply exclusive to the user, so access the variation before change into shadow user.

   Include a constraint to ensure group is set when function/accounting/{agent,manager} is used on
   assignments.

   This custom approach is used on Panel View and it is only working properly for Panel View since the gadget that will be redirected into is the slapos_panel_view.

   When use normal renderjs_runner it may cause confusion, so keep the URLs just pointing to normal locations.

   This helps administrator to identify which project the ticket is related too.

    This ticket list is broken and it is duplicated with Panel View anyway.

   This information is duplicated with Panel View anyway, no reason to keep it

   Whenever the serialization is JSON-in-xml the parameters are uploaded into {'_' : '{}') format.

    Add mixin to Remote Node and Instance Node, so they can respond to getAccessStatus calls.

    Despite we don't set data on it (same for Slave Instance), it is important to respond this API to display
    Property the Status Gadgets on the API.

    Later on we can setAccessStatus based on Software Instance (for Instance Node) or Compute Node
    based on some indirect criteria, but this not covered by this commit.

This part was not following the new design.
Use the new Type variation document to define how much a instance consumes the compute node capacity.