Feat/#190 migrate to vm#193
Conversation
There was a problem hiding this comment.
Pull request overview
This PR adds VM-migration operational tooling and introduces container/service health checks to support more reliable deployments outside the current LXC runtime.
Changes:
- Add Docker
HEALTHCHECKs for the Web and API containers and introduce an API/healthendpoint. - Add deployment helper scripts for preflight validation and post-deploy smoke checks.
- Make Cloudflare issuer configurable via
.env/compose and update docs/runbook for the VM migration target.
Reviewed changes
Copilot reviewed 9 out of 9 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| src/F1.Web/Dockerfile | Installs curl and adds a Docker healthcheck against the web root. |
| src/F1.Api/Program.cs | Adds a GET /health endpoint for health probing. |
| src/F1.Api/Dockerfile | Installs curl and adds a Docker healthcheck against the API health endpoint. |
| scripts/deploy-smoke-check.sh | Adds post-deploy smoke checks for API health, web reachability, and worker status. |
| scripts/deploy-preflight.sh | Adds pre-deploy validation for required env keys, log path writability, and disk space. |
| docker-compose.yml | Makes Cloudflare issuer configurable via CLOUDFLARE_ISSUER. |
| README.md | Updates infra docs and references the new deployment runbook and scripts. |
| DEPLOYMENT.md | Adds a VM migration/deploy runbook (setup, procedures, rollback, checks). |
| .env.example | Adds CLOUDFLARE_ISSUER and DEPLOY_MIN_FREE_DISK_MB defaults/documentation. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 12 out of 12 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| @@ -16,9 +16,10 @@ COPY --from=build /app/publish/wwwroot . | |||
| COPY nginx.conf /etc/nginx/templates/default.conf.template | |||
|
|
|||
| # Install jq for runtime configuration injection | |||
There was a problem hiding this comment.
The comment says only jq is installed, but the RUN now installs both jq and curl. Update the comment to reflect what’s actually being installed (or split into separate comments) to avoid confusion during image maintenance.
| # Install jq for runtime configuration injection | |
| # Install jq for runtime configuration injection and curl for health checks |
…flow config guards
No description provided.