From b72963d62b8695c04a69cd051a2f801484ecace4 Mon Sep 17 00:00:00 2001 From: Jamil Date: Fri, 19 Sep 2025 06:47:19 -0400 Subject: [PATCH] fix(ci): migrate with manual migrations (#10389) In 0b09d9f2f5, the `--migrations-path` arguments were added to the `Seed database` step. These only work for the `ecto.migrate` and family of commands, and not with `ecto.seed`. This meant all of our CI has essentially not been running the manual migrations. Then, #10377 we fixed the env var when starting the services so that manual migrations are run. That env var, however, only applies to "releases", which the `elixir` service is not. This caused the API service to try and run the missing manual migrations upon startup, which failed because they would then be run out-of-order. To fix all of the above, we fix the `Seed database` step of the perf-tests job to ensure it runs all migrations. The other places where we seed the database already handled this properly. --- .github/workflows/ci.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index dbefaeace..8deaa226b 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -315,7 +315,7 @@ jobs: # We need at least Docker v28.1 which is not yet available on GitHub actions runners - uses: docker/setup-docker-action@b60f85385d03ac8acfca6d9996982511d8620a19 # v4.3.0 - name: Seed database - run: docker compose run elixir /bin/sh -c 'cd apps/domain && mix ecto.seed --migrations-path priv/repo/migrations --migrations-path priv/repo/manual_migrations' + run: docker compose run elixir /bin/sh -c 'cd apps/domain && mix ecto.migrate --migrations-path priv/repo/migrations --migrations-path priv/repo/manual_migrations && mix ecto.seed' - name: Increase max UDP buffer sizes run: | sudo sysctl -w net.core.wmem_max=16777216 # 16 MB