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

[kvmtest]: Add dual ToR ARP tests to KVM test suite #2895

Merged
merged 2 commits into from
Feb 17, 2021

Conversation

theasianpianist
Copy link
Contributor

@theasianpianist theasianpianist commented Jan 30, 2021

Signed-off-by: Lawrence Lee [email protected]

Description of PR

Summary:
Fixes # (issue)

Type of change

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

Approach

What is the motivation for this PR?

How did you do it?

How did you verify/test it?

Any platform specific information?

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

Documentation

@theasianpianist theasianpianist marked this pull request as draft January 30, 2021 01:18
@lguohan
Copy link
Contributor

lguohan commented Jan 31, 2021

/Azurepipelines run

@azure-pipelines
Copy link

Pull request contains merge conflicts.

@lguohan
Copy link
Contributor

lguohan commented Jan 31, 2021

i have fixed the pr test issue, can you resolve the conflict?

@theasianpianist theasianpianist marked this pull request as ready for review February 1, 2021 17:18
@lguohan
Copy link
Contributor

lguohan commented Feb 2, 2021

looks like the test failed.

@yxieca
Copy link
Collaborator

yxieca commented Feb 17, 2021

/Azurepipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@yxieca
Copy link
Collaborator

yxieca commented Feb 17, 2021

@theasianpianist I restarted the test. can you check if the test fail again and address issues accordingly?

@theasianpianist theasianpianist merged commit eae4d47 into sonic-net:master Feb 17, 2021
@theasianpianist theasianpianist deleted the vs-arp-tests branch February 17, 2021 16:58
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.

3 participants