Skip to content

Implement translib backend logic for the gNSI Authz#200

Merged
sneelam20 merged 1 commit intosonic-net:masterfrom
niranjanivivek:gnsi_authz_be_pr
Mar 3, 2026
Merged

Implement translib backend logic for the gNSI Authz#200
sneelam20 merged 1 commit intosonic-net:masterfrom
niranjanivivek:gnsi_authz_be_pr

Conversation

@niranjanivivek
Copy link
Contributor

@niranjanivivek niranjanivivek commented Dec 5, 2025

This commit adds the necessary transformer code to read authorization policy counters and related state from the Redis databases.

HLD Reference: gNSI HLD

Frontend: sonic-gnmi sonic-net/sonic-gnmi#549

@mssonicbld
Copy link

/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.

@mssonicbld
Copy link

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link

/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.

@mssonicbld
Copy link

/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.

@mssonicbld
Copy link

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@niranjanivivek niranjanivivek marked this pull request as ready for review December 16, 2025 17:11
@mssonicbld
Copy link

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@niranjanivivek
Copy link
Contributor Author

@sreeiyer-1, Please review and approve

@niranjanivivek
Copy link
Contributor Author

niranjanivivek commented Dec 19, 2025

Hi @ndas7 and @rlucus , @kishanps Please review .

@sreeiyer-1
Copy link

@sreeiyer-1, Please review and approve

The changes look good

Copy link
Contributor

@ndas7 ndas7 left a comment

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, including the associated sonic-gnmi PR

@niranjanivivek
Copy link
Contributor Author

Please add more details in the description, including the associated sonic-gnmi PR

Description is updated.

@ndas7
Copy link
Contributor

ndas7 commented Jan 9, 2026

Could you please add a link to the HLD in the description?

@mssonicbld
Copy link

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

sneelam20
sneelam20 previously approved these changes Mar 2, 2026
@mssonicbld
Copy link

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

This commit adds the necessary transformer code to read authorization policy
counters and related state from the Redis databases.

Signed-off-by: kanchanavelusamy <velusamyk@google.com>
@mssonicbld
Copy link

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@sneelam20 sneelam20 merged commit 45b3556 into sonic-net:master Mar 3, 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.

7 participants