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

Debug dump utility dash objects update #19440

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

gpunathilell
Copy link
Contributor

Why I did it

This PR is required for the changes in sonic-net/sonic-utilities#3387. Adds libdashapi in the image and adds dependency of libdashapi to sonic-utilities

Work item tracking
  • Microsoft ADO (number only):

How I did it

Changed files/build_templates/sonic_debian_extension.j2 to install libdashapi to the image and changed rules/sonic-utilities.mk to add the dependency

How to verify it

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)

@gpunathilell
Copy link
Contributor Author

@Pterosaur Please review

@Pterosaur Pterosaur self-requested a review July 10, 2024 00:59
@gpunathilell
Copy link
Contributor Author

/azpw run Azure.buildimage

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.buildimage

Copy link

No pipelines are associated with this pull request.

Copy link

Commenter does not have sufficient privileges for PR 19440 in repo sonic-net/sonic-buildimage

@gpunathilell
Copy link
Contributor Author

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@gpunathilell
Copy link
Contributor Author

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@@ -1128,6 +1128,10 @@ SONIC_PLATFORM_PY3_WHEEL_NAME=$(basename {{platform_api_py3_wheel_path}})
sudo cp {{platform_api_py3_wheel_path}} $FILESYSTEM_ROOT/$SONIC_PLATFORM_PY3_WHEEL_NAME
sudo https_proxy=$https_proxy LANG=C chroot $FILESYSTEM_ROOT pip3 install $SONIC_PLATFORM_PY3_WHEEL_NAME
sudo rm -rf $FILESYSTEM_ROOT/$SONIC_PLATFORM_PY3_WHEEL_NAME

sudo dpkg --root=$FILESYSTEM_ROOT -i $debs_path/libdashapi_*.deb || \
Copy link
Collaborator

Choose a reason for hiding this comment

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

If this is sonic-utilities's dependency, you do not need to install it explicitly.

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.

5 participants