From 8efe4634a08cef76161226febd7d7dd5f217c620 Mon Sep 17 00:00:00 2001 From: Shu0T1an ChenG Date: Wed, 18 Sep 2019 16:03:32 -0700 Subject: [PATCH] [aclorch]: Fix table name in counter table for mirror rules In ACL combined mode, v4 and v6 rules are sharing the same physical table while having separated configuration tables. The daemon needs to use the configuration table name to store the counter information. Signed-off-by: Shu0T1an ChenG --- orchagent/aclorch.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/orchagent/aclorch.cpp b/orchagent/aclorch.cpp index 4839feebff8..21496c05fdd 100644 --- a/orchagent/aclorch.cpp +++ b/orchagent/aclorch.cpp @@ -3000,7 +3000,8 @@ void AclOrch::doTask(SelectableTimer &timer) swss::FieldValueTuple fvtb("Bytes", to_string(cnt.bytes)); values.push_back(fvtb); - AclOrch::getCountersTable().set(table_it.second.id + ":" + rule_it.second->getId(), values, ""); + AclOrch::getCountersTable().set(rule_it.second->getTableId() + ":" + + rule_it.second->getId(), values, ""); } values.clear(); }