Comment 1 for bug 352458

Revision history for this message
Thomas Herve (therve) wrote :

[1]
+ min(4096, resource.getrlimit(resource.RLIMIT_NOFILE)[1])):

Can you create an intermediate variable for the getrlimit value?

[2]
+from landscape.tests.helpers import LandscapeTest
+
+class CleanFDsTests(LandscapeTest):

Missing a blank line.

[3]
+ clean_fds()
+
+
+ def test_clean_fds_sanity(self):

Extra blank line :).

+1!