[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] domain uuid always change in xl list -l if no uuid in vm.cfg



On Thu, May 15, 2014 at 04:00:31PM -0400, Zhigang Wang wrote:
> Hi,
> 
> `xl list --long` will return a random domain UUID if no UUID is specified in 
> vm.cfg::
> 
>     # xl list -l |grep uuid
>                 "uuid": "bbf0c18e-3325-4fe9-b303-5fa1ef0751c5",
>     # xl list -l |grep uuid
>                 "uuid": "4982f6e2-0e17-4c88-8fa1-1d333d9f1f6e",
> 
> In this case, the domain handle (uuid) is set as 
> `ffc51cf9-2fb2-4838-b757-bcd65ace42b6` and will not change.
> 
> I think we may need always return the domain handle as shown in xenstore::
> 
>     # xenstore-read /local/domain/1/vm
>     /vm/ffc51cf9-2fb2-4838-b757-bcd65ace42b6
> 
> Related code tool/libxl/xl_cmdimpl.c:
> 
> static void parse_config_data(...
> 
>     if (!xlu_cfg_get_string (config, "uuid", &buf, 0) ) {
>         if ( libxl_uuid_from_string(&c_info->uuid, buf) ) {
>             fprintf(stderr, "Failed to parse UUID: %s\n", buf);
>             exit(1);
>         }
>     }else{
>         libxl_uuid_generate(&c_info->uuid);
>     }
> 
> I don't think it's right to just get the domain handle here. We may need to 
> refactor the logic a little.
> 
> If Wei or anyone else is working on related staff, please consider this.
> 

My series already covers this. ;-)

Weil


> Thanks,
> 
> Zhigang

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.