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

Changes to support bcmsh and swss logs on multi npu platforms #4783

Closed
wants to merge 30 commits into from

Commits on Jun 16, 2020

  1. Changes to support bcmsh and swss logs on multi npu platforms

    Signed-off-by: Arvindsrinivasan Lakshmi Narasimhan <arlakshm@microsoft.com>
    arlakshm committed Jun 16, 2020
    Configuration menu
    Copy the full SHA
    6a4d512 View commit details
    Browse the repository at this point in the history
  2. minor fix

    Signed-off-by: Arvindsrinivasan Lakshmi Narasimhan <arlakshm@microsoft.com>
    arlakshm committed Jun 16, 2020
    Configuration menu
    Copy the full SHA
    ea3607f View commit details
    Browse the repository at this point in the history

Commits on Jun 17, 2020

  1. minor fix and bcmsh changes

    Signed-off-by: Arvindsrinivasan Lakshmi Narasimhan <arlakshm@microsoft.com>
    arlakshm committed Jun 17, 2020
    Configuration menu
    Copy the full SHA
    8db1174 View commit details
    Browse the repository at this point in the history
  2. minor fixes

    Signed-off-by: Arvindsrinivasan Lakshmi Narasimhan <arlakshm@microsoft.com>
    arlakshm committed Jun 17, 2020
    Configuration menu
    Copy the full SHA
    c660fc9 View commit details
    Browse the repository at this point in the history

Commits on Jun 29, 2020

  1. Add the test signing certificates for secure boot (sonic-net#4866)

    * Add the test signing certificates for secure boot
    * Remove unnecessary ca key file
    * Regenerate the certificates to not expose the ca key
    xumia authored Jun 29, 2020
    Configuration menu
    Copy the full SHA
    5f16e96 View commit details
    Browse the repository at this point in the history

Commits on Jun 30, 2020

  1. syslog changes Multi ASIC platforms (sonic-net#4738)

    Add changes for syslog support for containers running in namespaces on multi ASIC platforms.
    On Multi ASIC platforms
    
    Rsyslog service is only running on the host. There is no rsyslog service running in each namespace.
    On multi ASIC platforms the rsyslog service on the host will be listening on the docker0 ip address instead of loopback address.
    The rsyslog.conf on the containers is modified to have omfwd target ip to be docker0 ipaddress instead of loopback ip
    
    Signed-off-by: Arvindsrinivasan Lakshmi Narasimhan <arlakshm@microsoft.com>
    arlakshm authored Jun 30, 2020
    Configuration menu
    Copy the full SHA
    ef994a1 View commit details
    Browse the repository at this point in the history
  2. [sonic-platform-common] Update submodule (sonic-net#4871)

    * src/sonic-platform-common 82bbeab...42781ff (1):
      > [SfpBase] Fix key name typo in docstring (sonic-net#99)
    
    Signed-off-by: Volodymyr Boyko <volodymyrx.boiko@intel.com>
    vboykox authored Jun 30, 2020
    Configuration menu
    Copy the full SHA
    22bf545 View commit details
    Browse the repository at this point in the history
  3. [mellanox]: Update SAI to 1.16.5 (sonic-net#4873)

    1.  Upgrade SAI headers to v1.6.3
    2.  Fix traffic lost during FFB related to buffer config + optimize buffer config timing for FB
    3.  Add ACL fields BTH, IP flags
    4.  Add ACL infrastructure of different fields per ASIC type
    keboliu authored Jun 30, 2020
    Configuration menu
    Copy the full SHA
    ba234ab View commit details
    Browse the repository at this point in the history
  4. Support for connecting to DB in namespace via TCP port in multi-asic …

    …platform. (sonic-net#4779)
    
    * Support for connecting to DB in namespace via IP:port ( using docker bridge network ) for applications in multi-asic platform.
    
    * Added the default IP as 127.0.0.1 if the IPaddress derivation from interface fails.
    Moved the localhost loopback IP binding logic into the supervisor.j2 file.
    judyjoseph authored Jun 30, 2020
    Configuration menu
    Copy the full SHA
    6e1ae35 View commit details
    Browse the repository at this point in the history
  5. [sonic-sairedis] sonic-sairedis submodule update (sonic-net#4847)

    * sonic-sairedis submodule update
    * Update BRCM SAI to 3.7.5.1
    smaheshm authored Jun 30, 2020
    Configuration menu
    Copy the full SHA
    ed7fafc View commit details
    Browse the repository at this point in the history

Commits on Jul 1, 2020

  1. [docker-nat]: Updated the NAT iptables patch for 4.19 buster (sonic-n…

    …et#4843)
    
    Updated the NAT iptables patch for 4.19 buster
    
    Depends on PR : sonic-net/sonic-linux-kernel#147
    
    1 Known issue:
    
    With both NAT patch files for 4.19 buster kernel, seeing 1 display issue in iptables like explained below
    
    On Docker NAT, iptables supported version is 1.6.0 and on base OS it’s 1.8.2. So seeing an display issue of which fullcone option is not showing in version 1.8.2 iptables output and no issues in functionality.
    
    Display issue – For example of comparsion:
    
    NAT Docker:
    root@sonic:/home/admin# docker exec -it nat bash
    root@sonic:/# iptables -t nat -nvL
    Chain PREROUTING (policy ACCEPT 0 packets, 0 bytes)
    pkts bytes target prot opt in out source destination
    0 0 DNAT all -- * * 0.0.0.0/0 0.0.0.0/0 to:1.1.1.1 fullcone
    
    Chain INPUT (policy ACCEPT 0 packets, 0 bytes)
    pkts bytes target prot opt in out source destination
    
    Chain OUTPUT (policy ACCEPT 30 packets, 2749 bytes)
    pkts bytes target prot opt in out source destination
    
    Chain POSTROUTING (policy ACCEPT 30 packets, 2749 bytes)
    pkts bytes target prot opt in out source destination
    root@sonic:/#
    
    Base OS:
    root@sonic:/home/admin# iptables-legacy -t nat -nvL
    Chain PREROUTING (policy ACCEPT 0 packets, 0 bytes)
    pkts bytes target prot opt in out source destination
    1 36 DNAT all -- * * 0.0.0.0/0 0.0.0.0/0 to:1.1.1.1
    
    Chain INPUT (policy ACCEPT 1 packets, 36 bytes)
    pkts bytes target prot opt in out source destination
    
    Chain OUTPUT (policy ACCEPT 41 packets, 3572 bytes)
    pkts bytes target prot opt in out source destination
    
    Chain POSTROUTING (policy ACCEPT 41 packets, 3572 bytes)
    pkts bytes target prot opt in out source destination
    root@sonic:/home/admin#
    
    To fix this issue, iptables need to update from 1.6.0 to 1.8.2 version and have to update the NAT docker from stretch to buster. Will raise a new PR with this.
    
    Signed-off-by: Akhilesh Samineni akhilesh.samineni@broadcom.com
    
    Signed-off-by: Akhilesh Samineni <akhilesh.samineni@broadcom.com>
    AkhileshSamineni authored Jul 1, 2020
    Configuration menu
    Copy the full SHA
    2187144 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    243268f View commit details
    Browse the repository at this point in the history
  3. [sonic-buildimage] Changes to make network specific sysctl common for…

    … both host and docker namespace (sonic-net#4838)
    
    * [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 <abdosi@microsoft.com>
    
    * Address Review Comments and made sure to invoke augtool
    only one and do string concatenation of all set commands
    
    * Address Review Comments.
    abdosi authored Jul 1, 2020
    Configuration menu
    Copy the full SHA
    e66cb47 View commit details
    Browse the repository at this point in the history

Commits on Jul 2, 2020

  1. [pfx_filter]: Add a prefix mask by default in pfx_filter, when there …

    …is no one (sonic-net#4860)
    
    If some table with a list of tuples (interface name, ip prefix) has ip prefixes without a mask length, it will cause issues in SONiC. For example quagga and frr will treat ipv4 address without a mask, so "10.20.30.40" address will be treated as "10.0.0.0/8", which is dangerous.
    
    The fix here is that when pfx_filter get a tuple (interface name, ip prefix), where the ip prefix doesn't have prefix mask length, add a mask by default: "/32 for ipv4 addresses, /128 for ipv6 addresses".
    
    Co-authored-by: Pavel Shirshov <pavel.contrib@gmail.com>
    pavel-shirshov and Pavel Shirshov authored Jul 2, 2020
    Configuration menu
    Copy the full SHA
    2e0aa4f View commit details
    Browse the repository at this point in the history
  2. Tests of FRR templates which rendered by sonic-cfggen (sonic-net#4875)

    * Tests of FRR templates which rendered by sonic-cfggen
    pavel-shirshov authored Jul 2, 2020
    Configuration menu
    Copy the full SHA
    b72fed7 View commit details
    Browse the repository at this point in the history
  3. [config]: Loopback Interface changes for multi ASIC devices (sonic-ne…

    …t#4825)
    
    * Loopback IP changes for multi ASIC devices
    multi ASIC will have 2 Loopback Interfaces
    
    - Loopback0 has globally unique IP address, which is advertised by the multi ASIC device to its peers.
    This way all the external devices will see this device as a single device.
    - Loopback4096 is assigned an IP address which has a scope is within the device. Each ASIC has a different ip address for Loopback4096. This ip address will be used as Router-Id by the bgp instance on multi ASIC devices.
    
    This PR implements this change for multi ASIC devices
    
    Signed-off-by: Arvindsrinivasan Lakshmi Narasimhan <arlakshm@microsoft.com>
    arlakshm authored Jul 2, 2020
    Configuration menu
    Copy the full SHA
    cae65a4 View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    a4253af View commit details
    Browse the repository at this point in the history

Commits on Jul 3, 2020

  1. Configuration menu
    Copy the full SHA
    5f31842 View commit details
    Browse the repository at this point in the history
  2. [docker-orchagent]: make build depends only on sairedis package (soni…

    …c-net#4880)
    
    make swss build depends only on libsairedis instead of syncd. This allows to build swss without depending
    on vendor sai library.
    
    Currently, libsairedis build also buils syncd which requires vendor SAI lib. This makes difficult to build
    swss docker in buster while still keeping syncd docker in stretch, as swss requires libsairedis which also
    build syncd and requires vendor to provide SAI for buster. As swss docker does not really contain syncd
    binary, so it is not necessary to build syncd for swss docker.
    
    * [submodule]: update sonic-sairedis
    
    * ccbb3bc 2020-06-28 | add option to build without syncd (HEAD, origin/master, origin/HEAD) [Guohan Lu]
    * 4247481 2020-06-28 | install saidiscovery into syncd package [Guohan Lu]
    * 61b8e8e 2020-06-26 | Revert "sonic-sairedis: Add support to sonic-sairedis for gearbox phys (sonic-net#624)" (sonic-net#630) [Danny Allen]
    * 85e543c 2020-06-26 | add a README to tests directory to describe how to run 'make check' (sonic-net#629) [Syd Logan]
    * 2772f15 2020-06-26 | sonic-sairedis: Add support to sonic-sairedis for gearbox phys (sonic-net#624) [Syd Logan]
    
    Signed-off-by: Guohan Lu <lguohan@gmail.com>
    lguohan authored Jul 3, 2020
    Configuration menu
    Copy the full SHA
    c4b5b00 View commit details
    Browse the repository at this point in the history
  3. [Dell]: DellEMC S6100 disable pericom/xlinx chipset (sonic-net#4868)

    - Xilinx/pericom peripherals are not actively used in DellEMC S6100 switch.
    - These peripherals are throwing PCIE corrected messages in some of the units and filling syslog.
    - Since it is not usable disabling it at startup.
    paavaanan authored Jul 3, 2020
    Configuration menu
    Copy the full SHA
    eebca91 View commit details
    Browse the repository at this point in the history
  4. [sonic-platform-daemons] Update submodule (sonic-net#4887)

    * src/sonic-platform-daemons abe115e...9b8bfa1 (1):
      > [xcvrd] Update key names in 'get_media_settings_value()' (sonic-net#63)
    jleveque authored Jul 3, 2020
    Configuration menu
    Copy the full SHA
    4240c8c View commit details
    Browse the repository at this point in the history
  5. [docker-orchagent]: upgrade docker-orchagent to buster (sonic-net#4889)

    also update submodule
    
    * 01f810f 2020-07-02 | fix compiling issue for gcc8.3 (sonic-net#1339) [lguohan]
    * 9b13120 2020-07-03 | Fix in script to avoid orchagent crash when port down followed by fdb delete (sonic-net#1340) [rupesh-k]
    * 9b01844 2020-07-01 | [qosorch] Update QoS scheduler params for shaping features (sonic-net#1296) [Michael Li]
    * 86b5e99 2020-07-02 | [mirrororch] Port Mirroring implementation (sonic-net#1314) [rupesh-k]
    * c05601c 2020-06-24 | [portsyncd]: add debug message if a port cannot be found in port able (sonic-net#1328) [lguohan]
    * a0b6412 2020-06-23 | COPP_DEL_fix: DEL for one trap group from SONIC is resetting all the trap IDs (sonic-net#1273) [SinghMinu]
    
    Signed-off-by: Guohan Lu <lguohan@gmail.com>
    lguohan authored Jul 3, 2020
    Configuration menu
    Copy the full SHA
    0a260b7 View commit details
    Browse the repository at this point in the history
  6. [barefoot][SAI v1.6.3] Update SAI and platform packages to 20200701 (s…

    …onic-net#4890)
    
    Signed-off-by: Volodymyr Boyko <volodymyrx.boiko@intel.com>
    vboykox authored Jul 3, 2020
    Configuration menu
    Copy the full SHA
    90dcbe1 View commit details
    Browse the repository at this point in the history

Commits on Jul 4, 2020

  1. Configuration menu
    Copy the full SHA
    405033b View commit details
    Browse the repository at this point in the history

Commits on Jul 5, 2020

  1. Skip thermalctld for arista platforms (sonic-net#4893)

    thermalctld throwing error messages because it is not yet fully configured, disabling it for now on arista platforms.
    
    Co-authored-by: Zhi Yuan Carl Zhao <zyzhao@arista.com>
    zzhiyuan and Zhi Yuan Carl Zhao authored Jul 5, 2020
    Configuration menu
    Copy the full SHA
    20698aa View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    e62b5f6 View commit details
    Browse the repository at this point in the history

Commits on Jul 6, 2020

  1. Address review comments.

    Signed-off-by: Arvindsrinivasan Lakshmi Narasimhan <arlakshm@microsoft.com>
    arlakshm committed Jul 6, 2020
    Configuration menu
    Copy the full SHA
    969a914 View commit details
    Browse the repository at this point in the history
  2. address review comment

    Signed-off-by: Arvindsrinivasan Lakshmi Narasimhan <arlakshm@microsoft.com>
    arlakshm committed Jul 6, 2020
    Configuration menu
    Copy the full SHA
    eb90fa0 View commit details
    Browse the repository at this point in the history
  3. address review comment

    Signed-off-by: Arvindsrinivasan Lakshmi Narasimhan <arlakshm@microsoft.com>
    arlakshm committed Jul 6, 2020
    Configuration menu
    Copy the full SHA
    afa4f1a View commit details
    Browse the repository at this point in the history
  4. address review comment

    Signed-off-by: Arvindsrinivasan Lakshmi Narasimhan <arlakshm@microsoft.com>
    arlakshm committed Jul 6, 2020
    Configuration menu
    Copy the full SHA
    32a34f9 View commit details
    Browse the repository at this point in the history