Skip to content

[202012]Added libpci and pciutils to the pmon docker #14053

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

[202012]Added libpci and pciutils to the pmon docker #14053

wants to merge 1 commit into from
+5 −1

Conversation

assrinivasan
Copy link
Contributor

Why I did it

This is part of a corresponding change to the pcie daemon that enables it to verify PCI peripherals on a platform against a preconfigured YAML file, and enables the pcied daemon to call the system commands needed for PCI peripheral verification

How I did it

Adding aforementioned libraries to the Dockerfile.j2 file

How to verify it

run 'which setpci' from the pmon docker - would show the path of the binary

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211

Description for the changelog

Modified pmon's Dockerfile.j2 to include pciutils and libpci libraries.

Ensure to add label/tag for the feature raised. example - PR#2174 under sonic-utilities repo. where, Generic Config and Update feature has been labelled as GCU.

Link to config_db schema for YANG module changes

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

@assrinivasan
Added libpci and pciutils to the pmon docker (sonic-net#12684)
cf92446 
This enables the pcied daemon to call the corresponding system commands needed for pci transactions
@linux-foundation-easycla
Copy link

CLA Missing ID CLA Not Signed

@assrinivasan assrinivasan changed the base branch from master to 202012 March 2, 2023 02:22
@assrinivasan
Copy link
Contributor Author

IMO we should revert the original commit: #12684 that caused the cherry-pick conflict. I will do a new PR on top of the latest changes. That way we can avoid this issue. Please advise @qiluo-msft @prgeor

@assrinivasan
Copy link
Contributor Author

Created in error

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@StormLiangMS StormLiangMS Awaiting requested review from StormLiangMS

@qiluo-msft qiluo-msft Awaiting requested review from qiluo-msft

@xumia xumia Awaiting requested review from xumia

@lguohan lguohan Awaiting requested review from lguohan

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@assrinivasan