diff options
author | Patrick Steinhardt <psteinhardt@gitlab.com> | 2022-03-15 14:49:15 +0300 |
---|---|---|
committer | Patrick Steinhardt <psteinhardt@gitlab.com> | 2022-03-18 16:39:04 +0300 |
commit | f1bc9b3c9c6b485439d3bc68127ed93439f1c538 (patch) | |
tree | 75f0119023876a264158fc40251d533610091d01 | |
parent | d2e2db2f03d2f9d9e788199606d5fe82790935ce (diff) |
ci: Increase test coverage of PgBouncer job
The PgBouncer CI job is currently only executing the `test-postgres`
target, which only exercises a subset of our test suite. It's much more
sensible thoguh to instead execute `test-with-praefect` so that we can
assert that a "real" Praefect proxy works alright in combination with
PgBouncer.
Do this conversion and remove the now-unused `test-postgres` target.
-rw-r--r-- | .gitlab-ci.yml | 2 | ||||
-rw-r--r-- | Makefile | 5 |
2 files changed, 1 insertions, 6 deletions
diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index 4a33e6d17..6c8576e16 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -218,7 +218,7 @@ test:pgbouncer: - *test_before_script - while ! psql -h "${PGHOST_PGBOUNCER}" -p "${PGPORT_PGBOUNCER}" -U "${PGUSER}" -c 'SELECT 1' > /dev/null; do echo "awaiting PgBouncer service to be ready..." && sleep 1 ; done && echo "PgBouncer service is ready!" script: - - make test-postgres + - make test-with-praefect test:nightly: <<: *test_definition @@ -416,11 +416,6 @@ test-with-proxies: prepare-tests test-with-praefect: prepare-tests ${Q}GITALY_TEST_WITH_PRAEFECT=YesPlease $(call run_go_tests) -.PHONY: test-postgres -## Run Go tests with Postgres. -test-postgres: TEST_PACKAGES := gitlab.com/gitlab-org/gitaly/v14/internal/praefect/... -test-postgres: test-go - .PHONY: race-go ## Run Go tests with race detection enabled. race-go: TEST_OPTIONS := ${TEST_OPTIONS} -race |