Skip to content

Commit

Permalink
Add dependency for tsa_enabled flag before populating peer config and…
Browse files Browse the repository at this point in the history
… bringing up BGP (#18556)

Why I did it
Ensure BGP peer bring-up and route advertisements are done only after checking TSA status during reload

Work item tracking
Microsoft ADO (number only): 27171112
How I did it
Add dependency on tsa_enabled flag before peer configuration

How to verify it
Validate that the BGP session bring up is not complete until tsa_enabled flag is populated.
Ensure no traffic is drawn to the device when in TSA and reboot with BGP unshut.
  • Loading branch information
tjchadaga authored and gechiang committed Apr 17, 2024
1 parent fc348a9 commit 462c618
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions src/sonic-bgpcfgd/bgpcfgd/managers_bgp.py
Original file line number Diff line number Diff line change
Expand Up @@ -107,6 +107,7 @@ def __init__(self, common_objs, db_name, table_name, peer_type, check_neig_meta)
deps = [
("CONFIG_DB", swsscommon.CFG_DEVICE_METADATA_TABLE_NAME, "localhost/bgp_asn"),
("CONFIG_DB", swsscommon.CFG_LOOPBACK_INTERFACE_TABLE_NAME, "Loopback0"),
("CONFIG_DB", swsscommon.CFG_BGP_DEVICE_GLOBAL_TABLE_NAME, "tsa_enabled"),
("LOCAL", "local_addresses", ""),
("LOCAL", "interfaces", ""),
]
Expand Down

0 comments on commit 462c618

Please sign in to comment.