Skip to content

Add docs for database auto user provisioning#26987

Merged
r0mant merged 7 commits intomasterfrom
roman/docs/dbuser
Jun 2, 2023
Merged

Add docs for database auto user provisioning#26987
r0mant merged 7 commits intomasterfrom
roman/docs/dbuser

Conversation

@r0mant
Copy link
Copy Markdown
Collaborator

@r0mant r0mant commented May 26, 2023

Documentation for #25614.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

This page just moved as-is from the old rbac.mdx.

@ptgott ptgott self-assigned this May 30, 2023
Comment thread docs/pages/database-access/rbac.mdx
Comment thread docs/pages/database-access/rbac/configuring-auto-user-provisioning.mdx Outdated
<Tabs>
<TabItem label="RDS PostgreSQL">
RDS PostgreSQL admin user must have `rds_iam` role attached to allow IAM
authentication as described in the [RDS guide](../guides/rds.mdx#step-56-create-a-database-iam-user).
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The description in the linked guide is very similar to the instructions here, so I'm not sure what a user is supposed to gather from the linked guide vs. this particular tab item. Should we remove the link and include all required information in this tab? The same goes for the other tab item.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I've added a bit more information but I think links to respective guide sections are still useful so I kept them.

Comment thread docs/pages/database-access/rbac/configuring-auto-user-provisioning.mdx Outdated
Comment thread docs/pages/database-access/rbac/configuring-auto-user-provisioning.mdx Outdated
Comment thread docs/pages/database-access/rbac/configuring-auto-user-provisioning.mdx Outdated
Comment thread docs/pages/database-access/rbac/configuring-auto-user-provisioning.mdx Outdated
Comment thread docs/pages/database-access/rbac/configuring-auto-user-provisioning.mdx Outdated
Comment thread docs/pages/database-access/rbac/configuring-auto-user-provisioning.mdx Outdated
@r0mant r0mant force-pushed the roman/docs/dbuser branch from 395f45b to b281b54 Compare May 31, 2023 18:45
@r0mant r0mant requested a review from ptgott May 31, 2023 18:47
@r0mant
Copy link
Copy Markdown
Collaborator Author

r0mant commented May 31, 2023

@ptgott I've addressed your feedback.

Copy link
Copy Markdown
Contributor

@ptgott ptgott left a comment

Choose a reason for hiding this comment

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

Approved with minor suggestions

Comment thread docs/pages/database-access/rbac.mdx Outdated
Comment thread docs/pages/database-access/rbac/configuring-auto-user-provisioning.mdx Outdated
Comment thread docs/pages/database-access/rbac/configuring-auto-user-provisioning.mdx Outdated
Comment thread docs/pages/database-access/rbac/configuring-auto-user-provisioning.mdx Outdated
Comment thread docs/pages/database-access/rbac/configuring-auto-user-provisioning.mdx Outdated
Comment thread docs/pages/database-access/rbac/configuring-auto-user-provisioning.mdx Outdated
Comment thread docs/pages/database-access/rbac/configuring-auto-user-provisioning.mdx Outdated
r0mant and others added 2 commits June 1, 2023 13:35
Co-authored-by: Paul Gottschling <paul.gottschling@goteleport.com>
@r0mant r0mant added this pull request to the merge queue Jun 2, 2023
Merged via the queue into master with commit fe46ac0 Jun 2, 2023
@r0mant r0mant deleted the roman/docs/dbuser branch June 2, 2023 17:39
@public-teleport-github-review-bot
Copy link
Copy Markdown

@r0mant See the table below for backport results.

Branch Result
branch/v13 Create PR

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants