Skip to content

Fix configuration after gnmi test#19150

Merged
wangxin merged 1 commit intosonic-net:masterfrom
ganglyu:gnmi_config
Jun 24, 2025
Merged

Fix configuration after gnmi test#19150
wangxin merged 1 commit intosonic-net:masterfrom
ganglyu:gnmi_config

Conversation

@ganglyu
Copy link
Contributor

@ganglyu ganglyu commented Jun 24, 2025 •
edited
Loading

Description of PR

Summary:
Fixes # (issue)
Microsoft ADO: 33375237

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?

Some end to end tests failed because the configuration is wrong after gnmi test.

How did you do it?

Run "config save" command to make sure configuration is persistent.

How did you verify/test it?

Run gnmi end to end test and check configuration.

Any platform specific information?

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

Documentation

@mssonicbld
Copy link
Collaborator

mssonicbld commented Jun 24, 2025

/azp run

@azure-pipelines
Copy link

azure-pipelines bot commented Jun 24, 2025

Azure Pipelines successfully started running 1 pipeline(s).

@wangxin wangxin merged commit a7c41d9 into sonic-net:master Jun 24, 2025
14 of 15 checks passed
mssonicbld pushed a commit to mssonicbld/sonic-mgmt that referenced this pull request Jun 24, 2025
@ganglyu @mssonicbld
Fix configuration after gnmi test (sonic-net#19150)
948973c 
What is the motivation for this PR?
Some end to end tests failed because the configuration is wrong after gnmi test.

How did you do it?
Run "config save" command to make sure configuration is persistent.

How did you verify/test it?
Run gnmi end to end test and check configuration.
mssonicbld pushed a commit to mssonicbld/sonic-mgmt that referenced this pull request Jun 24, 2025
@ganglyu @mssonicbld
Fix configuration after gnmi test (sonic-net#19150)
1c09186 
What is the motivation for this PR?
Some end to end tests failed because the configuration is wrong after gnmi test.

How did you do it?
Run "config save" command to make sure configuration is persistent.

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

mssonicbld commented Jun 24, 2025

Cherry-pick PR to 202411: #19156

@mssonicbld mssonicbld mentioned this pull request Jun 24, 2025
Merged
11 tasks
@mssonicbld
Copy link
Collaborator

mssonicbld commented Jun 24, 2025

Cherry-pick PR to 202505: #19157

@mssonicbld mssonicbld mentioned this pull request Jun 24, 2025
Merged
11 tasks
mssonicbld pushed a commit that referenced this pull request Jun 24, 2025
@ganglyu @mssonicbld
Fix configuration after gnmi test (#19150)
8aef4e3 
What is the motivation for this PR?
Some end to end tests failed because the configuration is wrong after gnmi test.

How did you do it?
Run "config save" command to make sure configuration is persistent.

How did you verify/test it?
Run gnmi end to end test and check configuration.
mssonicbld pushed a commit that referenced this pull request Jun 25, 2025
@ganglyu @mssonicbld
Fix configuration after gnmi test (#19150)
7625d06 
What is the motivation for this PR?
Some end to end tests failed because the configuration is wrong after gnmi test.

How did you do it?
Run "config save" command to make sure configuration is persistent.

How did you verify/test it?
Run gnmi end to end test and check configuration.
sdszhang pushed a commit to sdszhang/sonic-mgmt that referenced this pull request Aug 2, 2025
@r12f
[202412] Code sync sonic-net/sonic-mgmt:202411 => 202412 (sonic-net#477)
cd2e3fb 
Code sync sonic-net/sonic-mgmt:202411 => 202412

```
*   fb53396 (HEAD -> code-sync-202412, origin/code-sync-202412) r12f 250701:1725 - Merge remote-tracking branch 'base/202411' into code-sync-202412
|\
| * 54633cb (base/202411) prabhataravind 250701:0928 - Add t1-28-lag and t1-48-lag to tests_mark_conditions.yaml (sonic-net#19274)
| * 9a23c16 Cong Hou 250416:2355 - Align the sensors data for SN4280 with sonic-buildimage PR#21845 (sonic-net#17747)
| * 8b2d2cd Jibin Bao 250306:0835 - add watchdog test data for x86_64-nvidia_sn4280-r0 (sonic-net#17312)
| * 5c8d5f0 Jibin Bao 250701:0045 - [Mellanox] Update spf test related to error status when sw control is enabled (sonic-net#16573) (sonic-net#19264)
| * 0bea309 Yawen 250411:1620 - add hwsku V64 (sonic-net#17897)
| * 8ab9e6b zitingguo-ms 250628:1755 - [test_static_route] Skip PT0 neighbor when checking route redistribution (sonic-net#19226)
| * 4276be3 prabhataravind 250627:1827 - Support t1-48-lag topo for acl tests (sonic-net#19236)
| * e070035 Wenda Chu 250625:1719 - Skip multiple servers testbed parsing on branch 202411 (sonic-net#19158)
| * 0e5645d xwjiang-ms 250625:1619 - Roll back ceos image version since it's causing bgp failure (sonic-net#19180)
| * 7625d06 ganglv 250624:1548 - Fix configuration after gnmi test (sonic-net#19150)
| * 25e2e55 zitingguo-ms 250624:2000 - [ACL] Record service ports individually and add them to the acl_table_ports (sonic-net#19168)
```
nissampa pushed a commit to nissampa/sonic-mgmt_dpu_test that referenced this pull request Aug 7, 2025
@ganglyu @nissampa
Fix configuration after gnmi test (sonic-net#19150)
92fa8e0 
What is the motivation for this PR?
Some end to end tests failed because the configuration is wrong after gnmi test.

How did you do it?
Run "config save" command to make sure configuration is persistent.

How did you verify/test it?
Run gnmi end to end test and check configuration.
opcoder0 pushed a commit to opcoder0/sonic-mgmt that referenced this pull request Dec 8, 2025
@ganglyu
Fix configuration after gnmi test (sonic-net#19150)
c1bc845 
What is the motivation for this PR?
Some end to end tests failed because the configuration is wrong after gnmi test.

How did you do it?
Run "config save" command to make sure configuration is persistent.

How did you verify/test it?
Run gnmi end to end test and check configuration.

Signed-off-by: opcoder0 <[email protected]>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 16, 2025
@ganglyu @gshemesh2
Fix configuration after gnmi test (sonic-net#19150)
e5800b1 
What is the motivation for this PR?
Some end to end tests failed because the configuration is wrong after gnmi test.

How did you do it?
Run "config save" command to make sure configuration is persistent.

How did you verify/test it?
Run gnmi end to end test and check configuration.

Signed-off-by: Guy Shemesh <[email protected]>
AharonMalkin pushed a commit to AharonMalkin/sonic-mgmt that referenced this pull request Dec 16, 2025
@ganglyu @AharonMalkin
Fix configuration after gnmi test (sonic-net#19150)
2cffc48 
What is the motivation for this PR?
Some end to end tests failed because the configuration is wrong after gnmi test.

How did you do it?
Run "config save" command to make sure configuration is persistent.

How did you verify/test it?
Run gnmi end to end test and check configuration.

Signed-off-by: Aharon Malkin <[email protected]>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 21, 2025
@ganglyu @gshemesh2
Fix configuration after gnmi test (sonic-net#19150)
c86a015 
What is the motivation for this PR?
Some end to end tests failed because the configuration is wrong after gnmi test.

How did you do it?
Run "config save" command to make sure configuration is persistent.

How did you verify/test it?
Run gnmi end to end test and check configuration.

Signed-off-by: Guy Shemesh <[email protected]>
venu-nexthop pushed a commit to venu-nexthop/sonic-mgmt that referenced this pull request Jan 13, 2026
@ganglyu @venu-nexthop
Fix configuration after gnmi test (sonic-net#19150)
34233fb 
What is the motivation for this PR?
Some end to end tests failed because the configuration is wrong after gnmi test.

How did you do it?
Run "config save" command to make sure configuration is persistent.

How did you verify/test it?
Run gnmi end to end test and check configuration.
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Jan 26, 2026
@ganglyu @gshemesh2
Fix configuration after gnmi test (sonic-net#19150)
5fbefc7 
What is the motivation for this PR?
Some end to end tests failed because the configuration is wrong after gnmi test.

How did you do it?
Run "config save" command to make sure configuration is persistent.

How did you verify/test it?
Run gnmi end to end test and check configuration.

Signed-off-by: Guy Shemesh <[email protected]>
ytzur1 pushed a commit to ytzur1/sonic-mgmt that referenced this pull request Feb 2, 2026
@ganglyu @ytzur1
Fix configuration after gnmi test (sonic-net#19150)
9f6fe9e 
What is the motivation for this PR?
Some end to end tests failed because the configuration is wrong after gnmi test.

How did you do it?
Run "config save" command to make sure configuration is persistent.

How did you verify/test it?
Run gnmi end to end test and check configuration.

Signed-off-by: Yael Tzur <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@wangxin wangxin wangxin approved these changes

@wen587 wen587 wen587 approved these changes

Assignees

No one assigned

Labels

Approved for 202411 branch Approved for 202505 branch Included in 202411 branch Included in 202505 branch Request for 202411 branch Request for 202505 branch

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@ganglyu @mssonicbld @wangxin @wen587