Opened at 2007-11-15T03:52:06Z
Closed at 2020-01-16T19:36:16Z
#209 closed defect (wontfix)
solaris needs setrlimit code to increase max file descriptors
| Reported by: | zooko | Owned by: | zooko |
|---|---|---|---|
| Priority: | major | Milestone: | eventually |
| Component: | code-network | Version: | 0.7.0 |
| Keywords: | test solaris | Cc: | |
| Launchpad Bug: |
Change History (13)
comment:1 Changed at 2007-11-15T21:00:26Z by warner
comment:2 Changed at 2007-11-16T05:10:49Z by warner
- Owner changed from nobody to zooko
throwing this one to zooko: the solaris buildslave is having too-many-open-files errors
comment:3 Changed at 2007-11-17T00:34:40Z by warner
I fixed the underlying problem that prompted me to change test_system.SystemTest?.test_mutable to use 10 nodes instead of 5 . To exercise the node.py setrlimit() fix, you'll need to raise this back up to 10: there's a call to setup_nodes() that should become setup_nodes(10) while you play with this.
comment:4 Changed at 2007-11-19T17:49:37Z by warner
- Summary changed from unit test failure on downloading mutable files to solaris needs setrlimit code to increase max file descriptors
updating the description to point out that setrlimit() is the real goal of this ticket
comment:5 Changed at 2007-11-19T19:25:41Z by zooko
- Milestone changed from 0.7.0 to 0.7.1
comment:6 Changed at 2007-12-04T21:40:03Z by zooko
- Component changed from unknown to code-network
comment:7 Changed at 2008-01-23T02:38:41Z by zooko
- Milestone changed from 0.7.1 to undecided
comment:8 Changed at 2008-06-15T09:19:22Z by zooko
This file already mentions "solaris": src/allmydata/util/iputil.py, so maybe this ticket has already been fixed? I guess I need to try setting test_mutable to use 10 nodes instead of 5 on my solaris machine and see if it works.
comment:9 Changed at 2008-06-15T09:19:30Z by zooko
- Status changed from new to assigned
comment:10 Changed at 2008-06-16T18:19:20Z by warner
We could also add a test that creates 100 Tubs (perhaps using UnauthenticatedTub for speed) and has them all connect to a single target.. that would exercise rlimit > 64 at least. (Hmm, what was the default limit on solaris anyways?)
comment:11 Changed at 2009-12-20T01:25:06Z by davidsarah
- Keywords test solaris added
comment:12 Changed at 2011-08-17T05:16:48Z by davidsarah
The reference to Solaris was added in d5c0a85c68e0450a, when this code was in node.py. So there was an attempt to fix it then, but it isn't specifically tested for (and we don't currently have a Solaris buildslave).
comment:13 Changed at 2020-01-16T19:36:16Z by exarkun
- Description modified (diff)
- Resolution set to wontfix
- Status changed from assigned to closed
The project has no Solaris testing infrastructure and has had none for quite some while. If an enterprising Solaris enthusiast wishes to take up maintenance of Tahoe-LAFS on Solaris, this would be quite welcome. Until then, I'm closing Solaris-related tickets as they are unlikely to receive any attention.
Fixed by 59d6c3c8229d8457, but this is just a temporary workaround for the real issue, which is in #211 and has been pushed back to 0.7.1.
Except.. I spoke too soon, this workaround (using 10 peers in a test process instead of 5) causes our solaris buildslave to fail because of "too many open files". Sigh.