> +LUXI and IAllocator protocol extension
> +--------------
> +
> +The LUXI and IAllocator protocols are extended to include in the ``node``:
> +
> +* ``storage``: a list of objects (storage units) with
> +  # Storage unit, containing in order:
> +      # storage type
> +      # storage key (e.g. volume group name)
> +      # extra parameters (e.g. flag for exclusive storage) as a list.
> +  # Amount free in MiB
> +  # Amount total in MiB
> +
> +.. code-block:: javascript
> +
> +    {
> +      "storage": [
> +        { "sunit": ["drbd8", "xenvg", [false]]

what is this false supposed to signal? Exclusive storage is
not available for drbd, as drbd always depends on the network
(and hence not exclusive on operations of the very instance).

> +        , "free": 2000,
> +        , "total": 4000
> +        },
> +        { "sunit": ["file", "/path/to/storage1", []]
> +        , "free": 5000,
> +        , "total": 10000
> +        },
> +        { "sunit": ["file", "/path/to/storage2", []]
> +        , "free": 1000,
> +        , "total": 20000
> +        },
> +        { "sunit": ["lvm-vg", "xenssdvg", [false]]
> +        , "free": 1024,
> +        , "total": 1024
> +        }
> +      ]
> +    }
> +
> +is a node with an LVM volume group mirrored over DRBD, two file storage
> +directories, one half full, one mostly full, and a non-mirrored volume group.
> +
> +The storage type ``drbd8`` needs to be added in order to differentiate 
> between
> +mirrored storage and non-mirrored storage.
> +

Rest LGTM

-- 
Klaus Aehlig
Google Germany GmbH, Dienerstr. 12, 80331 Muenchen
Registergericht und -nummer: Hamburg, HRB 86891
Sitz der Gesellschaft: Hamburg
Geschaeftsfuehrer: Graham Law, Christine Elizabeth Flores

Reply via email to