Skip to content

Conversation

@GaladrielZhao
Copy link
Contributor

@GaladrielZhao GaladrielZhao commented Apr 14, 2025

What I did
Add related handling functions for SRv6 VPN Route and PIC Context in RouteSync.
And add some associated fields for database.

Why I did it
To supplement the functionality of SRv6 VPN Route and PIC Context.

How I verified it
This change is part of the PhoenixWing project. The functionality has already been tested and running in the PhoenixWing.
To improve coverage, unit tests using mock objects for the internal processing logic of Srv6 Vpn Route and Pic Context have been added.

@GaladrielZhao GaladrielZhao requested a review from prsunny as a code owner April 14, 2025 11:16
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@prsunny prsunny requested review from abdosi and kperumalbfn April 14, 2025 17:03
@GaladrielZhao GaladrielZhao force-pushed the add_pic_ctx_vpn_route_support branch from 69022cd to 008ad4c Compare April 21, 2025 11:52
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@GaladrielZhao GaladrielZhao force-pushed the add_pic_ctx_vpn_route_support branch from 008ad4c to d250a91 Compare April 22, 2025 11:32
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@GaladrielZhao GaladrielZhao force-pushed the add_pic_ctx_vpn_route_support branch from d250a91 to 56d38e8 Compare April 22, 2025 14:27
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@GaladrielZhao GaladrielZhao force-pushed the add_pic_ctx_vpn_route_support branch from 1730a1f to b2e290c Compare April 25, 2025 10:47
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@GaladrielZhao GaladrielZhao force-pushed the add_pic_ctx_vpn_route_support branch from b2e290c to 47c9d31 Compare May 9, 2025 14:18
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@GaladrielZhao GaladrielZhao force-pushed the add_pic_ctx_vpn_route_support branch from 47c9d31 to 60bbcd5 Compare May 14, 2025 09:57
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@GaladrielZhao GaladrielZhao force-pushed the add_pic_ctx_vpn_route_support branch 2 times, most recently from 0ee3cff to fa7ebe2 Compare June 4, 2025 06:00
@yuezhoujk
Copy link

/azp run Azure.sonic-swss

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@yuezhoujk
Copy link

/azp run Azure.sonic-swss

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@GaladrielZhao GaladrielZhao force-pushed the add_pic_ctx_vpn_route_support branch from a13e9a0 to 2a775c9 Compare October 17, 2025 02:24
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@GaladrielZhao
Copy link
Contributor Author

@GaladrielZhao Can you please resolve conflicts and address my comments?

@dgsudharsan Thanks for your review. I've resolved the conflicts and updated the code to use the new API. The branch has been rebased onto the latest mainline.

@yuezhoujk
Copy link

/azp run Azure.sonic-swss

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@eddieruan-alibaba
Copy link
Contributor

@dgsudharsan @abdosi can you help to finish the review ?

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@GaladrielZhao GaladrielZhao force-pushed the add_pic_ctx_vpn_route_support branch from ae46d0d to e24416c Compare October 24, 2025 07:12
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@prsunny
Copy link
Collaborator

prsunny commented Oct 28, 2025

@abdosi for viz , merging after PR checkers.

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@GaladrielZhao GaladrielZhao force-pushed the add_pic_ctx_vpn_route_support branch from b82d51a to cfb24bb Compare October 29, 2025 11:50
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Add support for SRv6 VPN Route message information parsing and processing,
supplementing related tables and entries.

Signed-off-by: Yuqing Zhao <[email protected]>
@GaladrielZhao GaladrielZhao force-pushed the add_pic_ctx_vpn_route_support branch from df7cf0b to 1195a13 Compare November 13, 2025 11:01
@mssonicbld
Copy link
Collaborator

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

8 participants