diff options
| author | zsdc <taras@vyos.io> | 2022-03-15 18:29:12 +0200 |
|---|---|---|
| committer | zsdc <taras@vyos.io> | 2022-03-15 18:39:05 +0200 |
| commit | df4b544c29974e36b52fc42bcbf617f50738a4a6 (patch) | |
| tree | 26793dbb950da6cdc24a2a0220a90fc73a62d1f7 /src/conf_mode/interfaces-bonding.py | |
| parent | 4924a82cbdc74c3bdaed60640ba805cc842f7f9c (diff) | |
| download | vyos-1x-df4b544c29974e36b52fc42bcbf617f50738a4a6.tar.gz vyos-1x-df4b544c29974e36b52fc42bcbf617f50738a4a6.zip | |
bonding: T4301: Fixed arp-monitor option
In verify function for arp-monitor option was used by mistake an extra
conversion for incoming data before comparing items. This commit removed
these unnecessary conversions and makes the option operable.
Diffstat (limited to 'src/conf_mode/interfaces-bonding.py')
| -rwxr-xr-x | src/conf_mode/interfaces-bonding.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/conf_mode/interfaces-bonding.py b/src/conf_mode/interfaces-bonding.py index 431d65f1f..d5be21949 100755 --- a/src/conf_mode/interfaces-bonding.py +++ b/src/conf_mode/interfaces-bonding.py @@ -132,10 +132,10 @@ def verify(bond): return None if 'arp_monitor' in bond: - if 'target' in bond['arp_monitor'] and len(int(bond['arp_monitor']['target'])) > 16: + if 'target' in bond['arp_monitor'] and len(bond['arp_monitor']['target']) > 16: raise ConfigError('The maximum number of arp-monitor targets is 16') - if 'interval' in bond['arp_monitor'] and len(int(bond['arp_monitor']['interval'])) > 0: + if 'interval' in bond['arp_monitor'] and int(bond['arp_monitor']['interval']) > 0: if bond['mode'] in ['802.3ad', 'balance-tlb', 'balance-alb']: raise ConfigError('ARP link monitoring does not work for mode 802.3ad, ' \ 'transmit-load-balance or adaptive-load-balance') |