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

Enable container autorestart for bgp/test_bgp_session.py #15766

Merged

Conversation

vivekverma-arista
Copy link
Contributor

@vivekverma-arista vivekverma-arista commented Nov 27, 2024

…ion.py

Description of PR

Summary:
Fixes #341

Type of change

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

Back port request

  • 202012
  • 202205
  • 202305
  • 202311
  • 202405

Approach

What is the motivation for this PR?

All testcases in bgp/test_bgp_session.py get skipped erroneously

How did you do it?

Enabled container autorestart for this module.

How did you verify/test it?

Ran the test on Arista 7260CX3 with dualtor topology.

Any platform specific information?

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

Documentation

@vivekverma-arista vivekverma-arista changed the title Enable container autorestart to prevent skipping of bgp/test_bgp_sess… Enable container autorestart for bgp/test_bgp_session.py Nov 27, 2024
@vivekverma-arista
Copy link
Contributor Author

While cherry picking to 202405, #15729 should merge before this.

Copy link
Collaborator

@StormLiangMS StormLiangMS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@StormLiangMS StormLiangMS merged commit d7999d0 into sonic-net:master Dec 5, 2024
16 checks passed
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.

2 participants