Skip to content

[Draft] Views documentation#6394

Closed
peternied wants to merge 2 commits intoopensearch-project:mainfrom
peternied:views
Closed

[Draft] Views documentation#6394
peternied wants to merge 2 commits intoopensearch-project:mainfrom
peternied:views

Conversation

@peternied
Copy link
Copy Markdown
Member

Description

Adds documentation for the views feature

Issues Resolved

Checklist

  • By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
    For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: Peter Nied <petern@amazon.com>
@stephen-crawford
Copy link
Copy Markdown
Contributor

@peternied maybe I missed it, but could you add a really clear explanation about the interaction between Views and roles? I think we want to make it really clear that we still use the normal permission system behind the scenes otherwise it may seem like Views can skip the normal check or are an alternative permission system.

@peternied
Copy link
Copy Markdown
Member Author

@peternied maybe I missed it, but could you add a really clear explanation about the interaction between Views and roles? I think we want to make it really clear that we still use the normal permission system behind the scenes otherwise it may seem like Views can skip the normal check or are an alternative permission system.

I have documented the feature as it is, as it changes I'll need to revise this PR.

There is no change or loosening to any permissions - you'll need both index permissions as normal in addition to cluster permissions that allow calling the different view APIs

Signed-off-by: Peter Nied <petern@amazon.com>
@peternied
Copy link
Copy Markdown
Member Author

I'm going to close this out - until there is a clear delivery data associated with this feature it doesn't make sense to clog up the PR queue. I'll reopen if it becomes relevant in the future.

@peternied peternied closed this Apr 25, 2024
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.

[DOC] Views

2 participants