Skip to content

Conversation

ArmandoLacerda
Copy link
Contributor

Because the Microsoft.Web/serverFarms resource provider shows on the portal even if the service is not registered for the subscription, it might become difficult to troubleshoot. The error message when it is not registered says the setup was not able to set the delegation, instead of saying the resource is not registered. Most of my customers assume it is a permission issue, not a service registration issue.

Because the Microsoft.Web/serverFarms resource provider shows on the portal even if the service is not registered for the subscription, it might become difficult to troubleshoot. The error message when it is not registered says the setup was not able to set the delegation, instead of saying the resource is not registered. Most of my customers assume it is a permission issue, not a service registration issue.
@Copilot Copilot AI review requested due to automatic review settings August 26, 2025 12:34
Copy link
Contributor

@ArmandoLacerda : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change.

Copy link
Contributor

@Copilot 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 documentation to clarify subnet delegation requirements for API Management VNet outbound integration. The change addresses a common troubleshooting confusion where users see the Microsoft.Web/serverFarms resource provider in the portal but encounter delegation errors when the service isn't properly registered.

  • Clarifies that the Microsoft.Web/serverFarms resource provider must be registered even if visible in the portal

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@@ -62,7 +62,7 @@ The subnet needs to be delegated to the **Microsoft.Web/serverFarms** service.


> [!NOTE]
> You might need to register the `Microsoft.Web/serverFarms` resource provider in the subscription so that you can delegate the subnet to the service.
> You might need to register the `Microsoft.Web/serverFarms` resource provider in the subscription so that you can delegate the subnet to the service, even if you see it on the list of available services in the subnet delegation setup in the portal.
Copy link
Preview

Copilot AI Aug 26, 2025

Choose a reason for hiding this comment

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

[nitpick] The sentence structure could be improved for clarity. Consider: 'You might need to register the Microsoft.Web/serverFarms resource provider in the subscription to delegate the subnet to the service, even if you see it listed as an available service in the portal's subnet delegation setup.'

Suggested change
> You might need to register the `Microsoft.Web/serverFarms` resource provider in the subscription so that you can delegate the subnet to the service, even if you see it on the list of available services in the subnet delegation setup in the portal.
> You might need to register the `Microsoft.Web/serverFarms` resource provider in the subscription to delegate the subnet to the service, even if you see it listed as an available service in the portal's subnet delegation setup.

Copilot uses AI. Check for mistakes.

Copy link
Contributor

Learn Build status updates of commit 2e19ffb:

✅ Validation status: passed

File Status Preview URL Details
articles/api-management/integrate-vnet-outbound.md ✅Succeeded

For more details, please refer to the build report.

@v-dirichards
Copy link
Contributor

@dlepow

Can you review the proposed changes?

Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.

#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 Aug 26, 2025
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.

3 participants