From: "J. Bruce Fields" <bfields@redhat.com>
To: linux-nfs@vger.kernel.org
Cc: "J. Bruce Fields" <bfields@redhat.com>
Subject: [PATCH 3/5] 4.1 server tests: don't run pnfs tests by default
Date: Wed, 15 Feb 2012 17:45:18 -0500 [thread overview]
Message-ID: <1329345920-16608-4-git-send-email-bfields@redhat.com> (raw)
In-Reply-To: <1329345920-16608-1-git-send-email-bfields@redhat.com>
From: "J. Bruce Fields" <bfields@redhat.com>
I'd rather the pynfs tests have clean results by default against a
correct 4.1 server. And it's correct for such a server not to implement
optional features. So leave pnfs tests off by default; you can still
get them by asking for "pnfs" tests.
Also, flags for individual pnfs operations seemed mild overkill for now,
given how few tests we have.
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
---
nfs4.1/server41tests/st_current_stateid.py | 2 +-
nfs4.1/server41tests/st_getdevicelist.py | 16 ++++++++--------
2 files changed, 9 insertions(+), 9 deletions(-)
diff --git a/nfs4.1/server41tests/st_current_stateid.py b/nfs4.1/server41tests/st_current_stateid.py
index aeaa8b1..7804900 100644
--- a/nfs4.1/server41tests/st_current_stateid.py
+++ b/nfs4.1/server41tests/st_current_stateid.py
@@ -119,7 +119,7 @@ def testOpenLayoutGet(t, env):
"""test current state id processing by having OPEN and LAYOUTGET
in a single compound
- FLAGS: currentstateid all
+ FLAGS: currentstateid pnfs
CODE: CSID7
"""
sess = env.c1.new_client_session(env.testname(t),
diff --git a/nfs4.1/server41tests/st_getdevicelist.py b/nfs4.1/server41tests/st_getdevicelist.py
index 8d53633..7a10c6c 100644
--- a/nfs4.1/server41tests/st_getdevicelist.py
+++ b/nfs4.1/server41tests/st_getdevicelist.py
@@ -10,7 +10,7 @@ from nfs4lib import state00
def testGetDevList(t, env):
"""Check devlist
- FLAGS: getdevicelist all
+ FLAGS: pnfs
CODE: GETDLIST1
"""
# Make sure E_ID returns MDS capabilities
@@ -36,7 +36,7 @@ def testGetDevList(t, env):
def testGetDevInfo(t, env):
"""Check devlist
- FLAGS: getdevicelist all
+ FLAGS: pnfs
DEPEND: GETDLIST1
CODE: GETDINFO1
"""
@@ -73,7 +73,7 @@ def testGetDevInfo(t, env):
## def xxxtestLayout(t, env):
## """Verify layout handling
-## FLAGS: layout all
+## FLAGS: pnfs
## CODE: GETLAYOUT1
## """
## # Make sure E_ID returns MDS capabilities
@@ -105,7 +105,7 @@ def testGetDevInfo(t, env):
def testGetLayout(t, env):
"""Verify layout handling
- FLAGS: getlayout all
+ FLAGS: pnfs
CODE: GETLAYOUT1
"""
# Make sure E_ID returns MDS capabilities
@@ -187,7 +187,7 @@ def testLayoutReturnFile(t, env):
"""
Return a file's layout
- FLAGS: layoutreturn all
+ FLAGS: pnfs
DEPEND: GETLAYOUT1
CODE: LAYOUTRET1
"""
@@ -220,7 +220,7 @@ def testLayoutReturnFsid(t, env):
"""
Return all of a filesystem's layouts
- FLAGS: layoutreturn all
+ FLAGS: pnfs
DEPEND: GETLAYOUT1
CODE: LAYOUTRET2
"""
@@ -252,7 +252,7 @@ def testLayoutReturnAll(t, env):
"""
Return all of a client's layouts
- FLAGS: layoutreturn all
+ FLAGS: pnfs
DEPEND: GETLAYOUT1
CODE: LAYOUTRET3
"""
@@ -283,7 +283,7 @@ def testLayoutCommit(t, env):
"""
Do some commits
- FLAGS: layoutcommit all
+ FLAGS: pnfs
CODE: LAYOUTCOMMIT1
"""
# Make sure E_ID returns MDS capabilities
--
1.7.5.4
next prev parent reply other threads:[~2012-02-15 22:45 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-02-15 22:45 pynfs: cleanup and turn off some tests by default J. Bruce Fields
2012-02-15 22:45 ` [PATCH 1/5] 4.0 tests: kill "Starting Call Back server stub" printk J. Bruce Fields
2012-02-15 22:45 ` [PATCH 2/5] 4.0 tests: remove 1-second wait on client creation J. Bruce Fields
2012-02-15 22:45 ` J. Bruce Fields [this message]
2012-02-15 22:45 ` [PATCH 4/5] 4.1 server tests: don't run utf8 tests by default J. Bruce Fields
2012-02-15 22:45 ` [PATCH 5/5] 4.0 server tests: remove RD12 (incorrect test) J. Bruce Fields
2012-02-16 0:28 ` pynfs: cleanup and turn off some tests by default Boaz Harrosh
2012-02-16 1:28 ` J. Bruce Fields
2012-02-16 3:31 ` Boaz Harrosh
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1329345920-16608-4-git-send-email-bfields@redhat.com \
--to=bfields@redhat.com \
--cc=linux-nfs@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.