chore(docker): dev-setup polish#860
Merged
Merged
Conversation
Resolves the 'FromAsCasing: as and FROM keywords casing do not match' buildkit warning and adds the missing newline at end of file. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
directus-sync looked for one of
directus-sync.config.{js,cjs,json}
on every run and warned 'No config file found'. Pinning dumpPath in a
proper config file silences the warning and gives us a single place to
hold the few defaults that don't belong in env vars.
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
A fresh checkout (or a 'sudo rm -rf data/' reset) hits an EACCES on /directus/uploads because the bind-mounted host directory is root-owned, while the Directus container runs as a non-root user. Add a one-shot 'init-uploads' service that runs before the backend and chmods ./data/uploads to 777, then make the backend depend on it via 'service_completed_successfully'. The matching 'sudo chmod 777 -R ./data/' step in the README is no longer needed. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
antontranelis
changed the title
- backend/README.md: bump directus-sync CLI examples to @3.5.1 to match the version pinned in push/pull/seed.sh. - backend/README.md: narrow the 'Access Data on local drive' chmod hint to ./data/database only; ./data/uploads is now handled automatically by the init-uploads service. - cypress/README.md: same adjustment in the troubleshooting section. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Summary
Three small follow-ups to #859 that smooth out the local dev experience without changing how production is run.
Commit 1 — Dockerfile casing + trailing newline
Resolves the
FromAsCasingbuildkit warning that's been ticking on every build, plus the missing final newline.Commit 2 —
backend/directus-sync.config.jsondirectus-syncprinted[config] No config file found. Tried path: …on every invocation. Adding a minimal config file silences that and gives us a single place for sync defaults that aren't environment-specific.Commit 3 —
init-uploadsservice auto-fixes permissionsA fresh checkout (or a
sudo rm -rf data/reset) hitsbecause the host-side
./data/uploadsis root-owned while the Directus container runs as a non-root user. Adding a one-shot busybox service that runs before the backend (service_completed_successfullygate) andchmods the directory removes the manualsudo chmod 777 -R ./data/step from the README.Test plan
docker compose up -d --build+cd backend && ./push.sh && ./seed.shsucceeds without manual chmod[config] No config file foundlog from directus-syncFromAsCasingwarning from docker build🤖 Generated with Claude Code