[PATCH] Fix backend error when creating multiple templates

Backend fails when user tries to create multiple templetes using local isos. The error occurs when Kimchi verifies if qemu has access to the local iso. Kimchi creates a tmp VM for each iso, but, they have same name and uuid, what causes the error. This patch changes the uuid for each VM, fixing the problem. Signed-off-by: Rodrigo Trujillo <rodrigo.trujillo@linux.vnet.ibm.com> --- src/kimchi/kvmusertests.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/kimchi/kvmusertests.py b/src/kimchi/kvmusertests.py index 1914a94..fac61de 100644 --- a/src/kimchi/kvmusertests.py +++ b/src/kimchi/kvmusertests.py @@ -39,7 +39,7 @@ class UserTests(object): </domain>""" def __init__(self): - self.vm_uuid = uuid.uuid3(uuid.NAMESPACE_DNS, 'vm-test.kimchi.org') + self.vm_uuid = uuid.uuid1() self.vm_name = "kimchi_test_%s" % self.vm_uuid def probe_user(self): -- 1.9.0

-- Reviewed-by: Paulo Vital <pvital@linux.vnet.ibm.com> On Fri, 2014-04-11 at 19:34 -0300, Rodrigo Trujillo wrote:
Backend fails when user tries to create multiple templetes using local isos. The error occurs when Kimchi verifies if qemu has access to the local iso. Kimchi creates a tmp VM for each iso, but, they have same name and uuid, what causes the error. This patch changes the uuid for each VM, fixing the problem.
Signed-off-by: Rodrigo Trujillo <rodrigo.trujillo@linux.vnet.ibm.com> --- src/kimchi/kvmusertests.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/kimchi/kvmusertests.py b/src/kimchi/kvmusertests.py index 1914a94..fac61de 100644 --- a/src/kimchi/kvmusertests.py +++ b/src/kimchi/kvmusertests.py @@ -39,7 +39,7 @@ class UserTests(object): </domain>"""
def __init__(self): - self.vm_uuid = uuid.uuid3(uuid.NAMESPACE_DNS, 'vm-test.kimchi.org') + self.vm_uuid = uuid.uuid1() self.vm_name = "kimchi_test_%s" % self.vm_uuid
def probe_user(self):

Applied. Thanks. Regards, Aline Manera
participants (3)
-
Aline Manera
-
Paulo Ricardo Paz Vital
-
Rodrigo Trujillo