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

Refactor docker compose, .env file doc #80

Merged
merged 4 commits into from
Mar 7, 2025

Conversation

Pascal-Delange
Copy link
Contributor

@Pascal-Delange Pascal-Delange commented Feb 26, 2025

Prerequisites

Related PRs renaming/modifying env vars: checkmarble/marble-frontend#721, checkmarble/marble-backend#877
I made some changes to variable naming & expected format so that the backend and frontend use the same conventions, and variables have the name that matches their expected format.

Link

View the doc on the branch: https://github.com/checkmarble/marble/blob/pascal/improve-deployment-doc/installation/README.md

Content

I tried to make it as complete as possible, while minimizing the duplication of source of truth (which was not always easy).
I reached the point where I need to let it rest a bit and perhaps get a fresh look on it.

All the detailed explanation on configuring the environment variables on frontend & backend are in .env.example + docker-compose.yaml (single source of truth).
A really simplified setup for local testing is in docker-compose-dev.yaml + env.dev.example.
The more high level explanations on dependencies, quick start, how to configure, target architecture, version support policy & migrations is in the installation folder, with README.md as the entry point that links to some more detailed files.

@Pascal-Delange Pascal-Delange force-pushed the pascal/improve-deployment-doc branch 2 times, most recently from c2e9e4e to 61a80f1 Compare February 26, 2025 20:43
@Pascal-Delange Pascal-Delange changed the title remove transfercheck env vars Refactor docker compose, .env file doc Feb 28, 2025
@Pascal-Delange Pascal-Delange force-pushed the pascal/improve-deployment-doc branch from 61a80f1 to ffc23a3 Compare March 1, 2025 22:02
@Pascal-Delange Pascal-Delange force-pushed the pascal/improve-deployment-doc branch from b9e35e5 to 1d77ddc Compare March 4, 2025 11:12
technical deployment documentation
reshuffle files

improvement of doc files continued

continued

more info on migrations

last fixes

rename files

update .env example

remove comment on postgres 15 depreciation for now
@Pascal-Delange Pascal-Delange force-pushed the pascal/improve-deployment-doc branch from 1d77ddc to 0aadf1d Compare March 4, 2025 11:14
@Pascal-Delange
Copy link
Contributor Author

@checkmarble/devs FYI I put the version bump for the release inside this PR

@Pascal-Delange Pascal-Delange merged commit 1b012bf into main Mar 7, 2025
@Pascal-Delange Pascal-Delange deleted the pascal/improve-deployment-doc branch March 7, 2025 20:52
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.

4 participants