Skip to content

Advance the broadcom SAI to 5.0.0.11 #8952

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

Closed
wants to merge 2 commits into from

Conversation

judyjoseph
Copy link
Contributor

@judyjoseph judyjoseph commented Oct 11, 2021

Why I did it

Advance the broadcom SAI to 5.0.0.11

How I did it

Includes the following commits/CSPs

9a1765cf CS00012203989: Max mtu not set at init knet interfaces
9b59e33f Fixes for CS00012195261, CS00012195512, CS00012194313, CS00012178692, CS00012192505, SDK SIDs

How to verify it

Verified on a DNX device, the interfaces are up - verified basic tests.

admin@str-a7280cr3-2:~$ bcmcmd "bsv"
bsv
BRCM SAI ver: [5.0.0.11], OCP SAI ver: [1.8.1], SDK ver: [sdk-6.5.22]
drivshell>
admin@str-a7280cr3-2:~$ bcmcmd "show unit"
show unit
Unit 0 chip BCM88690_B1 (current)
drivshell>
admin@str-a7280cr3-2:~$ docker ps
CONTAINER ID        IMAGE                                COMMAND                  CREATED             STATUS              PORTS               NAMES
d261bd2940b3        docker-sonic-telemetry:latest        "/usr/local/bin/supe…"   42 seconds ago      Up 40 seconds                           telemetry
8f7cb1e2d088        docker-sonic-mgmt-framework:latest   "/usr/local/bin/supe…"   42 seconds ago      Up 40 seconds                           mgmt-framework
dce3cfbea1a1        docker-snmp:latest                   "/usr/local/bin/supe…"   42 seconds ago      Up 40 seconds                           snmp
22c9701b33e4        docker-router-advertiser:latest      "/usr/bin/docker-ini…"   2 minutes ago       Up 2 minutes                            radv
d25e73bd6128        docker-lldp:latest                   "/usr/bin/docker-lld…"   2 minutes ago       Up 2 minutes                            lldp
c0e00779529c        docker-syncd-brcm-dnx:latest         "/usr/local/bin/supe…"   2 minutes ago       Up 2 minutes                            syncd
330659a98c38        docker-teamd:latest                  "/usr/local/bin/supe…"   2 minutes ago       Up 2 minutes                            teamd
9ba060690a98        docker-orchagent:latest              "/usr/bin/docker-ini…"   2 minutes ago       Up 2 minutes                            swss
f09e7837ad96        docker-fpm-frr:latest                "/usr/bin/docker_ini…"   4 minutes ago       Up 4 minutes                            bgp
e0b35b463a15        docker-platform-monitor:latest       "/usr/bin/docker_ini…"   4 minutes ago       Up 4 minutes                            pmon
b2734cf662bb        docker-database:latest               "/usr/local/bin/dock…"   4 minutes ago       Up 4 minutes                            database

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106

Description for the changelog

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

@judyjoseph judyjoseph requested a review from gechiang October 11, 2021 23:33
@@ -1,3 +1,9 @@
opennsl (5.0.0.11) unstable; urgency=medium
Copy link
Collaborator

Choose a reason for hiding this comment

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

Wondering if this file was changed and included when you built the libsaibcm debians? If not, this version will not be reflected when one issue bcmcmd "bsv" correct?

Copy link
Contributor Author

@judyjoseph judyjoseph Oct 12, 2021

Choose a reason for hiding this comment

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

We don't need to build with this file. libsaibcm is itself a debian and it has the version defined there in saibcm

networking-acs-saibcm$ git grep 5.0.0.11   
RELEASE:5.0.0.11
debian/changelog:saibcm (5.0.0.11) unstable; urgency=medium
include/brcm_sai_version.h:#define BRCM_SAI_VER "5.0.0.11"

We update this modules file -- so that we can track which saibcm release caused this change.

Copy link
Collaborator

Choose a reason for hiding this comment

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

I guess there are no minor involved yet this time so it is probably ok for now... it there are minor involved such as something like 5.0.0.11-1 then the "-1" will be missing if the debian/changelog is not included as part of the debian build...

@judyjoseph judyjoseph marked this pull request as ready for review October 15, 2021 22:26
@judyjoseph judyjoseph requested a review from lguohan as a code owner October 15, 2021 22:26
@judyjoseph
Copy link
Contributor Author

Closing this in favor of #9095, to Advance SAI 5.0 in branch 202106.

@judyjoseph judyjoseph closed this Oct 27, 2021
@judyjoseph judyjoseph deleted the SAI_5.0.0.11 branch October 29, 2021 18:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants