Skip to content

Commit 28dc042

Browse files
jipanyanglguohan
authored andcommitted
Remove default docker name value of swss. (#692)
* Remove default docker name value of swss. The default docker name caused a lot of confusion when the libraries used in other dockers Signed-off-by: Jipan Yang <[email protected]> * [VS]: fix syncd crash caused by meta_generic_validation_remove reference count error Signed-off-by: Jipan Yang <[email protected]> * [vs]: user sysfs to get mac of server eth0 Signed-off-by: Jipan Yang <[email protected]>
1 parent 067928d commit 28dc042

File tree

4 files changed

+10
-8
lines changed

4 files changed

+10
-8
lines changed

cfgmgr/vlanmgrd.cpp

+2-2
Original file line numberDiff line numberDiff line change
@@ -57,8 +57,8 @@ int main(int argc, char **argv)
5757
DBConnector appDb(APPL_DB, DBConnector::DEFAULT_UNIXSOCKET, 0);
5858
DBConnector stateDb(STATE_DB, DBConnector::DEFAULT_UNIXSOCKET, 0);
5959

60-
WarmStart::initialize("vlanmgrd");
61-
WarmStart::checkWarmStart("vlanmgrd");
60+
WarmStart::initialize("vlanmgrd", "swss");
61+
WarmStart::checkWarmStart("vlanmgrd", "swss");
6262

6363
/*
6464
* swss service starts after interfaces-config.service which will have

orchagent/main.cpp

+2-2
Original file line numberDiff line numberDiff line change
@@ -102,8 +102,8 @@ int main(int argc, char **argv)
102102

103103
SWSS_LOG_ENTER();
104104

105-
WarmStart::initialize("orchagent");
106-
WarmStart::checkWarmStart("orchagent");
105+
WarmStart::initialize("orchagent", "swss");
106+
WarmStart::checkWarmStart("orchagent", "swss");
107107

108108
if (signal(SIGHUP, sighup_handler) == SIG_ERR)
109109
{

portsyncd/portsyncd.cpp

+2-2
Original file line numberDiff line numberDiff line change
@@ -75,8 +75,8 @@ int main(int argc, char **argv)
7575
ProducerStateTable p(&appl_db, APP_PORT_TABLE_NAME);
7676
SubscriberStateTable portCfg(&cfgDb, CFG_PORT_TABLE_NAME);
7777

78-
WarmStart::initialize("portsyncd");
79-
WarmStart::checkWarmStart("portsyncd");
78+
WarmStart::initialize("portsyncd", "swss");
79+
WarmStart::checkWarmStart("portsyncd", "swss");
8080
const bool warm = WarmStart::isWarmStart();
8181

8282
LinkSync sync(&appl_db, &state_db);

tests/test_warm_reboot.py

+4-2
Original file line numberDiff line numberDiff line change
@@ -741,7 +741,8 @@ def test_OrchagentWarmRestartReadyCheck(dvs, testlog):
741741
assert result == "RESTARTCHECK failed\n"
742742

743743
# Cleaning previously pushed route-entry to ease life of subsequent testcases.
744-
del_entry_tbl(appl_db, swsscommon.APP_ROUTE_TABLE_NAME, "2.2.2.0/24")
744+
ps._del("2.2.2.0/24")
745+
time.sleep(1)
745746

746747
# recover for test cases after this one.
747748
dvs.stop_swss()
@@ -1521,7 +1522,8 @@ def test_system_warmreboot_neighbor_syncup(dvs, testlog):
15211522
dvs.runcmd("ip -6 addr add {}00::1/64 dev Ethernet{}".format(i*4,i*4))
15221523
dvs.servers[i].runcmd("ip link set up dev eth0")
15231524
dvs.servers[i].runcmd("ip addr flush dev eth0")
1524-
result = dvs.servers[i].runcmd_output("ifconfig eth0 | grep HWaddr | awk '{print $NF}'")
1525+
#result = dvs.servers[i].runcmd_output("ifconfig eth0 | grep HWaddr | awk '{print $NF}'")
1526+
result = dvs.servers[i].runcmd_output("cat /sys/class/net/eth0/address")
15251527
macs.append(result.strip())
15261528

15271529
#

0 commit comments

Comments
 (0)