bytelyst/learning_ai_notes
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(docker): clean up compose advisory warnings (Phase D follow-up)

Browse Source 
Mechanical fixes to address remaining docker-doctor advisory warnings:
  - Add GITEA_NPM_OWNER to existing build.args block (or full args block
    if missing) so Dockerfile receives the build arg explicitly
  - Add start_period: 30s to healthcheck blocks where missing
    (prevents false cold-start failures)

Both are advisory checks — neither blocks builds — but cleaning them up
eliminates noise from CI logs.

Refs: docker-build-optimization-roadmap.md \xc2\xa7Phase D follow-ups
This commit is contained in:
saravanakumardb1 2026-05-27 04:25:19 -07:00
parent 6b3bd0a84d
commit 5687e5ace1
2 changed files with 3 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
docker-compose.override.yml
View File

@ -40,6 +40,7 @@ services:
# effect on the already-bundled client code. # effect on the already-bundled client code.
build: build:
args: args:
GITEA_NPM_OWNER: ${GITEA_NPM_OWNER:-learning_ai_user}
NEXT_PUBLIC_NOTES_API_URL: "http://127.0.0.1:4016/api" NEXT_PUBLIC_NOTES_API_URL: "http://127.0.0.1:4016/api"
NEXT_PUBLIC_PLATFORM_SERVICE_URL: "http://127.0.0.1:4003/api" NEXT_PUBLIC_PLATFORM_SERVICE_URL: "http://127.0.0.1:4003/api"
NEXT_PUBLIC_EXTRACTION_SERVICE_URL: "http://127.0.0.1:4005" NEXT_PUBLIC_EXTRACTION_SERVICE_URL: "http://127.0.0.1:4005"

2
docker-compose.yml
View File

@ -40,12 +40,14 @@ services:
timeout: 5s timeout: 5s
retries: 3 retries: 3
start_period: 30s
web: web:
container_name: notelett-web container_name: notelett-web
build: build:
context: . context: .
dockerfile: web/Dockerfile dockerfile: web/Dockerfile
args: args:
GITEA_NPM_OWNER: ${GITEA_NPM_OWNER:-learning_ai_user}
# NEXT_PUBLIC_* values are baked into the Next.js bundle at build # NEXT_PUBLIC_* values are baked into the Next.js bundle at build
# time, so they MUST be passed as build args, not just runtime # time, so they MUST be passed as build args, not just runtime
# environment. The defaults below target a local stack; override # environment. The defaults below target a local stack; override