From 0dd86eb16c65b4ebbe866fe15d2ee5370a009252 Mon Sep 17 00:00:00 2001
From: wenyiz2021 <91497961+wenyiz2021@users.noreply.github.com>
Date: Mon, 3 Aug 2026 12:06:16 -0700
Subject: [PATCH] Enhance BGP peer shutdown test for new neighbor types

Added handling for LowerMgmtAggregator and UpperMgmtAggregator types in BGP peer shutdown test.
---
 tests/bgp/test_bgp_peer_shutdown.py | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/tests/bgp/test_bgp_peer_shutdown.py b/tests/bgp/test_bgp_peer_shutdown.py
index fb6c3fafd4d..2524b6cddae 100644
--- a/tests/bgp/test_bgp_peer_shutdown.py
+++ b/tests/bgp/test_bgp_peer_shutdown.py
@@ -77,6 +77,13 @@ def common_setup_teardown(
         neigh_type = "LowerRegionalHub"
         if confed_asn is not None:
             use_vtysh = True
+    elif dut_type in ["LowerMgmtAggregator", "UpperMgmtAggregator"]:
+        # Mgmt aggregators run BGP confederation. The test neighbor is an
+        # external eBGP peer, so it must be added directly to FRR via vtysh
+        # (the config_db path is not rendered for these peers).
+        neigh_type = "MgmtSpineRouter"
+        if confed_asn is not None:
+            use_vtysh = True
     else:
         neigh_type = "ToRRouter"
     logging.info(