Skip to content

Commit c28827a

Browse files
authored
[aclorch]: Change CFG_ACL_TABLE_NAME to CFG_ACL_TABLE_TABLE_NAME (sonic-net#978)
TABLE_NAME” is the suffix and “ACL_TABLE” is the table name. it is good to keep the convention here and align with all other table names. Please refer to sonic-swss-common pull request 296 Signed-off-by: Shu0T1an ChenG <[email protected]>
1 parent cb60eb1 commit c28827a

File tree

3 files changed

+4
-4
lines changed

3 files changed

+4
-4
lines changed

orchagent/aclorch.cpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -2135,7 +2135,7 @@ void AclOrch::doTask(Consumer &consumer)
21352135

21362136
string table_name = consumer.getTableName();
21372137

2138-
if (table_name == CFG_ACL_TABLE_NAME)
2138+
if (table_name == CFG_ACL_TABLE_TABLE_NAME)
21392139
{
21402140
unique_lock<mutex> lock(m_countersMutex);
21412141
doAclTableTask(consumer);

orchagent/orchdaemon.cpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -154,7 +154,7 @@ bool OrchDaemon::init()
154154
TableConnector confDbMirrorSession(m_configDb, CFG_MIRROR_SESSION_TABLE_NAME);
155155
MirrorOrch *mirror_orch = new MirrorOrch(stateDbMirrorSession, confDbMirrorSession, gPortsOrch, gRouteOrch, gNeighOrch, gFdbOrch, policer_orch);
156156

157-
TableConnector confDbAclTable(m_configDb, CFG_ACL_TABLE_NAME);
157+
TableConnector confDbAclTable(m_configDb, CFG_ACL_TABLE_TABLE_NAME);
158158
TableConnector confDbAclRuleTable(m_configDb, CFG_ACL_RULE_TABLE_NAME);
159159

160160
vector<TableConnector> acl_table_connectors = {

tests/mock_tests/aclorch_ut.cpp

+2-2
Original file line numberDiff line numberDiff line change
@@ -173,7 +173,7 @@ namespace aclorch_test
173173
PortsOrch *portsOrch, MirrorOrch *mirrorOrch, NeighOrch *neighOrch, RouteOrch *routeOrch) :
174174
config_db(config_db)
175175
{
176-
TableConnector confDbAclTable(config_db, CFG_ACL_TABLE_NAME);
176+
TableConnector confDbAclTable(config_db, CFG_ACL_TABLE_TABLE_NAME);
177177
TableConnector confDbAclRuleTable(config_db, CFG_ACL_RULE_TABLE_NAME);
178178

179179
vector<TableConnector> acl_table_connectors = { confDbAclTable, confDbAclRuleTable };
@@ -197,7 +197,7 @@ namespace aclorch_test
197197
void doAclTableTask(const deque<KeyOpFieldsValuesTuple> &entries)
198198
{
199199
auto consumer = unique_ptr<Consumer>(new Consumer(
200-
new swss::ConsumerStateTable(config_db, CFG_ACL_TABLE_NAME, 1, 1), m_aclOrch, CFG_ACL_TABLE_NAME));
200+
new swss::ConsumerStateTable(config_db, CFG_ACL_TABLE_TABLE_NAME, 1, 1), m_aclOrch, CFG_ACL_TABLE_TABLE_NAME));
201201

202202
consumerAddToSync(consumer.get(), entries);
203203

0 commit comments

Comments
 (0)