Skip to content

Add support for fdb flush in sairedis#288

Merged
kcudnik merged 1 commit intosonic-net:masterfrom
kcudnik:flush
Jan 24, 2018
Merged

Add support for fdb flush in sairedis#288
kcudnik merged 1 commit intosonic-net:masterfrom
kcudnik:flush

Conversation

@kcudnik
Copy link
Collaborator

@kcudnik kcudnik commented Jan 23, 2018

No description provided.

@kcudnik
Copy link
Collaborator Author

kcudnik commented Jan 23, 2018

test this please

return SAI_STATUS_INVALID_PARAMETER;
}

if (attr_count > MAX_LIST_COUNT)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should at least have one attribute in the list?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no, since list can be empty

@keboliu
Copy link
Collaborator

keboliu commented Jan 24, 2018

I tested this with my PRs, it works well.

@kcudnik
Copy link
Collaborator Author

kcudnik commented Jan 24, 2018

test this please

@kcudnik kcudnik merged commit f8b9385 into sonic-net:master Jan 24, 2018
@kcudnik kcudnik deleted the flush branch January 24, 2018 20:03
pettershao-ragilenetworks pushed a commit to pettershao-ragilenetworks/sonic-sairedis that referenced this pull request Nov 18, 2022
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.

3 participants