Yaniv Bronhaim has uploaded a new change for review.

Change subject: ppc: Over ppc arch vdsm reports UNKNOW as  operationSystem in 
getVdsCaps
......................................................................

ppc: Over ppc arch vdsm reports UNKNOW as  operationSystem in getVdsCaps

Vdsm currently check only /etc/redhat-release path which does not exist
over ppc arch, we should read /etc/ibm_powerkvm-release to retrieve the
release version

Change-Id: I5e6ef83353b4b47a22d91d222ef1a0122b9470d3
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1131965
Signed-off-by: Yaniv Bronhaim <[email protected]>
---
M vdsm/caps.py
1 file changed, 6 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/44/32444/1

diff --git a/vdsm/caps.py b/vdsm/caps.py
index 22b3b01..b337485 100644
--- a/vdsm/caps.py
+++ b/vdsm/caps.py
@@ -514,8 +514,13 @@
             version = linecache.getline('/etc/debian_version', 1).strip("\n")
             release = ""  # Debian just has a version entry
         else:
+            if platform.machine() == Architecture.PPC64:
+                release_path = '/etc/ibm_powerkvm-release'
+            else:
+                release_path = '/etc/redhat-release'
+
             ts = rpm.TransactionSet()
-            for er in ts.dbMatch('basenames', '/etc/redhat-release'):
+            for er in ts.dbMatch('basenames', release_path):
                 version = er['version']
                 release = er['release']
     except:


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I5e6ef83353b4b47a22d91d222ef1a0122b9470d3
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to