-
Notifications
You must be signed in to change notification settings - Fork 4k
Adding firewall prerequisite information #13385
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
Adding firewall prerequisite information #13385
Conversation
While trying to follow the example in this page I was getting GatewayTimeout errors. After updating the firewall to allow Azure services the examples work fine. This is important information to the success of the reader executing the example code. As this is essential information required for user success it has been highlighted as Important.
Thank you for your contribution johnmart82! We will review the pull request and get back to you soon. |
Thanks for your contribution again @johnmart82 ! |
@mikefrobbins Hello Mike could you help Sql team to review this PR? Thanks in advance 😀 |
/azp run |
Azure Pipelines successfully started running 3 pipeline(s). |
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
/azp run |
Azure Pipelines successfully started running 3 pipeline(s). |
/azp run azure-powershell - windows-powershell |
Azure Pipelines successfully started running 1 pipeline(s). |
While trying to follow the example in this page I was getting GatewayTimeout errors. After updating the firewall to allow Azure services the examples work fine. This is important information to the success of the reader executing the example code. As this is essential information required for user success it has been highlighted as Important.
Description
After trying to get the example code to work I tested with Allow azure services option enabled on the firewall. As this is important for the readers success when testing I have added it as Important information.
Checklist
CONTRIBUTING.md
ChangeLog.md
file(s) has been updated:ChangeLog.md
file can be found atsrc/{{SERVICE}}/{{SERVICE}}/ChangeLog.md
## Upcoming Release
header -- no new version header should be added