docs-website icon indicating copy to clipboard operation
docs-website copied to clipboard

Adding instructions for using a Group Managed Service Account.

Open phillipsj opened this issue 1 year ago • 4 comments
trafficstars

Currently, the documentation doesn't document how to use a gMSA to to connect to SQL Server. This attempts to add some basic guidance on how to configure it to work.

Please follow conventional commit standards in your commit messages and pull request title.

Give us some context

  • What problems does this PR solve?
  • Add any context that will help us review your changes such as testing notes, links to related docs, screenshots, etc.
  • If your issue relates to an existing GitHub issue, please link to it.

phillipsj avatar Jun 18 '24 01:06 phillipsj

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Jun 18 '24 01:06 CLAassistant

Hi @phillipsj 👋

Thanks for your pull request! Your PR is in a queue, and a writer will take a look soon. We generally publish small edits within one business day, and larger edits within three days.

We will automatically generate a preview of your request, and will comment with a link when the preview is ready (usually 10 to 20 minutes).

github-actions[bot] avatar Jun 18 '24 01:06 github-actions[bot]

Deploy Preview for docs-website-netlify ready!

Name Link
Latest commit e2b4a6a919afc24060b738bc77c75e4d439cf3e2
Latest deploy log https://app.netlify.com/sites/docs-website-netlify/deploys/6679a522c9f1290008974fcf
Deploy Preview https://deploy-preview-17681--docs-website-netlify.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

netlify[bot] avatar Jun 18 '24 01:06 netlify[bot]

Thanks so much for your contribution @phillipsj! I left some follow up comments to clarify the instructions, otherwise this is amazing stuff!

akristen avatar Jun 18 '24 14:06 akristen

@akristen Insee you closed this without it being merged. I did the updates requested, was there something else I needed to do?

phillipsj avatar Sep 23 '24 17:09 phillipsj

@phillipsj I started a new PR with these changes so that the branches were up to date without any major merge conflicts. Thought I'd be able to c/p the text even with the PR closed, but I jumped the gun 😅 I'll reply with the new PR here once I have your suggestions staged 🫡

akristen avatar Sep 23 '24 17:09 akristen

Since this PR is quite old, I've remade it over in this PR so it's more in sync with our most recent develop branch: https://github.com/newrelic/docs-website/pull/18745

akristen avatar Sep 23 '24 18:09 akristen