Skip to content

Update managed-instance-link-feature-overview.md - #8095

Merged
Masha Thomas (MSFT) (MashaMSFT) merged 2 commits into
MicrosoftDocs:livefrom
karthikyella:patch-2
Oct 11, 2022
Merged

Update managed-instance-link-feature-overview.md#8095
Masha Thomas (MSFT) (MashaMSFT) merged 2 commits into
MicrosoftDocs:livefrom
karthikyella:patch-2

Conversation

@karthikyella

Copy link
Copy Markdown
Contributor

One cannot deploy a VM to the same subnet as the managed instance. It should be corrected to vnet. Managed instance subnet cannot have any other resource types in its subnet. https://learn.microsoft.com/en-us/azure/azure-sql/managed-instance/connectivity-architecture-overview?view=azuresql#network-requirements

One cannot deploy a VM to the same subnet as the managed instance. It should be corrected to vnet.  Managed instance subnet cannot have any other resource types in its subnet.
https://learn.microsoft.com/en-us/azure/azure-sql/managed-instance/connectivity-architecture-overview?view=azuresql#network-requirements
@prmerger-automator

Copy link
Copy Markdown
Contributor

Karthik Yella (@karthikyella) : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@opbld33

opbld33 commented Sep 19, 2022

Copy link
Copy Markdown

Docs Build status updates of commit 6a2f514:

❌ Validation status: errors

Please follow instructions here which may help to resolve issue.

File Status Preview URL Details
❌Error Details

  • [Error: RuningBuildFailed] Some unexpected errors happened when running build, please open a ticket in https://aka.ms/SiteHelp and include the error report for our team to troubleshoot

For more details, please refer to the build report.

If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

@Jak-MS

Copy link
Copy Markdown
Contributor

Danimir Ljepava (@danimir)

  • Can you review this PR?
  • IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@prmerger-automator prmerger-automator Bot added the aq-pr-triaged tracking label for the PR review team label Sep 19, 2022

@danimir Danimir Ljepava (danimir) left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Jak Koke (@Jak-MS) Please capitalize V and N, therefore it should be "VNet" (not vnet), otherwise the change is fine (you can sign off upon this update). Thank you.

@opbld31

opbld31 commented Sep 20, 2022

Copy link
Copy Markdown

Docs Build status updates of commit 6a2f514:

✅ Validation status: passed

File Status Preview URL Details
azure-sql/managed-instance/managed-instance-link-feature-overview.md ✅Succeeded

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

@opbld30

opbld30 commented Oct 11, 2022

Copy link
Copy Markdown

Learn Build status updates of commit d2015dc:

✅ Validation status: passed

File Status Preview URL Details
azure-sql/managed-instance/managed-instance-link-feature-overview.md ✅Succeeded

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

@MashaMSFT
Masha Thomas (MSFT) (MashaMSFT) merged commit d0772ec into MicrosoftDocs:live Oct 11, 2022
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.

8 participants