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/ospf | |
| 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/ospf')
| -rwxr-xr-x | src/migration-scripts/ospf/0-to-1 | 2 | ||||
| -rwxr-xr-x | src/migration-scripts/ospf/1-to-2 | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/src/migration-scripts/ospf/0-to-1 b/src/migration-scripts/ospf/0-to-1 index 678569d9e..8f02acada 100755 --- a/src/migration-scripts/ospf/0-to-1 +++ b/src/migration-scripts/ospf/0-to-1 @@ -37,7 +37,7 @@ def ospf_passive_migration(config, ospf_base): config.set(ospf_base + ['interface', interface, 'passive', 'disable']) config.delete(ospf_base + ['passive-interface-exclude']) -if (len(argv) < 1): +if len(argv) < 2: print("Must specify file name!") exit(1) diff --git a/src/migration-scripts/ospf/1-to-2 b/src/migration-scripts/ospf/1-to-2 index a6beaf04e..ba9499c60 100755 --- a/src/migration-scripts/ospf/1-to-2 +++ b/src/migration-scripts/ospf/1-to-2 @@ -22,7 +22,7 @@ from sys import exit from vyos.configtree import ConfigTree -if (len(argv) < 1): +if len(argv) < 2: print("Must specify file name!") exit(1) |