Saggi Mizrahi has uploaded a new change for review.

Change subject: Revert "storage: Default to NFSv3 if version not specified"
......................................................................

Revert "storage: Default to NFSv3 if version not specified"

This reverts commit caff6146e7f9d7139b4c4ef41992958909731e47.

Change-Id: I404df524419c663accf0890eb860c5a3afb81583
Signed-off-by: Saggi Mizrahi <[email protected]>
---
M vdsm/storage/storageServer.py
1 file changed, 4 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/73/8573/1

diff --git a/vdsm/storage/storageServer.py b/vdsm/storage/storageServer.py
index 0dba0f1..269b8eb 100644
--- a/vdsm/storage/storageServer.py
+++ b/vdsm/storage/storageServer.py
@@ -229,7 +229,7 @@
 
     @property
     def version(self):
-        if self._version != 'auto':
+        if self._version is not None:
             return self._version
 
         # Version was not specified but if we are connected we can figure out
@@ -249,15 +249,16 @@
         return -1
 
 
-    def __init__(self, export, timeout=600, retrans=6, version=3):
+    def __init__(self, export, timeout=600, retrans=6, version=None):
         self._remotePath = normpath(export)
+        self._version = version
         options = self.DEFAULT_OPTIONS[:]
         self._timeout = timeout
         self._version = version
         self._retrans = retrans
         _addIntegerOption(options, "timeo", timeout)
         _addIntegerOption(options, "retrans", retrans)
-        if version != 'auto':
+        if version:
             _addIntegerOption(options, "vers", version)
 
         self._mountCon = MountConnection(export, "nfs", ",".join(options))


--
To view, visit http://gerrit.ovirt.org/8573
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I404df524419c663accf0890eb860c5a3afb81583
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Saggi Mizrahi <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to