Reviewed-by: Rodrigo Trujillo <rodrigo.trujillo@linux.vnet.ibm.com> On 12/02/2015 04:36 PM, Aline Manera wrote:

The /networks API was updated to allow user creates Linux bridges besides macvtap bridges.

But as the UI needs to be updated accordingly to add this option, Kimchi 2.0 will continue to use macvtap bridge as default.

This patch also fixes the issue while creating a brigded VLAN tagging.

Signed-off-by: Aline Manera <alinefm@linux.vnet.ibm.com> --- ui/js/src/kimchi.network.js | 2 +- ui/js/src/kimchi.network_add_main.js | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/ui/js/src/kimchi.network.js b/ui/js/src/kimchi.network.js index 539010e..c1c1ebd 100644 --- a/ui/js/src/kimchi.network.js +++ b/ui/js/src/kimchi.network.js @@ -35,7 +35,7 @@ kimchi.initNetworkListView = function() { in_use : data[i].in_use, state : data[i].state === "active" ? "up" : "down" }; - if (data[i].connection === "bridge") { + if (data[i].connection === "macvtap") { network.type = kimchi.NETWORK_TYPE_BRIDGE; } else { network.type = data[i].connection; diff --git a/ui/js/src/kimchi.network_add_main.js b/ui/js/src/kimchi.network_add_main.js index ea69421..c28c9ce 100644 --- a/ui/js/src/kimchi.network_add_main.js +++ b/ui/js/src/kimchi.network_add_main.js @@ -34,7 +34,7 @@ kimchi.startNetworkCreation = function() { connection: network.type }; if (network.type === kimchi.NETWORK_TYPE_BRIDGE) { - data.connection = "bridge"; + data.connection = "macvtap"; data.interface = network.interface; if ($("#enableVlan").prop("checked")) { data.vlan_id = network.vlan_id;