-
Notifications
You must be signed in to change notification settings - Fork 295
Closed
Description
I have done a pass over all of our CI workflows to remove duplicated work and have better reuse of work done in one place in other places.
In particular, on the cleanups side:
- Split TS and Elixir "check formatting" workflows since the two have different path dependencies
- Run vitest jobs in the
run mode. (More on that watch and run mode here). This is a minor change. I initially thought it would fix an issue with a stuck workflow we had but that turned out to be due to a different problem.- ci: Limit the runtime of TS test CI jobs to 10 minutes #3601 - a general timeout for TS tests enforced by GH Actions, to avoid the repeat of the situation where a job gets stuck running for 6 hours straight
- ci: Clean up the CI workflows for Elixir packages #3592
Resource reuse:
- ci: Build Postgres images preconfigured with logical replication for use in CI workflows #3588
- Remove the
Electric Docker Image Smoke Testworkflow and instead build a Docker image for the sync service on demand and use it as a service for TS tests and Lux integration tests. -
Run sync-service in a Docker container in the Lux integration tests workflowto achieve this, we need to modify the integration test running scripts to be able to start Electric both from source and as a Docker container; not a priority for now
Retiring Buildkite:
Performance:
- I've used Blacksmith's migration wizard to switch some of our workflows to their runners - .github/workflows: Migrate workflows to Blacksmith runners #3589. I had done this before all of the CI cleanup work so I'll go back it later once the above PRs have all been finalized.
msfstef
Metadata
Metadata
Assignees
Labels
No labels