Skip to content

[202411] Update timeout for gnmi subscribe API#19045

Merged
wangxin merged 1 commit intosonic-net:202411from
ganglyu:subscribe_timeout_202411
Jun 17, 2025
Merged

[202411] Update timeout for gnmi subscribe API#19045
wangxin merged 1 commit intosonic-net:202411from
ganglyu:subscribe_timeout_202411

Conversation

@ganglyu
Copy link
Contributor

@ganglyu ganglyu commented Jun 17, 2025

Description of PR

Summary:
Fixes # (issue)
Microsoft ADO: 33257276
Backport: #19019

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

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines will not run the associated pipelines, because the pull request was updated after the run command was issued. Review the pull request again and issue a new run command.

@ganglyu ganglyu changed the base branch from master to 202411 June 17, 2025 02:08
@ganglyu
Copy link
Contributor Author

ganglyu commented Jun 17, 2025

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@wangxin wangxin merged commit 66c7667 into sonic-net:202411 Jun 17, 2025
13 checks passed
sdszhang pushed a commit to sdszhang/sonic-mgmt that referenced this pull request Jun 30, 2025
Code sync sonic-net/sonic-mgmt:202411 => 202412

```
*   86163d9 (HEAD -> code-sync-202412, origin/code-sync-202412) r12f 250620:0224 - Merge remote-tracking branch 'base/202411' into code-sync-202412
|\
| * 6c7825f (base/202411) Justin Wong 250611:2109 - Skip test_bgp_multipath_relax.py on t1-isolated (sonic-net#18558)
| * 829aa40 Chun'ang Li 250619:1618 - [CI][202411]update pr test template reference (sonic-net#19059)
| * 6f2a1bd prabhataravind 250617:1800 - Fix port mapping for new 4280 hwskus (sonic-net#18802) (sonic-net#19030)
| * 66c7667 ganglv 250617:1623 - [202411] Update timeout for gnmi subscribe API (sonic-net#19045)
| * 8f0a97b Justin Wong 250611:2146 - Extend bgp command check to allow multipath routes and "best" routes (sonic-net#18910)
| * 12b7793 StormLiangMS 250611:1110 - [XFAIL: test_crm_available.py] set xfail of test_crm_available on dualtor platform (sonic-net#18874)
| * ab5fd70 rbpittman 250615:2148 - Shutdown BGP instead of stopping some processes. (sonic-net#18267)
| * 3f35fa2 Harish Kalyanaraman 250321:0332 - Increase GCUTIMEOUT value for nokia-armhf platforms (sonic-net#17420)
| * 48bf447 xwjiang-ms 250613:1238 - [202411] Support Ubuntu 24 server in KVM (sonic-net#18929)
| * 194b907 rbpittman 250612:2057 - Dualtor AA enablement for QOS SAI (sonic-net#17765) (sonic-net#17911)
```
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