diff options
author | Christian Poessinger <christian@poessinger.com> | 2020-07-26 11:16:13 +0200 |
---|---|---|
committer | Christian Poessinger <christian@poessinger.com> | 2020-07-26 15:07:27 +0200 |
commit | a156d2f1479affe4e7cfa56785e4d2d61a776cea (patch) | |
tree | 0da415f924fda0e0f29e4ec24dd4dc19fd0966ff /python | |
parent | 6b531e04474d7d976d7cee11e556c3fe3dc2b69f (diff) | |
download | vyos-1x-a156d2f1479affe4e7cfa56785e4d2d61a776cea.tar.gz vyos-1x-a156d2f1479affe4e7cfa56785e4d2d61a776cea.zip |
vxlan: ifconfig: T2653: move to get_interface_dict()
The current VyOS CLI parser code written in Python contains a ton of duplicates
which I can also hold myself accountable for - or maybe mainly me - depends on
the angle of judge.
Diffstat (limited to 'python')
-rw-r--r-- | python/vyos/ifconfig/vxlan.py | 20 |
1 files changed, 10 insertions, 10 deletions
diff --git a/python/vyos/ifconfig/vxlan.py b/python/vyos/ifconfig/vxlan.py index 973b4ef05..0dddab7b7 100644 --- a/python/vyos/ifconfig/vxlan.py +++ b/python/vyos/ifconfig/vxlan.py @@ -47,8 +47,8 @@ class VXLANIf(Interface): 'port': 8472, # The Linux implementation of VXLAN pre-dates # the IANA's selection of a standard destination port 'remote': '', - 'src_address': '', - 'src_interface': '', + 'source_address': '', + 'source_interface': '', 'vni': 0 } definition = { @@ -60,29 +60,29 @@ class VXLANIf(Interface): } } options = Interface.options + \ - ['group', 'remote', 'src_interface', 'port', 'vni', 'src_address'] + ['group', 'remote', 'source_interface', 'port', 'vni', 'source_address'] mapping = { 'ifname': 'add', 'vni': 'id', 'port': 'dstport', - 'src_address': 'local', - 'src_interface': 'dev', + 'source_address': 'local', + 'source_interface': 'dev', } def _create(self): cmdline = ['ifname', 'type', 'vni', 'port'] - if self.config['src_address']: - cmdline.append('src_address') + if self.config['source_address']: + cmdline.append('source_address') if self.config['remote']: cmdline.append('remote') - if self.config['group'] or self.config['src_interface']: - if self.config['group'] and self.config['src_interface']: + if self.config['group'] or self.config['source_interface']: + if self.config['group'] and self.config['source_interface']: cmdline.append('group') - cmdline.append('src_interface') + cmdline.append('source_interface') else: ifname = self.config['ifname'] raise ConfigError( |