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

SecurityContext seems to be breaking bedrock #164

Open
zepiaf opened this issue Jul 12, 2023 · 11 comments
Open

SecurityContext seems to be breaking bedrock #164

zepiaf opened this issue Jul 12, 2023 · 11 comments
Labels
bug Something isn't working

Comments

@zepiaf
Copy link

zepiaf commented Jul 12, 2023

Hi, updating the chart to the latest version seems to be breaking filesystem access within the entrypoint script.

DEBU[0000] Using /data to match uid and gid
DEBU[0000] Resolved UID=0 from match path
DEBU[0000] Resolved GID=0 from match path
Looking up latest version...
/opt/bedrock-entry.sh: line 23: restify.err: Permission denied
/opt/bedrock-entry.sh: line 23: restify.err: Permission denied
/opt/bedrock-entry.sh: line 23: restify.err: Permission denied
Updating permissions
/opt/bedrock-entry.sh: line 161: permissions.json: Permission denied

Am I missing something here ?

Thanks for all the work :)

@itzg
Copy link
Owner

itzg commented Jul 12, 2023

I added a question for @uhthomas on the PR. The group IDs don't seem right now that I'm looking again.

@itzg itzg added the bug Something isn't working label Jul 12, 2023
@itzg
Copy link
Owner

itzg commented Jul 12, 2023

@zepiaf are there any particular values you applied to chart or just going with defaults?

@zepiaf
Copy link
Author

zepiaf commented Jul 12, 2023

Only default, I didn't try to go farther and ended up reverting.

As reverting to previous version is fixing, it's a mild issue :)

@itzg
Copy link
Owner

itzg commented Jul 12, 2023

Odd. When I try the latest chart I get slightly different GID reporting at startup:

DEBU[0000] Using /data to match uid and gid             
DEBU[0000] Resolved UID=0 from match path
DEBU[0000] Resolved GID=2000 from match path
INFO[0000] Skipping uid/gid change since current user is not root

@uhthomas
Copy link
Contributor

Is it just bedrock? I'm able to run the standard Minecraft instance fine with this security context.

As explained in the reply on the PR, the different groups are deliberate and shouldn't cause issues like this.

@zepiaf
Copy link
Author

zepiaf commented Jul 12, 2023

As mentioned, I suspect it is the problem, however I did not push investigation too far, would be good to see if someone else has the problem deploying the chart. Might be a conflict with something else on my cluster. But hey, you know the rule, last PR always get the blame ;)

@zepiaf
Copy link
Author

zepiaf commented Jul 12, 2023

Is it just bedrock? I'm able to run the standard Minecraft instance fine with this security context.

As explained in the reply on the PR, the different groups are deliberate and shouldn't cause issues like this.

I'm only running bedrock, no idea for java

@zepiaf
Copy link
Author

zepiaf commented Jul 12, 2023

Odd. When I try the latest chart I get slightly different GID reporting at startup:


DEBU[0000] Using /data to match uid and gid             

DEBU[0000] Resolved UID=0 from match path

DEBU[0000] Resolved GID=2000 from match path

INFO[0000] Skipping uid/gid change since current user is not root

I'll try to deploy with the stock values later on a new namespace to double check, now that you mention it, it's weird indeed.

@eddedre
Copy link

eddedre commented Jan 6, 2024

I'm having the same issue with stock values. Is there a temp workaround? I noticed that this only happens when using a PVC

@itzg
Copy link
Owner

itzg commented Jan 6, 2024

Perhaps try using the Java image in bedrock mode

https://docker-minecraft-server.readthedocs.io/en/latest/misc/examples/#bedrock-compatible-server

@soekul
Copy link

soekul commented Feb 24, 2024

I encountered this error on a fresh setup w/ a pre-created PV at which I pointed the PVC. In my case I just needed to chown 1000 . on the underlying mount point.

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

No branches or pull requests

5 participants