Skip to content
New issue

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

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

Already on GitHub? # to your account

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

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
# for free to join this conversation on GitHub. Already have an account? # 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