New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Raise the resource limit on open files for fedmsg-hub #381

Merged
merged 1 commit into from Oct 28, 2016

Conversation

Projects
None yet
3 participants
@jeremycline
Member

jeremycline commented Oct 27, 2016

This is a short-term solution to systems running into the maximum open
file descriptor limit.

Signed-off-by: Jeremy Cline jeremy@jcline.org

@jeremycline jeremycline force-pushed the jeremycline:up-rlimit-nofile branch from 3ed13e4 to 3468cd3 Oct 27, 2016

@ralphbean

This comment has been minimized.

Contributor

ralphbean commented Oct 27, 2016

👍 from me!

@@ -17,11 +17,20 @@
#
# Authors: Ralph Bean <rbean@redhat.com>
#
import fedmsg
from gettext import gettext as _

This comment has been minimized.

@pypingou

pypingou Oct 27, 2016

Member

do we already depend on gettext?

This comment has been minimized.

@jeremycline

jeremycline Oct 27, 2016

Member

gettext is in the standard library. I don't know exactly when it was introduced, but it's definitely 2.6+ so we're good.

@pypingou

This comment has been minimized.

Member

pypingou commented Oct 28, 2016

Cool 👍 for me as well then :)

Raise the resource limit on open files for fedmsg-hub
This is a short-term solution to systems running into the maximum open
file descriptor limit.

Signed-off-by: Jeremy Cline <jeremy@jcline.org>

@jeremycline jeremycline force-pushed the jeremycline:up-rlimit-nofile branch from 3468cd3 to 3651cfc Oct 28, 2016

@jeremycline jeremycline merged commit caa5cc9 into fedora-infra:develop Oct 28, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@jeremycline jeremycline deleted the jeremycline:up-rlimit-nofile branch Oct 28, 2016

asdil12 added a commit to asdil12/fedmsg that referenced this pull request Nov 9, 2016

Merge pull request fedora-infra#381 from jeremycline/up-rlimit-nofile
Raise the resource limit on open files for fedmsg-hub
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment