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

[sonic-buildimage] Changes to make network specific sysctl common for both host and docker namespace #4838

Merged
merged 3 commits into from
Jul 1, 2020

Commits on Jun 23, 2020

  1. [sonic-buildimage] Changes to make network specific sysctl

    common for both host and docker namespace (in multi-npu).
    
    This change is triggered with issue found in multi-npu platforms
    where in docker namespace
    net.ipv6.conf.all.forwarding was 0 (should be 1) because of
    which RS/RA message were triggered and link-local router were learnt.
    
    Beside this there were some other sysctl.net.ipv6* params whose value
    in docker namespace is not same as host namespace.
    
    So to make we are always in sync in host and docker namespace
    created common file that list all sysctl.net.* params and used
    both by host and docker namespace. Any change will get applied
    to both namespace.
    
    Signed-off-by: Abhishek Dosi <[email protected]>
    abdosi committed Jun 23, 2020
    Configuration menu
    Copy the full SHA
    3af2577 View commit details
    Browse the repository at this point in the history

Commits on Jun 29, 2020

  1. Address Review Comments and made sure to invoke augtool

    only one and do string concatenation of all set commands
    abdosi committed Jun 29, 2020
    Configuration menu
    Copy the full SHA
    85943dd View commit details
    Browse the repository at this point in the history

Commits on Jul 1, 2020

  1. Address Review Comments.

    abdosi committed Jul 1, 2020
    Configuration menu
    Copy the full SHA
    329606d View commit details
    Browse the repository at this point in the history