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

[Docs] Add documentation on multiple tenants #103125

Merged
merged 3 commits into from
Jul 1, 2021

Conversation

jportner
Copy link
Contributor

@jportner jportner commented Jun 23, 2021

@jportner jportner added v8.0.0 release_note:skip Skip the PR/issue when compiling release notes docs v7.14.0 v7.13.0 labels Jun 23, 2021
@jportner jportner force-pushed the issue-96101-document-multitenancy branch from d84be6b to e6569c3 Compare June 23, 2021 16:58
@jportner jportner force-pushed the issue-96101-document-multitenancy branch from e6569c3 to d8a7e26 Compare June 23, 2021 17:02
@jportner jportner requested review from grabowskit, legrego and a team June 23, 2021 17:20
@jportner jportner marked this pull request as ready for review June 23, 2021 17:20
Comment on lines 21 to 23
Whichever approach you use, be aware that you need to be careful when granting cluster privileges and index privileges. Both of these
approaches share the same {es} cluster, and {kib} spaces do not prevent you from granting users of two different tenants access to the same
index.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Kibana's feature privileges alone won't grant direct access to any indices. This was the clearest and most concise way I could think of to address @grabowskit's concerns in #96101 (comment), but if anyone has suggestions on how this could be worded differently, please do share!

@jportner jportner added the auto-backport Deprecated - use backport:version if exact versions are needed label Jun 28, 2021
@thomheymann thomheymann self-requested a review July 1, 2021 16:56
Copy link
Contributor

@thomheymann thomheymann left a comment

Choose a reason for hiding this comment

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

Nice one! Reads well to me - clear and concise!

@jportner jportner merged commit c78c350 into elastic:master Jul 1, 2021
@jportner jportner deleted the issue-96101-document-multitenancy branch July 1, 2021 17:07
@kibanamachine
Copy link
Contributor

💔 Backport failed

Status Branch Result
7.13 Commit could not be cherrypicked due to conflicts
7.14 Commit could not be cherrypicked due to conflicts
7.x Commit could not be cherrypicked due to conflicts

To backport manually run:
node scripts/backport --pr 103125

jportner added a commit to jportner/kibana that referenced this pull request Jul 1, 2021
# Conflicts:
#	docs/user/security/authorization/index.asciidoc
jportner added a commit to jportner/kibana that referenced this pull request Jul 1, 2021
# Conflicts:
#	docs/user/security/authorization/index.asciidoc
jportner added a commit that referenced this pull request Jul 1, 2021
# Conflicts:
#	docs/user/security/authorization/index.asciidoc
jportner added a commit that referenced this pull request Jul 1, 2021
# Conflicts:
#	docs/user/security/authorization/index.asciidoc
jportner added a commit that referenced this pull request Jul 1, 2021
# Conflicts:
#	docs/user/security/authorization/index.asciidoc
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport Deprecated - use backport:version if exact versions are needed docs release_note:skip Skip the PR/issue when compiling release notes v7.13.0 v7.14.0 v7.15.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[DOCS] Clarify multitenancy in spaces
5 participants