diff options
| author | Thomas Mangin <thomas.mangin@exa.net.uk> | 2020-03-27 18:10:49 +0000 |
|---|---|---|
| committer | Thomas Mangin <thomas.mangin@exa.net.uk> | 2020-03-28 10:17:31 +0000 |
| commit | 627d1c704c83efe973b16db61005aacb1a96aead (patch) | |
| tree | 8fc2eb43bcaaf31d90985e3736248e958a26b4ce /src/conf_mode/interfaces-bonding.py | |
| parent | 583e9d907236a4a98fe40e97a378c1fb655f8a95 (diff) | |
| download | vyos-1x-627d1c704c83efe973b16db61005aacb1a96aead.tar.gz vyos-1x-627d1c704c83efe973b16db61005aacb1a96aead.zip | |
ifconfig: T2057: explicity name state functions
The Interface get_state/set_state were not clear about
if they edited the admin or operational state.
functions are now using admin_state and oper_state
for clarity.
Diffstat (limited to 'src/conf_mode/interfaces-bonding.py')
| -rwxr-xr-x | src/conf_mode/interfaces-bonding.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/conf_mode/interfaces-bonding.py b/src/conf_mode/interfaces-bonding.py index 03df6e16a..9b6401ab6 100755 --- a/src/conf_mode/interfaces-bonding.py +++ b/src/conf_mode/interfaces-bonding.py @@ -431,7 +431,7 @@ def apply(bond): # Some parameters can not be changed when the bond is up. if bond['shutdown_required']: # Disable bond prior changing of certain properties - b.set_state('down') + b.set_admin_state('down') # The bonding mode can not be changed when there are interfaces enslaved # to this bond, thus we will free all interfaces from the bond first! @@ -449,9 +449,9 @@ def apply(bond): # parameters we will only re-enable the interface if it is not # administratively disabled if not bond['disable']: - b.set_state('up') + b.set_admin_state('up') else: - b.set_state('down') + b.set_admin_state('down') # Configure interface address(es) # - not longer required addresses get removed first |