Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[syncd] Enable bulk api for next hop objects #1487

Merged
merged 4 commits into from
Dec 27, 2024

Conversation

Ndancejic
Copy link
Contributor

swss code change in 202411 uses nexthop bulk api to process dualtor mux switchover. Adds support in syncd for nexthop bulk api, which is present in SAI.

Currently blocking 202411 and master branch merges for swss. Needs to be backported to 202411 as well.

swss code change in 202411 uses nexthop bulk api to process dualtor mux
switchover. Adds support in syncd for nexthop bulk api, which is present
in SAI.

Signed-off-by: Nikola Dancejic <[email protected]>
@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@Ndancejic
Copy link
Contributor Author

@kcudnik please help review and add request tag for 202411 branch as well, thanks in advance!

this change is needed to unblock swss submodule update PR to buildimage

@prsunny for visibility

sonic-net/sonic-buildimage#21002
sonic-net/sonic-buildimage#21167

kcudnik
kcudnik previously approved these changes Dec 20, 2024
@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@kcudnik
Copy link
Collaborator

kcudnik commented Dec 22, 2024

please fix tests

@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@Ndancejic
Copy link
Contributor Author

Ndancejic commented Dec 26, 2024

Hi @kcudnik all checks are passing now, please help merge when available, thank you!

@prsunny prsunny merged commit 1684aec into sonic-net:master Dec 27, 2024
15 checks passed
@prsunny
Copy link
Contributor

prsunny commented Dec 27, 2024

Merging based on Kamil's previous approval and unblock submodule update

mssonicbld pushed a commit to mssonicbld/sonic-sairedis that referenced this pull request Dec 30, 2024
* [syncd] Enable bulk api for next hop objects

swss code change in 202411 uses nexthop bulk api to process dualtor mux
switchover. Adds support in syncd for nexthop bulk api, which is present
in SAI.
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202411: #1490

mssonicbld pushed a commit that referenced this pull request Dec 31, 2024
* [syncd] Enable bulk api for next hop objects

swss code change in 202411 uses nexthop bulk api to process dualtor mux
switchover. Adds support in syncd for nexthop bulk api, which is present
in SAI.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants