-
Notifications
You must be signed in to change notification settings - Fork 27
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Solved issue 45: Add optional private endpoints for AI Services and Storage Accounts #55
Solved issue 45: Add optional private endpoints for AI Services and Storage Accounts #55
Conversation
…turn the unique identifier of the created virtual network.
…cture for Blog Storage and File Shares.
…rt when the `use_private_endpoints` variable is set to `true`.
…Account services. This private endpoints are set when the variable `use_private_endpoints` is `true`. Also, the current IP address of the machine executing these templates is used as a allowed IP address to facilitate managing these services.
…ntly near 9 services requires them. - Increased the size of the subnet for the APIM as recommended by Microsoft's documentation. - Also, added delegation in the subnet for the APIM as specified by Microsoft's documentation.
…ge Account module.
…e_account_and_ai_services
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change the private DNS zone names and we'll be read to merge 🎉🎉🎉
…ource in APIM module.
…alize all cognitive service configuration in the same module, especially for the private endpoints and DNS configurations.
Done. Changed as agreed! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This PR would solve issue #45.
All private endpoints are set when the new variable
use_private_endpoints
istrue
. Also, the current IP address of the machine executing these templates is used as an allowed IP address to facilitate managing these services./29
to/28
since currently near 9 services requires them./27
as recommended by Microsoft's documentation.Forms
) module.use_private_endpoints
variable is set totrue
.