Skip to content

Allow other users to read the /opt/teku dir when using docker#6740

Closed
skylenet wants to merge 1 commit into
Consensys:masterfrom
skylenet:skylenet/fix-docker-dir-permissions
Closed

Allow other users to read the /opt/teku dir when using docker#6740
skylenet wants to merge 1 commit into
Consensys:masterfrom
skylenet:skylenet/fix-docker-dir-permissions

Conversation

@skylenet

Copy link
Copy Markdown
Contributor

PR Description

The problem is that if we try to run the teku container with any other user that has a differen uid than 1000, we won't be able to execute the teku binary due to the permissions of the working dir ( /opt/teku ) being too strict 0770. I'm changing the permissions to 0775 so that other users can also read the directory and execute the teku binary.

Example on how this fails currently with other users than 1000

$ docker pull consensys/teku 
$ docker run --rm -it --entrypoint ls --user=1000 consensys/teku 
LICENSE  bin  lib  licenses  teku.autocomplete.sh
$ docker run --rm -it --entrypoint ls --user=1003 consensys/teku 
ls: cannot open directory '.': Permission denied

Fixed Issue(s)

Documentation

  • I thought about documentation and added the doc-change-required label to this PR if updates are required.

Changelog

  • I thought about adding a changelog entry, and added one if I deemed necessary.

@CLAassistant

CLAassistant commented Jan 27, 2023

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

rolfyone added a commit to rolfyone/teku that referenced this pull request Jan 27, 2023
@lucassaldanha

lucassaldanha commented Jan 27, 2023

Copy link
Copy Markdown
Member

Recreated at #6743 to circumvent CircleCI permission issues.

rolfyone added a commit that referenced this pull request Jan 27, 2023
* hotfix from #6740

* cleanup permissions consistent with previous LTS
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