Re: [PATCH 02/17] CLNT: test lease period of unconfirmed record

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

On Thu, Mar 03, 2011 at 05:07:09PM +0800, Mi Jinlong wrote:
> 
> 
> J. Bruce Fields :
> > On Wed, Mar 02, 2011 at 02:27:22PM +0800, Mi Jinlong wrote:
> >> Signed-off-by: Mi Jinlong <mijinlong@cn.fujitsu.com>
> >> ---
> >>  nfs4.1/server41tests/st_exchange_id.py |   26 ++++++++++++++++++++++++++
> >>  1 files changed, 26 insertions(+), 0 deletions(-)
> >>
> >> diff --git a/nfs4.1/server41tests/st_exchange_id.py b/nfs4.1/server41tests/st_exchange_id.py
> >> index b5e34e6..85f3092 100644
> >> --- a/nfs4.1/server41tests/st_exchange_id.py
> >> +++ b/nfs4.1/server41tests/st_exchange_id.py
> >> @@ -1,5 +1,6 @@
> >>  from nfs4_const import *
> >>  import nfs4_ops as op
> >> +import time
> >>  from environment import check, fail
> >>  from nfs4_type import *
> >>  from rpc import RPCAcceptError, GARBAGE_ARGS, RPCTimeout
> >> @@ -436,3 +437,28 @@ def testNotOnlyOp(t, env):
> >>      res = c.compound([op.exchange_id(owner, 0, protect, [c.impl_id]), op.putrootfh()])
> >>      # per draft 21 18.35.3, server MUST return NFS4ERR_NOT_ONLY_OP
> >>      check(res, NFS4ERR_NOT_ONLY_OP)
> >> +
> >> +def testLeasePeriod(t, env):
> >> +    """Any unconfirmed record that is not confirmed within
> >> +       a lease period SHOULD be removed.
> >> +
> >> +    FLAGS: exchange_id all
> >> +    CODE: EID9
> >> +    """
> >> +    c1 = env.c1.new_client("%s_1" % env.testname(t))
> >> +    c2 = env.c1.new_client("%s_2" % env.testname(t))
> >> +    # CREATE_SESSION
> >> +    chan_attrs = channel_attrs4(0,8192,8192,8192,128,8,[])
> >> +    time.sleep(80)
> > 
> > Let's not hard-code any lease time.  pynfs should do a getattr to find
> > the server's lease time, and use that as the basis for calculations
> > (we need something like the getLeaseTime in the 4.0 pynfs.)
> > 
> > I actually usually set the server lease time to 15 seconds for testing,
> > just to make the tests go faster....
> 
>   Agree with you!
>   What about the following one?

Looks fine to me.

> +    chan_attrs = channel_attrs4(0,8192,8192,8192,128,8,[])
> +    time.sleep(lease - 10)

I changed this to

	time.sleep(min(lease-10, 1))

so it'll sill work with single-digit lease, and left it the same
otherwise.

Thanks!

--b.

> +    # Inside lease period, create_session will success.
> +    res1 = c1.c.compound([op.create_session(c1.clientid, c1.seqid, 0,
> +                                        chan_attrs, chan_attrs,
> +                                        123, [])], None)
> +    check(res1)
> +
> +    time.sleep(lease + 10)
> +    # After lease period, create_session will get error NFS4ERR_STALE_CLIENTID
> +    res2 = c2.c.compound([op.create_session(c2.clientid, c2.seqid, 0,
> +                                        chan_attrs, chan_attrs,
> +                                        123, [])], None)
> +    check(res2, NFS4ERR_STALE_CLIENTID)
> -- 
> 1.7.4.1
> 
> 
>