All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] CLNT: Turn on cache_this for specific tests
@ 2010-09-26 16:59 linux-nfs-owner
  0 siblings, 0 replies; only message in thread
From: linux-nfs-owner @ 2010-09-26 16:59 UTC (permalink / raw)
  To: Fred Isaman; +Cc: linux-nfs

From: J. Bruce Fields <bfields@redhat.com>

A previous patch turned this off by default; but we need it back on for
anything that tests the reply cache.

Signed-off-by: J. Bruce Fields <bfields@redhat.com>
---
 nfs4.1/server41tests/st_sequence.py |   10 +++++-----
 1 files changed, 5 insertions(+), 5 deletions(-)

Also available from:

	git://linux-nfs.org/~bfields/pynfs41.git

I've also done a little more triage of the results and turned on another
36 of the pynfs41 tests in my server regression testing:

	git://linux-nfs.org/~bfields/testd.git

--b.

diff --git a/nfs4.1/server41tests/st_sequence.py b/nfs4.1/server41tests/st_sequence.py
index f939408..a5e344d 100644
--- a/nfs4.1/server41tests/st_sequence.py
+++ b/nfs4.1/server41tests/st_sequence.py
@@ -191,7 +191,7 @@ def testReplayCache001(t, env):
     """
     c1 = env.c1.new_client(env.testname(t))
     sess1 = c1.create_session()
-    res1 = sess1.compound([op.putrootfh()])
+    res1 = sess1.compound([op.putrootfh()], cache_this=True)
     check(res1)
     res2 = sess1.compound([op.putrootfh()], seq_delta=0)
     check(res2)
@@ -211,7 +211,7 @@ def testReplayCache002(t, env):
     check(res)
     ops = env.home + [op.savefh(),\
           op.rename("%s_1" % env.testname(t), "%s_2" % env.testname(t))]
-    res1 = sess1.compound(ops)
+    res1 = sess1.compound(ops, cache_this=True)
     check(res1)
     res2 = sess1.compound(ops, seq_delta=0)
     check(res2)
@@ -227,7 +227,7 @@ def testReplayCache003(t, env):
     """
     c1 = env.c1.new_client(env.testname(t))
     sess1 = c1.create_session()
-    res1 = sess1.compound([op.putrootfh(), op.lookup("")])
+    res1 = sess1.compound([op.putrootfh(), op.lookup("")], cache_this=True)
     check(res1, NFS4ERR_INVAL)
     res2 = sess1.compound([op.putrootfh(), op.lookup("")], seq_delta=0)
     check(res2, NFS4ERR_INVAL)
@@ -244,7 +244,7 @@ def testReplayCache004(t, env):
     c1 = env.c1.new_client(env.testname(t))
     sess1 = c1.create_session()
     ops = [op.putrootfh(), op.savefh(), op.rename("", "foo")]
-    res1 = sess1.compound(ops)
+    res1 = sess1.compound(ops, cache_this=True)
     check(res1, NFS4ERR_INVAL)
     res2 = sess1.compound(ops, seq_delta=0)
     check(res2, NFS4ERR_INVAL)
@@ -260,7 +260,7 @@ def testReplayCache005(t, env):
     """
     c1 = env.c1.new_client(env.testname(t))
     sess1 = c1.create_session()
-    res1 = sess1.compound([op.illegal()])
+    res1 = sess1.compound([op.illegal()], cache_this=True)
     check(res1, NFS4ERR_OP_ILLEGAL)
     res2 = sess1.compound([op.illegal()], seq_delta=0)
     check(res2, NFS4ERR_OP_ILLEGAL)
-- 
1.7.0.4


^ permalink raw reply related	[flat|nested] only message in thread

only message in thread, other threads:[~2010-09-26 17:02 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-09-26 16:59 [PATCH] CLNT: Turn on cache_this for specific tests linux-nfs-owner

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.