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

[database] Fix docker-database flush_unused_database failed issue #11677

Merged
merged 1 commit into from
Aug 12, 2022

Commits on Aug 10, 2022

  1. Fix docker database flush_unused_database failed issue (#11600)

    Fix docker-database flush_unused_database failed issue: #11597
    When change flush_unused_database from use swsssdk to use swsscommon, get_instancelist() and get_dblist() name changed but not update.
    
    Change flush_unused_database code to use swsscommon API:
    Change get_instancelist to getInstanceList.
    Change get_dblist to getDbList.
    
    Pass all E2E test.
    Manually check syslog make sure error log not exist and swss, syncd, bgp service started.
    Search code in Azure make sure there all similer case are fixed in this PR.
    
    <!--
    - Note we only backport fixes to a release branch, *not* features!
    - Please also provide a reason for the backporting below.
    - e.g.
    - [x] 202006
    -->
    
    - [ ] 201811
    - [ ] 201911
    - [ ] 202006
    - [ ] 202012
    - [ ] 202106
    - [ ] 202111
    - [ ] 202205
    
    Fix docker-database flush_unused_database failed issue: #11597
    When change flush_unused_database from use swsssdk to use swsscommon, get_instancelist() and get_dblist() name changed but not update.
    
    <!--
    Provide a link to config_db schema for the table for which YANG model
    is defined
    Link should point to correct section on https://github.com/Azure/sonic-buildimage/blob/master/src/sonic-yang-models/doc/Configuration.md
    -->
    
    Co-authored-by: liuh-80 <azureuser@liuh-dev-vm-02.5fg3zjdzj2xezlx1yazx5oxkzd.hx.internal.cloudapp.net>
    liuh-80 and liuh-80 committed Aug 10, 2022
    Configuration menu
    Copy the full SHA
    9106980 View commit details
    Browse the repository at this point in the history