Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sam/oriole17 flake check #1399

Draft
wants to merge 11 commits into
base: develop
Choose a base branch
from
Draft

Sam/oriole17 flake check #1399

wants to merge 11 commits into from

Conversation

samrose
Copy link
Contributor

@samrose samrose commented Jan 7, 2025

What kind of change does this PR introduce?

This PR consolidates start-client flake app into start-server, and then uses start-server in the makeCheckHarness nix flake check, and in that flake check, brings in the rest of the migrations and config to match our AMI set up as closely as possible. start-client now just wraps psql for the version of postgres we're running. start-server inherits the --migration-file capability, and has an optional --skip-migrations

In addition, this PR refactors start-server to start/stop postgres with https://github.com/DarthSim/overmind so that managing the running service in foreground, background, etc is simpler. The script handles all interaction with overmind, so the nix run commands are unchanged other than the new flags.

This PR will remain in DRAFT mode until the flake checks pass.

@samrose samrose requested a review from olirice January 7, 2025 00:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants