[Kimchi-devel] [RFC] Problems with iso names while create new template

Rodrigo Trujillo rodrigo.trujillo at linux.vnet.ibm.com
Thu May 22 20:08:28 UTC 2014


Hi,

There is a problem while creating a new template with local isos.
If some iso is stored inside an active storagepool, then kimchi is going 
to show it for selection.

Kimchi looks for any file, with any name, but of type "iso" - 
IsoVolumesModel:get_list() in
src/kimchi/model/storagevolumes.py.

So, isos rename like  ubuntu.iso.1 or even teste.exe will be shown to user.
Error happens if user selects one of them, because the JSON schema 
restricts file names to
ends with ".iso".

According to my tests, Libvirt accepts any name, since the file is ISO 
type. So, I would like to
propose to remove the ".iso" checking in the JSON schema.

This is not going to break anything in UI.
Problems would happen if user tries to create a template using the API 
directly and gives a file
that is not an ISO. In this case, a error will be returned by libvirt 
and triggered to user.

Patch is quite simple. Any objection ?

Rodrigo Trujillo




More information about the Kimchi-devel mailing list