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

slurm 24.05.X support #345

Closed
wants to merge 3 commits into from
Closed

Conversation

basvandervlies
Copy link
Contributor

The code compiles and I can run our python slurm utils. For now I have disabled the epilog|prolog' configs they have changed from char *tochar **`. Could not find a function for it. So that is still on the todo list

The code compiles amd I can run our python slurm utils.
For now I have disabled the `epilog|prolgo' cconfigs they have changed
for `char *` tp `char **`. Could not find a function for it so that is
still on the todo list
so epilog/prolog//prolog_slurmctld/epilog_slurmctld working again
@tazend tazend self-assigned this Jul 14, 2024
@tazend
Copy link
Member

tazend commented Jul 14, 2024

Hi @basvandervlies

thank you for this. I will check it out soon in more detail, but looks good on a first quick glance.

@jvhaarst
Copy link

@basvandervlies: Thanks !

@fonsgithub
Copy link

Awesome @basvandervlies ! Thanks!
I've just installed your pr against our Slurm 24.05.1, and I can query Slurm just fine when I test.
I'll report here if anything weird shows up in our scripting

@tazend tazend mentioned this pull request Sep 25, 2024
@tazend
Copy link
Member

tazend commented Sep 25, 2024

Sorry that it took so long to respond here. I made a new PR (#348) based on this one with some additional fixes, which is now merged into main. Thanks again @basvandervlies for working on it! :)

@tazend tazend closed this Sep 25, 2024
@basvandervlies
Copy link
Contributor Author

No problem it is done and you improved it ;-)

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.

4 participants