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

missing job field 'comment' #244

Closed
Whitehorse2 opened this issue Oct 3, 2022 · 4 comments
Closed

missing job field 'comment' #244

Whitehorse2 opened this issue Oct 3, 2022 · 4 comments
Assignees

Comments

@Whitehorse2
Copy link

Details

  • Slurm Version: 20.11.8-1
  • Python Version: 3.8
  • Cython Version: 0.29.32
  • PySlurm Branch: slurm-20.11.8
  • Linux Distribution: ubuntu 20.04

Issue

From the pyslurm job database output, i am missing the 'comment' and tmp disk section.

Why is the data missing and can it be added?

Thanks in advance

@Whitehorse2 Whitehorse2 changed the title missing job data missing job fileds 'comment' Oct 5, 2022
@Whitehorse2 Whitehorse2 changed the title missing job fileds 'comment' missing job field 'comment' Oct 5, 2022
@tazend
Copy link
Member

tazend commented Oct 6, 2022

Hi @Whitehorse2

just had a look, yeah fetching the comment from the database output just isn't implemented currently. I can check it out on the weekend.

Can you clarify what you mean with "tmp disk"? Maybe you can tell me the exact name of the property you want to retrieve? (by checking with sacct -e)

@Whitehorse2
Copy link
Author

@tazend : Thank you, could you integrate this value at branch slurm-20.11.8 ?
I mean the MIN_TMP_DISK value, visible at squeue, but I could not find it at sacct.

@tazend tazend self-assigned this Oct 10, 2022
@tazend
Copy link
Member

tazend commented Oct 10, 2022

Hi @Whitehorse2

I have added a "comment" field in the slurmdb jobs output in the branch you mentioned (b3cc365)

In regards to the MIN_TMP_DISK - for Jobs fetched from the database this information isn't available.

But for jobs which are still running/pending, you can use job().get() and then the field is called pn_min_tmp_disk. (that what squeue reports)

@tazend
Copy link
Member

tazend commented Dec 12, 2024

Workaround has been added for the old slurm version in the branch mentioned above. But the code in pyslurm.pyx is deprecated and the workaround will not be added to subsequent versions. The new API should be used, documented on https://pyslurm.github.io

@tazend tazend closed this as completed Dec 12, 2024
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

No branches or pull requests

2 participants