Skip to content

Add SLURM cgroups support to system.go#510

Open
MichaelAronsen wants to merge 1 commit intohenrygd:cgroupsfrom
MichaelAronsen:main
Open

Add SLURM cgroups support to system.go#510
MichaelAronsen wants to merge 1 commit intohenrygd:cgroupsfrom
MichaelAronsen:main

Conversation

@MichaelAronsen
Copy link
Copy Markdown

No description provided.

@henrygd
Copy link
Copy Markdown
Owner

henrygd commented Jan 31, 2025

Thanks! Adding support for cgroups is a big help, but we might want to separate this functionality into its own package and use it only if the system supports reading from cgroup.

Looks like a lot of stuff was removed here that we need to keep (sensors, gpu, etc.).

I need to work on some high priority things for the next release but I'll get back to you after that. Probably next week.

@MichaelAronsen
Copy link
Copy Markdown
Author

Yeah, this code was a proof of concept to see if it would even work.

If I can do more to help, do let me know, I want to use this to run per job node monitoring in hpc clusters.

@tecosaur
Copy link
Copy Markdown

I wonder if this could be used to add support for monitoring Systemd services as well as containers?

@MichaelAronsen
Copy link
Copy Markdown
Author

I wonder if this could be used to add support for monitoring Systemd services as well as containers?

I think so yes, as those are generally started within their own cgroup.
So that would enable beszel to monitor just the resources assigned to such services and containers.

@tecosaur
Copy link
Copy Markdown

For what it's worth, this would catapult Beszel from "almost useless" to "really good" for me — since I use a NixOS server and most applications are managed as nix-provided systemd services instead of containers.

@tecosaur
Copy link
Copy Markdown

tecosaur commented May 5, 2025

Friendly poke @henrygd 🙂

As mentioned above, this looks really cool to me due to the capability to monitor systemd services. If you are able to spare the time to give this a good look, that would be great!

@MichaelAronsen
Copy link
Copy Markdown
Author

Friendly poke @henrygd 🙂

As mentioned above, this looks really cool to me due to the capability to monitor systemd services. If you are able to spare the time to give this a good look, that would be great!

I made a new version that i'm testing out, if it works well, i'll make another Pull Request.

@henrygd
Copy link
Copy Markdown
Owner

henrygd commented May 5, 2025

Thanks, if the feature can be more isolated (like in its own package) then I should be able to get to it more quickly.

I've had less spare time this year and more requests for support, so I'm behind where I hoped I'd be as far as development. We will get there eventually.

@tecosaur
Copy link
Copy Markdown

I made a new version that i'm testing out, if it works well, i'll make another Pull Request.

Hey @MichaelAronsen, just a gentle poke, it would be cool to hear how your revised effort went 🙂

@henrygd henrygd changed the base branch from main to cgroups September 4, 2025 17:49
@github-actions github-actions Bot added the Stale label Nov 4, 2025
@github-actions github-actions Bot closed this Nov 11, 2025
@github-project-automation github-project-automation Bot moved this from Feature backlog to Done in Beszel Roadmap Nov 11, 2025
@henrygd henrygd reopened this Nov 11, 2025
@github-actions github-actions Bot removed the Stale label Nov 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants