Change in ovirt-engine[master]: packaging: Updated getEngineVersion functionality

juan.hernandez at redhat.com juan.hernandez at redhat.com
Tue Feb 19 18:10:09 UTC 2013


Juan Hernandez has submitted this change and it was merged.

Change subject: packaging: Updated getEngineVersion functionality
......................................................................


packaging: Updated getEngineVersion functionality

If we use the getEngineVersion code on upgrades from
versions where the version file does not exist, we will
experience an error with opening the version file.

This patch tries to solve it by checking the presence of
the version file. If the file doesn't exist it will use
the getRpmVersion function to fetch the real version of
the engine package from the rpm.

Change-Id: I57c62cf64d0c44f824a8ec5071f2ef50ec97854e
Signed-off-by: Alex Lourie <alourie at redhat.com>
---
M packaging/fedora/setup/common_utils.py
1 file changed, 6 insertions(+), 2 deletions(-)

Approvals:
  Juan Hernandez: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I57c62cf64d0c44f824a8ec5071f2ef50ec97854e
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alex Lourie <alourie at redhat.com>
Gerrit-Reviewer: Alex Lourie <alourie at redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Moran Goldboim <mgoldboi at redhat.com>
Gerrit-Reviewer: Ofer Schreiber <oschreib at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>



More information about the Engine-commits mailing list