summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDaniil Baturin <daniil@baturin.org>2017-10-13 01:21:09 +0200
committerDaniil Baturin <daniil@baturin.org>2017-10-13 01:21:09 +0200
commit2e786bf3b54665a82ad04bc59b278db1331a37e2 (patch)
treeb01d59724f522d20f93164ea3d0be0e24884ce02
parent9e6aff9ec8a87d8d8fc9f645ed81ed29cf6c6999 (diff)
downloadvyatta-cfg-quagga-2e786bf3b54665a82ad04bc59b278db1331a37e2.tar.gz
vyatta-cfg-quagga-2e786bf3b54665a82ad04bc59b278db1331a37e2.zip
T87: make non-assigned update-source address a warning rather than an error.
Making it an error prevents configuration from loading, which has adverse interaction with non-permanent interfaces such as certain VPN tunnels that may not immediately go up. A dead BGP session is lesser of two evils compared to incomplete configuration.
-rwxr-xr-xscripts/bgp/vyatta-bgp.pl4
1 files changed, 2 insertions, 2 deletions
diff --git a/scripts/bgp/vyatta-bgp.pl b/scripts/bgp/vyatta-bgp.pl
index 1fc1e3c1..ce653683 100755
--- a/scripts/bgp/vyatta-bgp.pl
+++ b/scripts/bgp/vyatta-bgp.pl
@@ -1558,10 +1558,10 @@ sub check_source {
if ($ip) {
my $found = grep { my $a = new NetAddr::IP::Lite($_);
$a->addr() eq $ip->addr() } Vyatta::Misc::getIP();
- die "IP address $ip does not exist on this system\n" if ($found == 0);
+ print("Warning: IP address $ip does not exist on this system\n") if ($found == 0);
} else {
my $found = grep { $_ eq $src } Vyatta::Misc::getInterfaces();
- die "Interface $src does not exist on the system\n" if ($found == 0);
+ print("Warning: Interface $src does not exist on the system\n") if ($found == 0);
}
}