Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

Conversation

@emidoots
Copy link
Member

@emidoots emidoots requested review from davejrt and daxmc99 February 19, 2021 22:40
@sourcegraph-bot
Copy link
Contributor

Notifying subscribers in CODENOTIFY files for diff 5bfeafa...6578b4a.

Notify File(s)
@christinaforney doc/admin/updates/pure_docker.md
@sourcegraph/distribution doc/admin/updates/pure_docker.md

Confirm that `codeinsights-db-disk` has the correct file permissions:

```
sudo chown -R 999:999 ~/sourcegraph-docker/codeinsights-db-disk/
Copy link
Contributor

Choose a reason for hiding this comment

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

Should this directory exist before 3.25?

Copy link
Member Author

Choose a reason for hiding this comment

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

It will be a new directory created after running deploy.sh or manually. The primary customer running this deployment has handled this a few times in the past and knows they'll need to e.g. create that directory with the right permissions or similar.

Copy link
Contributor

@daxmc99 daxmc99 left a comment

Choose a reason for hiding this comment

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

LGTM

@emidoots emidoots merged commit c2caa5d into main Feb 19, 2021
@emidoots emidoots deleted the sg/insights-pure-docker branch February 19, 2021 22:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants