Daniel P. Berrange wrote:
On Sun, Dec 13, 2015 at 06:41:21AM +0300, Roman Bogorodskiy wrote:
> Make bhyveload respect boot order as specified by os.boot section of the
> domain XML or by "boot order" for specific devices. As bhyve does not
> support a real boot order specification right now, it's just about
> choosing a single device to boot from.
So if bhyve only lets you specify a single device to boot from,
then we should report an error to the user if they provide more
than one <boot> element in the XML.
The reason that I did it this way was because my original intention was
to make it work with virt-manager smoother. However, I think you're
right, it's better to error on configuration we cannot properly support
instead of letting users have false expectation.
It will make creation of new VM via virt-manager fail with the default
configuration, but it's not critical because user still have to modify
some hardware settings anyway to make it work with bhyve (i.e.: drop vnc
and pty console).
Thanks for the review, I'll upload v2 shortly.
Roman Bogorodskiy