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

[202205] Update Broadcom DNX SAI version to 7.1.60.4 #16351

Merged
merged 1 commit into from
Aug 31, 2023

Conversation

tjchadaga
Copy link
Contributor

@tjchadaga tjchadaga commented Aug 31, 2023

Why I did it

To include the following fixes:

DNX:

  1. CS00012287482 - Support for 1024 LAGs on DNX (Added back fix reverted in [202205] Update Broadcom DNX SAI version to 7.1.54.4 #15850)
  2. CS00012302400 - New SAI 7.1.50.4 caused regression in sonic-mgmt ACL test &
    ACL entry creation failing with SAI_STATUS_INVALID_PORT_NUMBER in SAI 7.1.50.4
    (CS00012302347)
  3. CS00012302163 - SAI_API_BRIDGE:_brcm_sai_bridge_port_learn_flag:1620 sai bridge lag port list get. failed with error -7.
  4. CS00012296571 - LACP packets are queued to Queue 0 instead of Queue 7
  5. CS00012301919 - The traffic is queued to VOQ 8 sometimes instead of destination port's VOQ
  6. CS00012297160 - [SONIC] [J2C+] Traffic to unknown destination route getting enqueued on VOQ 10
  7. CS00012298730 - [7.x][J2/J2C+] : Treat Q=0 as lowest priority and Q=7 as highest priority in Strict Priority Scheduling

Also includes -
XGS:

  1. Port SONIC-62323 to SAI 7.1, Use single NH instead of ecmp
  2. [SAI_BRANCH rel_ocp_sai_7_1] ECMP group expansion fail due to no resources
  3. Fix capability for Hostif queue on SAI version 7.1
  4. CS00012302193 - SAI_SWITCH_ATTR_SWITCH_HARDWARE_INFO attribute value changed
Work item tracking
  • Microsoft ADO (number only): 25007127

How I did it

Updated SAI version for DNX

How to verify it

Basic DNX chassis bringup and run SAI validation pipeline on DNX chassis

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

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

Copy link
Collaborator

@gechiang gechiang left a comment

Choose a reason for hiding this comment

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

LGTM

@gechiang
Copy link
Collaborator

@tjchadaga Build failed due to missing libsaibcm?

[ REASON ] :      target/debs/bullseye/libsaibcm_dnx_7.1.60.4_amd64.deb does not exist  
[ FLAGS  FILE    ] : [broadcom amd64 bullseye 287c1f61cdc4de217b899fceb4fb5e243d81ba94b9e03d7b09ed698c3b26d4cf bullseye] 
[ FLAGS  DEPENDS ] : [broadcom amd64 bullseye 6810c1f3a9f82455cb22095a9f6607e846c35a7fe2f7904362684783f6c20e42 bullseye] 
[ FLAGS  DIFF    ] : [6810c1f3a9f82455cb22095a9f6607e846c35a7fe2f7904362684783f6c20e42 287c1f61cdc4de217b899fceb4fb5e243d81ba94b9e03d7b09ed698c3b26d4cf] 
Warning: Failed to verify the package: https://sonicstorage.blob.core.windows.net/public/sai/bcmsai/REL_7.0_202205/7.1.60.4/libsaibcm_dnx_7.1.60.4_amd64.deb, the version is not specified
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed

  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
curl: (22) The requested URL returned error: 404 The specified blob does not exist.
[  FAIL LOG END  ] [ target/debs/bullseye/libsaibcm_dnx_7.1.60.4_amd64.deb ]
make: *** [slave.mk:487: target/debs/bullseye/libsaibcm_dnx_7.1.60.4_amd64.deb] Error 1
make: *** Waiting for unfinished jobs....
[ finished ] [ target/docker-base-bullseye.gz-load ] 

Maybe it was not ready yet... I will rerun the failed job and see if it gets lucky... you might want to check what happened to the libsaibcm if it fails again... might need build team to check why it is missing...

@tjchadaga
Copy link
Contributor Author

Thanks. Yes, it was not ready yet. Went through now.

@gechiang gechiang requested review from rlhui and yxieca August 31, 2023 15:14
@gechiang
Copy link
Collaborator

@lguohan , @yxieca appreciate if you can help review/approve this BRCM DNX bug fixes that is needed for 202205.
Thanks!

@yxieca yxieca merged commit 56d6ed1 into sonic-net:202205 Aug 31, 2023
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants