Skip to content

Comments

Ensure that timelimits for joined jobs takes priority over defaults#35

Merged
islas merged 1 commit intomainfrom
force-timelimit
Jan 16, 2025
Merged

Ensure that timelimits for joined jobs takes priority over defaults#35
islas merged 1 commit intomainfrom
force-timelimit

Conversation

@islas
Copy link
Owner

@islas islas commented Jan 16, 2025

When a host-specific set of submit options is given at the top-level, this is effectively "derived" for all tests and steps created including the in situ one created during joining of jobs. This results in a joined job calcuating a new required timelimit of N but at submission time the value being overwritten by the previous host-specific value M. This is especially important for joined jobs where the calcuated timelimit MUST be correct and cannot rely on defaults if provided.

The changes place all possible values able to be provided in a host-specific submit option block inside one using the current host, whether manually provided or queried.

When a host-specific set of submit options is given at the top-level,
this is effectively "derived" for all tests and steps created including
the in situ one created during joining of jobs. This results in a joined
job calcuating a new required timelimit of N but at submission time the
value being overwritten by the previous host-specific value M.
This is especially important for joined jobs where the calcuated
timelimit MUST be correct and cannot rely on defaults if provided.

The changes place all possible values able to be provided in a
host-specific submit option block inside one using the current host,
whether manually provided or queried.
@islas islas added the test Require testing label Jan 16, 2025
@github-actions github-actions bot removed the test Require testing label Jan 16, 2025
@islas islas added the test Require testing label Jan 16, 2025
@github-actions github-actions bot removed the test Require testing label Jan 16, 2025
@islas islas merged commit ef99a1f into main Jan 16, 2025
3 of 4 checks passed
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.

1 participant