Skip to content

Conversation

danbosscher
Copy link

Offers service tags as an alternative. The amount of IP ranges added by the script exceed what's allowed in AKS (200), but Service Tags work without issue.

Offers service tags as an alternative. The amount of IP ranges added by the script exceed what's allowed in AKS (200).
Copy link
Contributor

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

@v-dirichards
Copy link
Contributor

@mijacobs

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"
#assign: @mijacobs
@MicrosoftDocs/public-repo-pr-review-team

@v-dirichards
Copy link
Contributor

@mijacobs Could you review this proposed update to your article and enter #sign-off in a comment if it's ready to merge?

Thanks!

@@ -259,8 +259,19 @@ namespace WeeklyFileIPRanges
```

### Service tags

Microsoft-hosted agents can't be listed by service tags. If you're trying to grant hosted agents access to your resources, you'll need to follow the IP range allow listing method.
Where supported, customers can use the AzureCloud [service tag](https://learn.microsoft.com/en-us/azure/virtual-network/service-tags-overview) instead. To limit this to the United States Geography, you would use the following tags:

Choose a reason for hiding this comment

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

Suggested change
Where supported, customers can use the AzureCloud [service tag](https://learn.microsoft.com/en-us/azure/virtual-network/service-tags-overview) instead. To limit this to the United States Geography, you would use the following tags:
Where supported, customers can use the AzureCloud [service tag](/azure/virtual-network/service-tags-overview) instead. To limit this to the United States Geography, you would use the following tags:

Always use relative path to avoid enforcing the content language.

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.

5 participants