Re: [PATCH 2/2] Open files with OPEN4_SHARE_DENY_NONE for DELEG5 test

["J. Bruce Fields" <bfields@fieldses.org>]

Thanks, applying both.

On Mon, Jun 23, 2014 at 11:57:42AM -0500, Malahal Naineni wrote:
> DELEG5 test, while getting the READ delegation, uses
> OPEN4_SHARE_DENY_WRITE as that is the default.

That's a really dumb default, by the way, we should fix it to default to
DENY_NONE.

--b.

> Pass
> OPEN4_SHARE_DENY_NONE explicitly as that is assumed
> in the test.
> 
> Signed-off-by: Malahal Naineni <malahal@us.ibm.com>
> ---
>  nfs4.0/servertests/st_delegation.py | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/nfs4.0/servertests/st_delegation.py b/nfs4.0/servertests/st_delegation.py
> index 4da3a7a..eaff326 100644
> --- a/nfs4.0/servertests/st_delegation.py
> +++ b/nfs4.0/servertests/st_delegation.py
> @@ -249,6 +249,7 @@ def testManyReaddeleg(t, env, funct=_recall, response=NFS4_OK):
>              
>          # Get a read delegation
>          res = c.open_file(t.code, access=OPEN4_SHARE_ACCESS_READ,
> +                          deny=OPEN4_SHARE_DENY_NONE,
>                            set_recall=True,
>                            recall_funct=funct, recall_return=response)
>          fh, stateid = c.confirm(t.code, res)
> -- 
> 1.8.3.1
>