#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

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.

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.

Last edited at 2011-08-17T05:07:10Z by davidsarah (previous) (diff)

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.

Note: See TracTickets for help on using tickets.