Skip to content

Fix issue 158#161

Closed
chrispsommers wants to merge 3 commits intosonic-net:mainfrom
chrispsommers:fix-issue-158
Closed

Fix issue 158#161
chrispsommers wants to merge 3 commits intosonic-net:mainfrom
chrispsommers:fix-issue-158

Conversation

@chrispsommers
Copy link
Copy Markdown
Collaborator

@chrispsommers chrispsommers commented Jul 22, 2022

Fix #158 - libsai delete operation failure and log msg. The test for error code was backwards and log message used same write updateType string for all operations. I fixed the compare and used a dynamic enum print method.

…test for error code was backwards and log message used same write updateType string for all operations. I fixed the compare and used a dynamic enum print method.
…sg. THe test for error code was backwards and log message used same write updateType string for all operations. I fixed the compare and used a dynamic enum print method."

This reverts commit 820e08e.
…test for error code was backwards and log message used same write updateType string for all operations. I fixed the compare and used a dynamic enum print method.
@marian-pritsak
Copy link
Copy Markdown
Collaborator

Can we add the exact description of the fix instead of issue #? It's impossible to track in git.

@chrispsommers
Copy link
Copy Markdown
Collaborator Author

Can we add the exact description of the fix instead of issue #? It's impossible to track in git.

@marian-pritsak Sorry, the commit message didn't display. Please see revised description.

@chrispsommers
Copy link
Copy Markdown
Collaborator Author

Fixed in #181, no need for this PR any more.

@chrispsommers chrispsommers deleted the fix-issue-158 branch August 9, 2022 18:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

DASH libsai fails on remove() operations, wrong log message

2 participants