Skip to content

Fix MIRROR_SESSION table macro name#264

Merged
lguohan merged 2 commits intosonic-net:masterfrom
jipanyang:mirror
May 24, 2019
Merged

Fix MIRROR_SESSION table macro name#264
lguohan merged 2 commits intosonic-net:masterfrom
jipanyang:mirror

Conversation

@jipanyang
Copy link
Copy Markdown
Contributor

Signed-off-by: Jipan Yang [email protected]

"MIRROR_SESSION" table is in stateDB.

@yxieca
Copy link
Copy Markdown
Contributor

yxieca commented Apr 22, 2019

This change will cause warm-reboot issue. DB migration code is required to be in place.

@lguohan
Copy link
Copy Markdown
Contributor

lguohan commented Apr 24, 2019

that is in a different PR.

@lguohan lguohan merged commit 0fd1f22 into sonic-net:master May 24, 2019
@stcheng
Copy link
Copy Markdown
Contributor

stcheng commented May 30, 2019

ah this is also causing the current unit tests failed due to the naming of the tables get changed 😿

yxieca pushed a commit that referenced this pull request Sep 11, 2019
prgeor pushed a commit to prgeor/sonic-swss-common that referenced this pull request Feb 27, 2025
* Refactoring  read_porttab_mappings

Signed-off-by: Sudharsan Dhamal Gopalarathnam <[email protected]>

* Addressing review comment
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.

4 participants