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.

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

Extra resources #16785

Closed
wants to merge 2 commits into from
Closed

Extra resources #16785

wants to merge 2 commits into from

Commits on Dec 10, 2022

  1. Implement extra resources for test actions 

    Add support for user-specified resource types in the resource manager.
This generalizes the CPU, RAM and "test count" resource support for
other resource types such as the number of GPUs, available GPU memory,
the number of embedded devices connected to the host, etc.

The available amount of extra resources can be specified using the new
--local_extra_resources=<resourcename>=<amount> command line flag, which
is analoguous to the existing --local_cpu_resources and
--local_memory_resources flags.

Tests can then declare the amount of extra resources they need by using
a "resources:<resourcename>:<amount>" tag.
    @scele
    scele authored and Drew Macrae committed Dec 10, 2022
    Configuration menu
    Copy the full SHA
    1b13d2f View commit details
    Browse the repository at this point in the history

  2. fixup Implement extra resources for test actions 

    Addressed comments on bazelbuild#13996
Fixed issues in tests and built and tested with
lowRISC/opentitan#16436

Signed-off-by: Drew Macrae <drewmacrae@google.com>
    Drew Macrae committed Dec 10, 2022
    Configuration menu
    Copy the full SHA
    ce23a5f View commit details
    Browse the repository at this point in the history