On Tue, 2025-04-15 at 13:48 +0200, Tigran Mkrtchyan wrote: > The test assumes that the server can return either OK or DELAY, > however, the 'break' condition checks only for OK. > > Signed-off-by: Tigran Mkrtchyan <tigran.mkrtchyan@xxxxxxx> > --- > nfs4.1/server41tests/st_delegation.py | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/nfs4.1/server41tests/st_delegation.py b/nfs4.1/server41tests/st_delegation.py > index ea4c073..60b0de6 100644 > --- a/nfs4.1/server41tests/st_delegation.py > +++ b/nfs4.1/server41tests/st_delegation.py > @@ -181,8 +181,8 @@ def testDelegRevocation(t, env): > owner, how, claim) > while 1: > res = sess2.compound(env.home + [open_op]) > - if res.status == NFS4_OK: > - break; > + if res.status == NFS4_OK or res.status == NFS4ERR_DELAY: > + break > check(res, [NFS4_OK, NFS4ERR_DELAY]) > # just to keep sess1 renewed. This is a bit fragile, as we > # depend on the above compound waiting no longer than the Don't you want to loop on NFS4ERR_DELAY? It looks like this is supposed to loop (with no DELEGRETURN) until the open returns NFS4_OK. Presumably at that point the delegation will be revoked and you'll get the expected errors. -- Jeff Layton <jlayton@xxxxxxxxxx>
