Skip to content

Conversation

@dwrth
Copy link
Contributor

@dwrth dwrth commented Oct 28, 2025

Description

This PR adds user: root to the docker-compose.yaml file in the sample/ directory.

Validation

After adding the line, the permission error mentioned in the issue is no longer thrown.

Related Issues

Fixes #1284

Check List

  • I confirm that my contribution is made under the terms of the Apache 2.0
    license.
  • I have run pnpm checks to ensure code compiles and meets standards.
  • I have run pnpm test to check if all tests are passing.
  • I have covered new added functionality with unit tests if necessary.
  • I have added an entry in the Changelog.md.

@kmcginnes
Copy link
Collaborator

@dwrth Nice catch. Using the root user in the docker image seems a bit risky to me. I wonder if changing the folder where the sample data is placed might make a difference. Perhaps the gremlin folder is owned by a different user.

Here is an example of how I moved the sample files up to the root of the docker container filesystem. You might also try something like /usr.

@dwrth
Copy link
Contributor Author

dwrth commented Nov 7, 2025

@kmcginnes Sadly that also did not fix the issue for me and I still had to manually change permissions for that directory.

I came up with another potential solution that works straight out of the box but does not require using user: root

Let me know what you think!

main...dwrth:graph-explorer:fix/gremlin-server-permissions

@kmcginnes
Copy link
Collaborator

@kmcginnes Sadly that also did not fix the issue for me and I still had to manually change permissions for that directory.

That's unfortunate.

I came up with another potential solution that works straight out of the box but does not require using user: root

Let me know what you think!

I do like this solution better than the root user solution. I'll check with my team mates to make sure I'm not missing anything, but I think this is a fine approach.

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.

[Bug] Permission error when starting up the air_routes sample database

2 participants