diff options
author | Viacheslav Hletenko <v.gletenko@vyos.io> | 2023-08-02 09:12:23 +0000 |
---|---|---|
committer | Viacheslav Hletenko <v.gletenko@vyos.io> | 2023-08-02 09:12:23 +0000 |
commit | d1923b7b58795f0d9635ae0e8df110f591881bdf (patch) | |
tree | 4f8dc76154caf1f0104e583e3e5350fe2855512e /src/migration-scripts/interfaces/16-to-17 | |
parent | ad57339e9ce655823e1f8e7ad25dc5560359b8da (diff) | |
download | vyos-1x-d1923b7b58795f0d9635ae0e8df110f591881bdf.tar.gz vyos-1x-d1923b7b58795f0d9635ae0e8df110f591881bdf.zip |
T5427: Fix migration script arguments len expects 2 args
The script's name is always provided as the first argument sys.argv[0]
Expected length for argv is 2 (script itself + config file)
Change: 'if (len(argv) < 1)' to 'if len(argv) < 2'
Diffstat (limited to 'src/migration-scripts/interfaces/16-to-17')
-rwxr-xr-x | src/migration-scripts/interfaces/16-to-17 | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/migration-scripts/interfaces/16-to-17 b/src/migration-scripts/interfaces/16-to-17 index d123be06f..75f160686 100755 --- a/src/migration-scripts/interfaces/16-to-17 +++ b/src/migration-scripts/interfaces/16-to-17 @@ -21,7 +21,7 @@ import sys from vyos.configtree import ConfigTree if __name__ == '__main__': - if (len(sys.argv) < 1): + if len(sys.argv) < 2: print("Must specify file name!") sys.exit(1) @@ -35,7 +35,7 @@ if __name__ == '__main__': if not config.exists(base): # Nothing to do sys.exit(0) - + for interface in config.list_nodes(base): mirror_old_base = base + [interface, 'mirror'] if config.exists(mirror_old_base): @@ -43,7 +43,7 @@ if __name__ == '__main__': if config.exists(mirror_old_base): config.delete(mirror_old_base) config.set(mirror_old_base + ['ingress'],intf[0]) - + try: with open(file_name, 'w') as f: f.write(config.to_string()) |