topotests: fix default BFD peer shutdown state

The commit `bfdd: simplify and remove duplicated code` fixed a problem
that was causing the protocol configuration to override the user
configuration.

In this test case: the peer was configured to be disabled (default is
`shutdown`) and the test was expecting it to get activated (`no shutdown`)
when the protocol converged. I changed the peer default state to
`no shutdown`, however another way to get the same effect is to
configure the protocol to use a profile or don't configure a peer at all
(and use the defaults).

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

diff --git a/tests/topotests/bfd-isis-topo1/rt1/bfdd.conf b/tests/topotests/bfd-isis-topo1/rt1/bfdd.conf
index 4793155939b4a0f1cd5401690b5134938188cd2a..57f9cd9e3db4e102c777bfbb18243ff6ce004a23 100644 (file)
--- a/tests/topotests/bfd-isis-topo1/rt1/bfdd.conf
+++ b/tests/topotests/bfd-isis-topo1/rt1/bfdd.conf
@@ -8,10 +8,12 @@ bfd
   detect-multiplier 3
   receive-interval 300
   transmit-interval 300
+  no shutdown
  !
  peer 10.0.2.2 interface eth-rt3
   detect-multiplier 3
   receive-interval 300
   transmit-interval 300
+  no shutdown
  !
 !

diff --git a/tests/topotests/bfd-isis-topo1/rt2/bfdd.conf b/tests/topotests/bfd-isis-topo1/rt2/bfdd.conf
index a49cd4fa6bcca60cab883d945bcf8f6912e04063..6b34e337d3ccdc335fa580ac20f04918775f20bc 100644 (file)
--- a/tests/topotests/bfd-isis-topo1/rt2/bfdd.conf
+++ b/tests/topotests/bfd-isis-topo1/rt2/bfdd.conf
@@ -8,5 +8,6 @@ bfd
   detect-multiplier 3
   receive-interval 300
   transmit-interval 300
+  no shutdown
  !
 !

diff --git a/tests/topotests/bfd-isis-topo1/rt3/bfdd.conf b/tests/topotests/bfd-isis-topo1/rt3/bfdd.conf
index 600054a0bc35deb4e0a0e2c7ee28969a12106543..22937fe46fbbfaa421d07aad74f6812125eab475 100644 (file)
--- a/tests/topotests/bfd-isis-topo1/rt3/bfdd.conf
+++ b/tests/topotests/bfd-isis-topo1/rt3/bfdd.conf
@@ -8,5 +8,6 @@ bfd
   detect-multiplier 3
   receive-interval 300
   transmit-interval 300
+  no shutdown
  !
 !