Skip to content

Add config save after TSB to reset device state#9462

Merged
tjchadaga merged 1 commit intosonic-net:masterfrom
tjchadaga:tsa_config_reload_fix
Aug 16, 2023
Merged

Add config save after TSB to reset device state#9462
tjchadaga merged 1 commit intosonic-net:masterfrom
tjchadaga:tsa_config_reload_fix

Conversation

@tjchadaga
Copy link
Copy Markdown
Contributor

Description of PR

Summary:
Changes to fix the issue where after running bgp/test_traffic_sup.py and reload, the device came up with TSA enabled, causing subsequent test failures.

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • Test case(new/improvement)

Back port request

  • 201911
  • 202012
  • 202205

Approach

What is the motivation for this PR?

To fix the issue where after running bgp/test_traffic_sup.py and reload, the device came up with TSA enabled, causing subsequent test failures.

How did you do it?

In test_TSA_TSB_chassis_with_config_reload, config was saved after TSA, but not after subsequent TSB. Due to this, the device returned to isolated state when rebooted in subsequent tests

How did you verify/test it?

Manual run of bgp/test_traffic_shift_sup.py and validated config DB and current state on all LCs

Any platform specific information?

Supported testbed topology if it's a new test case?

Documentation

@tjchadaga
Copy link
Copy Markdown
Contributor Author

Adding @mannytaheri, @saksarav-nokia

@mssonicbld
Copy link
Copy Markdown
Collaborator

Cherry-pick PR to 202205: #9519

@mssonicbld
Copy link
Copy Markdown
Collaborator

Cherry-pick PR to 202305: #9520

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants