Skip to content

[acl/test_src_mac_rewrite]: Fix ACL counter and acl modification handling - #1374

Open
Aakash Tiwari (aakashtiwari2630-max) wants to merge 1 commit into
Azure:202511_azdfrom
aakashtiwari2630-max:aaktiwar/srcmac_test
Open

[acl/test_src_mac_rewrite]: Fix ACL counter and acl modification handling#1374
Aakash Tiwari (aakashtiwari2630-max) wants to merge 1 commit into
Azure:202511_azdfrom
aakashtiwari2630-max:aaktiwar/srcmac_test

Conversation

@aakashtiwari2630-max

Copy link
Copy Markdown

Porting master PR - sonic-net/sonic-mgmt#25114

Summary of the fix:

  1. In case of range test, the test aim is to send multiple packets and validate acl counters but after sending first packet, the wait_until logic was wrong due to which the count_after was read incorrectly. Fix is to pass the correct previous counter so that count after is incremented correctly.
  2. Modify ACL rule tries to modify Mac field in acl rule by doing a HSET call in APP_DB in redis. This doesn't update the STATE_DB. Correct way is to call setup_acl_rules with updated mac.
  3. _check_vxlan_tunnel_config was checking the APP_DB incorrectly. Fixed it.

…ling

Signed-off-by: Aakash Tiwari <aaktiwar@cisco.com>
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant