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

use :Z (selinux) for volume mounts (re. #201) #204

Closed
wants to merge 1 commit into from

Conversation

vorburger
Copy link
Contributor

This helps with, but doesn't entirely fix just yet,
running this project on Fedora using podman up -d instead of Docker.

This should not break regular docker-compose on systems without SELinux.

This helps with, but doesn't entirely fix just yet,
running this project on Fedora using `podman up -d` instead of Docker.

This should not break regular `docker-compose` on systems without SELinux.
@vorburger
Copy link
Contributor Author

@saghul this partially helps with #201, but doesn't entirely fix it yet.

You'll probably want to double check that it does not break your regular Docker (it should not).

@jmhunter
Copy link

jmhunter commented Mar 28, 2020

FWIW - I just tried using docker-compose to install jitsi (CentOS 7, selinux enabled) and it was not successful until I applied this patch.

Before the patch it was unable to create directories as below. After the patch, all works great!

[cont-init.d] 10-config: executing...
mkdir: cannot create directory '/config/nginx': Permission denied
mkdir: cannot create directory '/config/keys': Permission denied

If it helps:
CentOS Linux release 7.7.1908 (Core)
Docker version 1.13.1, build cccb291/1.13.1
docker-compose version 1.23.2, build 1110ad01

@vorburger
Copy link
Contributor Author

@saghul given the independent verification" (above) of this one, do you want to merge it?

@digital-codes
Copy link

I have the same setup (centos 7.7. with selinux). The :Z is required to make it work.

@sapkra
Copy link
Contributor

sapkra commented Oct 8, 2020

Closing because already applied in master.

@sapkra sapkra closed this Oct 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants