Skip to content
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

Flush RESTAPI_DB when config reload is performed #9037

Merged
merged 1 commit into from
Oct 22, 2021

Conversation

sumukhatv
Copy link
Contributor

Why I did it

The RESTAPI_DB consists of the reset_status and needs to be reset upon config reload

How I did it

Flush the RESTAPI_DB contents upon config reload

How to verify it

reset_status API will yield true after this operation
RESET_STATUS table in RESTAPI_DB will not be present after this operation.

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106

Description for the changelog

A picture of a cute animal (not mandatory but encouraged)

@@ -159,6 +159,7 @@ start() {
$SONIC_DB_CLI FLEX_COUNTER_DB FLUSHDB
$SONIC_DB_CLI GB_ASIC_DB FLUSHDB
$SONIC_DB_CLI GB_COUNTERS_DB FLUSHDB
$SONIC_DB_CLI RESTAPI_DB FLUSHDB
Copy link
Contributor

Choose a reason for hiding this comment

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

As discussed, lets add sonic-mgmt tests to cover this.

@sumukhatv sumukhatv merged commit 3971c20 into sonic-net:master Oct 22, 2021
@sumukhatv sumukhatv deleted the restapi_reload branch October 22, 2021 18:45
@qiluo-msft
Copy link
Collaborator

This commit could not be cleanly cherry-picked to 202012. Please submit another PR.

@prsunny prsunny added the P1 Priority of the issue, lower than P0 label Jan 18, 2023
@prsunny
Copy link
Contributor

prsunny commented Jan 19, 2023

Merged to 202012 as part of #9093

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