Updates:
        Status: Fixed

Comment #3 on issue 633 by [email protected]: Migration of ext type instances fail due to physical_id missing
http://code.google.com/p/ganeti/issues/detail?id=633

commit a63985f3d1612d7e4c05f9eeb65572e7c9ef11f2
Author: Dimitris Aragiorgis <[email protected]>
Date:   Mon Dec 9 14:00:27 2013 +0200

    SetDiskID() before accepting an instance

    SetDiskID() fills physical_id slot of a Disk object.

    LUInstanceSetParams() does not invoke SetDiskID() upon creation of a
    new disk. As a result the physical_id slot of the Disk object in
    config data is missing.

    In case of ext disk template, in AcceptInstance() we invoke
    _GatherAndLinkBlockDevices(). This takes `instance` as an argument
    which includes current disks info. So, after adding a disk,
    migration of ext instances will fail because FindDevice() expects
    the physical_id slot.

    With this patch we invoke SetDiskID() for every disk of the instance
    before accept_instance() RPC.

    Fixes Issue 633.

    Signed-off-by: Dimitris Aragiorgis <[email protected]>
    Signed-off-by: Michele Tartara <[email protected]>
    Reviewed-by: Michele Tartara <[email protected]>


--
You received this message because this project is configured to send all issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings

Reply via email to