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 <[email protected]>
  • Loading branch information
stcheng authored and yxieca committed Sep 19, 2019
1 parent ab7d8da commit 3cee6b8
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 @@ -2693,7 +2693,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 3cee6b8

Please sign in to comment.