Skip to content

Return duplicate prefix list entry#22164

Closed
sudhanshukumar22 wants to merge 3 commits intosonic-net:masterfrom
sudhanshukumar22:return_duplicate_prefix_list_entry
Closed

Return duplicate prefix list entry#22164
sudhanshukumar22 wants to merge 3 commits intosonic-net:masterfrom
sudhanshukumar22:return_duplicate_prefix_list_entry

Conversation

@sudhanshukumar22
Copy link
Contributor

Why I did it

Work item tracking
  • Microsoft ADO (number only):

How I did it

How to verify it

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

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
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

From: Donatas Abraitis <donatas@opensourcerouting.org>
Date: Tue, 25 Mar 2025 13:54:24 +0200
Subject: [PATCH] lib: Return duplicate prefix-list entry

If we do e.g.:

ip prefix-list PL_LoopbackV4 permit 10.1.0.32/32
ip prefix-list PL_LoopbackV4 permit 10.1.0.32/32
ip prefix-list PL_LoopbackV4 permit 10.1.0.32/32

We end up, having duplicate records with a different sequence number only.

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>
@sudhanshukumar22 sudhanshukumar22 force-pushed the return_duplicate_prefix_list_entry branch from b87dbc0 to f8ef016 Compare March 27, 2025 11:03
@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@sudhanshukumar22
Copy link
Contributor Author

Pushing another PR #22165

@sudhanshukumar22 sudhanshukumar22 deleted the return_duplicate_prefix_list_entry branch March 27, 2025 11:20
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