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

[vs][dash libsai] Use the latest DASH commit #21360

Merged
merged 3 commits into from
Jan 18, 2025

Conversation

jimmyzhai
Copy link
Contributor

@jimmyzhai jimmyzhai commented Jan 9, 2025

Why I did it

To have the latest DASH bmv2 pipeline and DASH libsai library for DPU KVM image.

Work item tracking
  • Microsoft ADO 30793749:

How I did it

Update DASH_SAI_COMMIT in rules/dash-sai.mk

How to verify it

Build dash libsai package successfully and it includes the latest DASH bmv2 pipeline, libsai library.

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)

@jimmyzhai jimmyzhai requested review from Pterosaur and r12f and removed request for lguohan, qiluo-msft and xumia January 9, 2025 03:11
@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@@ -1,7 +1,7 @@
# DASH SAI repo: https://github.com/sonic-net/DASH

DASH_SAI_VERSION = 1.0.0
DASH_SAI_COMMIT = cc281333e24ade258773ea843f669251a44d7c9a
DASH_SAI_COMMIT = 88a4771
Copy link
Contributor

Choose a reason for hiding this comment

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

do you mind to switch to a full hash?

Copy link
Contributor

Choose a reason for hiding this comment

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

nothing particular, just more stable. : D

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jimmyzhai jimmyzhai requested review from lguohan and r12f January 9, 2025 10:33
@r12f
Copy link
Contributor

r12f commented Jan 17, 2025

thanks Junhua! looks great to me!

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@lguohan lguohan merged commit 8433642 into sonic-net:master Jan 18, 2025
20 of 21 checks passed
VladimirKuk pushed a commit to Marvell-switching/sonic-buildimage that referenced this pull request Jan 21, 2025
To have the latest DASH bmv2 pipeline and DASH libsai library for DPU KVM image.

Work item tracking
Microsoft ADO 30793749:
BYGX-wcr pushed a commit to BYGX-wcr/sonic-buildimage that referenced this pull request Jan 21, 2025
To have the latest DASH bmv2 pipeline and DASH libsai library for DPU KVM image.

Work item tracking
Microsoft ADO 30793749:
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.

4 participants