118 platform production release deployment from GitHub #130
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
lmcdonough
added
feature work
There was a problem hiding this comment.
Pull Request Overview
This PR implements a GitHub-centric production release pipeline, streamlines container builds, and refines migration workflows.
- Introduces automated multi-arch production builds and manual Tailscale-based deployments with rollback support
- Refactors
Dockerfileandentrypoint.shto focus on required binaries and run SeaORM migrations - Cleans up
docker-compose.yaml, disables legacyseed_db, and updates ignore patterns
Reviewed Changes
Copilot reviewed 9 out of 9 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| entrypoint.sh | Adjust migration logic to run by default and call the new migration binary |
| docker-compose.yaml | Remove redundant environment variables, add comments, and refine depends_on usage |
| Dockerfile | Only build specified binaries and include a debug ls step |
| Cargo.toml | Comment out seed_db binary section with a TODO |
| .github/workflows/release.yml | New workflow for building and pushing multi-arch stable images |
| .github/workflows/deploy_to_do.yml | Manual deploy via Tailscale with rollback logic |
| .github/workflows/build_and_deploy_containers.yml | Rename workflow to "Build and Push Containers" |
| .dockerignore | Prune unnecessary files from build context |
Comments suppressed due to low confidence (2)
Dockerfile:24
- [nitpick] This debug listing likely isn’t needed in production builds—remove it to streamline the image and avoid leaking directory structure.
RUN echo "LIST OF CONTENTS" && ls -lahR /usr/src/app
.github/workflows/deploy_to_do.yml:38
BACKEND_IMAGE_NAMEisn’t defined in this workflow and no tag is specified, sodocker pullwill fail or pull the wrong image. Defineenv: BACKEND_IMAGE_NAME(including the:stabletag) or use the full image reference explicitly.
docker pull ${{ env.BACKEND_IMAGE_NAME }}
446fdd4 to
9b6693b
Compare
jhodapp
reviewed
May 23, 2025
jhodapp
reviewed
May 23, 2025
jhodapp
reviewed
May 26, 2025
…e interactively by a human user)
adds multiple change suggestions from pr review. Co-authored-by: Jim Hodapp <[email protected]> Co-authored-by: Copilot <[email protected]>
removes backtick symbals from workflow as per review suggestion. Co-authored-by: Jim Hodapp <[email protected]>
jhodapp
requested changes
May 28, 2025
jhodapp
requested changes
May 28, 2025
…in permissions Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
… robust entrypoint.sh script.
…y no hard coding main as the branch
jhodapp
approved these changes
May 29, 2025
…ontainers starting/stopping/restarting
… covering systemd service management.
…ocker compose down/up. This requires that /etc/systemd/system/refactor-platform.service exists.
jhodapp
deleted the
118-platform-production-release-deployment-from-github
branch
github-project-automation
bot
moved this from Review
to ✅ Done
in Refactor Coaching Platform
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request adds production deployment automation and refines containerization and database migration processes to support stable GitHub-based releases.
Deployment and CI/CD Enhancements:
.github/workflows/deploy_to_do.ymlfor manual deployments to a DigitalOcean droplet via Tailscale, with rollback support..github/workflows/release.ymlto build and push multi-arch:stableimages to GHCR..github/workflows/build_and_deploy_containers.ymlfor clarity.Docker and Containerization Updates:
Dockerfileto include only required binaries (refactor_platform_rs,migration) and removesseed_db..dockerignoreto reduce image size and build time.Database and Migration Adjustments:
entrypoint.shto runmigration upby default.seed_dbinCargo.tomlin favor of SeaORM migrations.Docker Compose Refinements:
docker-compose.ymlby removing redundant environment variables and commands.depends_onwith conditions.Branch:
118-platform-production-release-deployment-from-github