3:40 p.m.
From: Paulo Vital <pvital(a)linux.vnet.ibm.com>
This patchset adds support in backend to create templates and guests to netboot
without setting a cdrom path or URL as image to install. Once created a guest
to boot via network, the guest will request DHCP/TFTP/(NFS/HTTP/FTP) network
installation servers to download the configured images and start the install.
To test, use the curl commands:
curl -k -u test -H "Content-Type: application/json" -H \
"Accept: application/json"
'https://localhost:8001/plugins/kimchi/templates' \
-X POST -d '{"name": "test-netboot"}'
Enter host password for user 'test':
{
"cpu_info":{
"maxvcpus":1,
"vcpus":1
},
"graphics":{
"type":"vnc",
"listen":"127.0.0.1"
},
"cdrom":"netboot",
"networks":[
"default"
],
"icon":"plugins/kimchi/images/icon-vm.png",
"os_distro":"unknown",
"name":"test-netboot",
"disks":[
{
"index":0,
"format":"qcow2",
"pool":{
"type":"dir",
"name":"/plugins/kimchi/storagepools/default"
},
"size":10
}
],
"invalid":{},
"os_version":"unknown",
"memory":1024,
"folder":[]
}
curl -k -u test -H "Content-Type: application/json" -H \
"Accept: application/json" 'https://localhost:8001/plugins/kimchi/vms'
-X POST \
-d
'{"name":"1netboot-test","template":"/plugins/kimchi/templates/test-netboot"}'
Enter host password for user 'test':
{
"status":"running",
"message":"OK",
"id":"1",
"target_uri":"/plugins/kimchi/vms/1netboot-test"
}
Paulo Vital (3):
Add support to create templates without ISO image.
Add support to create guests without ISO image.
Update test cases to support netboot.
model/templates.py | 35 ++++++++++++++++++++++-------------
tests/test_template.py | 18 ++++++++++++++----
tests/test_vmtemplate.py | 16 ++++++++++++++++
vmtemplate.py | 29 ++++++++++++++++++-----------
xmlutils/bootorder.py | 44 ++++++++++++++++++++++++++++++++++++++++++++
5 files changed, 114 insertions(+), 28 deletions(-)
create mode 100644 xmlutils/bootorder.py
--
2.5.0
3:40 p.m.
New subject: [PATCH] [Kimchi 1/3] Add support to create templates without ISO image.
From: Paulo Vital <pvital(a)linux.vnet.ibm.com>
Changed API.json and model to accept empty parameters or missing the cdrom (ISO
image) parameter while creating a new template.
Now, when creating a new template and not specifying an ISO image it is assumed
the template will use netboot - PXE/DHCP/TFTP/(NFS/HTTP/FTP).
This is part of solution to Issue #372.
Signed-off-by: Paulo Vital <pvital(a)linux.vnet.ibm.com>
---
model/templates.py | 35 ++++++++++++++++++++++-------------
vmtemplate.py | 2 +-
2 files changed, 23 insertions(+), 14 deletions(-)
diff --git a/model/templates.py b/model/templates.py
index 8a29e02..bbf5081 100644
--- a/model/templates.py
+++ b/model/templates.py
@@ -41,18 +41,24 @@ class TemplatesModel(object):
def create(self, params):
name = params.get('name', '').strip()
- iso = params.get('cdrom')
- # check search permission
- if iso and iso.startswith('/') and os.path.exists(iso):
- st_mode = os.stat(iso).st_mode
- if stat.S_ISREG(st_mode) or stat.S_ISBLK(st_mode):
- user = UserTests().probe_user()
- run_setfacl_set_attr(iso, user=user)
- ret, excp = probe_file_permission_as_user(iso, user)
- if ret is False:
- raise InvalidParameter('KCHISO0008E',
- {'filename': iso, 'user':
user,
- 'err': excp})
+ iso = params.get('cdrom', '')
+
+ if not iso:
+ # set template to boot via network card (net boot)
+ params['cdrom'] = 'netboot'
+ params['os_distro'] = 'unknown'
+ params['os_version'] = 'unknown'
+ else:
+ # check search permission
+ if iso and iso.startswith('/') and os.path.exists(iso):
+ st_mode = os.stat(iso).st_mode
+ if stat.S_ISREG(st_mode) or stat.S_ISBLK(st_mode):
+ user = UserTests().probe_user()
+ run_setfacl_set_attr(iso, user=user)
+ ret, excp = probe_file_permission_as_user(iso, user)
+ if ret is False:
+ raise InvalidParameter('KCHISO0008E',
{'filename': iso,
+ 'user': user, 'err':
excp})
conn = self.conn.get()
for net_name in params.get(u'networks', []):
@@ -64,7 +70,10 @@ class TemplatesModel(object):
# Creates the template class with necessary information
# Checkings will be done while creating this class, so any exception
# will be raised here
- t = LibvirtVMTemplate(params, scan=True, conn=self.conn)
+ if params['cdrom'] == 'netboot':
+ t = LibvirtVMTemplate(params, conn=self.conn)
+ else:
+ t = LibvirtVMTemplate(params, scan=True, conn=self.conn)
# Validate cpu info
t.cpuinfo_validate()
diff --git a/vmtemplate.py b/vmtemplate.py
index ef17ff6..435bf42 100644
--- a/vmtemplate.py
+++ b/vmtemplate.py
@@ -501,7 +501,7 @@ class VMTemplate(object):
# validate iso integrity
# FIXME when we support multiples cdrom devices
iso = self.info.get('cdrom')
- if iso:
+ if iso and (iso != 'netboot'):
if os.path.exists(iso):
st_mode = os.stat(iso).st_mode
if not (stat.S_ISREG(st_mode) or stat.S_ISBLK(st_mode)):
--
2.5.0
3:40 p.m.
New subject: [PATCH] [Kimchi 2/3] Add support to create guests without ISO image.
From: Paulo Vital <pvital(a)linux.vnet.ibm.com>
Add support to create guests without CD-ROM device information and setting the
second boot order option as 'netboot'.
This is part of solution to Issue #372.
Signed-off-by: Paulo Vital <pvital(a)linux.vnet.ibm.com>
---
vmtemplate.py | 27 +++++++++++++++++----------
xmlutils/bootorder.py | 44 ++++++++++++++++++++++++++++++++++++++++++++
2 files changed, 61 insertions(+), 10 deletions(-)
create mode 100644 xmlutils/bootorder.py
diff --git a/vmtemplate.py b/vmtemplate.py
index 435bf42..ce99627 100644
--- a/vmtemplate.py
+++ b/vmtemplate.py
@@ -34,6 +34,7 @@ from wok.plugins.kimchi import imageinfo
from wok.plugins.kimchi import osinfo
from wok.plugins.kimchi.isoinfo import IsoImage
from wok.plugins.kimchi.utils import check_url_path, pool_name_from_uri
+from wok.plugins.kimchi.xmlutils.bootorder import get_bootorder_xml
from wok.plugins.kimchi.xmlutils.cpu import get_cpu_xml
from wok.plugins.kimchi.xmlutils.disk import get_disk_xml
from wok.plugins.kimchi.xmlutils.graphics import get_graphics_xml
@@ -365,15 +366,22 @@ class VMTemplate(object):
graphics.update(kwargs.get('graphics', {}))
params['graphics'] = get_graphics_xml(graphics)
- libvirt_stream_protocols = kwargs.get('libvirt_stream_protocols', [])
- cdrom_xml = self._get_cdrom_xml(libvirt_stream_protocols)
-
- if not urlparse.urlparse(self.info.get('cdrom', "")).scheme in
\
- libvirt_stream_protocols and \
- params.get('iso_stream', False):
- params['qemu-stream-cmdline'] = cdrom_xml
+ # Add information of CD-ROM device only if not setup to netboot.
+ # Also, setup the correct boot order
+ if params['cdrom'] == 'netboot':
+ params['boot_order'] = get_bootorder_xml(network=True)
else:
- params['cdroms'] = cdrom_xml
+ libvirt_stream_protocols = kwargs.get('libvirt_stream_protocols',
+ [])
+ cdrom_xml = self._get_cdrom_xml(libvirt_stream_protocols)
+
+ if (not urlparse.urlparse(self.info.get('cdrom',
"")).scheme in
+ libvirt_stream_protocols and
+ params.get('iso_stream', False)):
+ params['qemu-stream-cmdline'] = cdrom_xml
+ else:
+ params['cdroms'] = cdrom_xml
+ params['boot_order'] = get_bootorder_xml()
# max memory
params['max_memory'] = self._get_max_memory(params['memory'])
@@ -419,8 +427,7 @@ class VMTemplate(object):
%(cpu_info_xml)s
<os>
<type arch='%(arch)s'>hvm</type>
- <boot dev='hd'/>
- <boot dev='cdrom'/>
+ %(boot_order)s
</os>
<features>
<acpi/>
diff --git a/xmlutils/bootorder.py b/xmlutils/bootorder.py
new file mode 100644
index 0000000..269f052
--- /dev/null
+++ b/xmlutils/bootorder.py
@@ -0,0 +1,44 @@
+#
+# Project Kimchi
+#
+# Copyright IBM, Corp. 2016
+#
+# This library is free software; you can redistribute it and/or
+# modify it under the terms of the GNU Lesser General Public
+# License as published by the Free Software Foundation; either
+# version 2.1 of the License, or (at your option) any later version.
+#
+# This library is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+# Lesser General Public License for more details.
+#
+# You should have received a copy of the GNU Lesser General Public
+# License along with this library; if not, write to the Free Software
+# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
+
+import lxml.etree as ET
+from lxml.builder import E
+
+
+def get_bootorder_xml(network=False):
+ """
+ <boot dev='hd'/>
+ <boot dev='cdrom'/>
+
+ - For netboot configuration:
+
+ <boot dev='hd'/>
+ <boot dev='cdrom'/>
+ <boot dev='network'/>
+ """
+ boot_order = ['hd', 'cdrom']
+ if network:
+ boot_order.append('network')
+
+ boot_xml = ''
+ for device in boot_order:
+ boot = E.boot(dev=device)
+ boot_xml += ET.tostring(boot, encoding='utf-8', pretty_print=True)
+
+ return boot_xml
--
2.5.0
3:40 p.m.
New subject: [PATCH] [Kimchi 3/3] Update test cases to support netboot.
From: Paulo Vital <pvital(a)linux.vnet.ibm.com>
Update test cases to support the creation of templates without CD-ROM device
information and creation of guests with cdrom set up as 'netboot'.
This is part of solution to Issue #372.
Signed-off-by: Paulo Vital <pvital(a)linux.vnet.ibm.com>
---
tests/test_template.py | 18 ++++++++++++++----
tests/test_vmtemplate.py | 16 ++++++++++++++++
2 files changed, 30 insertions(+), 4 deletions(-)
diff --git a/tests/test_template.py b/tests/test_template.py
index da0037e..b7a9821 100644
--- a/tests/test_template.py
+++ b/tests/test_template.py
@@ -68,12 +68,22 @@ class TemplateTests(unittest.TestCase):
self.assertEquals(200, resp.status)
self.assertEquals(0, len(json.loads(resp.read())))
- # Create a template without cdrom and disk specified fails with 400
- t = {'name': 'test', 'os_distro': 'ImagineOS',
- 'os_version': '1.0', 'memory': 1024,
'cpu_info': {'vcpus': 1}}
+ # Create a template without cdrom - netboot template
+ t = {'name': 'test-netboot'}
req = json.dumps(t)
resp = self.request('/plugins/kimchi/templates', req, 'POST')
- self.assertEquals(400, resp.status)
+ self.assertEquals(201, resp.status)
+
+ # Verify the netboot template
+ tmpl = json.loads(
+ self.request('/plugins/kimchi/templates/test-netboot').read()
+ )
+ self.assertEquals(tmpl['cdrom'], 'netboot')
+
+ # Delete the netboot template
+ resp = self.request('/plugins/kimchi/templates/test-netboot',
'{}',
+ 'DELETE')
+ self.assertEquals(204, resp.status)
# Create a template
t = {'name': 'test', 'cdrom': '/tmp/mock.iso'}
diff --git a/tests/test_vmtemplate.py b/tests/test_vmtemplate.py
index e157198..6784529 100644
--- a/tests/test_vmtemplate.py
+++ b/tests/test_vmtemplate.py
@@ -130,3 +130,19 @@ class VMTemplateTests(unittest.TestCase):
self.assertEquals(['foo'], t.info.get('networks'))
self.assertEquals(self.iso, t.info.get('cdrom'))
self.assertEquals(graphics, t.info.get('graphics'))
+
+ def test_netboot_vmtemplate(self):
+ disk_bus = get_template_default('old', 'disk_bus')
+ memory = get_template_default('old', 'memory')
+ nic_model = get_template_default('old', 'nic_model')
+ fields = (('name', 'test'), ('cdrom',
'netboot'),
+ ('os_distro', 'unknown'), ('os_version',
'unknown'),
+ ('cpu_info', {'vcpus': 1, 'maxvcpus': 1}),
+ ('memory', memory), ('networks', ['default']),
+ ('disk_bus', disk_bus), ('nic_model', nic_model),
+ ('graphics', {'type': 'vnc', 'listen':
'127.0.0.1'}))
+
+ args = {'name': 'test', 'cdrom': 'netboot'}
+ t = VMTemplate(args)
+ for name, val in fields:
+ self.assertEquals(val, t.info.get(name))
--
2.5.0
4:47 a.m.
Reviewed-by: Daniel Barboza <dhbarboza82(a)gmail.com>
Minor comments to be looked at in the case you end up sending a v2
(not worth sending a v2 because of them alone):
- copyright format is wrong in new file xmlutils/bootorder.py. It must
be 'Copyright IBM Corp, 2016'. This copyright will be fixed by WoK's
copyright script when it is deployed in Kimchi, so don't need to rush.
- I am not fond of comments when the code is clear enough. This is
the case in your code below:
+ # Add information of CD-ROM device only if not setup to netboot.
+ # Also, setup the correct boot order
+ if params['cdrom'] == 'netboot':
+ params['boot_order'] = get_bootorder_xml(network=True)
The thing about this kind of comment is that it doesn't add up anything
(the code is easy to understand) but then we'll need to keep the comment
updated if the code logic changes for some reason.
Daniel
On 02/16/2016 07:40 PM, pvital(a)linux.vnet.ibm.com wrote:
From: Paulo Vital <pvital(a)linux.vnet.ibm.com>
This patchset adds support in backend to create templates and guests to netboot
without setting a cdrom path or URL as image to install. Once created a guest
to boot via network, the guest will request DHCP/TFTP/(NFS/HTTP/FTP) network
installation servers to download the configured images and start the install.
To test, use the curl commands:
curl -k -u test -H "Content-Type: application/json" -H \
"Accept: application/json"
'https://localhost:8001/plugins/kimchi/templates' \
-X POST -d '{"name": "test-netboot"}'
Enter host password for user 'test':
{
"cpu_info":{
"maxvcpus":1,
"vcpus":1
},
"graphics":{
"type":"vnc",
"listen":"127.0.0.1"
},
"cdrom":"netboot",
"networks":[
"default"
],
"icon":"plugins/kimchi/images/icon-vm.png",
"os_distro":"unknown",
"name":"test-netboot",
"disks":[
{
"index":0,
"format":"qcow2",
"pool":{
"type":"dir",
"name":"/plugins/kimchi/storagepools/default"
},
"size":10
}
],
"invalid":{},
"os_version":"unknown",
"memory":1024,
"folder":[]
}
curl -k -u test -H "Content-Type: application/json" -H \
"Accept: application/json" 'https://localhost:8001/plugins/kimchi/vms'
-X POST \
-d
'{"name":"1netboot-test","template":"/plugins/kimchi/templates/test-netboot"}'
Enter host password for user 'test':
{
"status":"running",
"message":"OK",
"id":"1",
"target_uri":"/plugins/kimchi/vms/1netboot-test"
}
Paulo Vital (3):
Add support to create templates without ISO image.
Add support to create guests without ISO image.
Update test cases to support netboot.
model/templates.py | 35 ++++++++++++++++++++++-------------
tests/test_template.py | 18 ++++++++++++++----
tests/test_vmtemplate.py | 16 ++++++++++++++++
vmtemplate.py | 29 ++++++++++++++++++-----------
xmlutils/bootorder.py | 44 ++++++++++++++++++++++++++++++++++++++++++++
5 files changed, 114 insertions(+), 28 deletions(-)
create mode 100644 xmlutils/bootorder.py
--
2.5.0
_______________________________________________
Kimchi-devel mailing list
Kimchi-devel(a)ovirt.org
http://lists.ovirt.org/mailman/listinfo/kimchi-devel
5:07 a.m.
On 02/17/2016 08:47 AM, Daniel Henrique Barboza wrote:
Reviewed-by: Daniel Barboza <dhbarboza82(a)gmail.com>
Minor comments to be looked at in the case you end up sending a v2
(not worth sending a v2 because of them alone):
- copyright format is wrong in new file xmlutils/bootorder.py. It must
be 'Copyright IBM Corp, 2016'. This copyright will be fixed by WoK's
copyright script when it is deployed in Kimchi, so don't need to rush.
I did a copy&paste of the header from a different file from xmlutils.
Since the new script will fix all files, I don't think it's necessary a
V2 for this.
- I am not fond of comments when the code is clear enough. This is
the case in your code below:
+ # Add information of CD-ROM device only if not setup to netboot.
+ # Also, setup the correct boot order
+ if params['cdrom'] == 'netboot':
+ params['boot_order'] = get_bootorder_xml(network=True)
The thing about this kind of comment is that it doesn't add up anything
(the code is easy to understand) but then we'll need to keep the comment
updated if the code logic changes for some reason.
I agree with you that comments for simple code is not necessary.
However, the comment is pertinent for all if-else block that is setting
the information related to cd-rom - the else part is more robust than
the if in this case.
In addition, all other information process in the method has comments to
inform what kind of information is being processed. Just added one more
to keep the understand of the method easier to anyone.
Daniel
On 02/16/2016 07:40 PM, pvital(a)linux.vnet.ibm.com wrote:
> From: Paulo Vital <pvital(a)linux.vnet.ibm.com>
>
> This patchset adds support in backend to create templates and guests
> to netboot
> without setting a cdrom path or URL as image to install. Once created
> a guest
> to boot via network, the guest will request DHCP/TFTP/(NFS/HTTP/FTP)
> network
> installation servers to download the configured images and start the
> install.
>
> To test, use the curl commands:
>
> curl -k -u test -H "Content-Type: application/json" -H \
> "Accept: application/json"
> 'https://localhost:8001/plugins/kimchi/templates' \
> -X POST -d '{"name": "test-netboot"}'
> Enter host password for user 'test':
> {
> "cpu_info":{
> "maxvcpus":1,
> "vcpus":1
> },
> "graphics":{
> "type":"vnc",
> "listen":"127.0.0.1"
> },
> "cdrom":"netboot",
> "networks":[
> "default"
> ],
> "icon":"plugins/kimchi/images/icon-vm.png",
> "os_distro":"unknown",
> "name":"test-netboot",
> "disks":[
> {
> "index":0,
> "format":"qcow2",
> "pool":{
> "type":"dir",
> "name":"/plugins/kimchi/storagepools/default"
> },
> "size":10
> }
> ],
> "invalid":{},
> "os_version":"unknown",
> "memory":1024,
> "folder":[]
> }
>
> curl -k -u test -H "Content-Type: application/json" -H \
> "Accept: application/json"
'https://localhost:8001/plugins/kimchi/vms'
> -X POST \
> -d
>
'{"name":"1netboot-test","template":"/plugins/kimchi/templates/test-netboot"}'
>
> Enter host password for user 'test':
> {
> "status":"running",
> "message":"OK",
> "id":"1",
> "target_uri":"/plugins/kimchi/vms/1netboot-test"
> }
>
>
> Paulo Vital (3):
> Add support to create templates without ISO image.
> Add support to create guests without ISO image.
> Update test cases to support netboot.
>
> model/templates.py | 35 ++++++++++++++++++++++-------------
> tests/test_template.py | 18 ++++++++++++++----
> tests/test_vmtemplate.py | 16 ++++++++++++++++
> vmtemplate.py | 29 ++++++++++++++++++-----------
> xmlutils/bootorder.py | 44
> ++++++++++++++++++++++++++++++++++++++++++++
> 5 files changed, 114 insertions(+), 28 deletions(-)
> create mode 100644 xmlutils/bootorder.py
>
> --
> 2.5.0
>
> _______________________________________________
> Kimchi-devel mailing list
> Kimchi-devel(a)ovirt.org
> http://lists.ovirt.org/mailman/listinfo/kimchi-devel
_______________________________________________
Kimchi-devel mailing list
Kimchi-devel(a)ovirt.org
http://lists.ovirt.org/mailman/listinfo/kimchi-devel
11:18 a.m.
Hi Paulo,
Ramon is working on a patch to change the way the Templates are created.
It will replace 'cdrom' and 'disks[base]' parameters to
'source_media'
I have talked to him to do something like:
source_media: {type: disk/netboot, path: <source-media-path>}
To create a Template based on disk (cdrom or base img) the type=disk and
a path must be specified.
To create a netboot Template, only the type is required.
That way we don't need to mix things in the 'cdrom' parameter as you did
with this patch.
The API will be as I described above (Ramon let us know if I am missing
something) so if you want, you can change this patch accordingly or work
on top of Ramon's patch.
Regards,
Aline Manera
On 02/16/2016 07:40 PM, pvital(a)linux.vnet.ibm.com wrote:
From: Paulo Vital <pvital(a)linux.vnet.ibm.com>
This patchset adds support in backend to create templates and guests to netboot
without setting a cdrom path or URL as image to install. Once created a guest
to boot via network, the guest will request DHCP/TFTP/(NFS/HTTP/FTP) network
installation servers to download the configured images and start the install.
To test, use the curl commands:
curl -k -u test -H "Content-Type: application/json" -H \
"Accept: application/json"
'https://localhost:8001/plugins/kimchi/templates' \
-X POST -d '{"name": "test-netboot"}'
Enter host password for user 'test':
{
"cpu_info":{
"maxvcpus":1,
"vcpus":1
},
"graphics":{
"type":"vnc",
"listen":"127.0.0.1"
},
"cdrom":"netboot",
"networks":[
"default"
],
"icon":"plugins/kimchi/images/icon-vm.png",
"os_distro":"unknown",
"name":"test-netboot",
"disks":[
{
"index":0,
"format":"qcow2",
"pool":{
"type":"dir",
"name":"/plugins/kimchi/storagepools/default"
},
"size":10
}
],
"invalid":{},
"os_version":"unknown",
"memory":1024,
"folder":[]
}
curl -k -u test -H "Content-Type: application/json" -H \
"Accept: application/json" 'https://localhost:8001/plugins/kimchi/vms'
-X POST \
-d
'{"name":"1netboot-test","template":"/plugins/kimchi/templates/test-netboot"}'
Enter host password for user 'test':
{
"status":"running",
"message":"OK",
"id":"1",
"target_uri":"/plugins/kimchi/vms/1netboot-test"
}
Paulo Vital (3):
Add support to create templates without ISO image.
Add support to create guests without ISO image.
Update test cases to support netboot.
model/templates.py | 35 ++++++++++++++++++++++-------------
tests/test_template.py | 18 ++++++++++++++----
tests/test_vmtemplate.py | 16 ++++++++++++++++
vmtemplate.py | 29 ++++++++++++++++++-----------
xmlutils/bootorder.py | 44 ++++++++++++++++++++++++++++++++++++++++++++
5 files changed, 114 insertions(+), 28 deletions(-)
create mode 100644 xmlutils/bootorder.py
--
2.5.0
noon
Ok.
I'll wait Rammon's patches and then resubmit the the patches
Paulo.
On 02/23/2016 02:18 PM, Aline Manera wrote:
Hi Paulo,
Ramon is working on a patch to change the way the Templates are created.
It will replace 'cdrom' and 'disks[base]' parameters to
'source_media'
I have talked to him to do something like:
source_media: {type: disk/netboot, path: <source-media-path>}
To create a Template based on disk (cdrom or base img) the type=disk and
a path must be specified.
To create a netboot Template, only the type is required.
That way we don't need to mix things in the 'cdrom' parameter as you did
with this patch.
The API will be as I described above (Ramon let us know if I am missing
something) so if you want, you can change this patch accordingly or work
on top of Ramon's patch.
Regards,
Aline Manera
On 02/16/2016 07:40 PM, pvital(a)linux.vnet.ibm.com wrote:
> From: Paulo Vital <pvital(a)linux.vnet.ibm.com>
>
> This patchset adds support in backend to create templates and guests
> to netboot
> without setting a cdrom path or URL as image to install. Once created
> a guest
> to boot via network, the guest will request DHCP/TFTP/(NFS/HTTP/FTP)
> network
> installation servers to download the configured images and start the
> install.
>
> To test, use the curl commands:
>
> curl -k -u test -H "Content-Type: application/json" -H \
> "Accept: application/json"
> 'https://localhost:8001/plugins/kimchi/templates' \
> -X POST -d '{"name": "test-netboot"}'
> Enter host password for user 'test':
> {
> "cpu_info":{
> "maxvcpus":1,
> "vcpus":1
> },
> "graphics":{
> "type":"vnc",
> "listen":"127.0.0.1"
> },
> "cdrom":"netboot",
> "networks":[
> "default"
> ],
> "icon":"plugins/kimchi/images/icon-vm.png",
> "os_distro":"unknown",
> "name":"test-netboot",
> "disks":[
> {
> "index":0,
> "format":"qcow2",
> "pool":{
> "type":"dir",
> "name":"/plugins/kimchi/storagepools/default"
> },
> "size":10
> }
> ],
> "invalid":{},
> "os_version":"unknown",
> "memory":1024,
> "folder":[]
> }
>
> curl -k -u test -H "Content-Type: application/json" -H \
> "Accept: application/json"
'https://localhost:8001/plugins/kimchi/vms'
> -X POST \
> -d
>
'{"name":"1netboot-test","template":"/plugins/kimchi/templates/test-netboot"}'
>
> Enter host password for user 'test':
> {
> "status":"running",
> "message":"OK",
> "id":"1",
> "target_uri":"/plugins/kimchi/vms/1netboot-test"
> }
>
>
> Paulo Vital (3):
> Add support to create templates without ISO image.
> Add support to create guests without ISO image.
> Update test cases to support netboot.
>
> model/templates.py | 35 ++++++++++++++++++++++-------------
> tests/test_template.py | 18 ++++++++++++++----
> tests/test_vmtemplate.py | 16 ++++++++++++++++
> vmtemplate.py | 29 ++++++++++++++++++-----------
> xmlutils/bootorder.py | 44
> ++++++++++++++++++++++++++++++++++++++++++++
> 5 files changed, 114 insertions(+), 28 deletions(-)
> create mode 100644 xmlutils/bootorder.py
>
> --
> 2.5.0
2978
days inactive
2985
days old
7 comments
4 participants
participants (4)
-
Aline Manera -
Daniel Henrique Barboza -
Paulo Ricardo Paz Vital -
pvital＠linux.vnet.ibm.com