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

Add documentation on repo table row-level security#22746

Merged
flying-robot merged 2 commits intomainfrom
flying-robot/rls-docs
Jul 16, 2021
Merged

Add documentation on repo table row-level security#22746
flying-robot merged 2 commits intomainfrom
flying-robot/rls-docs

Conversation

@flying-robot
Copy link
Copy Markdown
Contributor

@flying-robot flying-robot commented Jul 10, 2021

The repo table is currently protected by a row-level security policy,
which is associated with the sg_service role[0]. This change introduces
admin documentation for understanding the basics of RLS, how we use it
and how to bypass it if necessary.

Rendered Markdown view

Screen Shot 2021-07-10 at 9 43 30 AM

[0] https://github.com/sourcegraph/sourcegraph/pull/22303

@flying-robot flying-robot added team/core-application copy exclusively a copy change labels Jul 10, 2021
@flying-robot flying-robot requested a review from a team July 10, 2021 13:14
@flying-robot flying-robot force-pushed the flying-robot/rls-docs branch 5 times, most recently from 74f130a to ae2866b Compare July 10, 2021 13:52
@unknwon
Copy link
Copy Markdown
Contributor

unknwon commented Jul 15, 2021

Is this ready for review yet?

@flying-robot flying-robot marked this pull request as ready for review July 15, 2021 15:43
@flying-robot
Copy link
Copy Markdown
Contributor Author

@unknwon yeah we can go ahead and review this. I think the release is next week / not far off now.

@sourcegraph-bot
Copy link
Copy Markdown
Contributor

sourcegraph-bot commented Jul 15, 2021

Notifying subscribers in CODENOTIFY files for diff 7eb956e...1ab9975.

Notify File(s)
@christinaforney doc/admin/index.md
doc/admin/repo/index.md
doc/admin/repo/row_level_security.md
@sourcegraph/distribution doc/admin/index.md
doc/admin/repo/index.md
doc/admin/repo/row_level_security.md

Comment thread doc/admin/repo/row_level_security.md Outdated
The repo table is currently protected by a row-level security policy,
which is associated with the sg_service role[0]. This change introduces
admin documentation for understanding the basics of RLS, how we use it
and how to bypass it if necessary.

[0] https://github.com/sourcegraph/sourcegraph/pull/22303
@flying-robot flying-robot force-pushed the flying-robot/rls-docs branch from ae2866b to 1ab9975 Compare July 16, 2021 01:56
Copy link
Copy Markdown
Contributor

@unknwon unknwon left a comment

Choose a reason for hiding this comment

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

@flying-robot flying-robot merged commit d0742a2 into main Jul 16, 2021
@flying-robot flying-robot deleted the flying-robot/rls-docs branch July 16, 2021 12:13
@flying-robot
Copy link
Copy Markdown
Contributor Author

Live now, thanks for the review @unknwon: https://docs.sourcegraph.com/admin/repo/row_level_security

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

Labels

copy exclusively a copy change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants