diff options
author | Christian Poessinger <christian@poessinger.com> | 2021-08-31 23:03:01 +0200 |
---|---|---|
committer | Christian Poessinger <christian@poessinger.com> | 2021-08-31 23:36:14 +0200 |
commit | b0d4112bd6073e4a947869c3bd80f8e87783fbfa (patch) | |
tree | c8266c7b3c60625072eec0706d89ad0ab37cf4a5 /src/migration-scripts/interfaces | |
parent | 56fa0d4baaeb7624918d77172fa7e1500d694b93 (diff) | |
download | vyos-1x-b0d4112bd6073e4a947869c3bd80f8e87783fbfa.tar.gz vyos-1x-b0d4112bd6073e4a947869c3bd80f8e87783fbfa.zip |
vyos.ethtool: T3163: purify code to read and change flow-control settings
It makes no sense to have a parser for the ethtool values in ethtool.py
and ethernet.py - one instance ios more then enough!
(cherry picked from commit 0229645c8248decb5664056df8aa5cd5dff41802)
Diffstat (limited to 'src/migration-scripts/interfaces')
-rwxr-xr-x | src/migration-scripts/interfaces/20-to-21 | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/src/migration-scripts/interfaces/20-to-21 b/src/migration-scripts/interfaces/20-to-21 index bd89dcdb4..0bd858760 100755 --- a/src/migration-scripts/interfaces/20-to-21 +++ b/src/migration-scripts/interfaces/20-to-21 @@ -104,6 +104,14 @@ for ifname in config.list_nodes(base): config.delete(speed_path) config.delete(duplex_path) + # Also while processing the interface configuration, not all adapters support + # changing disabling flow-control - or change this setting. If disabling + # flow-control is not supported by the NIC, we remove the setting from CLI + flow_control_path = base + [ifname, 'disable-flow-control'] + if config.exists(flow_control_path): + if not eth.check_flow_control(): + config.delete(flow_control_path) + try: with open(file_name, 'w') as f: f.write(config.to_string()) |