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:
us...@infra.apache.org
