solaris needs setrlimit code to increase max file descriptors #209

New Issue

zooko · 2007-11-15T03:52:06Z

zooko commented

2007-11-15 03:52:06 +00:00

5.g.:

http://allmydata.org/buildbot/builders/OS-X/builds/113/steps/test/logs/stdio

5.g.: <http://allmydata.org/buildbot/builders/OS-X/builds/113/steps/test/logs/stdio>

zooko added the

labels 2007-11-15 03:52:06 +00:00

zooko added this to the 0.7.0 milestone 2007-11-15 03:52:06 +00:00

warner commented

2007-11-15 21:00:26 +00:00

Fixed by changeset: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.

Fixed by changeset: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.

warner commented

2007-11-16 05:10:49 +00:00

throwing this one to zooko: the solaris buildslave is having too-many-open-files errors

warner commented

2007-11-17 00:34:40 +00:00

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.

warner commented

2007-11-19 17:49:37 +00:00

updating the description to point out that setrlimit() is the real goal of this ticket

warner changed title from ~~unit test failure on downloading mutable files~~ to solaris needs setrlimit code to increase max file descriptors

2007-11-19 17:49:37 +00:00

zooko added

code-network

and removed

unknown

labels 2007-12-04 21:40:03 +00:00

zooko added this to the undecided milestone 2008-01-23 02:38:41 +00:00

zooko commented

2008-06-15 09:19:22 +00:00

This file already mentions "solaris": source: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.

warner commented

2008-06-16 18:19:20 +00:00

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?)

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?)

davidsarah commented

2011-08-17 05:16:48 +00:00

The reference to Solaris was added in changeset:d5c0a85c68e0450a, when this code was in [node.py]source:src/allmydata/node.py@1631. 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).

exarkun commented

2020-01-16 19:36:16 +00:00

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.

exarkun added the

wontfix

label 2020-01-16 19:36:16 +00:00

exarkun closed this issue

2020-01-16 19:36:16 +00:00

Sign in to join this conversation.