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

Add powerdrivers for AMT and IPMI powered PC's #1603

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

Conversation

sessl3r
Copy link

@sessl3r sessl3r commented Feb 12, 2025

Description

Adds resources/drivers to power on/off/get PC's which can be accessed via AMT/IPMI remotely.

  • how did you verify the feature works?
    Just by running labgrid-client and Pytest powering such devices on/off, no direct tests implemented.
  • if hardware is needed for the feature, which hardware is supported and which
    hardware did you test with?
    Tests were executed using HPz620/HPz640 workstations for AMT and for example ASRockRack EPYCD8-2T for IPMI.

Checklist

  • Documentation for the feature
  • Tests for the feature
  • The arguments and description in doc/configuration.rst have been updated
  • PR has been tested
  • Man pages have been regenerated

@Emantor
Copy link
Member

Emantor commented Feb 17, 2025

Has this PR been tested?

@sessl3r
Copy link
Author

sessl3r commented Feb 17, 2025

Has this PR been tested?

Testing exactly the PR is todo as this is a cherry-pick from our internal branch. Will update the checklist when tested.

This currently uses amtctrl as the backend as it is simpler then
importing the library of the same tool. If more flexibility is needed
the python amt library could be used.

Signed-off-by: Tobias Binkowski <[email protected]>
@sessl3r sessl3r force-pushed the pr-powerdrivers branch 3 times, most recently from b391062 to c4048eb Compare February 17, 2025 13:51
@sessl3r
Copy link
Author

sessl3r commented Feb 18, 2025

Testing is done. Missed the args attribute we added as for some machines we need to customize the executed IPMI command further.

this currently uses ipmi-power tool from freeipmi-tools

Signed-off-by: Tobias Binkowski <[email protected]>
@ep1cman
Copy link
Contributor

ep1cman commented Feb 20, 2025

This PR has some overlap with: #1567

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.

3 participants