test(Makefile): fix postgresql memory usage

mafredri · mafredri · commit 82def17960c0 · 2025-01-17T12:00:15.000Z
Since setting work mem will raise the memory consumption of _all_
connections, let's keep it in range of what we're actually giving the
container and how many total connections we allow.
diff --git a/Makefile b/Makefile
@@ -864,6 +864,12 @@ test-migrations: test-postgres-docker
 # NOTE: we set --memory to the same size as a GitHub runner.
 test-postgres-docker:
 	docker rm -f test-postgres-docker-${POSTGRES_VERSION} || true
+	# Make sure to not overallocate work_mem and max_connections as each
+	# connection will _always_ allocate this much:
+	# - work_mem=8MB * max_connections=1000 = 8GB
+	# - shared_buffers=2GB + effective_cache_size=1GB = 3GB
+	# This leaves 5GB for the rest of the system _and_ storing the
+	# database in memory (--tmpfs).
 	docker run \
 		--env POSTGRES_PASSWORD=postgres \
 		--env POSTGRES_USER=postgres \
@@ -876,9 +882,9 @@ test-postgres-docker:
 		--detach \
 		--memory 16GB \
 		gcr.io/coder-dev-1/postgres:${POSTGRES_VERSION} \
-		-c shared_buffers=1GB \
-		-c work_mem=1GB \
+		-c shared_buffers=2GB \
 		-c effective_cache_size=1GB \
+		-c work_mem=8MB \
 		-c max_connections=1000 \
 		-c fsync=off \
 		-c synchronous_commit=off \
