1
0
mirror of https://github.com/samba-team/samba.git synced 2024-12-22 13:34:15 +03:00

ctdb-scripts: Support monitoring of interestingly named VLANs on bonds

VLAN interfaces on bonds with a name other than <iface>.<id>@<iface>
are not currently supported.  That is, where the VLAN name isn't based
on the underlying bond name.  Such VLAN interfaces can be created with
the "ip link" command, as opposed to the "vconfig" command, or by
renaming a VLAN interface.

This is improved by determining the underlying interface name for a
VLAN from the output of "ip link".

No serious attempt is made to support VLANs with '@' in their name,
although this seems to be legal.  Why would you do that?

Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
This commit is contained in:
Martin Schwenke 2015-07-08 21:23:48 +10:00 committed by Amitay Isaacs
parent 87c5c96b76
commit bc71251433

View File

@ -52,6 +52,30 @@ get_all_interfaces ()
all_interfaces=$(echo $all_interfaces $ctdb_ifaces | tr ' ' '\n' | sort -u) all_interfaces=$(echo $all_interfaces $ctdb_ifaces | tr ' ' '\n' | sort -u)
} }
get_real_iface ()
{
# Output of "ip link show <iface>"
_iface_info="$1"
# Extract the full interface description to see if it is a VLAN
_t=$(echo "$_iface_info" |
awk 'NR == 1 { iface = $2; sub(":$", "", iface) ; \
print iface }')
case "$_t" in
*@*)
# VLAN: use the underlying interface, after the '@'
echo "${_t##*@}"
;;
*)
# Not a regular VLAN. For backward compatibility, assume
# there is some other sort of VLAN that doesn't have the
# '@' in the output and only use what is before a '.'. If
# there is no '.' then this will be the whole interface
# name.
echo "${_t%%.*}"
esac
}
monitor_interfaces() monitor_interfaces()
{ {
get_all_interfaces get_all_interfaces
@ -65,7 +89,7 @@ monitor_interfaces()
# problem with an interface then set fail=true and continue. # problem with an interface then set fail=true and continue.
for iface in $all_interfaces ; do for iface in $all_interfaces ; do
ip link show $iface 2>/dev/null >/dev/null || { _iface_info=$(ip link show $iface 2>&1) || {
echo "ERROR: Interface $iface does not exist but it is used by public addresses." echo "ERROR: Interface $iface does not exist but it is used by public addresses."
mark_down $iface mark_down $iface
continue continue
@ -74,7 +98,7 @@ monitor_interfaces()
# These interfaces are sometimes bond devices # These interfaces are sometimes bond devices
# When we use VLANs for bond interfaces, there will only # When we use VLANs for bond interfaces, there will only
# be an entry in /proc for the underlying real interface # be an entry in /proc for the underlying real interface
realiface=`echo $iface |sed -e 's/\..*$//'` realiface=$(get_real_iface "$_iface_info")
bi=$(get_proc "net/bonding/$realiface" 2>/dev/null) && { bi=$(get_proc "net/bonding/$realiface" 2>/dev/null) && {
echo "$bi" | grep -q 'Currently Active Slave: None' && { echo "$bi" | grep -q 'Currently Active Slave: None' && {
echo "ERROR: No active slaves for bond device $realiface" echo "ERROR: No active slaves for bond device $realiface"