diff mbox

4.1 server tests: make DELEG8 also check that an operation using a revoked stateid returns revoked status

Message ID 20171101120855.65421-1-aweits@rit.edu (mailing list archive)
State New, archived
Headers show

Commit Message

Andrew W Elble Nov. 1, 2017, 12:08 p.m. UTC 
Signed-off-by: Andrew Elble <aweits@rit.edu>
---
 nfs4.1/server41tests/st_delegation.py | 2 ++
 1 file changed, 2 insertions(+)

Comments

J. Bruce Fields Nov. 9, 2017, 2:13 p.m. UTC | #1 
Applied, thanks.--b.

On Wed, Nov 01, 2017 at 08:08:55AM -0400, Andrew Elble wrote:
> Signed-off-by: Andrew Elble <aweits@rit.edu>
> ---
>  nfs4.1/server41tests/st_delegation.py | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/nfs4.1/server41tests/st_delegation.py b/nfs4.1/server41tests/st_delegation.py
> index a5066926a89d..c5959f6cda84 100644
> --- a/nfs4.1/server41tests/st_delegation.py
> +++ b/nfs4.1/server41tests/st_delegation.py
> @@ -186,6 +186,8 @@ def testDelegRevocation(t, env):
>          # depend on the above compound waiting no longer than the
>          # server's lease period:
>          res = sess1.compound([])
> +    res = sess1.compound([op.putfh(fh), op.read(delegstateid, 0, 1000)])
> +    check(res, NFS4ERR_DELEG_REVOKED, "Read with a revoked delegation")
>      slot, seq_op = sess1._prepare_compound({})
>      res = sess1.c.compound([seq_op])
>      flags = res.resarray[0].sr_status_flags;
> -- 
> 2.10.1
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/nfs4.1/server41tests/st_delegation.py b/nfs4.1/server41tests/st_delegation.py
index a5066926a89d..c5959f6cda84 100644
--- a/nfs4.1/server41tests/st_delegation.py
+++ b/nfs4.1/server41tests/st_delegation.py
@@ -186,6 +186,8 @@  def testDelegRevocation(t, env):
         # depend on the above compound waiting no longer than the
         # server's lease period:
         res = sess1.compound([])
+    res = sess1.compound([op.putfh(fh), op.read(delegstateid, 0, 1000)])
+    check(res, NFS4ERR_DELEG_REVOKED, "Read with a revoked delegation")
     slot, seq_op = sess1._prepare_compound({})
     res = sess1.c.compound([seq_op])
     flags = res.resarray[0].sr_status_flags;