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

feat(slurm_ops): add cgroup configuration manager #33

Merged
merged 5 commits into from
Sep 20, 2024

Conversation

NucciTheBoss
Copy link
Member

This PR enhances the configuration management interface and adds a configuration manager for the cgroup.conf file. The configuration management interface is extended by adding the load and dump methods which make it much easier to build abstractions around the various configuration files on the machines.

Also includes some various quality of life fixes such as removing a random string that we had in the integration tests, minor documentation and formatting fixes here and there.

Closes #31

Changes:

- Add configuration manager for cgroup.conf on `SlurmctldManager`. This
  is required to reach feature parity with `slurmctld_ops`

Signed-off-by: Jason C. Nucciarone <[email protected]>
…interface

Chnages:

- Add unit tests for the cgroup.conf configuration manager.

Signed-off-by: Jason C. Nucciarone <[email protected]>
@NucciTheBoss NucciTheBoss added the enhancement New feature or request label Sep 20, 2024
Copy link
Contributor

@jedel1043 jedel1043 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work!

tests/integration/slurm_ops/test_manager.py Show resolved Hide resolved
lib/charms/hpc_libs/v0/slurm_ops.py Show resolved Hide resolved
@NucciTheBoss NucciTheBoss merged commit 796555a into charmed-hpc:main Sep 20, 2024
4 checks passed
@NucciTheBoss NucciTheBoss deleted the feat-config-editing branch October 8, 2024 21:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Enhancement]: Add support for editing Slurm's cgroup configuration
2 participants