Skip to content
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

Clarify default route breaks public comms for vnet #1145

Merged
merged 2 commits into from Apr 12, 2017

Conversation

PedroPerezMSFT
Copy link
Contributor

Just had a customer case and we have no clarification about this. I think it would be good to clarify beforehand and avoid customer pain.

Just had a customer case and we have no clarification about this
@msftclas
Copy link

@PedroPerezMSFT,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@jomolnar
Copy link
Contributor

@cherylmc Hi Cheryl Please review these proposed contributions. Thanks.

@cherylmc
Copy link
Contributor

@yushwang - can you please review?

@yushwang
Copy link
Contributor

Some wording changes required. Let me get to that later today.

@yushwang
Copy link
Contributor

I don't know how to change to the branch and update the PR. So here goes:

"Please note this will force all VNet egress traffic towards your on-premises site, and will prevent the VNet VMs from accepting public communication from the Internet directly, such RDP or SSH from the Internet to the VMs."

@PedroPerezMSFT could you update your PR? Or we can close this one and I can make the update directly.

Thanks,
Yushun

Copy link
Contributor Author

@PedroPerezMSFT PedroPerezMSFT left a comment

Choose a reason for hiding this comment

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

I have changed the text for Yushun's.

Thanks!

@cherylmc
Copy link
Contributor

#sign-off

@jomolnar jomolnar merged commit 7de68b4 into MicrosoftDocs:master Apr 12, 2017
rajumsys pushed a commit to rajumsys/azure-docs that referenced this pull request May 4, 2017
Design updates to all ref sections in landing pages
@PedroPerezMSFT PedroPerezMSFT deleted the patch-2 branch May 22, 2017 13:05
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.

None yet

5 participants