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

SONiC image is not ready, skip test_gnmi_configdb_full_01 #7251

Merged
merged 1 commit into from
Jan 12, 2023
Merged

SONiC image is not ready, skip test_gnmi_configdb_full_01 #7251

merged 1 commit into from
Jan 12, 2023

Conversation

ganglyu
Copy link
Contributor

@ganglyu ganglyu commented Jan 12, 2023

Description of PR

Summary:
SONiC image is not ready, skip test_gnmi_configdb_full_01, and we need to enable this test after merging the fix.

Fixes # (issue)

Type of change

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

Back port request

  • 201911
  • 202012
  • 202205

Approach

What is the motivation for this PR?

This issue is introduced by GNMI end2end test, and we need to wait for below PR:
sonic-net/sonic-buildimage#13333

How did you do it?

Skip test_gnmi_configdb_full_01 for now.

How did you verify/test it?

Check end2end test result.

Any platform specific information?

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

Documentation

@ganglyu ganglyu requested a review from wangxin January 12, 2023 05:33
@ganglyu ganglyu changed the title Skip test_gnmi_configdb_full_01 to fix smoke test SONiC image is not ready, skip test_gnmi_configdb_full_01 Jan 12, 2023
@ganglyu ganglyu merged commit 4dfec81 into sonic-net:master Jan 12, 2023
ganglyu added a commit that referenced this pull request Jan 30, 2023
…7347)

This reverts commit 4dfec81.

What is the motivation for this PR?
The fix for config reload issue is merged to sonic-buildimage, so we can enable gnmi test.
sonic-net/sonic-buildimage#13333

How did you do it?
Update conditional mark.

How did you verify/test it?
Run GNMI e2e test.
kellyyeh pushed a commit to kellyyeh/sonic-mgmt that referenced this pull request Mar 31, 2023
…#7251)" (sonic-net#7347)

This reverts commit 4dfec81.

What is the motivation for this PR?
The fix for config reload issue is merged to sonic-buildimage, so we can enable gnmi test.
sonic-net/sonic-buildimage#13333

How did you do it?
Update conditional mark.

How did you verify/test it?
Run GNMI e2e test.
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