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

Update SAI submodule to point to OCP SAI v1.15.3 release. #1495

Merged
merged 1 commit into from
Jan 18, 2025

Conversation

r12f
Copy link
Contributor

@r12f r12f commented Jan 9, 2025

Update SAI submodule to point to OCP SAI v1.15.3 release.

Commit id: 4d0982283feedd39365a0c5b7bd4f6c1de1be319

@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@kcudnik
Copy link
Collaborator

kcudnik commented Jan 9, 2025

Problem is that we moved 4 past back because I already moved pointers forward and we are waiting to bring that back and there will be conflicf

@kcudnik
Copy link
Collaborator

kcudnik commented Jan 9, 2025

#1481 @yxieca when we bring that back?

@r12f
Copy link
Contributor Author

r12f commented Jan 13, 2025

Update is merged in 202412. Closing this test PR for CI.

@r12f r12f closed this Jan 13, 2025
@r12f r12f reopened this Jan 14, 2025
@mssonicbld
Copy link
Collaborator

/azp run

@r12f r12f changed the title [TEST PR, DO NOT MERGE] Update SAI to v1.15.3 Update SAI to v1.15.3 Jan 14, 2025
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@r12f r12f force-pushed the user/r12f/test-sai branch from 0988d4a to 9de9e11 Compare January 17, 2025 02:08
@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@r12f r12f changed the title Update SAI to v1.15.3 Update SAI submodule to point to OCP SAI v1.15.3 release. Jan 17, 2025
Copy link
Contributor

@abdosi abdosi left a comment

Choose a reason for hiding this comment

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

Need to update PR title and description with SAI commit Id

@prabhataravind
Copy link
Contributor

@oleksandrivantsiv fyi

@r12f
Copy link
Contributor Author

r12f commented Jan 18, 2025

Pipeline is broken due to rsyslog error. This PR was passing the CI pipeline without any issue, but get closed and reopened, so the status is lost here.

Here is the previous run data: https://dev.azure.com/mssonic/build/_build/results?buildId=741545&view=results

image

@r12f
Copy link
Contributor Author

r12f commented Jan 18, 2025

@rlhui , is it ok to get it merged to unblock critical changes?

@rlhui rlhui merged commit 9137103 into sonic-net:master Jan 18, 2025
11 of 15 checks passed
@r12f r12f deleted the user/r12f/test-sai branch January 18, 2025 01:25
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.

6 participants