Skip to content

Commit

Permalink
[aclorch]: Fix table name in counter table for mirror rules (#1060)
Browse files Browse the repository at this point in the history
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 <shuche@microsoft.com>
  • Loading branch information
stcheng authored and yxieca committed Sep 19, 2019
1 parent 12c29b4 commit 543bd98
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion orchagent/aclorch.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -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();
}
Expand Down

0 comments on commit 543bd98

Please sign in to comment.