Skip to content

[mirrororch]: Remove mirror session state after it is remvoed#1066

Merged
stcheng merged 1 commit intosonic-net:masterfrom
stcheng:mirror_state_db
Sep 26, 2019
Merged

[mirrororch]: Remove mirror session state after it is remvoed#1066
stcheng merged 1 commit intosonic-net:masterfrom
stcheng:mirror_state_db

Conversation

@stcheng
Copy link
Contributor

@stcheng stcheng commented Sep 20, 2019

Remove deprecated mirror session states in the state database

Signed-off-by: Shu0t1an Cheng [email protected]

Remove deprecated mirror session states in the state database

Signed-off-by: Shu0t1an Cheng <[email protected]>
@stcheng
Copy link
Contributor Author

stcheng commented Sep 23, 2019

retest this please

1 similar comment
@stcheng
Copy link
Contributor Author

stcheng commented Sep 24, 2019

retest this please

@lguohan
Copy link
Contributor

lguohan commented Sep 24, 2019

can you describe the effort if a deprecated mirror session is not removed from state db? does it impact the warm reboot?

@stcheng
Copy link
Contributor Author

stcheng commented Sep 24, 2019

@lguohan it doesn't impact the warm reboot; it could clean up the deprecated information or extra information stored before the warm reboot.

Copy link
Contributor

@lguohan lguohan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please fix indentation.

@stcheng
Copy link
Contributor Author

stcheng commented Sep 26, 2019

Indentations are fixed.

@stcheng stcheng merged commit 182940d into sonic-net:master Sep 26, 2019
@stcheng stcheng deleted the mirror_state_db branch September 26, 2019 23:11
yxieca pushed a commit that referenced this pull request Sep 26, 2019
Remove deprecated mirror session states in the state database

Signed-off-by: Shu0t1an Cheng <[email protected]>
Janetxxx pushed a commit to Janetxxx/sonic-swss that referenced this pull request Nov 10, 2025
…net#1066)

Remove deprecated mirror session states in the state database

Signed-off-by: Shu0t1an Cheng <[email protected]>
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.

3 participants