Skip to content

update vault mssql docs #2267

Closed
sachin-chand01 wants to merge 30 commits intovault/202605from
VAULT-43670-update-mssql-docs
Closed

update vault mssql docs #2267
sachin-chand01 wants to merge 30 commits intovault/202605from
VAULT-43670-update-mssql-docs

Conversation

@sachin-chand01
Copy link
Copy Markdown

PR Description

Summary

Updates the permissions for the MSSQL docs for this bug fix

MSSql Server database plugin throws "Logins other than the current user can only be seen by members of the sysadmin role." when using the default revocation statement.

Please go to the Preview tab and select the appropriate template:

Terraform

@sachin-chand01 sachin-chand01 requested a review from a team as a code owner April 17, 2026 10:18
@github-actions github-actions Bot added the Vault Content update for Vault product docs label Apr 17, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 17, 2026

Vercel Previews Deployed

Name Status Preview Updated (UTC)
Dev Portal ✅ Ready (Inspect) Visit Preview Tue May 5 11:57:03 UTC 2026
Unified Docs API ✅ Ready (Inspect) Visit Preview Tue May 5 11:53:02 UTC 2026

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 17, 2026

Broken Link Checker

This PR contains broken links, but won't be blocked. Use this report to improve content quality:

Quick Actions

  • Internal links (HashiCorp sites): Please fix these - they impact user experience
  • External links: Consider if these are essential or can be updated/removed
  • Temporary issues: External sites may recover - check again before merging

Need Help?


Internal Links

Full Github Actions output

External Links

Summary

Status Count
🔍 Total 36
✅ Successful 10
⏳ Timeouts 0
🔀 Redirected 5
👻 Excluded 21
❓ Unknown 0
🚫 Errors 0
⛔ Unsupported 0

Redirects per input

Redirects in content/terraform-enterprise/2.0.x/docs/enterprise/releases/2.0.x/index.mdx

Redirects in content/vault/v1.19.x/content/docs/secrets/databases/mssql.mdx

Redirects in content/vault/v1.20.x/content/docs/secrets/databases/mssql.mdx

Redirects in content/vault/v1.21.x/content/docs/secrets/databases/mssql.mdx

Redirects in content/vault/v2.x/content/docs/secrets/databases/mssql.mdx

Full Github Actions output

@schavis
Copy link
Copy Markdown
Contributor

schavis commented Apr 17, 2026

@sachin-chand01 Is this fix part of 2.0.0?

@schavis
Copy link
Copy Markdown
Contributor

schavis commented Apr 27, 2026

@sachin-chand01 Is the bug fix part of the May release? If so, please change the merge target to vault/202505 and make sure to add an entry to the 2.x release notes

@sachin-chand01
Copy link
Copy Markdown
Author

sachin-chand01 commented Apr 28, 2026

@sachin-chand01 Is the bug fix part of the May release? If so, please change the merge target to vault/202505 and make sure to add an entry to the 2.x release notes

@schavis No, this fix is not part of May release. It’s planned for the 2.1.0 release.

sachin-chand01 and others added 4 commits April 28, 2026 10:34
This is an automated pull request to sync changes between the public and
private unified docs repos.

To preserve continuity across repos, _do not squash_ this pull request.
@schavis
Copy link
Copy Markdown
Contributor

schavis commented Apr 28, 2026

@schavis No, this fix is not part of May release. It’s planned for the 2.1.0 release.

So it's part of the October release? I didn't realize we needed a release branch for that already.

Let me create one and you can change the merge target. To clarify: the content repo doesn't work like the product repo. Merging something to main publishes those changes to the public docs immediately.

@sachin-chand01
Copy link
Copy Markdown
Author

So it's part of the October release? I didn't realize we needed a release branch for that already.

Let me create one and you can change the merge target. To clarify: the content repo doesn't work like the product repo. Merging something to main publishes those changes to the public docs immediately.

Got it, that makes sense! Thanks for catching that. I’ll switch the merge target once the release branch is ready.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the MSSQL database secrets engine documentation to reflect additional permissions required by the default revocation behavior and to clarify revocation behavior constraints.

Changes:

  • Add additional SQL Server permissions (VIEW ANY DEFINITION, VIEW SERVER STATE, VIEW DEFINITION) to the “minimum functionality” grant set.
  • Add a warning that the default revocation procedure’s metadata query is scoped to the database in connection_url, which can prevent cleanup when users are created in a different database.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
content/vault/v2.x/content/docs/secrets/databases/mssql.mdx Adds required grants and clarifies default revocation scoping behavior.
content/vault/v1.21.x/content/docs/secrets/databases/mssql.mdx Same as v2.x, but contains a duplicated-phrase typo in the newly added warning text.
content/vault/v1.19.x/content/docs/secrets/databases/mssql.mdx Adds required grants and clarifies default revocation scoping behavior.

@stuti-sr stuti-sr self-requested a review April 29, 2026 04:30
Copy link
Copy Markdown
Contributor

@stuti-sr stuti-sr left a comment

Choose a reason for hiding this comment

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

LGTM!!

@schavis
Copy link
Copy Markdown
Contributor

schavis commented May 1, 2026

@sachin-chand01 Please change the merge target to vault/202610

@hashicorp hashicorp deleted a comment from Copilot AI May 1, 2026
Comment thread content/vault/v1.21.x/content/docs/secrets/databases/mssql.mdx Outdated
Comment thread content/vault/v2.x/content/docs/secrets/databases/mssql.mdx Outdated
sachin-chand01 and others added 2 commits May 5, 2026 16:48
Co-authored-by: Sarah Chavis <62406755+schavis@users.noreply.github.com>
@sachin-chand01 sachin-chand01 changed the base branch from main to vault/202605 May 5, 2026 11:39
@sachin-chand01 sachin-chand01 requested review from a team as code owners May 5, 2026 11:39
@sachin-chand01
Copy link
Copy Markdown
Author

@schavis quick update, this fix will go into the May release. Apologies for the change.

Closing this PR due to unnecessary changes after the merge target update and have opened a new PR : #2374

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

Labels

do not merge Vault Content update for Vault product docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants