Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: move the docker-compose volume #507

Merged
merged 1 commit into from
Sep 5, 2023

Conversation

vpavlin
Copy link
Contributor

@vpavlin vpavlin commented Aug 5, 2023

I was playing around with Codex and noticed the datadir is created in root (i.e. /) of my disk, which is not great.

This PR moves the datadir to docker/ folder when you run something like

docker-compose -f docker/docker-compose.yaml up

I am also adding :z which is useful when you use SELinux to properly label the volume (https://prefetch.net/blog/2017/09/30/using-docker-volumes-on-selinux-enabled-servers/)

@dryajov
Copy link
Contributor

dryajov commented Aug 9, 2023

@benbierens looks like a good change and I'd like to merge it, unless this will mess your setup?

@benbierens
Copy link
Contributor

Nope, the docker-compose is just there as an example. None of the tests hinge on it.
Todo-Follow-up: volumes for the k8s setup.

@dryajov dryajov merged commit d399290 into codex-storage:master Sep 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants