Skip to content

Implement translib backend logic for the gNSI Certz - #198

Merged
sneelam20 merged 1 commit into
sonic-net:masterfrom
jayaragini-hcl:gnsi_certz_be_xfmr
Jan 28, 2026
Merged

Implement translib backend logic for the gNSI Certz#198
sneelam20 merged 1 commit into
sonic-net:masterfrom
jayaragini-hcl:gnsi_certz_be_xfmr

Conversation

@jayaragini-hcl

@jayaragini-hcl jayaragini-hcl commented Nov 19, 2025

Copy link
Copy Markdown
Contributor

This commit adds the necessary transformer code to provide certificate management for gNMI read/write access.

HLD Link

Dependency Chain for Merge

Please follow this merge order,

Base Dependency: #183 (Must be merged first)
Current PR: #198
Frontend Base PR: sonic-net/sonic-gnmi#528
Incremental PR: sonic-net/sonic-gnmi#559

@linux-foundation-easycla

linux-foundation-easycla Bot commented Nov 19, 2025

Copy link
Copy Markdown

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: ndas7 / name: Neha Das (28250c4)

@jayaragini-hcl
jayaragini-hcl marked this pull request as draft November 19, 2025 08:28
@mssonicbld

Copy link
Copy Markdown

/azp run

@azure-pipelines

Copy link
Copy Markdown
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.

@mssonicbld

Copy link
Copy Markdown

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld

Copy link
Copy Markdown

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld

Copy link
Copy Markdown

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld

Copy link
Copy Markdown

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld

Copy link
Copy Markdown

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld

Copy link
Copy Markdown

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld

Copy link
Copy Markdown

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld

Copy link
Copy Markdown

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@jayaragini-hcl
jayaragini-hcl marked this pull request as ready for review January 6, 2026 17:30
@jayaragini-hcl

Copy link
Copy Markdown
Contributor Author

Hi @ndas7 and @rlucas Please review the PR.
cc: @kishanps

@ndas7 ndas7 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add more details in the description such as dependent PRs and HLD

Comment thread translib/db/rcm.go Outdated
Comment thread translib/transformer/xfmr_system.go Outdated
@mssonicbld

Copy link
Copy Markdown

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

Comment thread translib/transformer/xfmr_system.go Outdated
Comment thread translib/transformer/xfmr_system.go
@mssonicbld

Copy link
Copy Markdown

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@jayaragini-hcl

jayaragini-hcl commented Jan 19, 2026

Copy link
Copy Markdown
Contributor Author

Hi @kwangsuk , @ndas7 and @rlucus Could you please review the updated PR
cc: @kishanps

@jayaragini-hcl

Copy link
Copy Markdown
Contributor Author

Hi @kwangsuk , @ndas7 and @rlucas Could you please review the updated PR and share your feedback
cc: @kishanps

@ndas7

ndas7 commented Jan 23, 2026

Copy link
Copy Markdown
Contributor

Adding @rlucus for review

kwangsuk
kwangsuk previously approved these changes Jan 23, 2026
Signed-off-by: Pattela JAYARAGINI <pattelaj@google.com>
@mssonicbld

Copy link
Copy Markdown

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@jayaragini-hcl

Copy link
Copy Markdown
Contributor Author

Hi @ndas7 and @rlucus Could you please review the updated PR
cc: @kishanps

@jayaragini-hcl

jayaragini-hcl commented Jan 27, 2026

Copy link
Copy Markdown
Contributor Author

Hi @kwangsuk Could you please re-approve the PR since previous approval got nullified after addressing Neha's review comments and PR has been rebased.
@rlucus Please review and approve the PR
cc: @kishanps, @ndas7

@sneelam20
sneelam20 merged commit 2744e68 into sonic-net:master Jan 28, 2026
3 checks passed
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.

6 participants