Skip to content

slurm24-11-5 with Rest+grafana+elasticsearch #60

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

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

JDCHAU
Copy link

@JDCHAU JDCHAU commented Jun 24, 2025

Hello Dear Giovanni Torres , I upgraded the version of slurm to 24.11.5 and added restd grafana elasticsearch, since this version supports both cgroupv1 and v2, I disabled cgroup by default. you can see the new folder for details. I didn't modify your existing files because I wanted the previous version to coexist with the new one. However, I have disabled the automatic selection of the new version for download based on the arg parameter, because the last few versions have changed so much that it's better to compile them manually.

@giovtorres
Copy link
Owner

Thank you for the contribution. I had a hard time viewing the changes on GitHub. It seems like there are over 2000 files changes and 1 million lines added. Was that expected?

@JDCHAU
Copy link
Author

JDCHAU commented Jul 28, 2025

Since the new version of slurm always has compilation dependency issues when installing on Rocky 9, I downloaded the slurm source code and placed it in a folder. I am currently developing additional features such as GPU support, cgroupv2, and adding an sshd authentication node. These features are currently working fine in local testing and are being prepared for testing in a server environment. I noticed that you have opened an issue regarding upgrading to Rocky 9, and I would be happy to work with you to

@JDCHAU
Copy link
Author

JDCHAU commented Jul 28, 2025

Or would you prefer to create a new branch for these features? Because they are indeed very different from your previous code.

@le-markus
Copy link

le-markus commented Jul 29, 2025

I also needed a more recent version of SLURM and the Rest API. I also had some problems getting it working. It draws on your approach for including the Web API, so it felt a bit weird opening up a pull request. Maybe it can help you with removing SLURM from your Codebase.
https://github.com/le-markus/slurm-docker-cluster

@giovtorres thanks for this project 😄

@JDCHAU
Copy link
Author

JDCHAU commented Jul 29, 2025

I also needed a more recent version of SLURM and the Rest API. I also had some problems getting it working. It draws on your approach for including the Web API, so it felt a bit weird opening up a pull request. Maybe it can help you with removing SLURM from your Codebase. https://github.com/le-markus/slurm-docker-cluster

@giovtorres thanks for this project 😄

I also needed a more recent version of SLURM and the Rest API. I also had some problems getting it working. It draws on your approach for including the Web API, so it felt a bit weird opening up a pull request. Maybe it can help you with removing SLURM from your Codebase. https://github.com/le-markus/slurm-docker-cluster

@giovtorres thanks for this project 😄

Hallo Markus! I see you are using the new version of slurm. The code is very neat and clean,geil. I suggest you manually compile the new version of slurmrestd. Theoretically, 24.11 should support v0.0.45, but after compilation, this file is missing, which is strange.

@le-markus
Copy link

le-markus commented Jul 29, 2025

I think slurmrestd should be compiled .

I don't know what you mean with v.0.0.45. Looking at https://github.com/SchedMD/slurm/tree/slurm-25-05-1-1/src/plugins/data_parser I can only see up to version v.0.0.43 (in master branch is up to v.0.0.44).

Those also seem to be included in the rest-api (only checked curl -H ... http://localhost:6820/openapi/v3 doc).

If you can tell me, what you mean with v0.0.45 I will try to look into it.

@JDCHAU
Copy link
Author

JDCHAU commented Jul 29, 2025 via email

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