-
Notifications
You must be signed in to change notification settings - Fork 182
[DOC-13180] Update the Roles page in the Server docs to include query_use_sequences and query_manage_sequence #3875
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
Conversation
| ^| image:introduction/no.png[] | ||
| ^| image:introduction/no.png[] | ||
|
|
||
| ^| App Telemetry |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is no app telemetry for 7.6. it will be introduced in 8.0.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Noted - I'll open a ticket to include it in 8.0 docs and remove it from this PR.
|
Just one comment everything else looks fine. Thanks for taking care of this. |
|
LGTM |
dgnicholson
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @Rebecca-Martinez007 ! 🚀
| The *Use Sequences* role allows users to use sequences within a specific scope. | ||
| Users with this role can execute {sqlpp} NEXTVAL and PREVVAL functions. | ||
| This role does not allow creating or managing sequences. | ||
| It does allow access to Couchbase Server Web Console. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We seem to also use Couchbase Web Console and Couchbase Server Web Console. Also, sometimes we use an article in front and sometimes not. Nothing worth slowing this down, but might be worth a style query. 🤔
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah good catch. Noted - I'll follow up on that.
Update this page to include role mappings query_use_sequences and query_manage_sequence for Server. As per AV-60009: [AUTH] CB 7.6, query_manage_sequences and query_use_sequences roles
Released
this got introduced in 7.6 for Capella.
Currently, the Capella Roles page is updated with the role mapping but was not able to include links to the Server roles page as they were not included yet.
This ticket is for Server to add roles:
Query Manage Sequences
Query Use Sequences
Working with the resources and role privileges shown in the commit here: MB-58036 Add query sequences roles. · couchbase/ns_server@fbedde2
Also found in the menalaus_roles.erl file: ns_server/apps/ns_server/src/menelaus_roles.erl at master · couchbase/ns_server
Images of local build:
