Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug]: [ACL][DualTor] test_acl.py ingress-uplink->downlink forward test cases are failing due to muxcable keepalive packets are droped by acl rules #15206

Open
echuawu opened this issue Oct 28, 2024 · 3 comments

Comments

@echuawu
Copy link
Contributor

echuawu commented Oct 28, 2024

Issue Description

On Dualtor setup, all the ingress-uplink->downlink forward test cases of test_acl.py are failing.
For dual ToR setups, it relies on a set of loopback interfaces for the servers to keep alive between each tor to understand their state (active or standby)
In the test, it generates a series of ACL entries to be applied and verified.
By default there is a "discard all" entry in each ACL table, which breaks the keep alive path on the dual ToR setup.

Results you see

  1. Expected packets would not be received.
  2. Muxcable status keeps in standby mode in both upper and lower tor.

Results you expected to see

In ingress-uplink->downlink forward test cases:

  1. The acl rules should cares the muxcable keep alive packets between test server and switches.
  2. The muxcable status should not in standby mode in both side.
  3. Test expected packets should be received correctly.

Is it platform specific

generic

Relevant log output

No response

Output of show version

No response

Attach files (if any)

No response

@bingwang-ms
Copy link
Collaborator

Hi @echuawu , do you mean the heartbeat from node to ToR is blocked by the testing ACLs? If that's the case, then it's probably a specific issue or limitation on Mellanox device. We didn't see similar issues on other platforms.

@bingwang-ms
Copy link
Collaborator

@echuawu Can you update the test to allow heartbeat?

@echuawu
Copy link
Contributor Author

echuawu commented Nov 6, 2024

@echuawu Can you update the test to allow heartbeat?

Sure, it's in plan.

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

No branches or pull requests

2 participants