On 6/12/2014 05:17, J. Bruce Fields wrote: > I tried to apply this, and the ipv6 patch (could we do that for 4.1 > too?) and the 3 other patches, but: nfs4.1 has supports IPv6, the patch for nfs4.0 is a back-port from nfs4.1. > > - there are some conflicts with the latest pynfs (e.g. I already > added a few acl tests, apologies) > - I'm getting failures to write to a read-only fs, apparently > one of the tests is trying to write to the export root? I > didn't check which one. > > Could you look into those and resend your pynfs patches? Thanks! Yes, I will check those patch and resend them. thanks, Kinglong Mee > > --b. > > > On Tue, Jun 10, 2014 at 07:55:47PM +0800, Kinglong Mee wrote: >> There are many sessions left after testing, they hold drc memory at server. >> This patch tries to clean session after each testcase. >> >> v2, >> adds empty function of clean_sessions for nfs4.0, >> because they are using the same testmod.py >> >> Signed-off-by: Kinglong Mee <kinglongmee@xxxxxxxxx> >> --- >> nfs4.0/servertests/environment.py | 3 +++ >> nfs4.1/server41tests/environment.py | 7 ++++++- >> nfs4.1/testmod.py | 1 + >> 3 files changed, 10 insertions(+), 1 deletion(-) >> >> diff --git a/nfs4.0/servertests/environment.py b/nfs4.0/servertests/environment.py >> index 993320d..9852178 100644 >> --- a/nfs4.0/servertests/environment.py >> +++ b/nfs4.0/servertests/environment.py >> @@ -226,6 +226,9 @@ class Environment(testmod.Environment): >> cmd += ' ' + args >> os.system(cmd); >> >> + def clean_sessions(self): >> + return >> + >> ######################################### >> debug_fail = False >> >> diff --git a/nfs4.1/server41tests/environment.py b/nfs4.1/server41tests/environment.py >> index b2df732..7632655 100644 >> --- a/nfs4.1/server41tests/environment.py >> +++ b/nfs4.1/server41tests/environment.py >> @@ -248,7 +248,12 @@ class Environment(testmod.Environment): >> def testname(self, t): >> """Returns a name for the test that is unique between runs""" >> return "%s_%i" % (t.code, self.timestamp) >> - >> + >> + def clean_sessions(self): >> + """Destroy client name env.c1""" >> + for sessionid in self.c1.sessions.keys(): >> + self.c1.compound([op.destroy_session(sessionid)]) >> + >> ######################################### >> debug_fail = False >> >> diff --git a/nfs4.1/testmod.py b/nfs4.1/testmod.py >> index 5908066..661aed0 100644 >> --- a/nfs4.1/testmod.py >> +++ b/nfs4.1/testmod.py >> @@ -220,6 +220,7 @@ class Test(object): >> environment.startUp() >> self.runtest(self, environment) >> self.result = self._pass_result >> + environment.clean_sessions() >> except KeyboardInterrupt: >> raise >> except TestException, e: >> -- >> 1.9.3 >> > -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
