Improve GNMI_CLIENT_CERT table to support multiple roles.#21849
Merged
qiluo-msft merged 11 commits intosonic-net:masterfrom Apr 28, 2025
Merged
Improve GNMI_CLIENT_CERT table to support multiple roles.#21849qiluo-msft merged 11 commits intosonic-net:masterfrom
qiluo-msft merged 11 commits intosonic-net:masterfrom
Conversation
Collaborator
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Collaborator
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Collaborator
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Collaborator
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Collaborator
|
/azp run Azure.sonic-buildimage |
Contributor
Author
|
/azpw run Azure.sonic-buildimage |
Contributor
Author
|
azpw command does not work, close and reopen to trigger build |
Collaborator
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Contributor
Author
|
test case in gnmi/test_gnmi_configdb.py failed because yang validation break. |
Collaborator
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Collaborator
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
liuh-80
force-pushed
the
dev/liuh/gnmi_multiple_role
branch
from
9d04798 to
fcfff8d
Compare
11 tasks
Collaborator
|
Cherry-pick PR to 202411: #23448 |
11 tasks
Collaborator
|
Cherry-pick PR to 202405: #23449 |
11 tasks
Collaborator
|
Cherry-pick PR to 202311: #23456 |
11 tasks
Collaborator
|
Cherry-pick PR to msft-202412: Azure/sonic-buildimage-msft#1420 |
r12f
added
Included in msft-202412 Branch
and removed
Created PR to msft-202412 Branch
labels
opcoder0
pushed a commit
to opcoder0/sonic-mgmt
that referenced
this pull request
Dec 8, 2025
Why I did it GNMI service will change to mapping cname to a role list: sonic-net/sonic-buildimage#21849 To make sure GNMI test case in sonic-mgmt can pass with/without this change, we need improve test case to handle both case. Work item tracking Microsoft ADO: 31561802 How I did it Ignore test case bug github issue: sonic-net#17876 Change GNMI setup code to handle role list by check yang model How to verify it Pass all test case. Signed-off-by: opcoder0 <[email protected]>
AharonMalkin
pushed a commit
to AharonMalkin/sonic-mgmt
that referenced
this pull request
Dec 16, 2025
Why I did it GNMI service will change to mapping cname to a role list: sonic-net/sonic-buildimage#21849 To make sure GNMI test case in sonic-mgmt can pass with/without this change, we need improve test case to handle both case. Work item tracking Microsoft ADO: 31561802 How I did it Ignore test case bug github issue: sonic-net#17876 Change GNMI setup code to handle role list by check yang model How to verify it Pass all test case. Signed-off-by: Aharon Malkin <[email protected]>
gshemesh2
pushed a commit
to gshemesh2/sonic-mgmt
that referenced
this pull request
Dec 21, 2025
Why I did it GNMI service will change to mapping cname to a role list: sonic-net/sonic-buildimage#21849 To make sure GNMI test case in sonic-mgmt can pass with/without this change, we need improve test case to handle both case. Work item tracking Microsoft ADO: 31561802 How I did it Ignore test case bug github issue: sonic-net#17876 Change GNMI setup code to handle role list by check yang model How to verify it Pass all test case. Signed-off-by: Guy Shemesh <[email protected]>
gshemesh2
pushed a commit
to gshemesh2/sonic-mgmt
that referenced
this pull request
Jan 26, 2026
Why I did it GNMI service will change to mapping cname to a role list: sonic-net/sonic-buildimage#21849 To make sure GNMI test case in sonic-mgmt can pass with/without this change, we need improve test case to handle both case. Work item tracking Microsoft ADO: 31561802 How I did it Ignore test case bug github issue: sonic-net#17876 Change GNMI setup code to handle role list by check yang model How to verify it Pass all test case. Signed-off-by: Guy Shemesh <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
9 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Improve GNMI_CLIENT_CERT table to support multiple roles.
Why I did it
GNMI need support multiple roles.
HLD: sonic-net/SONiC#1967
Work item tracking
How I did it
Change GNMI_CLIENT_CERT table yang model.
How to verify it
Pass all test case.
This PR depends on following PRs:
sonic-net/sonic-gnmi#366
sonic-net/sonic-mgmt#17866
Which release branch to backport (provide reason below if selected)
Tested branch (Please provide the tested image version)
Description for the changelog
Improve GNMI_CLIENT_CERT table to support multiple roles.
Link to config_db schema for YANG module changes
A picture of a cute animal (not mandatory but encouraged)