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] Fix bulk api object type for next hop group members #685

Merged
merged 1 commit into from
Nov 3, 2020

Conversation

kcudnik
Copy link
Collaborator

@kcudnik kcudnik commented Oct 30, 2020

SAI_OBJECT_TYPE_NEXT_HOP_GROUP_MEMBER object type should be used for m_apis.next_hop_group_api->create_next_hop_group_members

NEXT_HOP_GROUP object type has no bulk api at all

@lguohan
Copy link
Contributor

lguohan commented Oct 30, 2020

retest this please

@kcudnik
Copy link
Collaborator Author

kcudnik commented Oct 30, 2020

retest vs please

@lguohan
Copy link
Contributor

lguohan commented Oct 30, 2020

retest this please

1 similar comment
@lguohan
Copy link
Contributor

lguohan commented Oct 31, 2020

retest this please

@kcudnik
Copy link
Collaborator Author

kcudnik commented Oct 31, 2020

retest vs please

1 similar comment
@kcudnik
Copy link
Collaborator Author

kcudnik commented Nov 1, 2020

retest vs please

@lguohan
Copy link
Contributor

lguohan commented Nov 1, 2020

retest this please

1 similar comment
@lguohan
Copy link
Contributor

lguohan commented Nov 1, 2020

retest this please

@lguohan
Copy link
Contributor

lguohan commented Nov 2, 2020

i think there are still some problem on the bulk api, can you temporarily enable -l option and check if vstest passing?

@kcudnik
Copy link
Collaborator Author

kcudnik commented Nov 2, 2020

for this PR ? or other PR ?
can you point to errors?

@lguohan
Copy link
Contributor

lguohan commented Nov 3, 2020

for example, the previous pr checker results. https://sonic-jenkins.westus2.cloudapp.azure.com/job/vs/job/sonic-sairedis-build-pr/656/

@lguohan
Copy link
Contributor

lguohan commented Nov 3, 2020

well, let me do this. let's merge this pr and open a new pr to do the test.

@lguohan lguohan merged commit 797197c into sonic-net:master Nov 3, 2020
dmytroxshevchuk added a commit to dmytroxshevchuk/sonic-sairedis that referenced this pull request Nov 5, 2020
@kcudnik kcudnik deleted the nhgmfix branch November 5, 2020 16:50
dmytroxshevchuk added a commit to dmytroxshevchuk/sonic-sairedis that referenced this pull request Mar 4, 2021
KostiantynYarovyiBf added a commit to KostiantynYarovyiBf/sonic-sairedis that referenced this pull request Sep 21, 2021
pettershao-ragilenetworks pushed a commit to pettershao-ragilenetworks/sonic-sairedis that referenced this pull request Nov 18, 2022
…t#685)

SAI_OBJECT_TYPE_NEXT_HOP_GROUP_MEMBER object type should be used for m_apis.next_hop_group_api->create_next_hop_group_members

NEXT_HOP_GROUP object type has no bulk api at all
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants