Skip to content

Simplify the broadcom sai packages reference #9151

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

Merged
merged 2 commits into from
Dec 2, 2021

Conversation

xumia
Copy link
Collaborator

@xumia xumia commented Nov 2, 2021

Why I did it

For broadcom sai, we only need to upgrade the version, not necessary the token part in the url.

How I did it

How to verify it

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)

@xumia
Copy link
Collaborator Author

xumia commented Nov 2, 2021

@gechiang, it is only a sample PR, please skip the version 6.0 -> 4.3 change, we will not merge it.

@xumia xumia changed the title Support broadcom sai container token Simplify the broadcom sai packages reference Nov 28, 2021
@xumia xumia requested a review from gechiang November 28, 2021 14:55
@xumia xumia marked this pull request as ready for review November 28, 2021 14:56
@xumia
Copy link
Collaborator Author

xumia commented Nov 29, 2021

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@lguohan lguohan merged commit 0ff96a0 into sonic-net:master Dec 2, 2021
@judyjoseph judyjoseph added the Request for 202111 Branch For PRs being requested for 202111 branch label Jan 6, 2022
judyjoseph pushed a commit that referenced this pull request Jan 7, 2022
For broadcom sai, we only need to upgrade the version, not necessary the token part in the url.

Co-authored-by: Ubuntu <xumia@xumia-vm1.jqzc3g5pdlluxln0vevsg3s20h.xx.internal.cloudapp.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Included in 202111 Branch Request for 202111 Branch For PRs being requested for 202111 branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants