Skip to content

Revert "[TableBase] Make channel name from both table name and databa…#574

Merged
qiluo-msft merged 1 commit intosonic-net:masterfrom
qiluo-msft:qiluo/revert568
Jan 15, 2022
Merged

Revert "[TableBase] Make channel name from both table name and databa…#574
qiluo-msft merged 1 commit intosonic-net:masterfrom
qiluo-msft:qiluo/revert568

Conversation

@qiluo-msft
Copy link
Contributor

…se ID (#568)"

This reverts commit bd4d0ce.

@qiluo-msft qiluo-msft marked this pull request as ready for review January 15, 2022 07:43
@qiluo-msft qiluo-msft merged commit 14f068e into sonic-net:master Jan 15, 2022
@qiluo-msft qiluo-msft deleted the qiluo/revert568 branch January 15, 2022 18:43
@qiluo-msft
Copy link
Contributor Author

We revert this commit, because after merging it, all the PR build jobs and master build jobs in sonic-swss repo and sonic-utilities repo will lead to many ERROR in vstest step.

For example: https://dev.azure.com/mssonic/build/_build/results?buildId=65407&view=logs&j=3f6395b2-1619-5ebe-f305-2aedcf353cb5&t=35460df8-21be-5313-3ee4-50900c6db765

test_pbh.py::TestPbhInterfaceBinding::test_PbhTablePortBinding ERROR     [ 60%]
test_pbh.py::TestPbhInterfaceBinding::test_PbhTablePortChannelBinding ERROR [ 60%]
test_pbh.py::TestPbhBasicFlows::test_PbhHashFieldCreationDeletion ERROR  [ 60%]
test_pbh.py::TestPbhBasicFlows::test_PbhHashCreationDeletion ERROR       [ 61%]
test_pbh.py::TestPbhBasicFlows::test_PbhTableCreationDeletion ERROR      [ 61%]
test_pbh.py::TestPbhBasicFlows::test_PbhRuleCreationDeletion ERROR       [ 61%]
test_pbh.py::TestPbhExtendedFlows::test_PbhNvgreVxlanConfiguration ERROR [ 61%]
test_pbh.py::TestPbhDependencyFlows::test_PbhHashCreationDeletionWithDependencies ERROR [ 61%]
test_pbh.py::TestPbhDependencyFlows::test_PbhRuleCreationDeletionWithDependencies ERROR [ 61%]

jimmyzhai added a commit that referenced this pull request Jan 24, 2022
qiluo-msft pushed a commit that referenced this pull request Jan 25, 2022
…d database ID (#568)" (#574)" (#578)

Reverts #574

The real cause is fixed at sonic-net/sonic-sairedis#995. Continue to commit bd4d0ce again.
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.

2 participants