Add LXC version implementation

This patch adds an implementation of the version function to the LXC driver.
The providers use the hypervisor version in a field of one of the instances,
so we need to have something meaningful here.  AFAICT, the only real option
we have (considering the limitations of the libvirt version information) is
to use the kernel version.

Fix qemu media connect/eject to work with latest syntax. Add support for ejecting floppy devices and scsi cdroms.

Update domain xml after usb device hotplug.

Scrap media eject output to try and determine success/fail.

Fix disk ordering and add target duplication checking when parsing domain xml.

avoid a segfault on CD eject in KVM/QEmu
* src/qemu_driver.c: patch from Cole Robinson to avoid a segfault
  on KVM CD eject
Daniel

add output on attach and detach success
* src/virsh.c: patch from Cole Robinson to add output on attach
  and detach success
daniel

* AUTHORS: add Cole Robinson as commiter
daniel

* src/storage_conf.c: add one missing check in virStoragePoolDefParseDoc
Daniel

* src/domain_conf.c: fix a parsing error for input devices
  patch by Cole Robinson
daniel

check availbility of emulators before reporting them in capabilities
* src/qemu_conf.c: patch from Cole Robinson to report in the
  capabilities only the ones where the emulators is actually found
* po/libvirt.pot: regenerated
* src/libvirt.c: small typo fix
Daniel

Adds storage source element for pools
* src/storage_backend.h src/storage_backend_logical.c
src/storage_conf.c src/storage_conf.h src/virsh.c:
Applied patches from David Lively to add storage source
elements needed for storage pool
* docs/formatstorage.html docs/formatstorage.html.in: associated
documentation
Daniel

qemu_conf.c: use new function, virFileReadLimFD

* src/qemu_conf.c (qemudExtractVersionInfo): Use virFileReadLimFD
and VIR_FREE in place of an open-coded loop and a static buffer.

util.c: add a file-descriptor-based wrapper for fread_file_lim

* src/util.c (virFileReadLimFP): New function.
(__virFileReadLimFD): New function.
* src/util.h (__virFileReadLimFD): Declare.
(virFileReadLimFD): Define.
(virFileReadAll): Rewrite to use virFileReadLimFP.

Switch over fork/exec code to use virExec

Allow for private root filessytem in containers using pivot_root

Fix lookup of xen VM after define (Cole Robinson)

improve parallel build support

* autobuild.sh: Append -j$N to MAKEFLAGS, if no -j option is there.
* libvirt.spec.in: Use %{?_smp_mflags} with "make".
Based on a patch from James Morris:
http://thread.gmane.org/gmane.comp.emulators.libvirt/8201

Add storage pool source discovery support (patch from David Lively)

Fix QEMU domain creation by allowing virExec to preserve certain FDs

Fix QEMU domain creation by allowing virExec to preserve certain FDs

Fix EOF handling when reading openvz domain list

Remove use of virState apis from openvz driver

[LXC] Cleanup devices on container shutdown

Use libvirt error message for python exceptions

configure.in: link with -lpthread when necessary

* configure.in: Explicitly add -lpthread to $LIBS.
With an empty "ACTION_IF_FOUND", AC_CHECK_LIB would have done
that for us automatically, but when there is an explicit third
argument, those commands are run instead of the default.
Reported by Jun Koi <junkoi2004@gmail.com>.

autobuild.sh: Fix minor shell-quoting bugs.

* autobuild.sh: Remove unnecessary quotes.
Don't choke on a file name argument containing a space.
Don't misbehave for $AUTOBUILD_INSTALL_ROOT containing
a shell meta-character.

Forgotten ChangeLog entry

Improve error reporting from remote driver

Propagate virConnectOpen errors to global error object

Remove unused lx_controller.h file

Add libvirt_lxc to file list

Ignore libvirt_lxc binary

Create lxc_controller standalone binary

Do conditional driver config in Makefile.am, not source files

Remove configFile/autostartLink vars from virDomainObj struct

add cpu management functionality to OpenVZ driver
* src/openvz_conf.c src/openvz_driver.c: patch from Evgeniy Sokolov
  to limit the number of CPUs used by OpenVZ domains.
Daniel

Fix creating and cleaning up logical volumes if a target path is specified
* src/storage_backend_logical.c: Patch from Cole Robinson, fix
  creating and cleaning up logical volumes if a target path is
  specified.
Daniel

Fix size reporting for disks without partitions.
* src/parthelper.c: Patch from Cole Robinson, fix size reporting for
  disks without partitions.
Daniel

Handle HTML generation in the absence of XHTML1 DTDs
* configure.in libvirt.spec.in docs/Makefile.am: fix the problem
  of HTML generation in the absence of XHTML1 DTDs in the system
  catalog.
Daniel

* src/network_conf.c src/network_conf.h src/qemu_driver.c: allow to
  add static host definition for dnsmasq
Daniel

Allow non-blocking/daemon mode to be specified via virExec flags

Allow a pre-opened FD to be be pass to virExec for stdout/err

Avoid signal race in virExec

Improve virExec error reporting

Remove duplicated call to virInitialize

Fri Aug 15 11:00:38 IST 2008 Mark McLoughlin <markmc@redhat.com>

        * src/qemu_conf.c: fix serial/telnet to use "server" option
        rather than "listen"

        * tests/qemuxml2argvdata/qemuxml2argv-serial-tcp-telnet.args:
        fix the test case too

fix typos src/virsh.c from John Levon,
  and 3 more files(src/domain_conf.c src/openvz_driver.c src/qemu_driver.c)

Update LXC driver in POTFILES.in

Switch LXC driver over to generic domain XML processing APIs

Make LXC I/O controller process a parent of the container process

Re-arrange code between LXC driver files

Refactor LXC driver to pass tty/socket state directly

* docs/storage.html[.in] src/storage_backend_disk.c: revert previous
  msdos patch and apply new one from Cole Robinson
daniel

rewrite virFileLinkPointsTo

* src/util.c (SAME_INODE): Define.
(virFileLinkPointsTo): Rewrite to be more portable and more efficient.

make sure parted doesn't ask for confirmation
* src/storage_backend_disk.c: make sure parted doesn't ask
  confirmation,  patch by Cole Robinson
Daniel

rename the disk type from 'dos' to 'msdos'
* docs/storage.html[.in] src/storage_backend_disk.c: patch from
  Cole Robinson to rename the disk type from 'dos' to 'msdos'
Daniel

add _fd to stdin, stderr stdout in struct virDomainObj for compilation on MinGW

fix compilation for MinGW

Fix connection lookup in python storage instances
* python/generator.py: patch from Cole Robinson trying to fix
  problem of connection lookup when creating storage instances
Daniel

* po/*: regeneration of pot file, updates of .po
Daniel

documentation for USB passthrough
* docs/formatdomain.html docs/formatdomain.html.in: documentation
  for the USB format option by Guido Günther
daniel

tests: append a newline to expected-output files lacking NL-at-EOF

* tests/qemuxml2argvtest.c (testCompareXMLToArgvFiles): Adjust the
code that creates "actual" output, so that it too produces a
newline-terminated buffer.
* tests/qemuxml2argvdata/*.args: Append a newline to each, via:
for i in $(find|grep '\.args$'); do echo >> $i;done

Fix const-correctness of virRUn and virExec

llow to add a disk as an USB device (QEmu/KVM)
* src/domain_conf.c src/domain_conf.h src/qemu_conf.c
  src/qemu_driver.c: patch from Guido Günther allowing to add
  disk as USB devices
* docs/libvirt.rng: extend the schemas for the new value
* tests/qemuxml2argvdata/qemuxml2argv-disk-usb.args
  tests/qemuxml2argvdata/qemuxml2argv-disk-usb.xml
  tests/qemuxml2argvtest.c tests/qemuxml2xmltest.c: add a specific
  test to the regression suite
Daniel

Extend parser and add support for USB devices in QEmu/KVM
* src/domain_conf.c src/domain_conf.h src/qemu_conf.c
  src/qemu_driver.c: Patch from Guido Günther allowing to pass
  usb devices to qemu/kvm
* docs/libvirt.rng: add the new functionality to the grammar
* tests/qemuxml2argvdata/qemuxml2argv-hostdev-usb-address.args
  tests/qemuxml2argvdata/qemuxml2argv-hostdev-usb-address.xml
  tests/qemuxml2argvdata/qemuxml2argv-hostdev-usb-product.args
  tests/qemuxml2argvdata/qemuxml2argv-hostdev-usb-product.xml
  tests/qemuxml2argvtest.c tests/qemuxml2xmltest.c: adding examples
  to the regression tests
* libvirt.spec.in: fix the licence tag
Daniel

fix typos in docs
docs/formatdomain.html docs/formatdomain.html.in docs/java.html docs/java.html.in

* tests/domainschematest: patch from Guido Günther to fix RNG checking
  in out of tree builds.
daniel

Fix device name clash in src/veth.c

add the mkstemp module from gnulib

* bootstrap (modules): Add mkstemp.
The remainder of these changes are the result of running
./bootstrap, adding new files, and committing the result.
* gnulib/lib/gettimeofday.c: New file.
* gnulib/lib/mkstemp.c: New file.
* gnulib/lib/tempname.c: New file.
* gnulib/lib/tempname.h: New file.
* gnulib/m4/gettimeofday.m4: New file.
* gnulib/m4/mkstemp.m4: New file.
* gnulib/m4/tempname.m4: New file.
* gnulib/tests/test-gettimeofday.c: New file.
* gnulib/lib/Makefile.am: Update.
* gnulib/m4/gnulib-cache.m4: Likewise.
* gnulib/m4/gnulib-comp.m4: Likewise.
* gnulib/m4/inet_pton.m4: Likewise.
* gnulib/tests/Makefile.am: Likewise.

* src/bridge.c: fix a build failure on RHEL4
Daniel

update from gnulib

* build-aux/mktempd (rand_bytes, mktempd):
* build-aux/useless-if-before-free (FILE):
* build-aux/vc-list-files:
* gnulib/lib/.cvsignore:
* gnulib/lib/Makefile.am:
* gnulib/lib/poll.c (poll):
* gnulib/lib/stdbool.in.h (_GL_STDBOOL_H, true):
* gnulib/lib/stdio-impl.h (fp_, fp_ub):
* gnulib/lib/stdio.in.h (vasprintf, obstack_printf)
(obstack_vprintf):
* gnulib/lib/vasnprintf.c (_GNU_SOURCE, sprintf, IF_LINT, exp)
(remainder, scale10_round_decimal_long_double)
(scale10_round_decimal_double, pad_ourselves):
* gnulib/m4/gnulib-cache.m4:
* gnulib/m4/gnulib-common.m4 (gl_MODULE_INDICATOR):
* gnulib/m4/gnulib-tool.m4 (gl_LOCAL_DIR, gl_M4_BASE, gl_PO_BASE)
(gl_DOC_BASE, gl_TESTS_BASE, gl_WITH_TESTS, gl_LIB, gl_LGPL)
(gl_MAKEFILE_NAME, gl_MACRO_PREFIX, gl_PO_DOMAIN, gl_VC_FILES):
* gnulib/m4/lock.m4 (gl_LOCK_EARLY_BODY, gl_PREREQ_LOCK)
(gl_DISABLE_THREADS):
* gnulib/m4/onceonly.m4:
* gnulib/m4/posix-shell.m4 (gl_POSIX_SHELL):
* gnulib/m4/stdio_h.m4 (gl_STDIO_H_DEFAULTS):
* gnulib/tests/test-getaddrinfo.c (ENABLE_DEBUGGING, dbgprintf)
(simple):
* gnulib/tests/test-stdint.c (UINTMAX_MAX, or):
* gnulib/tests/test-vc-list-files-cvs.sh (compare):
* gnulib/tests/test-vc-list-files-git.sh (compare):

complete the MinGW inet_pton portability fix

* gnulib/m4/inet_pton.m4: New file.  Required for MinGW.
* gnulib/m4/gnulib-comp.m4: Regenerate via gnulib-tool.

fix failing "make syntax-check"

* src/stats_linux.c: Don't include <c-ctype.h> no longer used.

work around MinGW build failure due to its lack of inet_aton

Use inet_pton instead; pull in gnulib's module by the same name.
* src/network_conf.c (virNetworkDefParseXML): Use inet_pton,
rather than inet_aton.
* bootstrap (modules): Add inet_pton.
* gnulib/lib/inet_pton.c: New file, from gnulib.

* docs/formatdomain.html docs/formatdomain.html.in: fix memory unit
  as pointed out by matthew chan it's kilobytes
Daniel

DanB pointed out that my sexpr xend_internal patch from yesterday broke one of
the regression tests.  The problem is that the xenDaemonFormatSxpr{Disk,Net}
functions are shared between domain creation time and attaching disk time.
Unfortunately, though, Xend expects something different during these two times.
 During domain creation time, it wants the "(device" in front of the sexpr,
while during attach time it does not.  To remedy this situation, I added a flag
to these two functions to differentiate between these two modes.  With this
patch in place, all of the regression tests pass.

Signed-off-by: Chris Lalancette <clalance@redhat.com>

With the recent refactoring of the domain code, plus the changes with the Xend
code, a couple of bugs were introduced into the attach-disk and attach-interface
functionality.  This patch fixes 3 bugs:

1)  In xenDaemonAttachDevice(), there is a switch statement to determine which
of the xenDaemonFormatSxpr{Disk,Net} functions to call.  Unfortunately, the case
statements are all missing the corresponding "break", so we always fall-through
to the default error case.  This patch just adds the appropriate break statements.

2)  (minor) In xenDaemonDomainDefineXML (that's a mouthful!), there is a stray
"fprintf".  This is now converted to a proper virXendError().

3)  xenDaemonFormatSxpr{Disk,Net} were adding an extra (device to the front of
the sexpr expressions that xend did not expect (this is Xend on RHEL 5.2).
Because of this, the attaches would fail.  The patch fixes this by removing the
(device from the front, which makes attach-disk and attach-interface work again.

Signed-off-by: Chris Lalancette <clalance@redhat.com>

Recently upstream Xen added support for having xvd devices > 16.  For the most
part, this doesn't really concern libvirt, since for things like attach and
detach we just pass it through and let xend worry about whether it is supported
or not.  The one place this breaks down is in the stats collecting code, where
we need to figure out the device number so we can go digging in /sys for the
statistics.

To remedy this, I've re-written xenLinuxDomainDeviceID() to use regular
expressions to figure out the device number from the name.  The major advantage
is that now xenLinuxDomainDeviceID() looks fairly identical to
tools/python/xen/util/blkif.py (in the Xen sources), so that adding additional
devices in the future should be much easier.  It also reduces the size of the
code, and, in my opinion, the code complexity.

With this patch in place, I was able to get block statistics both on older style
devices (/dev/xvda) and on the new, expanded devices (/dev/xvdaa).

Signed-off-by: Chris Lalancette <clalance@redhat.com>

saving UUID of new OpenVZ domains
* src/openvz_conf.c src/openvz_conf.h src/openvz_driver.c: patch
  from Evgeniy Sokolov saving the UUID when creating or defining
  a domain.
Daniel

make distclean: remove generated source files

* src/Makefile.am (DISTCLEANFILES): Define to $(BUILT_SOURCES)
so "make distcheck" passes.

fix syntax-check error (replace TAB to space)

src/xend_internal.c (xenDaemonParseSxpr): Add const to "root" parameter.

This piece was mistakenly omitted from the previous patch.

xend_internal.c, sexpr.c: const-correctness fixes

Incorrect attribution.

'virsh edit' and related commands
* src/virsh.c: Implement 'virsh edit', 'virsh net-edit' and
  'virsh pool-edit' commands.  These edit the XML for domains,
  networks and storage pools respectively, and are the
  equivalent of doing 'virsh dumpxml; vi foo.xml; virsh define'
* src/Makefile.am, src/.cvsignore: Auto-generate the net-edit
  and pool-edit commands.
* docs/virsh.pod: Updated the documentation.

Fix initial VM state

virsh.c: more const-correctness fixes

Support for container related domain XML extensions

Always set forward-delay on bridge, even if 0

Const-correctness fixes in virsh.c
* src/virsh.c: Const-correctness.

Link to "how to contribute to open source"
* HACKING: Link to external document which describes how to
  contribute to open source projects.

* src/qemu_driver.c: patch from Chris Lalancette adding missing
  carriage returns to logs
Daniel

Added test to validate all domain XML files against schema

Improve error messages when XML is not well-formed.
* src/domain_conf.c, src/network_conf.c, src/storage_conf.c:
  Improve error messages from commands such as 'virsh define'
  when the XML is not well-formed by passing libxml2 errors
  back out through virterror.

* docs/formatdomain.html docs/formatdomain.html.in docs/libvirt-api.xml
  docs/libvirt-refs.xml docs/html/libvirt-libvirt.html src/libvirt.c:
  fixed somedocs and API entry point descriptions as suggested by
  Charles Duffy and regenerated the API descriptions.
Daniel

Adding Charles Duffy as author, Daniel

* src/qemu_driver.c: apply patch from Charles Duffy fixing erro
  messages when trying to crate a domain already defined.
Daniel

Fix autoport when domain is inactive

From Charles Duffy:

Per subject; if autoport is in use for a host, the current
virDomainGraphicsDefFormat code always emits "port=-1", even if a port
is assigned to the host; this leaves no way for a client to find the VNC
port assigned to the host in question.

This patches fixes it by putting the vncport in place if there is one, and
only putting "-1" if we fail that test.