RHEL 8: v2v: rhv-upload: Remove restriction on -oa sparse.

See: https://bugzilla.redhat.com/show_bug.cgi?id=1565681
and the v2v-devel private thread "Do we already support migration using FC?"
This commit is contained in:
Richard W.M. Jones
2018-05-14 10:16:58 +01:00
committed by Pino Toscano
parent 845411f894
commit d4d45f4a8a
3 changed files with 2 additions and 19 deletions

View File

@@ -177,19 +177,11 @@ See also \"OUTPUT TO RHV\" in the virt-v2v(1) manual.")
error (f_"nbdkit was compiled without SELinux support. You will have to recompile nbdkit with libselinux-devel installed, or else set SELinux to Permissive mode while doing the conversion.")
in
(* Output format/sparse must be raw/sparse. We may be able to
* lift this limitation in future, but it requires changes on the
* RHV side. See TODO file for details. XXX
*)
(* Output format must be raw. *)
let error_current_limitation required_param =
error (f_"rhv-upload: currently you must use %s. This restriction will be loosened in a future version.") required_param
in
let error_unless_output_alloc_sparse () =
if output_alloc <> Sparse then
error_current_limitation "-oa sparse"
in
(* JSON parameters which are invariant between disks. *)
let json_params = [
"verbose", JSON.Bool (verbose ());
@@ -242,7 +234,6 @@ object
error_unless_ovirtsdk4_module_available ();
error_unless_nbdkit_working ();
error_unless_nbdkit_python3_working ();
error_unless_output_alloc_sparse ();
if have_selinux then
error_unless_nbdkit_compiled_with_selinux ()

View File

@@ -139,10 +139,8 @@ def open(readonly):
format = disk_format,
initial_size = params['disk_size'],
provisioned_size = params['disk_size'],
# XXX Ignores params['output_sparse'].
# Handling this properly will be complex, see:
# https://www.redhat.com/archives/libguestfs/2018-March/msg00177.html
sparse = True,
sparse = params['output_sparse'],
storage_domains = [
types.StorageDomain(
name = params['output_storage'],

View File

@@ -1822,12 +1822,6 @@ username is not specified then virt-v2v defaults to using
C<admin@internal> which is the typical superuser account for oVirt
instances.
=item I<-of raw>
Currently you must use I<-of raw> and you cannot use I<-oa preallocated>.
These restrictions will be loosened in a future version.
=item I<-op> F<password-file>
A file containing a password to be used when connecting to the oVirt