diff options
| author | Christian Poessinger <christian@poessinger.com> | 2020-03-26 07:31:46 +0100 |
|---|---|---|
| committer | Christian Poessinger <christian@poessinger.com> | 2020-03-26 07:32:55 +0100 |
| commit | 583e9d907236a4a98fe40e97a378c1fb655f8a95 (patch) | |
| tree | cfc58842d0a8eaff6946c14317ed1ba86a0d1c4f /src/migration-scripts/interfaces/5-to-6 | |
| parent | 1dec776850781279866a65c1b2d6a09ae218b874 (diff) | |
| download | vyos-1x-583e9d907236a4a98fe40e97a378c1fb655f8a95.tar.gz vyos-1x-583e9d907236a4a98fe40e97a378c1fb655f8a95.zip | |
router-advert: T2162: bugfix migrating link-mtu nodes with 0 value
... those nodes mean link-mtu advertisement is disabled in radvd and thus we
delete the node completely as it's useless.
Diffstat (limited to 'src/migration-scripts/interfaces/5-to-6')
| -rwxr-xr-x | src/migration-scripts/interfaces/5-to-6 | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/src/migration-scripts/interfaces/5-to-6 b/src/migration-scripts/interfaces/5-to-6 index 9dbfd30e1..85a1994c6 100755 --- a/src/migration-scripts/interfaces/5-to-6 +++ b/src/migration-scripts/interfaces/5-to-6 @@ -75,6 +75,14 @@ def copy_rtradv(c, old_base, interface): if tmp == 'false': c.set(new_base + ['no-send-advert']) + # link-mtu advertisement was formerly disabled by setting its value to 0 + # ... this makes less sense - if it should not be send, just do not + # configure it + if c.exists(new_base + ['link-mtu']): + tmp = c.return_value(new_base + ['link-mtu']) + if tmp == '0': + c.delete(new_base + ['link-mtu']) + if __name__ == '__main__': if (len(sys.argv) < 1): print("Must specify file name!") |