Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add dependency for tsa_enabled flag before populating peer config and… #335

Merged
merged 2 commits into from
Apr 19, 2024
Merged

Add dependency for tsa_enabled flag before populating peer config and… #335

merged 2 commits into from
Apr 19, 2024

Conversation

gechiang
Copy link
Contributor

… 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.

Why I did it

Work item tracking
  • Microsoft ADO (number only):

How I did it

How to verify it

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

… 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.
@gechiang gechiang requested a review from StormLiangMS as a code owner April 17, 2024 18:44
@gechiang gechiang requested a review from tjchadaga April 17, 2024 18:45
@gechiang gechiang enabled auto-merge April 19, 2024 02:31
@gechiang gechiang merged commit c49f39b into Azure:202205 Apr 19, 2024
3 checks passed
@gechiang gechiang deleted the 202205 branch April 19, 2024 02:31
@gechiang
Copy link
Contributor Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants