Skip to content

Commit

Permalink
Don't set ulimit
Browse files Browse the repository at this point in the history
1ES pools should now have ulimit set already now
  • Loading branch information
dsplaisted committed Nov 30, 2021
1 parent 6d87160 commit f8e115f
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 7 deletions.
4 changes: 1 addition & 3 deletions build.sh
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,5 @@ dockerbuild()
if [ ! -z "$BUILD_IN_DOCKER" ]; then
dockerbuild $args
else
# Run under sudo so we can set ulimit
# See https://github.com/dotnet/core-eng/issues/14808
sudo -E $DIR/run-build.sh $args
$DIR/run-build.sh $args
fi
4 changes: 0 additions & 4 deletions run-build.sh
Original file line number Diff line number Diff line change
Expand Up @@ -25,10 +25,6 @@ CUSTOM_BUILD_ARGS=
# Set nuget package cache under the repo
[ -z $NUGET_PACKAGES ] && export NUGET_PACKAGES="$REPOROOT/.nuget/packages"

# Set max number of files open, helps avoid errors during NuGet restore
# See https://github.com/dotnet/core-eng/issues/14808
ulimit -n 16384

args=( )

while [[ $# > 0 ]]; do
Expand Down

0 comments on commit f8e115f

Please sign in to comment.