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

plugin: add estimation of cores-per-node count on system during initialization #469

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

plugin: add estimation of cores-per-node count on system during initialization #469

wants to merge 3 commits into from

Commits on Sep 18, 2024

  1. configure: fix dots, add more paths 

    Problem: The paths for FLUX_IDSET_LIBS/FLUX_IDSET_CFLAGS are not listed
in the output of ./configure. There is also slight misformatting in the
output of the FLUX_CORE_CFLAGS label.

Switch the period and colon at the end of the FLUX_CORE_CFLAGS line.

Add FLUX_IDSET_LIBS and FLUX_IDSET_CFLAGS to the output of ./configure.
    @cmoussa1
    cmoussa1 committed Sep 18, 2024
    Configuration menu
    Copy the full SHA
    52a006d View commit details
    Browse the repository at this point in the history

  2. plugin: initialize plugin w/ cores-per-node count 

    Problem: The priority plugin does not know about basic system
information it will need in order to enforce a max-cores limit per
association, such as the number of cores on a node.

Add an estimation of a cores-per-node count estimate during the
initialization of the priority plugin by fetching resource.R from the
KVS. Store this estimate in a global variable in the plugin.

Add this estimate to the list of information returned in the
plugin.query callback.
    @cmoussa1
    cmoussa1 committed Sep 18, 2024
    Configuration menu
    Copy the full SHA
    18b3867 View commit details
    Browse the repository at this point in the history

  3. t: add test for querying cores-per-node info 

    Problem: flux-accounting has no tests for estimating the cores-per-node
count by fetching resource.R from the KVS and then querying it from the
plugin.

Add some basic tests.
    @cmoussa1
    cmoussa1 committed Sep 18, 2024
    Configuration menu
    Copy the full SHA
    409649f View commit details
    Browse the repository at this point in the history