diff --git a/lib/backend.py b/lib/backend.py index 5bf45a918972b62a033c71c9cc23144c91421bc3..b19b4012e51b6c4b06a24e462209137702b2c7f5 100644 --- a/lib/backend.py +++ b/lib/backend.py @@ -1490,11 +1490,7 @@ def _OSOndiskAPIVersion(name, os_dir): " a regular file" % os_dir) try: - f = open(api_file) - try: - api_versions = f.readlines() - finally: - f.close() + api_versions = utils.ReadFile(api_file).splitlines() except EnvironmentError, err: return False, ("Error while reading the API version file at %s: %s" % (api_file, _ErrnoOrStr(err))) diff --git a/lib/hypervisor/hv_base.py b/lib/hypervisor/hv_base.py index 237088747139124fa88e694df05a3b0dceb76d0b..cdb7702f26b47f13ae681e73704c0400232bebcb 100644 --- a/lib/hypervisor/hv_base.py +++ b/lib/hypervisor/hv_base.py @@ -315,11 +315,7 @@ class BaseHypervisor(object): """ try: - fh = file("/proc/meminfo") - try: - data = fh.readlines() - finally: - fh.close() + data = utils.ReadFile("/proc/meminfo").splitlines() except EnvironmentError, err: raise errors.HypervisorError("Failed to list node info: %s" % (err,)) diff --git a/tools/lvmstrap b/tools/lvmstrap index 8af2f61e139694dc4e5110fd13b3b576625c96e7..0a17a5f7c27b39b7bfe95107c1f12f60fa1dfca1 100755 --- a/tools/lvmstrap +++ b/tools/lvmstrap @@ -44,7 +44,7 @@ import sys import optparse import time -from ganeti.utils import RunCmd +from ganeti.utils import RunCmd, ReadFile from ganeti import constants USAGE = ("\tlvmstrap diskinfo\n" @@ -424,9 +424,7 @@ def GetMountInfo(): a mountpoint: device number dictionary """ - f = open("/proc/mounts", "r") - mountlines = f.readlines() - f.close() + mountlines = ReadFile("/proc/mounts").splitlines() mounts = {} for line in mountlines: device, mountpoint, fstype, rest = line.split(None, 3)