7a4cac2ed8
0c223ab2-guest-Dont-set-default-uefi-if-firmware-is-set.patch 414ffa5e-virt-install-Use-minutes-instead-of-seconds-on-get_time_string.patch 53245827-urlfetcher-Force-a-flush-after-writing-to-a-file.patch 3009888a-urlfetcher-Dont-override-fullurl-when-its-explicitly-set.patch OBS-URL: https://build.opensuse.org/package/show/Virtualization/virt-manager?expand=0&rev=482
30 lines
1.1 KiB
Diff
30 lines
1.1 KiB
Diff
Subject: virt-install: Use minutes instead of seconds on get_time_string()
|
|
From: Fabiano Fidêncio fidencio@redhat.com Thu Jul 25 14:09:53 2019 +0200
|
|
Date: Thu Jul 25 19:13:13 2019 -0400:
|
|
Git: 414ffa5ef82548d502a4de98d40228a68bdb29c1
|
|
|
|
get_time_string() currently uses self._wait_secs, while it should use
|
|
self._wait_mins, resulting on confusing messages when using `--wait`
|
|
option, as shown below:
|
|
|
|
fidencio@laerte ~/src/upstream/virt-manager $ ./virt-install --install fedora30 --unattended --wait 20
|
|
...
|
|
Waiting 1200 minutes for installation to complete.
|
|
|
|
Reviewed-by: Cole Robinson <crobinso@redhat.com>
|
|
Signed-off-by: Fabiano Fidêncio <fidencio@redhat.com>
|
|
|
|
diff --git a/virt-install b/virt-install
|
|
index af6c75bf..8efa4960 100755
|
|
--- a/virt-install
|
|
+++ b/virt-install
|
|
@@ -610,7 +610,7 @@ class WaitHandler:
|
|
self._start_time = time.time()
|
|
|
|
def get_time_string(self):
|
|
- timestr = _(" %d minutes") % self._wait_secs
|
|
+ timestr = _(" %d minutes") % self._wait_mins
|
|
if self._wait_forever:
|
|
timestr = ""
|
|
ret = _("Waiting%(time_string)s for installation to complete.") % {
|