Skip to content

[SRV6] Sonic-swss changes for SRV6#1964

Merged
prsunny merged 38 commits intosonic-net:masterfrom
kperumalbfn:swss_srv6
Nov 11, 2021
Merged

[SRV6] Sonic-swss changes for SRV6#1964
prsunny merged 38 commits intosonic-net:masterfrom
kperumalbfn:swss_srv6

Conversation

@kperumalbfn
Copy link
Copy Markdown
Contributor

@kperumalbfn kperumalbfn commented Oct 15, 2021

SONiC swss support for SRV6

  • RouteOrch changes to trigger SRV6 nexthops and update route entries
  • SRV6Orch changes to create SRV6 nexthops, tunnel and SRV6 MY_SID_ENTRY objects
  • CRMOrch changes to include SRV6 MY_SID

sonic-swss-common changes for SRV6 APPL_DB tables: sonic-net/sonic-swss-common#538
sonic-utiliities changes for SRV6 CRM: sonic-net/sonic-utilities#1883
sonic-buildimage changes for SRV6 CRM: sonic-net/sonic-buildimage#9004

SRV6 HLD: sonic-net/SONiC#795

@kperumalbfn kperumalbfn requested a review from prsunny as a code owner October 15, 2021 19:27
@kperumalbfn
Copy link
Copy Markdown
Contributor Author

sonic-swss srv6 test scripts will be pushed soon.

@zhangyanzhao
Copy link
Copy Markdown

@caizhenghui-juniper would you please help to review?

@hzheng5 hzheng5 mentioned this pull request Oct 21, 2021
Comment thread orchagent/nexthopkey.h
Copy link
Copy Markdown
Collaborator

@prsunny prsunny 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 VS tests

@prsunny
Copy link
Copy Markdown
Collaborator

prsunny commented Oct 26, 2021

@venkatmahalingam , please review

Comment thread orchagent/neighorch.cpp
Comment thread orchagent/srv6orch.cpp Outdated
Comment thread orchagent/srv6orch.cpp
@kperumalbfn
Copy link
Copy Markdown
Contributor Author

@caizhenghui-juniper Please review this PR and approve it.

@kperumalbfn
Copy link
Copy Markdown
Contributor Author

/AzurePipelines run

@azure-pipelines
Copy link
Copy Markdown

Commenter does not have sufficient privileges for PR 1964 in repo Azure/sonic-swss

@kperumalbfn
Copy link
Copy Markdown
Contributor Author

/azpw run

@mssonicbld
Copy link
Copy Markdown
Collaborator

/AzurePipelines run

Comment thread orchagent/orchdaemon.cpp Outdated
Comment thread orchagent/crmorch.cpp Outdated
Comment thread orchagent/routeorch.cpp Outdated
Comment thread orchagent/routeorch.cpp
Comment thread orchagent/routeorch.cpp Outdated
Comment thread orchagent/routeorch.cpp Outdated
Comment thread orchagent/srv6orch.cpp Outdated
Comment thread orchagent/srv6orch.cpp
Comment thread orchagent/nexthopgroupkey.h Outdated
Comment thread orchagent/routeorch.cpp
Comment thread orchagent/routeorch.cpp
@prsunny
Copy link
Copy Markdown
Collaborator

prsunny commented Nov 11, 2021

@venkatmahalingam , @caizhenghui-juniper , please review/sign-off.

@kperumalbfn
Copy link
Copy Markdown
Contributor Author

Thanks @prsunny @venkatmahalingam

@prsunny @caizhenghui-juniper's approval got cleared after code changes.

@kperumalbfn
Copy link
Copy Markdown
Contributor Author

@prsunny Please merge

@reshmaintel
Copy link
Copy Markdown

@prsunny @venkatmahalingam @caizhenghui-juniper @zhangyanzhao
Thank you for your review help so far. Could you please help with the last step review/merge for this PR as well.

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