Skip to content

Deduplicate Subscriptions#586

Open
niranjanivivek wants to merge 1 commit intosonic-net:masterfrom
niranjanivivek:deduplicate_pr
Open

Deduplicate Subscriptions#586
niranjanivivek wants to merge 1 commit intosonic-net:masterfrom
niranjanivivek:deduplicate_pr

Conversation

@niranjanivivek
Copy link
Contributor

@niranjanivivek niranjanivivek commented Feb 25, 2026

Implement Deduplicate subscriptions.

Dependency PR:
Base BE PR to be merged first : Implement RCM - PR#202

UT Test Report

gnmi_server

=== RUN TestSubscriptionDeduplication
--- PASS: TestSubscriptionDeduplication (5.69s)

sonic_data_client

=== RUN TestCreateSuperSubscription
=== RUN TestCreateSuperSubscription/CreateSuperSubscription
=== RUN TestCreateSuperSubscription/CreateNilSuperSubscription
--- PASS: TestCreateSuperSubscription (0.00s)
--- PASS: TestCreateSuperSubscription/CreateSuperSubscription (0.00s)
--- PASS: TestCreateSuperSubscription/CreateNilSuperSubscription (0.00s)
=== RUN TestFindSuperSubscription
=== RUN TestFindSuperSubscription/FindExistingSuperSubscription
=== RUN TestFindSuperSubscription/FindNonExistingSuperSubscription
=== RUN TestFindSuperSubscription/FindNilSuperSubscription
--- PASS: TestFindSuperSubscription (0.00s)
--- PASS: TestFindSuperSubscription/FindExistingSuperSubscription (0.00s)
--- PASS: TestFindSuperSubscription/FindNonExistingSuperSubscription (0.00s)
--- PASS: TestFindSuperSubscription/FindNilSuperSubscription (0.00s)
=== RUN TestDeleteSuperSubscription
=== RUN TestDeleteSuperSubscription/DeleteExistingSuperSubscription
=== RUN TestDeleteSuperSubscription/DeleteNonExistingSuperSubscription
=== RUN TestDeleteSuperSubscription/DeleteNilSuperSubscription
--- PASS: TestDeleteSuperSubscription (0.00s)
--- PASS: TestDeleteSuperSubscription/DeleteExistingSuperSubscription (0.00s)
--- PASS: TestDeleteSuperSubscription/DeleteNonExistingSuperSubscription (0.00s)
--- PASS: TestDeleteSuperSubscription/DeleteNilSuperSubscription (0.00s)

Why I did it

How I did it

How to verify it

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

@mssonicbld
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@niranjanivivek
Copy link
Contributor Author

@sreeiyer-1, Please review.

@niranjanivivek
Copy link
Contributor Author

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

@niranjanivivek niranjanivivek marked this pull request as ready for review March 3, 2026 01:07
@mssonicbld
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Signed-off-by: Niranjani Vivek <niranjaniv@google.com>
@mssonicbld
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

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