Sandro Bonazzola has posted comments on this change.
Change subject: packaging: setup: support OVF import
......................................................................
Patch Set 5:
(2 comments)
....................................................
File src/plugins/ovirt-hosted-engine-setup/vm/boot_disk.py
Line 252: )
Line 253: for filename in tar.getnames():
Line 254: self.logger.debug(filename)
Line 255: if (
Line 256: filename.startswith('master') and
Well' Ive not a deep knowledge of OVF format, but at least in image-uploader it
is required that the ovf is in a subdirectory called master.
|-- images
| |-- <Image Group UUID>
| |--- <Image UUID (this is the disk image)>
| |--- <Image UUID (this is the disk image)>.meta
|-- master
| |---vms
| |--- <UUID>
| |--- <UUID>.ovf
Line 257: os.path.splitext(filename)[1] == '.ovf'
Line 258: ):
Line 259: ovf_xml = filename
Line 260: break
....................................................
File src/plugins/ovirt-hosted-engine-setup/vm/configurevm.py
Line 101: 'from (@VALUES@) [@DEFAULT@]: '
Line 102: ),
Line 103: prompt=True,
Line 104: caseSensitive=True,
Line 105: validValues=self.BOOT_DEVICE.keys(),
should be already fixed by that patch fixing keys calls.
Line 106: default=ohostedcons.Defaults.DEFAULT_BOOT,
Line 107: )
Line 108:
Line 109: if self.environment[
--
To view, visit http://gerrit.ovirt.org/17521
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I04b1de72c19e558b20d46e9fe6a435c594b0005d
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-hosted-engine-setup
Gerrit-Branch: master
Gerrit-Owner: Sandro Bonazzola <[email protected]>
Gerrit-Reviewer: Alex Lourie <[email protected]>
Gerrit-Reviewer: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Greg Padgett <[email protected]>
Gerrit-Reviewer: Lev Veyde <[email protected]>
Gerrit-Reviewer: Moran Goldboim <[email protected]>
Gerrit-Reviewer: Ofer Schreiber <[email protected]>
Gerrit-Reviewer: Oved Ourfali <[email protected]>
Gerrit-Reviewer: Sandro Bonazzola <[email protected]>
Gerrit-Reviewer: Yedidyah Bar David <[email protected]>
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
