-
Notifications
You must be signed in to change notification settings - Fork 1.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[swss]: Clear MUX-related state DB tables on start #6759
[swss]: Clear MUX-related state DB tables on start #6759
Conversation
* Add *MUX_CABLE_TABLE* to set of tables to clear on SWSS start, which will clear HW_MUX_CABLE_TABLE and MUX_CABLE_TABLE * Order swss to start before pmon to ensure that DBs are cleared before xcvrd (running inside pmon) starts and re-populates the tables Signed-off-by: Lawrence Lee <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cleanup table lgtm.. @vdahiya12 , @lguohan to check service dependency.
@jleveque , now pmon is affected by swss. |
Thanks for the heads-up! Do we have any concerns that SwSS startup issues could prevent PMon from starting, thus causing thermal overload issues? I believe since this is only a |
@jleveque This dependency should not prevent PMON from starting if SWSS fails, since |
/Azurepipelines run |
Azure Pipelines successfully started running 1 pipeline(s). |
* Add *MUX_CABLE_TABLE* to set of tables to clear on SWSS start, which will clear HW_MUX_CABLE_TABLE and MUX_CABLE_TABLE * Order swss to start before pmon to ensure that DBs are cleared before xcvrd (running inside pmon) starts and re-populates the tables Signed-off-by: Lawrence Lee <[email protected]>
in the chassis supervisor card environment, the pmon needs to before swss. |
Signed-off-by: Lawrence Lee [email protected]
- Why I did it
Port configuration changes can leave keys for unused interfaces behind in STATE DB
- How I did it
will clear HW_MUX_CABLE_TABLE and MUX_CABLE_TABLE
xcvrd (running inside pmon) starts and re-populates the tables
- How to verify it
- Which release branch to backport (provide reason below if selected)
- Description for the changelog
- A picture of a cute animal (not mandatory but encouraged)