Alon Bar-Lev has posted comments on this change.
Change subject: vdsm bootstrap changes for kvm on ppc64 support
......................................................................
Patch Set 6: (6 inline comments)
....................................................
File vds_bootstrap/vds_bootstrap.py
Line 300: self.res = "Intel"
Line 301: elif "AuthenticAMD" == deployUtil.cpuVendorID():
Line 302: self.res = "AMD"
Line 303: elif "IBM_POWER" == deployUtil.cpuVendorID():
Line 304: self.res = "IBM"
else:
exception, error?
Line 305:
Line 306: if self.vt_svm is None:
Line 307: self.vt_svm = "NA"
Line 308:
Line 396:
Line 397: def _avoidPKGConflict(self):
Line 398: arch = deployUtil.getPlatformArch()
Line 399: for pack in CONFL_PACK:
Line 400: pack_arch = pack + "." + arch
what about leaving keep using the pack variable only changing the:
for pack_prefix in CONFL_PACK:
pack = pack_prefix + "." + arch
Line 401: self.res, self.message =
deployUtil.getPackageInfo("CONFL", pack_arch, 'status')
Line 402: res = self.res #Reverse display status
Line 403: if res == "WARN":
Line 404: res = "OK"
Line 408: logging.debug('>>> Conflicting package %s installed',
pack_arch)
Line 409:
Line 410: def _delPKG(self):
Line 411: arch = deployUtil.getPlatformArch()
Line 412: for pack in DEL_PACK:
same here
Line 413: pack_arch = pack + "." + arch
Line 414: self.res, self.message = deployUtil.getPackageInfo("DEL",
pack_arch, 'status')
Line 415: res = self.res #Reverse display status
Line 416: if res == "WARN":
Line 424: def _getAllPackages(self):
Line 425: logging.debug('Check required packages ...')
Line 426: arch = deployUtil.getPlatformArch()
Line 427: for pack in REQ_PACK:
Line 428: pack_arch = pack + "." + arch
same here.
Line 429: self.res, self.message = deployUtil.getPackageInfo("REQ",
pack_arch, 'status')
Line 430: self._xmlOutput('REQ PACKAGES', self.res, "result",
pack_arch, self.message)
Line 431: if self.res == "WARN":
Line 432: self.req_pack.append(pack_arch)
....................................................
File vdsm_reg/deployUtil.py.in
Line 321:
Line 322: if ret == 0 and "Not" not in out: #Avoid error string- 'Not
Settable' or 'Not Present'
Line 323: strReturn = out.replace ("\n", "")
Line 324: elif getPlatformArch() == "ppc64":
Line 325: if os.path.exists('/proc/device-tree/system-id'):
if not?
Line 326: strReturn =
file('/proc/device-tree/system-id').readline().replace(",","")
Line 327: else:
Line 328: logging.error("getMachineUUID: Could not find machine's
UUID.")
Line 329:
Line 322: if ret == 0 and "Not" not in out: #Avoid error string- 'Not
Settable' or 'Not Present'
Line 323: strReturn = out.replace ("\n", "")
Line 324: elif getPlatformArch() == "ppc64":
Line 325: if os.path.exists('/proc/device-tree/system-id'):
Line 326: strReturn =
file('/proc/device-tree/system-id').readline().replace(",","")
Can you please paste a sample?
Line 327: else:
Line 328: logging.error("getMachineUUID: Could not find machine's
UUID.")
Line 329:
Line 330: return strReturn
--
To view, visit http://gerrit.ovirt.org/7072
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I06ca5dc3adba6183657afed778f999715ee332e0
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Pradipta Banerjee <[email protected]>
Gerrit-Reviewer: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Douglas Schilling Landgraf <[email protected]>
Gerrit-Reviewer: Itamar Heim <[email protected]>
Gerrit-Reviewer: Pradipta Banerjee <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
