Allow multiple alternative directory names for distributions (or
distribution familiy) when installing Linux guest tools packages.
Original naming required that there is a separate directory for every
version of a distribution (e.g. fc28, fc29, ...). This is inconvenient
when users want to keep just a single version of the package for the
distribution.
For each distribution one can have either a common directory (e.g.
fedora) or a versioned directory (fedora28). This can also be combined.
I.e. one can have both `fedora` and `fedora28` in which case `fedora28`
will be used when converting Fedora 28 guest wheres `fedora` will be
used when converting guests with any other Fedora version.
To have better names for unversioned directories the original names
were changed this way:
fc -> fedora
el -> rhel
lp -> suse
The original directory names are kept for backward compatibility and are
aliased to new names as described below. When both new and old name are
present on file system the new name takes precedence.
fc28 -> fedora
el6 -> rhel6
el7 -> rhel7
lp151 -> suse
Signed-off-by: Tomáš Golembiovský
---
v2v/windows_virtio.ml | 65 +--
1 file changed, 38 insertions(+), 27 deletions(-)
diff --git a/v2v/windows_virtio.ml b/v2v/windows_virtio.ml
index 94c4774b7..cc33d9502 100644
--- a/v2v/windows_virtio.ml
+++ b/v2v/windows_virtio.ml
@@ -186,14 +186,18 @@ let rec install_drivers ((g, _) as reg) inspect rcaps =
and install_linux_tools g inspect =
let os =
match inspect.i_distro with
-| "fedora" -> Some "fc28"
+| "fedora" -> Some [
+ (sprintf "fedora%d" inspect.i_major_version); "fedora"; "fc28"]
| "rhel" | "centos" | "scientificlinux" | "redhat-based"
-| "oraclelinux" ->
- (match inspect.i_major_version with
- | 6 -> Some "el6"
- | 7 -> Some "el7"
- | _ -> None)
-| "sles" | "suse-based" | "opensuse" -> Some "lp151"
+| "oraclelinux" -> Some (
+ [(sprintf "rhel%d" inspect.i_major_version)]
+ @ (match inspect.i_major_version with
+ | 6 -> ["el6"]
+ | 7 -> ["el7"]
+ | _ -> [])
+ @ ["rhel"])
+| "sles" | "suse-based" | "opensuse" -> Some [
+ (sprintf "fedora%d" inspect.i_major_version); "suse"; "lp151"]
| _ -> None in
match os with
@@ -201,15 +205,15 @@ and install_linux_tools g inspect =
warning (f_"don't know how to install guest tools on %s-%d")
inspect.i_distro inspect.i_major_version
| Some os ->
- let src_path = "linux" // os in
+ let src_paths = List.map ((//) "linux") os in
let dst_path = "/var/tmp" in
- debug "locating packages in %s" src_path;
+ debug "locating packages in: %s" (String.concat ", " src_paths);
let packages =
-copy_from_virtio_win g inspect src_path dst_path
+copy_from_virtio_win g inspect src_paths dst_path
(fun _ _ -> true)
(fun () ->
- warning (f_"guest tools directory ‘%s’ is
missing from the virtio-win directory or ISO.\n\nGuest tools are only provided
in the RHV Guest Tools ISO, so this can happen if you are using the version of
virtio-win which contains just the virtio drivers. In this case only virtio
drivers can be installed in the guest, and installation of Guest Tools will be
skipped.")
- src_path) in
+ warning (f_"none of the guest tools directories
‘[%s]’ was found on the virtio-win directory or ISO.\n\nGuest tools are only
provided in the oVirt/RHV Guest Tools ISO, so this can happen if you are using
the version of virtio-win which contains just the virtio drivers. In this case
only virtio drivers can be installed in the guest, and installation of Guest
Tools will be skipped.")
+ (String.concat ", " src_paths)) in
debug "done copying %d files" (List.length packages);
let packages = List.map ((//) dst_path) packages in
try
@@ -290,29 +294,34 @@ and ddb_regedits inspect drv_name drv_pciid =
* been copied.
*)
and copy_drivers g inspect driverdir =
- [] <> copy_from_virtio_win g inspect "/" driverdir
+ [] <> copy_from_virtio_win g inspect ["/"] driverdir
virtio_iso_path_matches_guest_os
(fun () ->
error (f_"root directory ‘/’ is missing from the virtio-win directory or
ISO.\n\nThis should not happen and may indicate that virtio-win or virt-v2v is
broken in some way. Please report this as a bug with a full debug log."))
-(* Copy all files from virtio_win directory/ISO located in [srcdir]
- * subdirectory and all its subdirectories to the [destdir]. The directory
- * hierarchy is not preserved, meaning all files will be directly in [destdir].
- * The file list is filtered based on [filter] function.
+(* Find first existing direcotry from [scrdirs] list located in virtio_win
+ *
