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

[202205][Dual-ToR] handle 'mux_tunnel_ingress_acl' attrib in order to change ACL configuration (drop on ingress/egress) on standby ToR #2703

Conversation

ayurkiv-nvda
Copy link
Contributor

What I did
Use "mux_tunnel_ingress_acl" to set ACL rules on ingress/egress side depending on attribute value ("disabled/enabled").

Why I did it
We need to drop data-plane traffic and handle Control-plane traffic in the Dual-ToR scenario.
But we can't do it on Mellanox platform and process traffic on ingress.
To workaround it we can set ACL rules on egress ports, so will process control plane on ingress and drop Data-plane traffic that came from standby port on egress.

How I verified it
check "show mux status" on standby ToR - Mux status should be healthy.
check "show what-just-happened" on standby ToR - no ICMP drop expected on standby ports.

Details if related

…ACL configuration (drop on ingress/egress) on standby ToR (202205)

Signed-off-by: Andriy Yurkiv <[email protected]>
@prsunny
Copy link
Collaborator

prsunny commented Mar 14, 2023

Please have the branch name tagged first in the title

@ayurkiv-nvda ayurkiv-nvda changed the title [Dual-ToR] handle 'mux_tunnel_ingress_acl' attrib in order to change ACL configuration (drop on ingress/egress) on standby ToR (202205) [202205][Dual-ToR] handle 'mux_tunnel_ingress_acl' attrib in order to change ACL configuration (drop on ingress/egress) on standby ToR Mar 14, 2023
@ayurkiv-nvda
Copy link
Contributor Author

ayurkiv-nvda commented Mar 14, 2023

This PR requires YANG model for SYSTEM_DEFAULT.
Waiting for sonic-net/sonic-buildimage#11117 to be merged to 202205

@ayurkiv-nvda
Copy link
Contributor Author

Related PRs
#2646 - logic implementation in master
sonic-net/sonic-buildimage#13547 - config_db implementation in master

@liat-grozovik
Copy link
Collaborator

No need to backport that for 202205. We will use the current implementation

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.

3 participants