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

Fix iBGP skip in bgp/test_4-byte_asn_community.py #15614

Conversation

arista-nwolfe
Copy link
Contributor

Description of PR

Pretty much exactly the same issue as #15411 but in a different test.
The fix is identical to so I'll leave this description short as the description in #15411 explains this issue as well.

Summary:
Fixes #15613

Type of change

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

Back port request

  • 202012
  • 202205
  • 202305
  • 202311
  • 202405

Copy link
Contributor

@Javier-Tan Javier-Tan left a comment

Choose a reason for hiding this comment

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

Same as #15411, didn't run the test but LGTM

@arlakshm arlakshm merged commit 733a24f into sonic-net:master Nov 22, 2024
16 checks passed
@rlhui rlhui added the chassis label Nov 27, 2024
mssonicbld pushed a commit to mssonicbld/sonic-mgmt that referenced this pull request Nov 27, 2024
)

Description of PR
Pretty much exactly the same issue as sonic-net#15411 but in a different test.
The fix is identical to so I'll leave this description short as the description in sonic-net#15411 explains this issue as well.

Summary:
Fixes sonic-net#15613

Type of change
 Bug fix
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202405: #15774

mssonicbld pushed a commit that referenced this pull request Nov 27, 2024
Description of PR
Pretty much exactly the same issue as #15411 but in a different test.
The fix is identical to so I'll leave this description short as the description in #15411 explains this issue as well.

Summary:
Fixes #15613

Type of change
 Bug fix
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[T2]: bgp/test_4-byte_asn_community.py Fails with KeyError
6 participants