Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@hv15
Copy link
Contributor

@hv15 hv15 commented Jan 3, 2023

Hi, just came across this project via SLURM-user mailing list. Really useful tool, thank you! For our cluster we're looking to add it. We'd like to by default have job history days and timeout set to different values.

With this PR we add entries to the scom.conf file to set new defaults for both job history query parameter days and command timeout. This allows changing the default value of these parameters while still allowing users to specify -d and -t flags.

Possibly being able to set (or pass) a -S args might also be worth exploring.

A default value for job history query parameter days and command timeout
can be set through the scom.conf file now. This allows changing the
default value of these parameters while still allowing users to specify
`-d` and `-t` flags.
@pja237 pja237 self-requested a review January 3, 2023 17:25
@pja237 pja237 added the enhancement New feature or request label Jan 3, 2023
Copy link
Contributor

@pja237 pja237 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @hv15,

sorry for the delayed response, we're still in the holiday/restructuring mode of work.
Love your additions, thank you very much for the contribution, all looks perfect to me.
Just to make sure we're on the same page regarding the -S param? That would be like slurm start time specifier? or i misunderstood it?
In principle, i see any addition that makes people lives easier as a good one/worth exploring. Time-wise, might take me some to get to attacking that, so safest place for such ideas would be to just queue them in Issues, and we can do them as we find free time.

Also, would you have anything against signing yourself as a contributor in the "thank you" section of the about tab before i merge and release? 😃

A special thank you goes to:
Seren Ümit
Kilian Cavalotti
Killian Murphy
`

@hv15
Copy link
Contributor Author

hv15 commented Jan 4, 2023

@pja237 Awesome, thanks! Yes for -S I do mean the slurm start time specifier, I'll add the suggestion as an issue (see #15).

@pja237 pja237 merged commit 21cee5d into CLIP-HPC:main Jan 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants