[node-patches] Change in ovirt-node[master]: installer: Fix loading installed version

fabiand at fedoraproject.org fabiand at fedoraproject.org
Wed Mar 6 11:24:52 UTC 2013


Fabian Deutsch has uploaded a new change for review.

Change subject: installer: Fix loading installed version
......................................................................

installer: Fix loading installed version

Previously the installer tried to load version informations about the
installed system even in the case that no system was installed.

Change-Id: I2d53615807703e04e8e88d5b9f95d8f5d4f83ba8
Signed-off-by: Fabian Deutsch <fabiand at fedoraproject.org>
---
M scripts/tui/src/ovirt/node/installer/welcome_page.py
1 file changed, 3 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-node refs/changes/55/12755/1

diff --git a/scripts/tui/src/ovirt/node/installer/welcome_page.py b/scripts/tui/src/ovirt/node/installer/welcome_page.py
index a400891..61ad53f 100644
--- a/scripts/tui/src/ovirt/node/installer/welcome_page.py
+++ b/scripts/tui/src/ovirt/node/installer/welcome_page.py
@@ -83,9 +83,6 @@
         if self.application.args.dry:
             return ui.Button("button.install", "Install Hypervisor (dry)")
 
-        media = utils.system.InstallationMedia()
-        installed = utils.system.InstalledMedia()
-
         has_hostvg = utils.system.has_hostvg()
         has_root = os.path.exists("/dev/disk/by-label/ROOT")
 
@@ -94,6 +91,9 @@
                             "uninstall existing version first")
 
         if has_hostvg:
+            media = utils.system.InstallationMedia()
+            installed = utils.system.InstalledMedia()
+
             try:
                 if media > installed:
                     return ui.Button("button.upgrade",


--
To view, visit http://gerrit.ovirt.org/12755
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I2d53615807703e04e8e88d5b9f95d8f5d4f83ba8
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Fabian Deutsch <fabiand at fedoraproject.org>



More information about the node-patches mailing list