diff options
author | Andy Grover <agrover@redhat.com> | 2013-05-30 17:03:24 -0700 |
---|---|---|
committer | Andy Grover <agrover@redhat.com> | 2013-05-30 17:03:24 -0700 |
commit | 1dc5db26660c598d20edeed3e133ae2114147e18 (patch) | |
tree | def20f78fc19b03d3458db699361edf0d23e3ea6 | |
parent | 5d74e9ca758af16a85cd77747e7287d7b28b59e4 (diff) | |
download | rtslib-fb-1dc5db26660c598d20edeed3e133ae2114147e18.tar.gz |
Fix is_configured()
It would return always-true since */info was always present. Change it
to check the value of */enable (only available in 3.10+) or
keep returning True.
Signed-off-by: Andy Grover <agrover@redhat.com>
-rw-r--r-- | rtslib/tcm.py | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/rtslib/tcm.py b/rtslib/tcm.py index c39d9bd..4152a98 100644 --- a/rtslib/tcm.py +++ b/rtslib/tcm.py @@ -245,14 +245,14 @@ class StorageObject(CFSNode): ''' self._check_self() - path = "%s/info" % self.path + path = "%s/enable" % self.path try: - fread(path) + configured = fread(path) except IOError: - return False - else: return True + return bool(int(configured)) + version = property(_get_version, doc="Get the version of the StorageObject's backstore") name = property(_get_name, |