make test: set the ulimit required for the test

Previously the fd limit was bumped in scripts/common.sh, run by
the CI. This meant that `make test` as run by a developer would fail.

This patch moves the `ulimit` into the Makefile so developers can
easily run the stress tests.

Signed-off-by: David Scott <[email protected]>

Author: djs55

Makefile

@@ -42,7 +42,8 @@ vpnkit.exe:
 .PHONY: test
 test:
 	jbuilder build --dev src/hostnet_test/main.exe
-	./_build/default/src/hostnet_test/main.exe
+	# One test requires 1026 file descriptors
+	ulimit -n 1500 && ./_build/default/src/hostnet_test/main.exe
 
 .PHONY: OSS-LICENSES
 OSS-LICENSES:

scripts/common.sh

@@ -37,8 +37,6 @@ opam install $(ls -1 ${OPAM_REPO}/packages/upstream) -y
 OPAMVERBOSE=1 opam install --deps-only -t vpnkit -y
 
 OPAMVERBOSE=1 make
-# One test requires 1026 file descriptors
-ulimit -n 1500
 OPAMVERBOSE=1 make test
 OPAMVERBOSE=1 make artefacts
 OPAMVERBOSE=1 make OSS-LICENSES