[libvirt] [PATCH 1/3] Parallels: remove disk serial number check

Dmitry Guryanov dguryanov at parallels.com
Thu Mar 26 18:06:40 UTC 2015


On 03/26/2015 08:51 PM, Alexander Burluka wrote:
> OpenStack needs disk serial number setup because
> nova boot --block-device-mapping command generates that param in
> libvirt xml. I took QEMU libvirt driver behavior as a base.
> QEMU driver skips inability to set serial and continues work.
> So Parallels driver will ignore this param too and let domain
> boot.
> ---
>   src/parallels/parallels_sdk.c |    4 +---
>   1 files changed, 1 insertions(+), 3 deletions(-)
>
> diff --git a/src/parallels/parallels_sdk.c b/src/parallels/parallels_sdk.c
> index c36b772..fa5c44d 100644
> --- a/src/parallels/parallels_sdk.c
> +++ b/src/parallels/parallels_sdk.c
> @@ -2358,10 +2358,8 @@ static int prlsdkCheckDiskUnsupportedParams(virDomainDiskDefPtr disk)
>       }
>   
>       if (disk->serial) {
> -        virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
> -                       _("Setting disk serial number is not "
> +        VIR_INFO("%s", _("Setting disk serial number is not "
>                            "supported by parallels driver."));
> -        return -1;
>       }
>   
>       if (disk->wwn) {
ACK

-- 
Dmitry Guryanov




More information about the libvir-list mailing list