Skip to content

Fix blazegraph permission issues#126

Merged
oxiez merged 3 commits into
masterfrom
volume_permissions
Sep 18, 2019
Merged

Fix blazegraph permission issues#126
oxiez merged 3 commits into
masterfrom
volume_permissions

Conversation

@oxiez

@oxiez oxiez commented Sep 18, 2019

Copy link
Copy Markdown
Collaborator

Some work still needs to be done for the initial startup, but redis should no longer overwrite the permissions of the data folder.

@oxiez oxiez requested a review from gordom6 September 18, 2019 21:42
Comment thread docs/docker.md Outdated
#### Prerequisites
* Docker
* `docker-compose`
* A data folder in the same directory as your app and Whyis with permissions 777

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should be /data alongside /apps, so I don't think this is right.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You're right, just checked.

Comment thread docs/docker.md Outdated

### Data directory

By default, the split `docker-compose.yml` will mount a directory named `data` in the same directory as your Whyis folder on the host machine. Due to how Docker allocates volumes, this folder will have the wrong permissions and cause Blazegraph to fail on the first setup. To avoid this, run the following instructions before running `docker-compose`:

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think this is right, either. Look at the relative path.

@minorg

minorg commented Sep 18, 2019

Copy link
Copy Markdown

I'm on my personal user. Please consider this approved. Good work, Owen.

@oxiez oxiez merged commit c212328 into master Sep 18, 2019
@pdwood pdwood deleted the volume_permissions branch October 23, 2019 21:09
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