Skip to content

[ci]: Support the rpc image for mellanox#7905

Merged
lguohan merged 2 commits intosonic-net:masterfrom
xumia:support-mellanox-rpc
Jun 21, 2021
Merged

[ci]: Support the rpc image for mellanox#7905
lguohan merged 2 commits intosonic-net:masterfrom
xumia:support-mellanox-rpc

Conversation

@xumia
Copy link
Copy Markdown
Collaborator

@xumia xumia commented Jun 17, 2021

Why I did it

Support the rpc image for mellanox.

Jenkins Pipeline reference:
https://github.com/Azure/sonic-build-tools/blob/master/jenkins/mellanox/buildimage-mlnx-202012-rpc/Jenkinsfile
https://github.com/Azure/sonic-build-tools/blob/master/jenkins/mellanox/buildimage-mlnx-201911-rpc/Jenkinsfile

How I did it

Rename the sync_rpc_image to docker_syncd_rpc_image
Add a new parameter syncd_rpc_image to build sonic-mellanox-rpc.bin

How to verify it

See the artifacts in https://dev.azure.com/mssonic/build/_build/results?buildId=20099&view=results

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

  • 201811
  • 201911
  • 202006
  • 202012

Description for the changelog

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

@xumia xumia requested a review from qiluo-msft June 18, 2021 03:17
@lguohan lguohan merged commit 0b53a60 into sonic-net:master Jun 21, 2021
qiluo-msft pushed a commit that referenced this pull request Jun 21, 2021
@abdosi
Copy link
Copy Markdown
Contributor

abdosi commented Jun 23, 2021

@xumia have conflict. create pr for 201911.

xumia added a commit to xumia/sonic-buildimage-1 that referenced this pull request Jun 23, 2021
lguohan pushed a commit that referenced this pull request Jun 23, 2021
Rename the sync_rpc_image to docker_syncd_rpc_image
Add a new parameter syncd_rpc_image to build sonic-mellanox-rpc.bin
carl-nokia pushed a commit to carl-nokia/sonic-buildimage that referenced this pull request Aug 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants