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

Enable ipv6 for namespaces in multi-npu system #4426

Merged
merged 1 commit into from
Apr 22, 2020

Conversation

arlakshm
Copy link
Contributor

Signed-off-by: Arvindsrinivasan Lakshmi Narasimhan [email protected]

- What I did
Enable ipv6 in Kernel when dockers are in bridge network. This is applicable for multi-npu platforms.

- How I did it

- How to verify it
Verify ipv6 is enabled in kernel on single and multi NPU VS platforms.

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

Signed-off-by: Arvindsrinivasan Lakshmi Narasimhan <[email protected]>
@arlakshm arlakshm requested review from SuvarnaMeenakshi, gxy001 and lguohan and removed request for gxy001 April 14, 2020 15:57
@lguohan
Copy link
Collaborator

lguohan commented Apr 14, 2020

suggest to change the pr title to [docker-database]: enable ipv6 for namespaces in multi-npu system

@arlakshm arlakshm changed the title Enable ipv6 in Kernel when dockers are in bridge network Enable ipv6 for namespaces in multi-npu system Apr 14, 2020
@arlakshm
Copy link
Contributor Author

suggest to change the pr title to [docker-database]: enable ipv6 for namespaces in multi-npu system

Modified the title

@lguohan
Copy link
Collaborator

lguohan commented Apr 15, 2020

retest vsimage please

2 similar comments
@arlakshm
Copy link
Contributor Author

retest vsimage please

@arlakshm
Copy link
Contributor Author

retest vsimage please

@lguohan lguohan merged commit 3a82ade into sonic-net:master Apr 22, 2020
abdosi pushed a commit that referenced this pull request Apr 27, 2020
@arlakshm arlakshm deleted the multi_npu_ipv6 branch May 12, 2020 23:26
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.

4 participants