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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).