search for: getdefaultsection

On Tue, Jul 18, 2017 at 03:46:38PM +0200, Pino Toscano wrote: > On Tuesday, 18 July 2017 12:01:12 CEST Richard W.M. Jones wrote: > > In SUSE guests, handle the case where > > Bootloader::Tools::GetDefaultSection () returns undef. > > > > Previously this would return an empty string and cause a bogus error > > in subsequent code: > > > > virt-v2v: error: libguestfs error: statns: statns_stub: path must start > > with a / character > > --- > > I'm torn a...

In SUSE guests, handle the case where Bootloader::Tools::GetDefaultSection () returns undef. Previously this would return an empty string and cause a bogus error in subsequent code: virt-v2v: error: libguestfs error: statns: statns_stub: path must start with a / character --- v2v/linux_bootloaders.ml | 17 ++++++++++++++--- 1 file changed, 14 insertions(+), 3 deletions...

On Tuesday, 18 July 2017 19:59:20 CEST Richard W.M. Jones wrote: > In SUSE guests, handle the case where > Bootloader::Tools::GetDefaultSection () returns undef. > > Previously this would return an empty string and cause a bogus error > in subsequent code: > > virt-v2v: error: libguestfs error: statns: statns_stub: path must start > with a / character > --- I saw this was pushed, even though v1 was NACKed by me, and...

On Tuesday, 18 July 2017 15:57:47 CEST Richard W.M. Jones wrote: > On Tue, Jul 18, 2017 at 03:46:38PM +0200, Pino Toscano wrote: > > On Tuesday, 18 July 2017 12:01:12 CEST Richard W.M. Jones wrote: > > > In SUSE guests, handle the case where > > > Bootloader::Tools::GetDefaultSection () returns undef. > > > > > > Previously this would return an empty string and cause a bogus error > > > in subsequent code: > > > > > > virt-v2v: error: libguestfs error: statns: statns_stub: path must start > > > with a / character > >...

...f g#exists "/sbin/grubby" then - [| "grubby"; "--default-kernel" |] - else - [| "/usr/bin/perl"; "-MBootloader::Tools"; "-e"; " - InitLibrary(); - my $default = Bootloader::Tools::GetDefaultSection(); - print $default->{image}; - " |] in - match g#command cmd with - | "" -> None - | k -> + let res = + match get_default_method with + | MethodGrubby -> + Some (g#command [| "grubby"; "...

Hi, this series adds a couple bits more in v2v to convert Debian/Ubuntu (and derived) guests. The series does not complete the support (see known issues below), but all the patches here should be fit for review and inclusion. The series does not enable the conversion, yet. Known issues: * currently tested with simple local guest images, hence needs testing with real guests on

Hi, this series adds a couple bits more in v2v to convert Debian/Ubuntu (and derived) guests. The series does not complete the support (see known issues below), but all the patches here should be fit for review and inclusion. The series does not enable the conversion, yet. Known issues: * currently tested with simple local guest images, hence needs testing with real guests on

On Tuesday, 18 July 2017 12:01:12 CEST Richard W.M. Jones wrote: > In SUSE guests, handle the case where > Bootloader::Tools::GetDefaultSection () returns undef. > > Previously this would return an empty string and cause a bogus error > in subsequent code: > > virt-v2v: error: libguestfs error: statns: statns_stub: path must start > with a / character > --- I'm torn about this one: the test case for this comes f...

On Fri, Jul 28, 2017 at 10:02:34AM +0200, Pino Toscano wrote: > On Tuesday, 18 July 2017 19:59:20 CEST Richard W.M. Jones wrote: > > In SUSE guests, handle the case where > > Bootloader::Tools::GetDefaultSection () returns undef. > > > > Previously this would return an empty string and cause a bogus error > > in subsequent code: > > > > virt-v2v: error: libguestfs error: statns: statns_stub: path must start > > with a / character > > --- > > I saw this was...

In SUSE guests, handle the case where Bootloader::Tools::GetDefaultSection () returns undef. Previously this would return an empty string and cause a bogus error in subsequent code: virt-v2v: error: libguestfs error: statns: statns_stub: path must start with a / character --- v2v/linux_bootloaders.ml | 17 ++++++++++++++--- 1 file changed, 14 insertions(+), 3 deletions...

...f g#exists "/sbin/grubby" then + [| "grubby"; "--default-kernel" |] + else + [| "/usr/bin/perl"; "-MBootloader::Tools"; "-e"; " + InitLibrary(); + my $default = Bootloader::Tools::GetDefaultSection(); + print $default->{image}; + " |] in + match g#command cmd with + | "" -> None + | k -> + let len = String.length k in + let k = + if len > 0 && k.[len-1] = '\n' then + String.s...

...f g#exists "/sbin/grubby" then + [| "grubby"; "--default-kernel" |] + else + [| "/usr/bin/perl"; "-MBootloader::Tools"; "-e"; " + InitLibrary(); + my $default = Bootloader::Tools::GetDefaultSection(); + print $default->{image}; + " |] in + match g#command cmd with + | "" -> None + | k -> + let len = String.length k in + let k = + if len > 0 && k.[len-1] = '\n' then + String.s...

...ubby" then > + [| "grubby"; "--default-kernel" |] > + else > + [| "/usr/bin/perl"; "-MBootloader::Tools"; "-e"; " > + InitLibrary(); > + my $default = Bootloader::Tools::GetDefaultSection(); > + print $default->{image}; > + " |] in > + match g#command cmd with > + | "" -> None > + | k -> > + let len = String.length k in > + let k = > + if len > 0 && k.[len-1] =...

...in/grubby" then - [| "grubby"; "--default-kernel" |] - else - [| "/usr/bin/perl"; "-MBootloader::Tools"; "-e"; " - InitLibrary(); - my $default = Bootloader::Tools::GetDefaultSection(); - print $default->{image}; - " |] in - match g#command cmd with - | "" -> None - | k -> - let len = String.length k in - let k = - if len > 0 && k.[len-1] = '\n...

v1 was: https://www.redhat.com/archives/libguestfs/2017-July/msg00154.html There is no change in the first patch, but I added a second patch adding slow tests of opensuse guests (which pass, but require the first patch). Rich.

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

...and(['grubby', '--default-kernel']); + } else { + $default = eval { $g->command(['/usr/bin/perl', + '-MBootloader::Tools', + '-e', 'InitLibrary(); '. + 'my $default=Bootloader::Tools::GetDefaultSection(); '. + 'print $default->{image};']) }; + # If the image starts with (hdX,X), remove it. + $default =~ s/^\(hd.*\)//; + } + + chomp($default); + return $default; +} + +sub set_default_image +{ + my $self = shift; + my ($path) = @_; + +...

...and(['grubby', '--default-kernel']); + } else { + $default = eval { $g->command(['/usr/bin/perl', + '-MBootloader::Tools', + '-e', 'InitLibrary(); '. + 'my $default=Bootloader::Tools::GetDefaultSection(); '. + 'print $default->{image};']) }; + # If the image starts with (hdX,X), remove it. + $default =~ s/^\(hd.*\)//; + } + + chomp($default); + return $default; +} + +sub set_default_image +{ + my $self = shift; + my ($path) = @_; + +...

...) = @_; > + > + my $g = $self->{g}; > + > + my $default = $g->command(['/usr/bin/perl', > + '-MBootloader::Tools', > + '-e', 'InitLibrary(); '. > + 'my $default=Bootloader::Tools::GetDefaultSection(); '. > + 'print $default->{image};']); > + # If the image starts with (hd*,*), remove it. > + $default =~ s/^\(hd.*\)//; > + > + return $default; > +} How confident can you be that perl and Bootloader::Tools are available on the guest? As...

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