[GitHub] [cloudstack-documentation] GabrielBrascher commented on a change in pull request #194: clearification of the advanced network setup in section "Host KVM installation"

[GitBox <gi...@apache.org>]

GabrielBrascher commented on a change in pull request #194:
URL: https://github.com/apache/cloudstack-documentation/pull/194#discussion_r588507443



##########
File path: source/installguide/hypervisor/kvm.rst
##########
@@ -645,11 +645,10 @@ although a reboot is recommended to see if everything works properly.
 Network Example for Advanced Networks
 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
 
-In the Advanced networking mode is most common to have (at least) two physical interfaces.
-In this example we will again have the hypervisor management interface on cloudbr0 on the 
-untagged (native) VLAN. But now we will have a bridge on top of our additional interface (eth1)
-for public and guest traffic with no VLANs applied by us - CloudStack will add the VLANs
-as required.
+In the Advanced networking mode is most common to have (at least) two physical interfaces per hypervior-host.
+We will use the interface eth0 linked to the bridge 'cloudbr0' using the untagged (native) VLAN for hypervisor management.
+Additionally we configure a the second interface for usage with the bridge 'cloudbr1' for public and guest traffic.

Review comment:
       `Additionally we configure a the second interface`
   :arrow_down: I would suggest removing `a`, but you might remove `the` instead, if you prefer
   `Additionally, we configure the second interface`




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org