-
Notifications
You must be signed in to change notification settings - Fork 84
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Browse files
Browse the repository at this point in the history
* Husam's review for new Network Guidance @fguerri @sblair01 @prasad3017 -bullet #4 at explaining Figure 1 - My question is: Leveraging DNAT and SNAT rules at NSX-T cannot we control if Public IP can be used only for Outbound but not for Inbound? * Husam's review for avs-networking-basics.md Looks great! mainly formatting changes! * Husam review for onprem-connectivity.md Looks great! Thanks @fguerri! I just made few formatting changes. @fguerri might be helpful to add a note describing the difference between ExpressRoute “Private” peering and ExpressRoute “Microsoft” peering to give reader a quick glance on the main difference. * Husam review vnet-connectivity.md In the diagram, it says "Read section: Connectivity with Azure VNet when ER Transit not used"... would probably make sense to actually point to that explicitly in the documentation page. Also, there is no specific paragraph talking about UDR (following the diagram) in that section. @fguerri @prasad3017 @sblair01 * Husam review internet-inbound-connectivity.md added note about Application Gateway and Azure firewalls. + minor formatting changes @fguerri * Husam review internet-outbound-connectivity.md minor formatting changes * Updated a Note in avs-networking-basics.md added a new line "<br>" * Update avs-networking-basics.md * Update readme.md
- Loading branch information
1 parent
f0480ea
commit e98c9a7
Showing
6 changed files
with
99 additions
and
93 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.