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

Container initialization changes top directory permissions to 'rwx------' #154

Open
lucekdudek opened this issue Feb 9, 2023 · 0 comments
Labels
bug Something isn't working impact: low impact: some users affected severity: minor severity: loss of minor functionality

Comments

@lucekdudek
Copy link

lucekdudek commented Feb 9, 2023

When investigating https://github.com/golemfactory/yagna-sdk-team/issues/205 we found out that ya-runtime-vm container instillation creates top directory with set permissions ignoring ones in provided image.


Exact line to be responsible per @prekucki: https://github.com/golemfactory/ya-runtime-vm/blob/master/runtime/init-container/src/init.c#L1677


Having a nginx-latest-cd6d7a1750.gvmi gvmi image:

mkdir /tmp/foo
sudo mount nginx-latest-cd6d7a1750.gvmi /tmp/foo
ls -al /tmp/foo

Will output proper permissions (same as original docker image permissions)

drwxr-xr-x 22 root root   409 lut  8 12:18 .
...

But using the same image in ya-runtime-vm causes permisons to change

sudo ya-runtime-dbg --runtime ya-runtime-vm --task-package nginx-latest-cd6d7a1750.gvmi --workdir /tmp/workdir --exec-shell bash
ls -al

outputs

drwx------   1 root root   80 Feb  9 08:44 .
...

The same behavior was observe running given image as payload in dapp-runner. dApp-runner payloads examples: https://github.com/golemfactory/dapp-experiments/pull/10/files

@MrDarthShoe MrDarthShoe added bug Something isn't working severity: minor severity: loss of minor functionality impact: low impact: some users affected labels Feb 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working impact: low impact: some users affected severity: minor severity: loss of minor functionality
Projects
None yet
Development

No branches or pull requests

2 participants