-
Notifications
You must be signed in to change notification settings - Fork 132
Fixed ip6table internal_docker_ip_traffic rule command for multi-asic #94
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
Conversation
Signed-off-by: anamehra <anamehra@cisco.com>
I am working on test case, will update soon. |
2550573
to
5ae479c
Compare
Signed-off-by: anamehra <anamehra@cisco.com>
5ae479c
to
fe142d1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@anamehra this is also needed in 202211, 202305 branches |
…sonic-net#94) * Fixed ip6table internal_docker_ip_traffic rule command for multi-asic * Added test coverage Signed-off-by: anamehra <anamehra@cisco.com>
Cherry-pick PR to 202305: #96 |
…#94) * Fixed ip6table internal_docker_ip_traffic rule command for multi-asic * Added test coverage Signed-off-by: anamehra <anamehra@cisco.com>
…sonic-net#94) * Fixed ip6table internal_docker_ip_traffic rule command for multi-asic * Added test coverage Signed-off-by: anamehra <anamehra@cisco.com>
Cherry-pick PR to 202311: #102 |
…#94) * Fixed ip6table internal_docker_ip_traffic rule command for multi-asic * Added test coverage Signed-off-by: anamehra <anamehra@cisco.com>
Signed-off-by: anamehra anamehra@cisco.com
Fixes: sonic-net/sonic-buildimage#17431
The self.namespace_docker_mgmt_ipv6[namespace] variable was being used as a string causing the cmd to fail.
Fixed the command.
Which release branch to backport (provide reason below if selected)