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

Add previous GCU tests to kvm daily test #4896

Merged
merged 2 commits into from
Jan 21, 2022
Merged

Conversation

wen587
Copy link
Contributor

@wen587 wen587 commented Jan 6, 2022

Description of PR

Summary:
Add previous Generic Configer Updater tests into kvm test suite
Fixes # (issue)

Type of change

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

Back port request

  • 201911

Approach

What is the motivation for this PR?

Move GCU test into kvm daily test

How did you do it?

Add GCU test case into kvmtest.sh

How did you verify/test it?

Run test of sonic-mgmt/tests/generic_config_updater/*.py on KVM

Any platform specific information?

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

Documentation

@liushilongbuaa
Copy link
Contributor

/azpw run Azure.sonic-mgmt

@liushilongbuaa
Copy link
Contributor

/azpw

@wen587
Copy link
Contributor Author

wen587 commented Jan 13, 2022

sonic-net/sonic-utilities#2006
Loopback test can only pass after above PR merge.

@qiluo-msft
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

qiluo-msft
qiluo-msft previously approved these changes Jan 13, 2022
@wen587
Copy link
Contributor Author

wen587 commented Jan 18, 2022

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@wen587 wen587 marked this pull request as ready for review January 20, 2022 08:53
@wen587 wen587 requested a review from a team as a code owner January 20, 2022 08:53
@wen587 wen587 merged commit 8169e18 into sonic-net:master Jan 21, 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.

3 participants