Skip to content

scripts/entrypoint.sh: Fix ioncorrect variable name#6

Open
miczyg1 wants to merge 1 commit into
mainfrom
group_id_fix
Open

scripts/entrypoint.sh: Fix ioncorrect variable name#6
miczyg1 wants to merge 1 commit into
mainfrom
group_id_fix

Conversation

@miczyg1
Copy link
Copy Markdown
Contributor

@miczyg1 miczyg1 commented Aug 6, 2024

To set the GROUP_ID for the coreboot user one has to pass two variables to docker environment: GROUP and GROUP_ID. Most likely it was a copy-pasta error, which can be fixed by using GROUP_ID in the IF condition.

To set the GROUP_ID for the coreboot user one has to pass
two variables to docker environment: GROUP and GROUP_ID.
Most likely it was a copy-pasta error, which can be fixed
by using GROUP_ID in the IF condition.

Signed-off-by: Michał Żygowski <michal.zygowski@3mdeb.com>
@miczyg1 miczyg1 requested review from macpijan and mkopec August 6, 2024 10:21
@pietrushnic
Copy link
Copy Markdown
Contributor

@miczyg1 I think it may already be in the main thanks to merging my changes, but just let you know there is merge conflict here and we probably could merge it or close.

@miczyg1
Copy link
Copy Markdown
Contributor Author

miczyg1 commented Jun 30, 2025

I see you are creating a user coreboot here: https://github.com/Dasharo/dasharo-sdk/blob/main/Dockerfile#L149

But I am not sure if it will get the same ID and group as the user on the host. So the changes may still actually be valid. I have to think how I can test it. Probably creating a second user on a machine and see if the container invoked by that user does not have permission problems.

@pietrushnic
Copy link
Copy Markdown
Contributor

Good point; I don't know what your initial motivation was, nor how you planned to validate it. Take your time. My question was triggered by bounty and community questions related to rootless Docker.

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