Skip to content

Commit 9de93ae

Browse files
committed
Merged PR 6572849: [database] Fix docker-database flush_unused_database failed issue
#### Why I did it Fix docker-database flush_unused_database failed issue: sonic-net#11597 When change flush_unused_database from use swsssdk to use swsscommon, get_instancelist() and get_dblist() name changed but not update. #### How I did it Change flush_unused_database code to use swsscommon API: Change get_instancelist to getInstanceList. Change get_dblist to getDbList. #### How to verify it 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. #### Which release branch to backport (provide reason below if selected) <!-- - 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 #### Description for the changelog Fix docker-database flush_unused_database failed issue: sonic-net#11597 When change flush_unused_database from use swsssdk to use swsscommon, get_instancelist() and get_dblist() name changed but not update. #### Link to config_db schema for YANG module changes <!-- 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 --> #### A picture of a cute animal (not mandatory but encouraged)
1 parent 3fff015 commit 9de93ae

File tree

2 files changed

+7
-7
lines changed

2 files changed

+7
-7
lines changed

dockers/docker-database/docker-database-init.sh

+1-1
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,7 @@ rm $db_cfg_file_tmp
7979

8080
# copy dump.rdb file to each instance for restoration
8181
DUMPFILE=/var/lib/redis/dump.rdb
82-
redis_inst_list=`/usr/bin/python3 -c "import swsssdk; print(' '.join(swsssdk.SonicDBConfig.get_instancelist().keys()))"`
82+
redis_inst_list=`/usr/bin/python3 -c "from swsscommon import swsscommon; print(' '.join(swsscommon.SonicDBConfig.getInstanceList().keys()))"`
8383
for inst in $redis_inst_list
8484
do
8585
mkdir -p /var/lib/$inst

dockers/docker-database/flush_unused_database

+6-6
Original file line numberDiff line numberDiff line change
@@ -11,15 +11,15 @@ while(True):
1111
break
1212
time.sleep(1)
1313

14-
instlists = swsscommon.SonicDBConfig.get_instancelist()
14+
instlists = swsscommon.SonicDBConfig.getInstanceList()
1515
for instname, v in instlists.items():
16-
insthost = v['hostname']
17-
instsocket = v['unix_socket_path']
16+
insthost = v.hostname
17+
instsocket = v.unixSocketPath
1818

19-
dblists = swsscommon.SonicDBConfig.get_dblist()
19+
dblists = swsscommon.SonicDBConfig.getDbList()
2020
for dbname in dblists:
21-
dbid = swsscommon.SonicDBConfig.get_dbid(dbname)
22-
dbinst = swsscommon.SonicDBConfig.get_instancename(dbname)
21+
dbid = swsscommon.SonicDBConfig.getDbId(dbname)
22+
dbinst = swsscommon.SonicDBConfig.getDbInst(dbname)
2323

2424
# this DB is on current instance, skip flush
2525
if dbinst == instname:

0 commit comments

Comments
 (0)