diff options
4 files changed, 12 insertions, 1 deletions
| diff --git a/data/templates/frr/bgpd.frr.j2 b/data/templates/frr/bgpd.frr.j2 index e9422b257..7d1241f5a 100644 --- a/data/templates/frr/bgpd.frr.j2 +++ b/data/templates/frr/bgpd.frr.j2 @@ -537,6 +537,9 @@ router bgp {{ system_as }} {{ 'vrf ' ~ vrf if vrf is vyos_defined }}  {% if parameters.allow_martian_nexthop is vyos_defined %}   bgp allow-martian-nexthop  {% endif %} +{% if parameters.disable_ebgp_connected_route_check is vyos_defined %} + bgp disable-ebgp-connected-route-check +{% endif %}  {% if parameters.always_compare_med is vyos_defined %}   bgp always-compare-med  {% endif %} diff --git a/interface-definitions/include/bgp/neighbor-disable-connected-check.xml.i b/interface-definitions/include/bgp/neighbor-disable-connected-check.xml.i index cb8b610b4..aef5a55e9 100644 --- a/interface-definitions/include/bgp/neighbor-disable-connected-check.xml.i +++ b/interface-definitions/include/bgp/neighbor-disable-connected-check.xml.i @@ -1,7 +1,7 @@  <!-- include start from bgp/neighbor-disable-connected-check.xml.i -->  <leafNode name="disable-connected-check">    <properties> -    <help>Disable check to see if eBGP peer address is a connected route</help> +    <help>Allow peerings between eBGP peer using loopback/dummy address</help>      <valueless/>    </properties>  </leafNode> diff --git a/interface-definitions/include/bgp/protocol-common-config.xml.i b/interface-definitions/include/bgp/protocol-common-config.xml.i index ca67eaf3c..0f05625a7 100644 --- a/interface-definitions/include/bgp/protocol-common-config.xml.i +++ b/interface-definitions/include/bgp/protocol-common-config.xml.i @@ -1249,6 +1249,12 @@          <valueless/>        </properties>      </leafNode> +    <leafNode name="disable-ebgp-connected-route-check"> +      <properties> +        <help>Disable checking if nexthop is connected on eBGP session</help> +        <valueless/> +      </properties> +    </leafNode>      <leafNode name="always-compare-med">        <properties>          <help>Always compare MEDs from different neighbors</help> diff --git a/smoketest/scripts/cli/test_protocols_bgp.py b/smoketest/scripts/cli/test_protocols_bgp.py index e8556cf44..e26ac0646 100755 --- a/smoketest/scripts/cli/test_protocols_bgp.py +++ b/smoketest/scripts/cli/test_protocols_bgp.py @@ -321,6 +321,7 @@ class TestProtocolsBGP(VyOSUnitTestSHIM.TestCase):          tcp_keepalive_probes = '22'          self.cli_set(base_path + ['parameters', 'allow-martian-nexthop']) +        self.cli_set(base_path + ['parameters', 'disable-ebgp-connected-route-check'])          self.cli_set(base_path + ['parameters', 'no-hard-administrative-reset'])          self.cli_set(base_path + ['parameters', 'log-neighbor-changes'])          self.cli_set(base_path + ['parameters', 'labeled-unicast', 'explicit-null']) @@ -372,6 +373,7 @@ class TestProtocolsBGP(VyOSUnitTestSHIM.TestCase):          self.assertIn(f'router bgp {ASN}', frrconfig)          self.assertIn(f' bgp router-id {router_id}', frrconfig)          self.assertIn(f' bgp allow-martian-nexthop', frrconfig) +        self.assertIn(f' bgp disable-ebgp-connected-route-check', frrconfig)          self.assertIn(f' bgp log-neighbor-changes', frrconfig)          self.assertIn(f' bgp default local-preference {local_pref}', frrconfig)          self.assertIn(f' bgp conditional-advertisement timer {cond_adv_timer}', frrconfig) | 
