[patch NetworkManager 5/6] cli: add team driver support
- From: Jiri Pirko <jiri resnulli us>
- To: networkmanager-list gnome org
- Cc: fbl redhat com
- Subject: [patch NetworkManager 5/6] cli: add team driver support
- Date: Thu, 11 Jul 2013 17:31:57 +0200
Signed-off-by: Jiri Pirko <jiri resnulli us>
---
cli/src/connections.c | 116 +++++++++++++++++++++++++++++++++++++++++---------
1 file changed, 97 insertions(+), 19 deletions(-)
diff --git a/cli/src/connections.c b/cli/src/connections.c
index 24cb4b5..3e36243 100644
--- a/cli/src/connections.c
+++ b/cli/src/connections.c
@@ -43,6 +43,7 @@
#include <nm-device-olpc-mesh.h>
#include <nm-device-infiniband.h>
#include <nm-device-bond.h>
+#include <nm-device-team.h>
#include <nm-device-bridge.h>
#include <nm-device-vlan.h>
#include <nm-remote-settings.h>
@@ -54,7 +55,7 @@
#include "settings.h"
#include "connections.h"
-/* Activation timeout waiting for bond/bridge slaves (in seconds) */
+/* Activation timeout waiting for bond/team/bridge slaves (in seconds) */
#define BB_SLAVES_TIMEOUT 10
/* Available fields for 'connection show configured' */
@@ -99,6 +100,8 @@ static NmcOutputField nmc_fields_settings_names[] = {
SETTING_FIELD (NM_SETTING_ADSL_SETTING_NAME, 0), /* 19 */
SETTING_FIELD (NM_SETTING_BRIDGE_SETTING_NAME, 0), /* 20 */
SETTING_FIELD (NM_SETTING_BRIDGE_PORT_SETTING_NAME, 0), /* 21 */
+ SETTING_FIELD (NM_SETTING_TEAM_SETTING_NAME, 0), /* 22 */
+ SETTING_FIELD (NM_SETTING_TEAM_PORT_SETTING_NAME, 0), /* 23 */
{NULL, NULL, 0, NULL, FALSE, FALSE, 0}
};
#define NMC_FIELDS_SETTINGS_NAMES_ALL_X NM_SETTING_CONNECTION_SETTING_NAME","\
@@ -121,7 +124,9 @@ static NmcOutputField nmc_fields_settings_names[] = {
NM_SETTING_BOND_SETTING_NAME","\
NM_SETTING_VLAN_SETTING_NAME","\
NM_SETTING_BRIDGE_SETTING_NAME","\
- NM_SETTING_BRIDGE_PORT_SETTING_NAME
+ NM_SETTING_BRIDGE_PORT_SETTING_NAME","\
+ NM_SETTING_TEAM_SETTING_NAME","\
+ NM_SETTING_TEAM_PORT_SETTING_NAME
#if WITH_WIMAX
#define NMC_FIELDS_SETTINGS_NAMES_ALL NMC_FIELDS_SETTINGS_NAMES_ALL_X","\
NM_SETTING_WIMAX_SETTING_NAME
@@ -257,6 +262,9 @@ usage_connection_add (void)
" [arp_interval <num>]\n"
" [arp_ip_traget <num>]\n\n"
" bond-slave: master <master (ifname or connection UUID)\n\n"
+ " team: [config <json config>]\n\n"
+ " team-slave: master <master (ifname or connection UUID)\n"
+ " [config <json config>]\n\n"
" bridge: [stp yes|no>]\n"
" [priority <num>]\n"
" [forward-delay <2-30>]\n"
@@ -1383,7 +1391,7 @@ typedef struct {
} ActivateConnectionInfo;
static gboolean
-bond_bridge_slaves_check (gpointer user_data)
+master_iface_slaves_check (gpointer user_data)
{
ActivateConnectionInfo *info = (ActivateConnectionInfo *) user_data;
NmCli *nmc = info->nmc;
@@ -1393,6 +1401,8 @@ bond_bridge_slaves_check (gpointer user_data)
if (strcmp (con_type, NM_SETTING_BOND_SETTING_NAME) == 0)
slaves = nm_device_bond_get_slaves (NM_DEVICE_BOND (device));
+ else if (strcmp (con_type, NM_SETTING_TEAM_SETTING_NAME) == 0)
+ slaves = nm_device_team_get_slaves (NM_DEVICE_TEAM (device));
else if (strcmp (con_type, NM_SETTING_BRIDGE_SETTING_NAME) == 0)
slaves = nm_device_bridge_get_slaves (NM_DEVICE_BRIDGE (device));
else
@@ -1464,12 +1474,13 @@ activate_connection_cb (NMClient *client, NMActiveConnection *active, GError *er
/* Start timer not to loop forever when signals are not emitted */
g_timeout_add_seconds (nmc->timeout, timeout_cb, nmc);
- /* Check for bond or bridge slaves */
+ /* Check for bond or team or bridge slaves */
if ( !strcmp (info->con_type, NM_SETTING_BOND_SETTING_NAME)
+ || !strcmp (info->con_type, NM_SETTING_TEAM_SETTING_NAME)
|| !strcmp (info->con_type, NM_SETTING_BRIDGE_SETTING_NAME)) {
- g_timeout_add_seconds (BB_SLAVES_TIMEOUT, bond_bridge_slaves_check, info);
- return; /* info will be freed in bond_bridge_slaves_check () */
+ g_timeout_add_seconds (BB_SLAVES_TIMEOUT, master_iface_slaves_check, info);
+ return; /* info will be freed in master_iface_slaves_check () */
}
}
}
@@ -1590,6 +1601,7 @@ do_connection_up (NmCli *nmc, int argc, char **argv)
con_type = nm_setting_connection_get_connection_type (s_con);
if ( nm_connection_is_type (connection, NM_SETTING_BOND_SETTING_NAME)
+ || nm_connection_is_type (connection, NM_SETTING_TEAM_SETTING_NAME)
|| nm_connection_is_type (connection, NM_SETTING_VLAN_SETTING_NAME)
|| nm_connection_is_type (connection, NM_SETTING_BRIDGE_SETTING_NAME))
is_virtual = TRUE;
@@ -1730,8 +1742,10 @@ static const NameItem nmc_valid_connection_types[] = {
{ NM_SETTING_OLPC_MESH_SETTING_NAME, "olpc-mesh" },
{ NM_SETTING_VLAN_SETTING_NAME, NULL },
{ NM_SETTING_BOND_SETTING_NAME, NULL },
+ { NM_SETTING_TEAM_SETTING_NAME, NULL },
{ NM_SETTING_BRIDGE_SETTING_NAME, NULL },
{ "bond-slave", NULL },
+ { "team-slave", NULL },
{ "bridge-slave", NULL },
{ NULL, NULL }
};
@@ -1869,20 +1883,18 @@ check_and_convert_mac (const char *mac,
}
static char *
-unique_bond_bridge_ifname (GSList *list, const char *type, const char *try_name)
+unique_master_iface_ifname (GSList *list,
+ const char *type,
+ const char *ifname_property,
+ const char *try_name)
{
NMConnection *connection;
NMSetting *setting;
char *new_name;
unsigned int num = 1;
GSList *iterator = list;
- const char *ifname_property;
char *ifname_val = NULL;
- ifname_property = strcmp (type, NM_SETTING_BOND_SETTING_NAME) == 0 ?
- NM_SETTING_BOND_INTERFACE_NAME :
- NM_SETTING_BRIDGE_INTERFACE_NAME;
-
new_name = g_strdup (try_name);
while (iterator) {
connection = NM_CONNECTION (iterator->data);
@@ -1946,6 +1958,7 @@ complete_connection_by_type (NMConnection *connection,
NMSettingBluetooth *s_bt;
NMSettingVlan *s_vlan;
NMSettingBond *s_bond;
+ NMSettingTeam *s_team;
NMSettingBridge *s_bridge;
NMSettingBridgePort *s_bridge_port;
NMSettingVPN *s_vpn;
@@ -2457,9 +2470,10 @@ cleanup_vlan:
/* Use connection's ifname as 'bond' ifname if exists, else generate one */
ifname = nm_setting_connection_get_interface_name (s_con);
if (!ifname)
- bond_ifname = unique_bond_bridge_ifname (all_connections,
- NM_SETTING_BOND_SETTING_NAME,
- "nm-bond");
+ bond_ifname = unique_master_iface_ifname (all_connections,
+ NM_SETTING_BOND_SETTING_NAME,
+ NM_SETTING_BOND_INTERFACE_NAME,
+ "nm-bond");
else
bond_ifname = g_strdup (ifname);
@@ -2530,6 +2544,67 @@ cleanup_vlan:
g_free (master_ask);
+ } else if (!strcmp (con_type, NM_SETTING_TEAM_SETTING_NAME)) {
+ /* Build up the settings required for 'team' */
+ char *team_ifname = NULL;
+ const char *ifname = NULL;
+
+ /* Use connection's ifname as 'team' ifname if exists, else generate one */
+ ifname = nm_setting_connection_get_interface_name (s_con);
+ if (!ifname)
+ team_ifname = unique_master_iface_ifname (all_connections,
+ NM_SETTING_TEAM_SETTING_NAME,
+ NM_SETTING_TEAM_INTERFACE_NAME,
+ "nm-team");
+ else
+ team_ifname = g_strdup (ifname);
+
+ /* Add 'team' setting */
+ s_team = (NMSettingTeam *) nm_setting_team_new ();
+ nm_connection_add_setting (connection, NM_SETTING (s_team));
+
+ /* Set team options */
+ g_object_set (s_team, NM_SETTING_TEAM_INTERFACE_NAME, team_ifname, NULL);
+
+ g_free (team_ifname);
+
+ } else if (!strcmp (con_type, "team-slave")) {
+ /* Build up the settings required for 'team-slave' */
+ const char *master = NULL;
+ char *master_ask = NULL;
+ const char *type = NULL;
+ nmc_arg_t exp_args[] = { {"master", TRUE, &master, !ask},
+ {"type", TRUE, &type, FALSE},
+ {NULL} };
+
+ if (!nmc_parse_args (exp_args, TRUE, &argc, &argv, error))
+ return FALSE;
+
+ if (!master && ask)
+ master = master_ask = nmc_get_user_input (_("Team master: "));
+ if (!master) {
+ g_set_error_literal (error, NMCLI_ERROR, NMC_RESULT_ERROR_USER_INPUT,
+ _("Error: 'master' is required."));
+ return FALSE;
+ }
+
+ if (type)
+ printf (_("Warning: 'type' is currently ignored. "
+ "We only support ethernet slaves for now.\n"));
+
+ /* Change properties in 'connection' setting */
+ g_object_set (s_con,
+ NM_SETTING_CONNECTION_TYPE, NM_SETTING_WIRED_SETTING_NAME,
+ NM_SETTING_CONNECTION_MASTER, master,
+ NM_SETTING_CONNECTION_SLAVE_TYPE, NM_SETTING_TEAM_SETTING_NAME,
+ NULL);
+
+ /* Add ethernet setting */
+ s_wired = (NMSettingWired *) nm_setting_wired_new ();
+ nm_connection_add_setting (connection, NM_SETTING (s_wired));
+
+ g_free (master_ask);
+
} else if (!strcmp (con_type, NM_SETTING_BRIDGE_SETTING_NAME)) {
/* Build up the settings required for 'bridge' */
gboolean success = FALSE;
@@ -2558,9 +2633,10 @@ cleanup_vlan:
/* Use connection's ifname as 'bridge' ifname if exists, else generate one */
ifname = nm_setting_connection_get_interface_name (s_con);
if (!ifname)
- bridge_ifname = unique_bond_bridge_ifname (all_connections,
- NM_SETTING_BRIDGE_SETTING_NAME,
- "nm-bridge");
+ bridge_ifname = unique_master_iface_ifname (all_connections,
+ NM_SETTING_BRIDGE_SETTING_NAME,
+ NM_SETTING_BRIDGE_INTERFACE_NAME,
+ "nm-bridge");
else
bridge_ifname = g_strdup (ifname);
@@ -2817,6 +2893,7 @@ cleanup_bridge_slave:
/* Read and add IP configuration */
if ( strcmp (con_type, "bond-slave") != 0
+ && strcmp (con_type, "team-slave") != 0
&& strcmp (con_type, "bridge-slave") != 0) {
NMSettingIP4Config *s_ip4 = NULL;
@@ -3007,8 +3084,9 @@ do_connection_add (NmCli *nmc, int argc, char **argv)
}
}
- /* ifname is mandatory for all connection types except virtual ones (bond, bridge, vlan) */
+ /* ifname is mandatory for all connection types except virtual ones (bond, team, bridge, vlan) */
if ( strcmp (type, NM_SETTING_BOND_SETTING_NAME) == 0
+ || strcmp (type, NM_SETTING_TEAM_SETTING_NAME) == 0
|| strcmp (type, NM_SETTING_BRIDGE_SETTING_NAME) == 0
|| strcmp (type, NM_SETTING_VLAN_SETTING_NAME) == 0)
ifname_mandatory = FALSE;
--
1.8.1.4
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]
