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

remove chown COPY #693

Closed
wants to merge 2 commits into from
Closed

remove chown COPY #693

wants to merge 2 commits into from

Conversation

martadinata666
Copy link
Collaborator

@martadinata666 martadinata666 commented Jul 2, 2022

there is 243 'access denied issues'. Looking further, we copy as 1000:1000 it good if we start as USER but as we had UID:GID modification using /entrypoint.sh to change permission /app dir, it not changing dir accordingly, try stay copy as root 0 so 0 can change permission /app according UID:GID.

Dedy Martadinata S added 2 commits July 2, 2022 20:35
According this #671 there is 243 'access denied issues'. Looking further we copy as 1000:1000 but when /entrypoint.sh interfere to change permission `/app` dir not changing dir accordingly, try stay copy as root `0` so `0` can change permission to `/app` as UID:GID.
@martadinata666 martadinata666 marked this pull request as ready for review July 2, 2022 13:51
@Tzahi12345
Copy link
Owner

Just curious, why would changing permissions be possible from root but not from UID 1000? Is UID 0 not always able to chmod a folder owned by 1000?

Apologies if I'm just confused, lots of this UNIX permissions stuff messes with my brain.

@martadinata666 martadinata666 marked this pull request as draft July 3, 2022 05:29
@martadinata666
Copy link
Collaborator Author

martadinata666 commented Jul 3, 2022

I also pretty confused, the dir changed correctly but somehow gosu command not executed with correct UID. At first i thought the /app dir not changed permission correctly, so i tried mount the /app locally, it changed fine. So i think the exec gosu somehow not respecting UID?

Update: Yep, it gosu behaviour that we dont aware before. #696

@martadinata666 martadinata666 deleted the patch-1 branch July 3, 2022 06:19
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.

2 participants