Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

move and clarify ulimit check #286

Merged
merged 1 commit into from

3 participants

@evanmcc

partial fix for #278

@jj1bdx

+1. Friendlier code to the sysadmin.

@jaredmorrow

This commit is fine with me, +1.

@evanmcc evanmcc merged commit 8e83af6 into master
@evanmcc

Although this should likely be reverted or altered if we don't go with the other commit on #278

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Mar 4, 2013
  1. @evanmcc

    move and clarify ulimit check

    evanmcc authored
This page is out of date. Refresh to see the latest.
Showing with 11 additions and 8 deletions.
  1. +11 −8 rel/files/riak
View
19 rel/files/riak
@@ -40,14 +40,6 @@ if ([ "$RUNNER_USER" ] && [ "x$WHOAMI" != "x$RUNNER_USER" ]); then
exec sudo -H -u $RUNNER_USER -i $RUNNER_SCRIPT_DIR/$RUNNER_SCRIPT $@
fi
-# Warn the user if ulimit -n is less than 4096
-ULIMIT_F=`ulimit -n`
-if [ "$ULIMIT_F" -lt 4096 ]; then
- echo "!!!!"
- echo "!!!! WARNING: ulimit -n is ${ULIMIT_F}; 4096 is the recommended minimum."
- echo "!!!!"
-fi
-
# Make sure CWD is set to runner base dir
cd $RUNNER_BASE_DIR
@@ -139,6 +131,17 @@ case "$1" in
echo $RES
exit 1
fi
+
+ # Warn the user if ulimit -n is less than 4096
+ ULIMIT_F=`ulimit -n`
+ if [ "$ULIMIT_F" -lt 4096 ]; then
+ echo "!!!!"
+ echo "!!!! WARNING: shell ulimit -n is ${ULIMIT_F}; 4096 is the recommended minimum."
+ echo "!!!! riak user or erlang ulimit may differ, please check"
+ echo "!!!! console.log for more detailed file limit information."
+ echo "!!!!"
+ fi
+
HEART_COMMAND="$RUNNER_SCRIPT_DIR/$SCRIPT start"
export HEART_COMMAND
mkdir -p $PIPE_DIR
Something went wrong with that request. Please try again.