[Engine-devel] getDeviceList should report partitioned devices, too
Livnat Peer
lpeer at redhat.com
Sun Jan 22 07:29:03 UTC 2012
On 22/01/12 08:28, Eduardo Warszawski wrote:
>
>
> ----- Original Message -----
>>
>>
>> ----- Original Message -----
>>> On Wed, Jan 18, 2012 at 06:17:53AM -0500, Ayal Baron wrote:
>>>> Can we broaden the scope and also allow passing createVG
>>>> partitioned devices with an override flag or something? (we'd
>>>> need
>>>> to check the devices and run "kpartx -d" and fdisk to clean the
>>>> devices before calling pvcreate).
>>>
>>> We can, and we should. My initial patch is just the bare minimum;
>>> I'd
>>> like
>>> Douglas to carry it on, and I am still waiting to meet his Engine
>>> counterpart.
>>> Currently, a LUN that was once used as a raw hard disk cannot be
>>> used
>>> by RHEV;
>>> that's sad.
>>>
>>> How about this for API:
>>>
>>> createVG(self, vgname, devlist, options={trashpart_devlist: []})
>>
>> No, stop using options as a trash can.
>> If we're changing the API, it should already be just passing a LUNs
>> dictionary to createStorageDomain and start deprecating createVG
>
> Was agreed that createVG and all vg-uuid aware functions will be removed shortly.
> Use only createStorageDomain.
>
When you write 'removed'? you don't actually mean remove, right?
>>
>>>
>>> createVG would honor an optional list of devices (subset of
>>> devlist)
>>> whose
>>> partition tables should be trashed.
>>>
>>> Dan.
>>>
>>
More information about the Engine-devel
mailing list