<html>
<head>
<meta content="text/html; charset=windows-1252"
http-equiv="Content-Type">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<br>
Hi Ramon,<br>
<br>
Before starting coding again, let's clarify some points:<br>
<br>
1) You are going to change the way a Template is created.<br>
So we are adding a new parameter "source_media"<br>
<br>
<b>Question 1:</b> Will this source_media parameter replace
cdrom and base disk or user will continue be able to use cdrom and
base disk instead of source_media?<br>
I prefer to have a single way to create a template, ie, when
addind 'source_media', cdrom and base disk parameters will be
removed.<br>
<br>
2) VMTemplate does not rely on Model or libvirt or anything else
rather than get the default values on osinfo.py and merge data in a
data structure (dict).<br>
<br>
3) To avoid scan 'source_media' multiple times, VMTemplate has a
'scan' parameter.<br>
When set to 'true', source_media will be scanned to get OS and
OS version. Otherwise, VMTemplate will only merge data.<br>
<br>
4) Always take a look in the existing functions. There are some
useful functions to identify a URL path or local path.<br>
<br>
Hope it helps! =)<br>
<br>
Let me know if you need any help to do that.<br>
<br>
Regards,<br>
Aline Manera<br>
<br>
<div class="moz-cite-prefix">On 03/22/2016 04:20 PM, Ramon Medeiros
wrote:<br>
</div>
<blockquote
cite="mid:1458674415-5872-1-git-send-email-ramonn@linux.vnet.ibm.com"
type="cite">
<pre wrap="">Instead of specify if the media is cdrom or disk, use source_media to create a template
Ramon Medeiros (6):
Create a single field to pass the installation media
Method to retrieve stored templates at object store
Fix checking duplicate template before creating it
Identify installation media while creating template
Only use source_media as boot media
Update tests
API.json | 5 ++
i18n.py | 2 +-
model/templates.py | 13 ++--
tests/test_authorization.py | 4 +-
tests/test_livemigration.py | 7 +-
tests/test_mockmodel.py | 14 ++--
tests/test_model.py | 69 ++++++++++++--------
tests/test_rest.py | 32 ++++------
tests/test_template.py | 39 +++++-------
tests/test_vmtemplate.py | 24 +++----
utils.py | 30 +++++++++
vmtemplate.py | 152 ++++++++++++++++++++++++++++++++++++++++----
12 files changed, 279 insertions(+), 112 deletions(-)
</pre>
</blockquote>
<br>
</body>
</html>