diff options
author | Christian Poessinger <christian@poessinger.com> | 2020-01-26 12:20:39 +0100 |
---|---|---|
committer | Christian Poessinger <christian@poessinger.com> | 2020-01-26 12:55:11 +0100 |
commit | 66f8be0757f27ab77abde070644a7b4310a799a2 (patch) | |
tree | e0e85de5256bc7aee433425e881f9bbdc1dc90ea /src/migration-scripts | |
parent | 07d002e3be03babf1c3faaee43a90fe513969134 (diff) | |
download | vyos-1x-66f8be0757f27ab77abde070644a7b4310a799a2.tar.gz vyos-1x-66f8be0757f27ab77abde070644a7b4310a799a2.zip |
lldp: T1896: remove MED civic based location information
Civic based location information was not working for quiet some time and
as an unused feature we should drop it to keep the codebase more simple
and maintainable.
Diffstat (limited to 'src/migration-scripts')
-rwxr-xr-x | src/migration-scripts/lldp/0-to-1 | 35 |
1 files changed, 35 insertions, 0 deletions
diff --git a/src/migration-scripts/lldp/0-to-1 b/src/migration-scripts/lldp/0-to-1 new file mode 100755 index 000000000..5f66570e7 --- /dev/null +++ b/src/migration-scripts/lldp/0-to-1 @@ -0,0 +1,35 @@ +#!/usr/bin/env python3 + +# Delete "set service lldp interface <interface> location civic-based" option +# as it was broken most of the time anyways + +import sys + +from vyos.configtree import ConfigTree + +if (len(sys.argv) < 1): + print("Must specify file name!") + sys.exit(1) + +file_name = sys.argv[1] + +with open(file_name, 'r') as f: + config_file = f.read() + +config = ConfigTree(config_file) +base = ['service', 'lldp', 'interface'] +if not config.exists(base): + # Nothing to do + sys.exit(0) +else: + # Delete nodes with abandoned CLI syntax + for interface in config.list_nodes(base): + if config.exists(base + [interface, 'location', 'civic-based']): + config.delete(base + [interface, 'location', 'civic-based']) + + try: + with open(file_name, 'w') as f: + f.write(config.to_string()) + except OSError as e: + print("Failed to save the modified config: {}".format(e)) + sys.exit(1) |