Skip to content

[Bug][Build]: Fix the mlnx-platform-api dpkg cache config error #9705

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 1 commit into from
Jan 9, 2022

Conversation

xumia
Copy link
Collaborator

@xumia xumia commented Jan 8, 2022

Why I did it

Failed to build the mellanox image when dpkg enabled, it has impact on PR checks.

xumia@501e7c58198f:/sonic$ cat target/python-wheels/bullseye/mlnx_platform_api-1.0-py3-none-any.whl.dep.tmp |xargs git hash-object > target/tmp.1
fatal: Unable to hash device/arista/x86_64-arista_7050_qx32/plugins
fatal: Unable to hash device/mellanox/x86_64-mlnx_lssn2700-r0/LS-SN2700
fatal: Unable to hash device/virtual/x86_64-kvm_x86_64_6_asic-r0/plugins
fatal: Unable to hash platform/broadcom/sonic-platform-modules-nokia
fatal: Unable to hash src/linkmgrd

The variable $(SONIC_PLATFORM_API_PY2)_SRC_PATH is not always defined, need to change to use $(SONIC_PLATFORM_API_PY3)_SRC_PATH.

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 xumia requested a review from lguohan as a code owner January 8, 2022 01:06
@xumia xumia added Build Bug 🐛 Request for 202111 Branch For PRs being requested for 202111 branch labels Jan 8, 2022
@xumia xumia merged commit 3e46582 into sonic-net:master Jan 9, 2022
@xumia xumia deleted the fix-mlnx-dpkg-cache-issue branch January 9, 2022 01:22
judyjoseph pushed a commit that referenced this pull request Jan 9, 2022
Failed to build the mellanox image when dpkg enabled, it has impact on PR checks.
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.

3 participants