Skip to content

Revert "Skip test_gnmi_configdb_full_01 to fix test issue. (#7251)"#7347

Merged
ganglyu merged 1 commit intosonic-net:masterfrom
ganglyu:recover_gnmi_test
Jan 30, 2023
Merged

Revert "Skip test_gnmi_configdb_full_01 to fix test issue. (#7251)"#7347
ganglyu merged 1 commit intosonic-net:masterfrom
ganglyu:recover_gnmi_test

Conversation

@ganglyu
Copy link
Copy Markdown
Contributor

@ganglyu ganglyu commented Jan 29, 2023

This reverts commit 4dfec81.

Description of PR

Summary:
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?

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.

Any platform specific information?

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

Documentation

@ganglyu ganglyu requested a review from wangxin January 29, 2023 01:40
@ganglyu ganglyu merged commit c4afc04 into sonic-net:master Jan 30, 2023
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