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

feat: Add slurm prometheus exporter to snapcraft.yaml recipe #9

Merged
merged 3 commits into from
Jun 4, 2024
Merged

feat: Add slurm prometheus exporter to snapcraft.yaml recipe #9

merged 3 commits into from
Jun 4, 2024

Commits on Jun 4, 2024

  1. feat: Add part to build slurm prometheus exporter 

    `CPATH` must be set in build environment as the exporter
uses `swig` to bind the Golang code to slurm/slurm.h

Signed-off-by: Jason C. Nucciarone <[email protected]>
    @NucciTheBoss
    NucciTheBoss committed Jun 4, 2024
    Configuration menu
    Copy the full SHA
    1d420b9 View commit details
    Browse the repository at this point in the history

  2. fix(ci): Bump cyclomatic complexity to 15 

    15 is the standard number used by HPC Engineering

Signed-off-by: Jason C. Nucciarone <[email protected]>
    @NucciTheBoss
    NucciTheBoss committed Jun 4, 2024
    Configuration menu
    Copy the full SHA
    adcfb40 View commit details
    Browse the repository at this point in the history

  3. refactor: Declare SLURM_CONF as global env var 

    Better than dozens of individual declarations for each command.
If change needs to be made to the `slurm.conf` path, only one env
var needs to be updated instead of many.

Signed-off-by: Jason C. Nucciarone <[email protected]>
    @NucciTheBoss
    NucciTheBoss committed Jun 4, 2024
    Configuration menu
    Copy the full SHA
    4280aa5 View commit details
    Browse the repository at this point in the history