Skip to content

Add renamed script test_ip_bgp.py to PR test#14977

Merged
StormLiangMS merged 1 commit intosonic-net:masterfrom
wangxin:add-test-ip-bgp
Oct 14, 2024
Merged

Add renamed script test_ip_bgp.py to PR test#14977
StormLiangMS merged 1 commit intosonic-net:masterfrom
wangxin:add-test-ip-bgp

Conversation

@wangxin
Copy link
Copy Markdown
Collaborator

@wangxin wangxin commented Oct 12, 2024

Description of PR

Summary:
Fixes # (issue)

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?

Script tests/generic_config_updater/test_ipv6.py was renamed to tests/generic_config_updater/test_ip_bgp.py in PR #13650

How did you do it?

Need to update file name in pr_test_scripts.yaml to ensure that renamed script is covered by PR testing.

How did you verify/test it?

Any platform specific information?

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

Documentation

Script tests/generic_config_updater/test_ipv6.py was renamed to tests/generic_config_updater/test_ip_bgp.py in PR sonic-net#13650

Need to update file name in pr_test_scripts.yaml to ensure that renamed script is covered by PR testing.

Signed-off-by: Xin Wang <[email protected]>
Copy link
Copy Markdown
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 f711dc5 into sonic-net:master Oct 14, 2024
vikshaw-Nokia pushed a commit to vikshaw-Nokia/sonic-mgmt that referenced this pull request Oct 23, 2024
Script tests/generic_config_updater/test_ipv6.py was renamed to tests/generic_config_updater/test_ip_bgp.py in PR sonic-net#13650

Need to update file name in pr_test_scripts.yaml to ensure that renamed script is covered by PR testing.

Signed-off-by: Xin Wang <[email protected]>
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.

3 participants