scarver2 / ehr-portal / 23326651432

77%

push

github

fix: Kamal monorepo deployment configuration and Dockerfile paths (#21)

* fix: use local Docker images instead of ghcr

Revert to using local Docker Hub images (scarver2/ehr-api, scarver2/ehr-portal)
instead of GHCR. This simplifies the deployment and removes the need for
GitHub token authentication in the build environment.

Also removed:
- Registry configuration (not needed for local Docker)
- Cache configuration (local Docker doesn't support registry caching)
- Simplified builder to just specify context directory

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

* fix: add Docker Hub registry for Kamal

Kamal requires a registry configuration even when using local Docker images.
Point to docker.io (Docker Hub) as the registry server.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

* fix: add DOCKER_PASSWORD to registry config

Kamal requires both username and password for registry authentication.
References DOCKER_PASSWORD secret for Docker Hub credentials.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

* fix: add --skip-push to kamal deploy for local Docker builds

Use --skip-push flag to build Docker images locally without pushing to
a registry. This allows deployment to work with local Docker images
without requiring Docker Hub authentication credentials.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

* fix: remove registry config for local Docker builds

Since we use --skip-push for local Docker builds, we don't need a
registry configuration. This eliminates the need for Docker Hub
authentication credentials.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

* fix: use localhost registry for local Docker builds

Kamal requires a registry server to be specified. Use localhost as a
placeholder for local Docker builds with --skip-push flag.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

* fix: add local username to localhost registry

Kamal requires a username for all registries. Us... (continued)

75 of 115 branches covered (65.22%)

Branch coverage included in aggregate %.

116 of 198 relevant lines covered (58.59%)

5.5 hits per line