Skip to content

[syncd] Comparison logic workaround for empty buffer profile #906

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

Merged
merged 9 commits into from
Sep 4, 2021

Conversation

kcudnik
Copy link
Collaborator

@kcudnik kcudnik commented Aug 25, 2021

Fixes #899

@kcudnik kcudnik changed the title [syncd] Comparison logic workaround for empty buffer profile. [syncd] Comparison logic workaround for empty buffer profile Aug 25, 2021
@kcudnik
Copy link
Collaborator Author

kcudnik commented Aug 31, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@kcudnik kcudnik merged commit 509b888 into sonic-net:master Sep 4, 2021
@kcudnik kcudnik deleted the wipbf1 branch September 4, 2021 09:22
@svsivm
Copy link

svsivm commented Sep 22, 2021

Hi Kamil, thanks for this fix. Would it be possible to double commit to 202012 branch also please?

kcudnik added a commit to kcudnik/sonic-sairedis that referenced this pull request Sep 22, 2021
…et#906)

* [syncd] WIP buffer profile GET

* Add object status serialize method

* Add buffer profile get test

* [syncd] Allow AsicView return dummy created object

* Update aspell

* [syncd] Transfer not processed objects to temp view

Check issue sonic-net#899
for details.

* [tests] Add missing recording file for tests
@kcudnik
Copy link
Collaborator Author

kcudnik commented Sep 22, 2021

#941

pettershao-ragilenetworks pushed a commit to pettershao-ragilenetworks/sonic-sairedis that referenced this pull request Nov 18, 2022
…et#906)

* [syncd] WIP buffer profile GET

* Add object status serialize method

* Add buffer profile get test

* [syncd] Allow AsicView return dummy created object

* Update aspell

* [syncd] Transfer not processed objects to temp view

Check issue sonic-net#899
for details.

* [tests] Add missing recording file for tests
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.

Warmboot reconciliation fails when zero buffer profiles are attached to some queues
2 participants