Skip to content

Add previous GCU tests to kvm daily test#4896

Merged
wen587 merged 2 commits intosonic-net:masterfrom
wen587:gcu_kvm_test
Jan 21, 2022
Merged

Add previous GCU tests to kvm daily test#4896
wen587 merged 2 commits intosonic-net:masterfrom
wen587:gcu_kvm_test

Conversation

@wen587
Copy link
Copy Markdown
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
Copy Markdown
Contributor

/azpw run Azure.sonic-mgmt

@liushilongbuaa
Copy link
Copy Markdown
Contributor

/azpw

@wen587
Copy link
Copy Markdown
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
Copy Markdown
Contributor

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

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

wen587 commented Jan 18, 2022

/azp run

@azure-pipelines
Copy link
Copy Markdown

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