Signed-off-by: Malahal Naineni <[email protected]>
---
nfs4.0/servertests/st_delegation.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/nfs4.0/servertests/st_delegation.py b/nfs4.0/servertests/st_delegation.py
index 4965f5a..4da3a7a 100644
--- a/nfs4.0/servertests/st_delegation.py
+++ b/nfs4.0/servertests/st_delegation.py
@@ -35,7 +35,7 @@ def _recall(c, op, cbid):
print "CALLBACK error in _recall:", e
res = None
_lock.release()
- if res.status != NFS4_OK:
+ if res is not None and res.status != NFS4_OK:
t_error = _handle_error(c, res, ops)
t = threading.Thread(target=t_error.run)
t.setDaemon(1)
--
1.8.3.1
DELEG5 test, while getting the READ delegation, uses
OPEN4_SHARE_DENY_WRITE as that is the default. Pass
OPEN4_SHARE_DENY_NONE explicitly as that is assumed
in the test.
Signed-off-by: Malahal Naineni <[email protected]>
---
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
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 <[email protected]>
> ---
> 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
>