Skip to content

[action] [PR:19019] Update timeout for gnmi subscribe API#19044

Merged
mssonicbld merged 1 commit intosonic-net:202505from
mssonicbld:cherry/202505/19019
Jun 17, 2025
Merged

[action] [PR:19019] Update timeout for gnmi subscribe API#19044
mssonicbld merged 1 commit intosonic-net:202505from
mssonicbld:cherry/202505/19019

Conversation

@mssonicbld
Copy link
Collaborator

Description of PR

Summary:
Fixes # (issue)
Microsoft ADO: 33257276

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • New Test case
  • Skipped for non-supported platforms
  • Test case improvement

Back port request

  • 202205
  • 202305
  • 202311
  • 202405
  • 202411
  • 202505

Approach

What is the motivation for this PR?

test_gnmi_configdb_streaming_onchange_01 failed on KVM DUT, and test logs show that it needs more than 30 seconds to get all the results from GNMI server.

How did you do it?

Update timeout from 30 seconds to 120 seconds.

How did you verify/test it?

Run gnmi end to end test.

Any platform specific information?

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

Documentation

What is the motivation for this PR?
test_gnmi_configdb_streaming_onchange_01 failed on KVM DUT, and test logs show that it needs more than 30 seconds to get all the results from GNMI server.

How did you do it?
Update timeout from 30 seconds to 120 seconds.

How did you verify/test it?
Run gnmi end to end test.
@mssonicbld
Copy link
Collaborator Author

Original PR: #19019

@mssonicbld
Copy link
Collaborator Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld mssonicbld merged commit eb7ca12 into sonic-net:202505 Jun 17, 2025
12 checks passed
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.

2 participants