Discussion:
[farm-report] Build python-HP_UX-B.11.00-9000_829-taylor was successful.
Guido van Rossum
2002-08-16 15:29:35 UTC
Permalink
Subject: [farm-report] Build python-HP_UX-B.11.00-9000_829-taylor was
successful.
[...]
Exception exceptions.AttributeError: "mkstemped instance has no attribute 'fd'" in <bound method mkstemped.__del__ of <test.test_tempfile.mkstemped instance at 0x4074ccb0>> ignored
Exception exceptions.AttributeError: "mkstemped instance has no attribute 'fd'" in <bound method mkstemped.__del__ of <test.test_tempfile.mkstemped instance at 0x40b11a58>> ignored
I thought I had checked in a fix for that (reducing the number of temp
files created from 1000 to 100). Why would that not be sufficient, if
the open file limit is 200? Or did you reduce it to 100? That
wouldn't work (I guess you'd need at least 103 :-).

--Guido van Rossum (home page: http://www.python.org/~guido/)
Kalle Svensson
2002-08-16 15:49:24 UTC
Permalink
[Guido van Rossum]
Post by Guido van Rossum
Subject: [farm-report] Build python-HP_UX-B.11.00-9000_829-taylor was
successful.
[...]
Exception exceptions.AttributeError: "mkstemped instance has no attribute 'fd'" in <bound method mkstemped.__del__ of <test.test_tempfile.mkstemped instance at 0x4074ccb0>> ignored
Exception exceptions.AttributeError: "mkstemped instance has no attribute 'fd'" in <bound method mkstemped.__del__ of <test.test_tempfile.mkstemped instance at 0x40b11a58>> ignored
I thought I had checked in a fix for that (reducing the number of temp
files created from 1000 to 100). Why would that not be sufficient, if
the open file limit is 200? Or did you reduce it to 100? That
wouldn't work (I guess you'd need at least 103 :-).
Revision 1.5 of test_tempfile.py still has this:

def test_basic_many(self):
"""_mkstemp_inner can create many files (stochastic)"""
extant = range(1000)
for i in extant:
extant[i] = self.do_create(pre="aa")

I think you changed the wrong test. Here's a patch.

*** Lib/test/test_tempfile.py Thu Aug 15 00:01:43 2002
- --- /home/kalle/test_tempfile.py Fri Aug 16 17:45:56 2002
***************
*** 290,296 ****

def test_basic_many(self):
"""_mkstemp_inner can create many files (stochastic)"""
! extant = range(1000)
for i in extant:
extant[i] = self.do_create(pre="aa")

- --- 290,296 ----

def test_basic_many(self):
"""_mkstemp_inner can create many files (stochastic)"""
! extant = range(100)
for i in extant:
extant[i] = self.do_create(pre="aa")

By the way, I haven't forgotten about the AIX problems, but I'll
probably have no time to work on them for two weeks now. Very much
going on.

Peace,
Kalle
- --
Kalle Svensson, http://www.juckapan.org/~kalle/
Student, root and saint in the Church of Emacs.
Neal Norwitz
2002-08-16 18:34:24 UTC
Permalink
Post by Kalle Svensson
[Guido van Rossum]
Post by Guido van Rossum
Subject: [farm-report] Build python-HP_UX-B.11.00-9000_829-taylor was
successful.
[...]
Exception exceptions.AttributeError: "mkstemped instance has no attribute 'fd'" in <bound method mkstemped.__del__ of <test.test_tempfile.mkstemped instance at 0x4074ccb0>> ignored
Exception exceptions.AttributeError: "mkstemped instance has no attribute 'fd'" in <bound method mkstemped.__del__ of <test.test_tempfile.mkstemped instance at 0x40b11a58>> ignored
I thought I had checked in a fix for that (reducing the number of temp
files created from 1000 to 100). Why would that not be sufficient, if
the open file limit is 200? Or did you reduce it to 100? That
wouldn't work (I guess you'd need at least 103 :-).
There are 3 other 1000's in the file:
test__RandomNameSequence.test_many # this is 100 now
test__mkstemp_inner.test_basic_many
test_mkdtemp.test_basic_many
test_mktemp.test_many

Should I make a global that defines the max # of files:

TEST_FILES = 100

and use it throughout the test?

Neal
Guido van Rossum
2002-08-16 18:48:58 UTC
Permalink
Post by Neal Norwitz
test__RandomNameSequence.test_many # this is 100 now
test__mkstemp_inner.test_basic_many
test_mkdtemp.test_basic_many
test_mktemp.test_many
TEST_FILES = 100
and use it throughout the test?
Yes please!

Great idea.

--Guido van Rossum (home page: http://www.python.org/~guido/)

Loading...