Skip to content

[aclorch]: Fix table name in counter table for mirror rules#1060

Merged
yxieca merged 1 commit intosonic-net:masterfrom
stcheng:acl_counter
Sep 19, 2019
Merged

[aclorch]: Fix table name in counter table for mirror rules#1060
yxieca merged 1 commit intosonic-net:masterfrom
stcheng:acl_counter

Conversation

@stcheng
Copy link
Contributor

@stcheng stcheng commented Sep 18, 2019

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]

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]>
@yxieca
Copy link
Contributor

yxieca commented Sep 19, 2019

checked another pr that has policer failure, looks like syncd is slow and is not able to process the request when we end the test.

@yxieca yxieca merged commit 543bd98 into sonic-net:master Sep 19, 2019
yxieca pushed a commit that referenced this pull request Sep 19, 2019
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]>
@stcheng stcheng deleted the acl_counter branch September 19, 2019 04:44
Janetxxx pushed a commit to Janetxxx/sonic-swss that referenced this pull request Nov 10, 2025
…t#1060)

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]>
jianyuewu pushed a commit to jianyuewu/sonic-swss that referenced this pull request Dec 24, 2025
Adding a change to delete multiple keys per one DEL operation instead of one key per DEL
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants