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

[python3]migrate dir_bcast_test and fdb scripts to python3 #5540

Merged
merged 1 commit into from
May 18, 2022

Conversation

ZhaohuiS
Copy link
Contributor

Description of PR

Summary:
migrate the following scripts to python3, this PR can't be merged until #5534 is merged which has dependency of ptf_runner.py

ansible/roles/test/files/ptftests/dir_bcast_test.py
ansible/roles/test/files/ptftests/fdb.py
ansible/roles/test/files/ptftests/fdb_mac_expire_test.py
ansible/roles/test/files/ptftests/fdb_test.py

Signed-off-by: Zhaohui Sun zhaohuisun@microsoft.com

Type of change

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

Back port request

  • 201911
  • 202012

Approach

What is the motivation for this PR?

migrate dir_bcast_test and fdb scripts to python3

How did you do it?

Update test cases accordingly.

tests/fdb/test_fdb_mac_expire.py
tests/ipfwd/test_dir_bcast.py

How did you verify/test it?

run tests/fdb/test_fdb_mac_expire.py and tests/ipfwd/test_dir_bcast.py

Any platform specific information?

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

Documentation

Signed-off-by: Zhaohui Sun <zhaohuisun@microsoft.com>
@ZhaohuiS ZhaohuiS requested a review from a team as a code owner April 20, 2022 09:38
@lgtm-com
Copy link

lgtm-com bot commented Apr 20, 2022

This pull request introduces 1 alert when merging e158572 into 3212c2f - view on LGTM.com

new alerts:

  • 1 for Wrong name for an argument in a call

@ZhaohuiS ZhaohuiS changed the title migrate dir_bcast_test and fdb scripts to python3 [python3]migrate dir_bcast_test and fdb scripts to python3 Apr 21, 2022
@ZhaohuiS
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@ZhaohuiS
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@ZhaohuiS ZhaohuiS merged commit 2c6d38a into sonic-net:master May 18, 2022
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