-
Notifications
You must be signed in to change notification settings - Fork 124
20.11 support #180
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
20.11 support #180
Conversation
Thank you @basvandervlies for this contribution! We'll take a look at this very soon. |
cd /root | ||
git clone https://github.com/bikerdanny/pyslurm.git | ||
cd pyslurm | ||
git checkout 20.02.5 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
20.02.5
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this has to be adjusted it is not an official release so I can adjust it
Hi, I'm getting an error when running https://github.com/PySlurm/pyslurm/blob/0d5e23aeb85797795d907641479b411626a7739b/examples/listdb_reservations.py Not sure if this is caused by pyslurm, but we're seeing
gdb shows the following
Offending code:
This was with slurm 20.11.6. |
Hi! any updates on supporting 20.11? |
I did not test this for us the basics works so I expected something has been chamged in the reserveration code. |
any update on supporting slurm 20.11 ? Thank you. |
@basvandervlies Did this branch ever compile successfully? I pulled down your branch and I think autopxd is missing many declarations. I created a branch based off this PR: https://github.com/PySlurm/pyslurm/tree/basvandervlies-20.11 It doesn't compile yet, but I'll keep working on it. /cc @tazend Let me know if you would like push access to this branch. |
@giovtorres yes this branch compiled successfully other people also used this and reported that they could compile and run it. Maybe other things are changes it has been a while ago when I created this patch. We use this code on our cluster where 20.11 is installed . |
@giovtorres Yes I can also have a look at it and prepare it in the same way like the 20.2 release |
Closed by #193 |
This is based on pull request:
Maybe its useful for someone