search for: _configure_kernel

...V2V/HVTarget/Linux.pm | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/lib/Sys/VirtV2V/HVTarget/Linux.pm b/lib/Sys/VirtV2V/HVTarget/Linux.pm index 53784ea..dc546de 100644 --- a/lib/Sys/VirtV2V/HVTarget/Linux.pm +++ b/lib/Sys/VirtV2V/HVTarget/Linux.pm @@ -349,7 +349,7 @@ sub _configure_kernel } # If we didn't install a new kernel, pick the default kernel - $boot_kernel = $guestos->get_default_kernel() if(!defined($boot_kernel)); + $boot_kernel ||= $guestos->get_default_kernel(); return $boot_kernel; } -- 1.6.5.2

...ng patch adds augeas_error to the Sys::VirtConvert::Util use statement for GrubLegacy (matching what is used in the Grub2 package). This is required to prevent an undefined subroutine error for the augeas_error call in list_kernels(). There are also a few minor typo fixups: - A message string in _configure_kernel was missing the $ before a variable - In _install_capability, the expression that filters out 'xen' should actually filter out '-xen' - Two typos in comments were irritating me ;) --- lib/Sys/VirtConvert/Converter/RedHat.pm | 10 +++++----- 1 file changed, 5 insertions(+), 5...

...defined($dom); - die("remap_block_devices called without desc argument") - unless defined($desc); - die("remap_block_devices called without virtio argument") - unless defined($virtio); my %map = (); @@ -167,12 +159,6 @@ sub _remap_block_devices sub _configure_kernel_modules { my ($guestos, $desc, $virtio) = @_; - die("configure_kernel_modules called without guestos argument") - unless defined($guestos); - die("configure_kernel_modules called without desc argument") - unless defined($desc); - die("configure_...

...-------------------- 1 files changed, 9 insertions(+), 57 deletions(-) diff --git a/lib/Sys/VirtV2V/Converter/RedHat.pm b/lib/Sys/VirtV2V/Converter/RedHat.pm index d8f9141..61e81dc 100644 --- a/lib/Sys/VirtV2V/Converter/RedHat.pm +++ b/lib/Sys/VirtV2V/Converter/RedHat.pm @@ -538,21 +538,11 @@ sub _configure_kernel { my ($virtio, $g, $config, $desc, $dom) = @_; - my @remove_kernels = (); - - # Remove foreign hypervisor specific kernels from the list of available - # kernels - foreach my $kernel (_find_hv_kernels($desc)) { - # Don't actually try to remove them yet in case we remo...

...all grub2 in the BIOS boot partition. This overwrites the previous > + # Install grub2 in the BIOS beot partition. This overwrites the previous Typo > # contents of the EFI boot partition. > $g->command(['grub2-install', $device]); > @@ -1002,8 +1101,9 @@ sub _configure_kernel > last; > } > > - # There should be an installed virtio capable kernel if virtio was installed > - die("virtio configured, but no virtio kernel found") > + # Warn if there is no installed virtio capable kernel. It is safe to > + # continue...

...virtio capability - my $virtio = _install_capability('virtio', $g, $root, $config, $meta, $grub); + my $virtio = + _install_capability('virtio', $g, $root, $config, $meta, $grub); # Get an appropriate kernel, or install one if none is available my $kernel = _configure_kernel($virtio, $g, $root, $config, $meta, $grub); # Install user custom packages - if (! _install_capability('user-custom', $g, $root, $config, $meta, $grub)) { + if (! _install_capability('user-custom', + $g, $root, $config, $meta, $grub)) {...

...gure_display_driver($g, $root, $config, $meta, $grub); - _remap_block_devices($meta, $virtio, $g, $root); + my $driver = _get_display_driver($g, $root); + _configure_display_driver($g, $root, $config, $meta, $grub, $driver); + _remap_block_devices($meta, $virtio, $g, $root, $grub); _configure_kernel_modules($g, $virtio); _configure_boot($kernel, $virtio, $g, $root, $grub); @@ -842,7 +937,7 @@ sub _configure_console sub _configure_display_driver { - my ($g, $root, $config, $meta, $grub) = @_; + my ($g, $root, $config, $meta, $grub, $driver) = @_; # Update the display dri...

The changelog action needs to checkout git2cl as a submodule. For some reason, when called through the Git module, command('submodule', 'update') was doing something unfathomable, and different to just calling 'git submodule update'. As Git is just a command line wrapper anyway, I've sidestepped this by just calling the command directly. --- Build.PL | 18

...> > found.')> > > if ($virtio && !defined($boot_kernel)); > > No. This indicates an error in the program rather than something the > user can fix. This is also why it's a plain die with no translation. If a virtio kernel must be installed prior to _configure_kernel, why is there the ability to install a replacement kernel immediately after this message? It doesn't really matter to SUSE though - adding 'kernel' to the deps for virtio in virt-v2v.db will trigger the installation of the correct kernel in the install_capabilities step. > >...

On Friday, October 04, 2013 09:38:58 AM Matthew Booth wrote: > It's specifically an error if we're attempting to configure virtio, and > there's no detected virtio kernel. It shouldn't have been possible to > get here in that state, hence it's a programmer error. The code below > attempts to install *any* kernel in the case that we aren't configuring >

...t; > > if ($virtio && !defined($boot_kernel)); > > > > No. This indicates an error in the program rather than something the > > user can fix. This is also why it's a plain die with no translation. > > If a virtio kernel must be installed prior to _configure_kernel, why is there > the ability to install a replacement kernel immediately after this message? It > doesn't really matter to SUSE though - adding 'kernel' to the deps for virtio > in virt-v2v.db will trigger the installation of the correct kernel in the > install_capabiliti...

The following series of patches adds support for converting SUSE guests through virt-v2v. These changes should not impact non-SUSE guest conversions. Mike Latimer (4): Add perl-Bootloader support to grub packages Add conversion support for SUSE guests Add SUSE to capabilities db and conf Add SUSE support documentation lib/Sys/VirtConvert/Converter/Linux.pm | 480

If a volume is not an exact multiple of 512 bytes, qemu-img will report its size rounded down to a 512 byte boundary. However, when copying, the file is still read until EOF, which will return more data than was expected. This change prevents that causing a failure in itself. The situation is still not resolved, however, as there are still situations where this will cause a failure. For example,

This patch series includes a repost of Milan's unmodified RHN support patch because I haven't pushed it yet. On top of that patch, it includes the capabilities patch in as many bits as I could make it into. The big one is 7/8. I've tested all of the following guests both with and without RHN registration: Xen RHEL 54 PV Xen RHEL 51 PV Xen RHEL 48 PV ESX RHEL 54 FV

...9;t require a direct + # replacement + _unconfigure_hv($g, $root); + + # Try to install the virtio capability + my $virtio = _install_capability('virtio', $g, $root, $config, $meta, $grub); + + # Get an appropriate kernel, or install one if none is available + my $kernel = _configure_kernel($virtio, $g, $root, $config, $meta, $grub); + + # Install user custom packages + if (! _install_capability('user-custom', $g, $root, $config, $meta, $grub)) { + logmsg WARN, __('Failed to install user-custom packages'); + } + + # Configure the rest of the system +...

...); + _unconfigure_hv($g, $root, $desc); # Try to install the virtio capability my $virtio = _install_capability('virtio', $g, $config, $meta, $desc); @@ -125,7 +130,7 @@ sub convert _configure_display_driver($g); _remap_block_devices($meta, $virtio, $g, $desc); _configure_kernel_modules($g, $desc, $virtio, $modpath); - _configure_boot($kernel, $virtio, $g, $desc); + _configure_boot($kernel, $virtio, $g, $root, $desc); my %guestcaps; @@ -850,18 +855,18 @@ sub _configure_kernel sub _configure_boot { - my ($kernel, $virtio, $g, $desc) = @_; + my ($ker...

These patches combine HVSource and HVTarget into a single Converter. This should make it more obvious where to hack without losing any practical flexibility. GuestOS remains separate. GuestOS is now a misnomer, because it's really only a Linux distro abstraction. It will be useless for Windows, for example. Functions which you'd expect to be different on a non-RH distro should live in

...ment > + _unconfigure_hv($g, $root); > + > + # Try to install the virtio capability > + my $virtio = _install_capability('virtio', $g, $root, $config, $meta, $grub); > + > + # Get an appropriate kernel, or install one if none is available > + my $kernel = _configure_kernel($virtio, $g, $root, $config, $meta, $grub); > + > + # Install user custom packages > + if (! _install_capability('user-custom', $g, $root, $config, $meta, $grub)) { > + logmsg WARN, __('Failed to install user-custom packages'); > + } > + > + #...

This is a new conversion module to convert SUSE Linux and openSUSE guests. The converter is based on the RedHat module, and should offer the same functionality on both SUSE and RedHat hosts. There are a few additional messages in this module, such as reporting of packages when installing through zypper or the local virt-v2v repo. These messages don't necessarily flow unless verbose switches

In preparation for an upcoming patch which adds support for SUSE guest conversions, it makes sense to have an intermediate steps that changes the RedHat.pm converter into a more generic Linux converter. The SUSE changes will then be limited in scope to only what is required for SUSE support. This series of patches accomplishes the following: - Renames RedHat.pm to Linux.pm - Modifies Linux.pm