Discussion:
[Libstoragemgmt-devel] [PATCH] _client.py: fix regression of LSM_UDS_PATH environment support
Gris Ge
2014-03-22 15:35:22 UTC
Permalink
* Fix regression of running user defined IPC folder via LSM_UDS_PATH
environment variable.

Signed-off-by: Gris Ge <***@redhat.com>
---
lsm/lsm/_client.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lsm/lsm/_client.py b/lsm/lsm/_client.py
index 5562219..197a7e9 100644
--- a/lsm/lsm/_client.py
+++ b/lsm/lsm/_client.py
@@ -131,7 +131,7 @@ class Client(INetworkAttachedStorage):
(plug, proto) = scheme.split("+")
scheme = plug

- self.plugin_path = os.path.join(_UDS_PATH, scheme)
+ self.plugin_path = os.path.join(self._uds_path, scheme)

if os.path.exists(self.plugin_path):
self._tp = _TransPort(_TransPort.get_socket(self.plugin_path))
--
1.8.3.1
Gris Ge
2014-03-25 02:50:33 UTC
Permalink
Post by Gris Ge
* Fix regression of running user defined IPC folder via LSM_UDS_PATH
environment variable.
Tony,

Can you review this patch and commit if OK?

It's a regression which fail 'make check' at step 1.

Thanks.
Post by Gris Ge
---
lsm/lsm/_client.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lsm/lsm/_client.py b/lsm/lsm/_client.py
index 5562219..197a7e9 100644
--- a/lsm/lsm/_client.py
+++ b/lsm/lsm/_client.py
(plug, proto) = scheme.split("+")
scheme = plug
- self.plugin_path = os.path.join(_UDS_PATH, scheme)
+ self.plugin_path = os.path.join(self._uds_path, scheme)
self._tp = _TransPort(_TransPort.get_socket(self.plugin_path))
--
1.8.3.1
--
Gris Ge
Tony Asleson
2014-03-25 04:46:00 UTC
Permalink
Committed!

Thanks,
Tony
Post by Gris Ge
Post by Gris Ge
* Fix regression of running user defined IPC folder via LSM_UDS_PATH
environment variable.
Tony,
Can you review this patch and commit if OK?
It's a regression which fail 'make check' at step 1.
Thanks.
Post by Gris Ge
---
lsm/lsm/_client.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lsm/lsm/_client.py b/lsm/lsm/_client.py
index 5562219..197a7e9 100644
--- a/lsm/lsm/_client.py
+++ b/lsm/lsm/_client.py
(plug, proto) = scheme.split("+")
scheme = plug
- self.plugin_path = os.path.join(_UDS_PATH, scheme)
+ self.plugin_path = os.path.join(self._uds_path, scheme)
self._tp = _TransPort(_TransPort.get_socket(self.plugin_path))
--
1.8.3.1
Loading...