Skip to contentSkip to conversation

nfs4.1: check whether old session is discard for EID5f

PATCH In review Kinglong Mee 2 participants 2 messages Dec 24, 2013 – Jan 6, 2014

Subsystems (2)
NFS client NFS server
lore
Share & tools
lore

Kinglong Mee

author
#0 lore raw

PATCH 1/1 nfs4.1: check whether old session is discard for EID5f

Signed-off-by Kinglong Mee <[email protected]>
---
 nfs4.1/server41tests/st_exchange_id.py | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)
nfs4.1/server41tests/st_exchange_id.py +6 -1
index c31af95..d2c0c97 100644
@@ -244,7 +244,12 @@ def testNoUpdate101(t, env):
     # Old session state should not be discarded until confirm:
     res = sess1.compound([])
     check(res)
-    # FIXME - more checks here
+
+    # Old session state should be discarded after confirm:
+    sess2 = c2.create_session()
+    check(res)
+    res = sess1.compound([])
+    check(res, NFS4ERR_BADSESSION)
 
 def testNoUpdate101b(t, env):
     """
Signature · 1 line
1.8.4.2
#1 lore raw
Quoted 2 lines · Kinglong Mee
Signed-off-by Kinglong Mee <[email protected]>

Applied and pushed out with the previous pynfs patch; let me know if you see anything missing in my pynfs repo.

--b.

Quoted 24 lines
---
 nfs4.1/server41tests/st_exchange_id.py | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)
nfs4.1/server41tests/st_exchange_id.py +6 -1
index c31af95..d2c0c97 100644
@@ -244,7 +244,12 @@ def testNoUpdate101(t, env):
     # Old session state should not be discarded until confirm:
     res = sess1.compound([])
     check(res)
-    # FIXME - more checks here
+
+    # Old session state should be discarded after confirm:
+    sess2 = c2.create_session()
+    check(res)
+    res = sess1.compound([])
+    check(res, NFS4ERR_BADSESSION)
 
 def testNoUpdate101b(t, env):
     """
Signature · 1 line
1.8.4.2