Skip to content

Conversation

@dliappis
Copy link
Contributor

@dliappis dliappis commented Apr 13, 2017

Elasticsearch runs as user elasticsearch with uid:gid 1000:1000 inside
the Docker container. Clarify that bind mounted local directories or files
need to be accessible by this user.

Closes #24058

Elasticsearch runs as user elasticsearch with uid:gid 1000:1000 inside
the Docker container. Clarify that bind mounted local directories need
to be accessible by this user.
@dliappis dliappis requested review from jasontedor and joshuar April 13, 2017 16:54
@dliappis dliappis added the >docs General docs changes label Apr 13, 2017
Copy link
Member

@jasontedor jasontedor left a comment

Choose a reason for hiding this comment

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

LGTM.

@jasontedor
Copy link
Member

Can you add to the PR description "Closes #24058"?

@dliappis
Copy link
Contributor Author

Can you add to the PR description "Closes #24058"?

@jasontedor Yes I was planning to add that line at the bottom of the squash+merge message.

@jasontedor
Copy link
Member

jasontedor commented Apr 13, 2017

Yes I was planning to add that line at the bottom of the squash+merge message.

Ah, okay. So we normally refer to the PR in the commit message, and add the "Closes #24058" (issue number) in the PR description. This gives traceability back to the PR from the commit message, and GitHub will automatically close the issue when the PR is merged.

@dliappis
Copy link
Contributor Author

Thanks for the clarifications @jasontedor will do so.

@dliappis dliappis merged commit c0ac50e into elastic:master Apr 19, 2017
dliappis added a commit that referenced this pull request Apr 19, 2017
Elasticsearch runs as user elasticsearch with uid:gid 1000:1000 inside
the Docker container. Clarify that bind mounted local directories need
to be accessible by this user.

Relates #24092
dliappis added a commit that referenced this pull request Apr 19, 2017
Elasticsearch runs as user elasticsearch with uid:gid 1000:1000 inside
the Docker container. Clarify that bind mounted local directories need
to be accessible by this user.

Relates #24092
dliappis added a commit that referenced this pull request Apr 19, 2017
Elasticsearch runs as user elasticsearch with uid:gid 1000:1000 inside
the Docker container. Clarify that bind mounted local directories need
to be accessible by this user.

Relates #24092
dliappis added a commit that referenced this pull request Apr 19, 2017
Elasticsearch runs as user elasticsearch with uid:gid 1000:1000 inside
the Docker container. Clarify that bind mounted local directories need
to be accessible by this user.

Relates #24092
dliappis added a commit that referenced this pull request Apr 19, 2017
Elasticsearch runs as user elasticsearch with uid:gid 1000:1000 inside
the Docker container. Clarify that bind mounted local directories need
to be accessible by this user.

Relates #24092
dliappis added a commit that referenced this pull request Apr 19, 2017
Elasticsearch runs as user elasticsearch with uid:gid 1000:1000 inside
the Docker container. Clarify that bind mounted local directories need
to be accessible by this user.

Relates #24092
@dliappis
Copy link
Contributor Author

Also backported:

Branch Commit ID
5.x 8562092
5.4 543868e
5.3 ee8aa89
5.2 47b1c8f
5.1 087fdd3
5.0 ea10430

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

>docs General docs changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants