January 2010 - Libguestfs - libguestfs List Archives

[PATCH] Add 'filesize' call.
by Richard W.M. Jones 25 Jan '10

25 Jan '10

-- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones libguestfs lets you edit virtual machines. Supports shell scripting, bindings from many languages. http://et.redhat.com/~rjones/libguestfs/ See what it can do: http://et.redhat.com/~rjones/libguestfs/recipes.html

1 0

[PATCH] guestfish: Use xstrtol to parse integers (RHBZ#557655).
by Richard W.M. Jones 22 Jan '10

22 Jan '10

See also: https://bugzilla.redhat.com/show_bug.cgi?id=557655 -- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones virt-p2v converts physical machines to virtual machines. Boot with a live CD or over the network (PXE) and turn machines into Xen guests. http://et.redhat.com/~rjones/virt-p2v

1 0

[PATCH] Don't run new-kernel-pkg or nscd (RHBZ#557262).
by Richard W.M. Jones 21 Jan '10

21 Jan '10

This is related to this message I posted yesterday: http://www.redhat.com/archives/libguestfs/2010-January/msg00016.html fakechroot doesn't sufficiently contain dracut, which is what was very indirectly causing that problem. However we don't need to run /sbin/new-kernel-package (in the kernel %post script). It takes a long time because dracut is so slow, and it builds an initrd that we never use. So the easiest thing here is to disable this (and also nscd which some %post scripts are trying to run). Rich. -- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones libguestfs lets you edit virtual machines. Supports shell scripting, bindings from many languages. http://et.redhat.com/~rjones/libguestfs/ See what it can do: http://et.redhat.com/~rjones/libguestfs/recipes.html

1 0

[PATCH] hivex: Add missing le32toh conversion around field access.
by Richard W.M. Jones 21 Jan '10

21 Jan '10

This is a bit of a oops. Any suggestions for how to test endian-dependent access code, w/o access to a big endian machine? Rich. -- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones New in Fedora 11: Fedora Windows cross-compiler. Compile Windows programs, test, and build Windows installers. Over 70 libraries supprt'd http://fedoraproject.org/wiki/MinGW http://www.annexia.org/fedora_mingw

1 0

[PATCH] hivex: Clarify some more fields.
by Richard W.M. Jones 21 Jan '10

21 Jan '10

-- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones Read my programming blog: http://rwmj.wordpress.com Fedora now supports 80 OCaml packages (the OPEN alternative to F#) http://cocan.org/getting_started_with_ocaml_on_red_hat_and_fedora

1 0

pvcreate fails with "Write locks are prohibited with --ignorelockingfailure. Can't get lock for orphan PVs"
by Richard W.M. Jones 20 Jan '10

20 Jan '10

Attached is a build.log from RHEL 6. All the failures are of the same sort: /sbin/lvm pvcreate /dev/sda1 Write locks are prohibited with --ignorelockingfailure. Can't get lock for orphan PVs Just noting this issue in case anyone else comes across it. Rich. -- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones virt-df lists disk usage of guests without needing to install any software inside the virtual machine. Supports Linux and Windows. http://et.redhat.com/~rjones/virt-df/

1 0

[PATCH] Missing crc kernel module on RHEL 6 / Linux 2.6.32
by Richard W.M. Jones 20 Jan '10

20 Jan '10

This patch is a confirmed fix for: https://bugzilla.redhat.com/show_bug.cgi?id=557195 I felt it was worth including all possible CRC* kernel modules in the appliance, since they are all very small (~3k each). Full details in the bug. Rich. -- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones virt-p2v converts physical machines to virtual machines. Boot with a live CD or over the network (PXE) and turn machines into Xen guests. http://et.redhat.com/~rjones/virt-p2v

1 0

[PATCH 0/7] Prepare for adding write support to hivex (windows registry) library
by Richard W.M. Jones 19 Jan '10

19 Jan '10

This series of patches prepares the way for a later series which will add write support for Windows Registry 'hive' files to our 'hivex' mini library. About hivex: http://libguestfs.org/hivex.3.html About the Windows Registry: Forget it - there are no reliable references. However the Wikipedia page explains some of the top level concepts: https://secure.wikimedia.org/wikipedia/en/wiki/Windows_Registry And this page explains the format, although it is frequently inaccurate (just less so than other documents): http://www.sentinelchicken.com/data/TheWindowsNTRegistryFileFormat.pdf These patches are ready for review right now. I'll post the second series which add write support later this week. Rich. -- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones Read my programming blog: http://rwmj.wordpress.com Fedora now supports 80 OCaml packages (the OPEN alternative to F#) http://cocan.org/getting_started_with_ocaml_on_red_hat_and_fedora

1 7

[PATCH] qemu: Upstream regression of -stdio serial option.
by Richard W.M. Jones 07 Jan '10

07 Jan '10

I've tested this with both upstream qemu from git, and the old non-broken qemu from Fedora 12. Rich. -- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones Read my programming blog: http://rwmj.wordpress.com Fedora now supports 80 OCaml packages (the OPEN alternative to F#) http://cocan.org/getting_started_with_ocaml_on_red_hat_and_fedora

1 1

[PATCH] Converter: Fixes to Xen metadata conversion
by Matthew Booth 06 Jan '10

06 Jan '10

Specifically fixes the issue where <script path='vif-bridge'/> would be corrupted rather than removed properly. Makes metadata conversion less generic. --- lib/Sys/VirtV2V/Converter.pm | 144 +++++++++++++++++++++--------------------- 1 files changed, 73 insertions(+), 71 deletions(-) diff --git a/lib/Sys/VirtV2V/Converter.pm b/lib/Sys/VirtV2V/Converter.pm index a6eba45..64a5a46 100644 --- a/lib/Sys/VirtV2V/Converter.pm +++ b/lib/Sys/VirtV2V/Converter.pm @@ -194,37 +194,7 @@ sub _unconfigure_hvs die("unconfigure_hvs called without default_dom argument") unless defined($default_dom); - # Get a list of source HV specific metadata nodes - my @nodeinfo = _find_hv_metadata($dom); - - for(my $i = 0; $i < $#nodeinfo; $i += 2) { - my $node = $nodeinfo[$i]; - my $xpath = $nodeinfo[$i + 1]->[0]; - my $required = $nodeinfo[$i + 1]->[1]; - - # Look for a replacement in the defaults - my ($default) = $default_dom->findnodes($xpath); - if(defined($default)) { - if($node->isa('XML::DOM::Attr')) { - $node->setNodeValue($default->getNodeValue()); - } else { - my $replacement = $default->cloneNode(1); - $replacement->setOwnerDocument($dom); - - $node->getParentNode()->replaceChild($replacement, $node); - } - } - - else { - # Warn if a replacement is required, but none was found - print STDERR user_message - (__x("WARNING: No replacement found for {xpath} in ". - "domain XML. The node was removed.", - xpath => $xpath)) if($required); - - $node->getParentNode()->removeChild($node); - } - } + _unconfigure_xen_metadata($dom, $default_dom); } sub _configure_os @@ -303,7 +273,8 @@ sub _configure_capabilities if(!$found) { print STDERR user_message (__x("The connected hypervisor does not support a ". - "machine type of {machine}.", + "machine type of {machine}. It will be set to the ". + "current default.", machine => $machine->getValue())); my ($type) = $dom->findnodes('/domain/os/type'); @@ -347,7 +318,9 @@ sub _unconfigure_bootloaders '/domain/os/kernel', '/domain/os/initrd', '/domain/os/root', - '/domain/os/cmdline' + '/domain/os/cmdline', + '/domain/bootloader', + '/domain/bootloader_args' ); foreach my $path (@bootloader_paths) { @@ -396,64 +369,93 @@ sub _configure_drivers } } -sub _find_hv_metadata +sub _replace_with_default_metadata { - my ($dom) = @_; + my ($node, $xpath, $default_dom) = @_; + + # Look for a replacement in the defaults + my ($default) = $default_dom->findnodes($xpath); + if(defined($default)) { + if($node->isa('XML::DOM::Attr')) { + $node->setNodeValue($default->getNodeValue()); + } else { + my $replacement = $default->cloneNode(1); + $replacement->setOwnerDocument($node->getOwnerDocument()); + + $node->getParentNode()->replaceChild($replacement, $node); + } + } + + else { + # Warn if no replacement was found + print STDERR user_message + (__x("WARNING: No replacement found for {xpath} in ". + "domain XML. The node was removed.", + xpath => $xpath)); - return _find_xen_metadata($dom); + $node->getParentNode()->removeChild($node); + } } -sub _find_xen_metadata +sub _unconfigure_xen_metadata { - my $dom = shift; - defined($dom) or carp("find_metadata called without dom argument"); - - # List of nodes requiring changes if they exist and match a particular - # pattern, and whether they need to be replaced for a guest to function - # Most of this is taken from inspection of domain.rng - my @check_nodes = ( - [ '/domain/@type', 'xen', 1 ], - [ '/domain/devices/emulator', 'xen', 0 ], - [ '/domain/devices/input/@bus', 'xen', 1 ], - [ '/domain/devices/interface/script/@path', 'vif-bridge', 0], - [ '/domain/os/loader', 'xen', 0 ], - [ '/domain/os/type/@machine', '(xenpv|xenfv|xenner)', 0 ], - [ '/domain/devices/disk/target/@bus', 'xen', 0 ], - [ '/domain/bootloader', undef, 0], - [ '/domain/bootloader_args', undef, 0] - ); + my ($dom, $default_dom) = @_; + + # The list of target xen-specific nodes is mostly taken from inspection of + # domain.rng - my @nodeinfo = (); - foreach my $check_node (@check_nodes) { - my $xpath = $check_node->[0]; - my $pattern = $check_node->[1]; - my $required = $check_node->[2]; + # Nodes which should be replaced with a default if they are present + foreach my $hv_node ( + [ '/domain/@type', 'xen' ], + [ '/domain/devices/input/@bus', 'xen' ] + ) { + my $xpath = $hv_node->[0]; + my $pattern = $hv_node->[1]; foreach my $node ($dom->findnodes($xpath)) { if(defined($pattern)) { - my $value; - if($node->isa('XML::DOM::Attr')) { - $value = $node->getNodeValue(); - } else { - my ($text) = $node->findnodes('text()'); - $value = $text->getNodeValue(); - } - - next unless($value =~ m{$pattern}); + next unless($node->getNodeValue() =~ m{$pattern}); } - push(@nodeinfo, $node => [ $xpath, $required ]); + _replace_with_default_metadata($node, $xpath, $default_dom); + } + } + + # Remove machine if it has a xen-specific value + # We could replace it with the generic 'pc', but 'pc' is a moving target + # across QEMU releases. By removing it entirely, libvirt will automatically + # add the latest machine type (e.g. pc-0.11), which is stable. + foreach my $machine_type ($dom->findnodes('/domain/os/type/@machine')) { + if ($machine_type->getNodeValue() =~ /(xenpv|xenfv|xenner)/) { + my ($type) = $dom->findnodes('/domain/os/type[@machine = "'. + $machine_type->getNodeValue().'"]'); + $type->getAttributes()->removeNamedItem("machine"); } } - return @nodeinfo; + # Remove emulator if it is defined + foreach my $emulator ($dom->findnodes('/domain/devices/emulator')) { + $emulator->getParent()->removeChild($emulator); + } + + # Remove the script element if its path attribute is 'vif-bridge' + foreach my $script ($dom->findnodes('/domain/devices/interface/script[@path = "vif-bridge"]')) + { + $script->getParent()->removeChild($script); + } + + # Other Xen related metadata is handled separately + # /domain/devices/disk/target/@bus = 'xen' + # /domain/os/loader = 'xen' + # /domain/bootloader + # /domain/bootloader_args } =back =head1 COPYRIGHT -Copyright (C) 2009 Red Hat Inc. +Copyright (C) 2009,2010 Red Hat Inc. =head1 LICENSE -- 1.6.5.2

1 0