Dan Kenigsberg has posted comments on this change.
Change subject: Modify getOSVersion to use python 'platform' module
......................................................................
Patch Set 1: (1 inline comment)
....................................................
File vdsm_reg/deployUtil.py.in
Line 449: try:
Line 450: osVer = platform.linux_distribution()
Line 451: logging.debug("OS Release Ver = %s", osVer[1])
Line 452: return osVer[1]
Line 453: except:
our current behaviour (returning "Unknown OS") is stupid, but please maintain
it in this patch.
Line 454: logging.error('failed to get OS release')
Line 455: return "Unknown OS"
Line 456:
Line 457: def getKernelVR():
--
To view, visit http://gerrit.ovirt.org/7930
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia58284ce75dc6d09f0302181ea35ed3817dd861d
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Pradipta Banerjee <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Pradipta Banerjee <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
