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.

Sign up for GitHub

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

Jump to bottom

[sonic-sairedis] sonic-sairedis submodule update #4847

Merged
merged 2 commits into from
Jun 30, 2020
Merged

[sonic-sairedis] sonic-sairedis submodule update #4847

merged 2 commits into from
Jun 30, 2020

Conversation

smaheshm
Copy link
Contributor

@smaheshm smaheshm commented Jun 25, 2020 •
edited

- Why I did it
sonic-sairedis submodule update to use SAI1.6.3 headers.

Update BRCM SAI to 3.7.5.1 built upon SAI1.6.3. headers.

- How I did it
sonic-sairedis - submodule update
BRCM SAI - Update link to libsaibcm debian package.

- How to verify it
Built sonic-broadcom.bin and verified basic functionality on a switch.

admin@sonic:~$ show version

SONiC Software Version: SONiC.HEAD.3708-004146f03
Distribution: Debian 10.4
Kernel: 4.19.0-6-2-amd64
Build commit: 004146f03
Build date: Thu Jun 25 02:19:18 UTC 2020
Built by: johnar@jenkins-worker-1

Platform: x86_64-dell_s6000_s1220-r0
HwSKU: Force10-S6000
ASIC: broadcom
Serial Number: FPBRX42
Uptime: 16:56:41 up 25 min,  2 users,  load average: 0.77, 0.93, 1.08

Docker images:
REPOSITORY                    TAG                   IMAGE ID            SIZE
docker-teamd                  HEAD.3708-004146f03   256f9fab6275        315MB
docker-teamd                  latest                256f9fab6275        315MB
docker-syncd-brcm             HEAD.3708-004146f03   9c356625eea0        442MB
docker-syncd-brcm             latest                9c356625eea0        442MB
docker-sflow                  HEAD.3708-004146f03   a424d663dfbc        318MB
docker-sflow                  latest                a424d663dfbc        318MB
docker-orchagent              HEAD.3708-004146f03   cb585e25e065        333MB
docker-orchagent              latest                cb585e25e065        333MB
docker-nat                    HEAD.3708-004146f03   7cb500a84d0b        316MB
docker-nat                    latest                7cb500a84d0b        316MB
docker-fpm-frr                HEAD.3708-004146f03   de9ca653fb14        335MB
docker-fpm-frr                latest                de9ca653fb14        335MB
docker-platform-monitor       HEAD.3708-004146f03   651b988732ae        357MB
docker-platform-monitor       latest                651b988732ae        357MB
docker-router-advertiser      HEAD.3708-004146f03   266940f372b3        349MB
docker-router-advertiser      latest                266940f372b3        349MB
docker-database               HEAD.3708-004146f03   68c416dae5b5        349MB
docker-database               latest                68c416dae5b5        349MB
docker-lldp                   HEAD.3708-004146f03   54b4494ce861        375MB
docker-lldp                   latest                54b4494ce861        375MB
docker-dhcp-relay             HEAD.3708-004146f03   dba5c3e48ec2        356MB
docker-dhcp-relay             latest                dba5c3e48ec2        356MB
docker-sonic-telemetry        HEAD.3708-004146f03   9890a1c81c42        412MB
docker-sonic-telemetry        latest                9890a1c81c42        412MB
docker-sonic-mgmt-framework   HEAD.3708-004146f03   21a7e0ce23b3        471MB
docker-sonic-mgmt-framework   latest                21a7e0ce23b3        471MB
docker-snmp                   HEAD.3708-004146f03   8d46df849b9a        388MB
docker-snmp                   latest                8d46df849b9a        388MB

admin@sonic:~$ sudo monit status | grep ^Process
Process 'rsyslog'
Process 'telemetry'
Process 'dialout_client'
Process 'syncd'
Process 'dsserve'
Process 'orchagent'
Process 'portsyncd'
Process 'neighsyncd'
Process 'vrfmgrd'
Process 'vlanmgrd'
Process 'intfmgrd'
Process 'portmgrd'
Process 'buffermgrd'
Process 'nbrmgrd'
Process 'vxlanmgrd'
Process 'snmpd'
Process 'snmp_subagent'
Process 'sflowmgrd'
Process 'lldpd_monitor'
Process 'lldp_syncd'
Process 'lldpmgrd'
Process 'redis_server'
Process 'zebra'
Process 'fpmsyncd'
Process 'bgpd'
Process 'staticd'
Process 'bgpcfgd'
admin@sonic:~$ sudo monit status | grep ^Process | wc -l
27

admin@str-s6000-acs-8:~$ show ip bgp summary

IPv4 Unicast Summary:
BGP router identifier 10.1.0.32, local AS number 65100 vrf-id 0
BGP table version 12909
RIB entries 13011, using 2338 KiB of memory
Peers 24, using 490 KiB of memory
Peer groups 4, using 256 bytes of memory

Neighbor        V         AS MsgRcvd MsgSent   TblVer  InQ OutQ  Up/Down State/PfxRcd   NeighborName
10.0.0.1        4      65200    3293    3329        0    0    0 00:04:21         6402   ARISTA01T2
10.0.0.5        4      65200    3294    3331        0    0    0 00:04:25         6402   ARISTA03T2
10.0.0.9        4      65200    3295    3333        0    0    0 00:04:28         6402   ARISTA05T2
10.0.0.13       4      65200    3299    4221        0    0    0 00:04:39         6402   ARISTA07T2
10.0.0.17       4      65200    3299    4221        0    0    0 00:04:42         6402   ARISTA09T2
10.0.0.21       4      65200    3299    4221        0    0    0 00:04:40         6402   ARISTA11T2
10.0.0.25       4      65200    3290    3325        0    0    0 00:04:13         6402   ARISTA13T2
10.0.0.29       4      65200    3291    3326        0    0    0 00:04:15         6402   ARISTA15T2
10.0.0.33       4      64001     102    6542        0    0    0 00:04:45            7   ARISTA01T0
10.0.0.35       4      64002      93    3330        0    0    0 00:04:23            6   ARISTA02T0
10.0.0.37       4      64003     101    6541        0    0    0 00:04:43            7   ARISTA03T0
10.0.0.39       4      64004     101    6542        0    0    0 00:04:46            6   ARISTA04T0
10.0.0.41       4      64005     106    6547        0    0    0 00:05:03            6   ARISTA05T0
10.0.0.43       4      64006     107    6548        0    0    0 00:05:04            6   ARISTA06T0
10.0.0.45       4      64007     106    6547        0    0    0 00:05:01            6   ARISTA07T0
10.0.0.47       4      64008      96    3334        0    0    0 00:04:30            6   ARISTA08T0
10.0.0.49       4      64009     105    6546        0    0    0 00:04:58            6   ARISTA09T0
10.0.0.51       4      64010      98    3337        0    0    0 00:04:38            6   ARISTA10T0
10.0.0.53       4      64011      98    3337        0    0    0 00:04:36            6   ARISTA11T0
10.0.0.55       4      64012      97    3335        0    0    0 00:04:34            6   ARISTA12T0
10.0.0.57       4      64013      96    3334        0    0    0 00:04:32            6   ARISTA13T0
10.0.0.59       4      64014     100    4222        0    0    0 00:04:42            6   ARISTA14T0
10.0.0.61       4      64015     107    6548        0    0    0 00:05:05            6   ARISTA15T0
10.0.0.63       4      64016     108    6548        0    0    0 00:05:06            6   ARISTA16T0

Total number of neighbors 24


admin@str-s6000-acs-8:~$ show ip bgp network

BGP table version is 12909, local router ID is 10.1.0.32, vrf id 0
Default local pref 100, local AS 65100
Status codes:  s suppressed, d damped, h history, * valid, > best, = multipath,
               i internal, r RIB-failure, S Stale, R Removed
Nexthop codes: @NNN nexthop's vrf id, < announce-nh-self
Origin codes:  i - IGP, e - EGP, ? - incomplete

   Network          Next Hop            Metric LocPrf Weight Path
*= 0.0.0.0/0        10.0.0.25                              0 65200 ?
*=                  10.0.0.29                              0 65200 ?
*=                  10.0.0.1                               0 65200 ?
*=                  10.0.0.5                               0 65200 ?
*=                  10.0.0.9                               0 65200 ?
*=                  10.0.0.13                              0 65200 ?
*=                  10.0.0.21                              0 65200 ?
*>                  10.0.0.17                              0 65200 ?
*> 10.1.0.32/32     0.0.0.0                  0         32768 i
*> 100.1.0.1/32     10.0.0.1                               0 65200 i
*> 100.1.0.3/32     10.0.0.5                               0 65200 i
*> 100.1.0.5/32     10.0.0.9                               0 65200 i
*> 100.1.0.7/32     10.0.0.13                              0 65200 i
*> 100.1.0.9/32     10.0.0.17                              0 65200 i
*> 100.1.0.11/32    10.0.0.21                              0 65200 i
*> 100.1.0.13/32    10.0.0.25                              0 65200 i
.
.
.

- Description for the changelog

sonic-sairedis

- [submodule update] SAI submodule update to v1.6.3 (#628) …
- Update .gitignore file (#625)
- [saiplayer] Add support for profile.ini (#622) …
- [flex counter] add rif plugins (#562) …
  • Updated link to libsaibcm debian package.

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

@smaheshm
Copy link
Contributor Author

smaheshm commented Jun 25, 2020 •
edited

Mellanox build fail is expected. Will receive an SDK update from Mellanox soon.

@smaheshm
Copy link
Contributor Author

retest vsimage please

2 similar comments
@smaheshm
Copy link
Contributor Author

retest vsimage please

@smaheshm
Copy link
Contributor Author

retest vsimage please

@gechiang gechiang mentioned this pull request Jun 26, 2020
Closed
@smaheshm
Copy link
Contributor Author

@rlhui How are we getting changes for Mellanox. Should I move this PR to "ready".

@smaheshm smaheshm marked this pull request as ready for review June 30, 2020 04:31
@smaheshm
Copy link
Contributor Author

Mellanox working on updating to SAI 1.6.3. Update expected within few days.

@rlhui rlhui self-assigned this Jun 30, 2020
rlhui
rlhui reviewed Jun 30, 2020
View reviewed changes
Copy link
Contributor

@rlhui rlhui left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could we please list what are basic functionalities verified? Thanks.

@ciju-juniper
Copy link
Contributor

@smaheshm @rlhui This doesn't fix the link OIR issue mentioned in #4646

@lguohan
Copy link
Collaborator

lguohan commented Jun 30, 2020

retest mellanox please

@smaheshm
Copy link
Contributor Author

Could we please list what are basic functionalities verified? Thanks.

Added the output in the description above.

  • process and container health
  • visual scan of errors in the log.
  • BGP neighborship
  • routing table entries.
  • ping to directly connected I/F

@smaheshm smaheshm merged commit ed7fafc into sonic-net:master Jun 30, 2020
qiluo-msft pushed a commit to qiluo-msft/sonic-buildimage that referenced this pull request Jul 12, 2020
@smaheshm @qiluo-msft
[sonic-sairedis] sonic-sairedis submodule update (sonic-net#4847)
74389cb 
* sonic-sairedis submodule update
* Update BRCM SAI to 3.7.5.1
qiluo-msft pushed a commit to qiluo-msft/sonic-buildimage that referenced this pull request Jul 12, 2020
@smaheshm @qiluo-msft
[sonic-sairedis] sonic-sairedis submodule update (sonic-net#4847)
96fedf1 
* sonic-sairedis submodule update
* Update BRCM SAI to 3.7.5.1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rlhui rlhui rlhui approved these changes

@lguohan lguohan lguohan approved these changes

Assignees

@rlhui rlhui

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@smaheshm @ciju-juniper @lguohan @rlhui