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.

Sign up for GitHub

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

Added 24h default time limit #156

Merged
merged 6 commits into from
Mar 10, 2023
Merged

Added 24h default time limit #156

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
c2cc287
Added 24h default time limit
jonas-lq Feb 28, 2023
e9169f4
Fixed whitespace to be pep8 compliant
jonas-lq Feb 28, 2023
78ecdb8
Added checks for short and long form of time flag
jonas-lq Mar 6, 2023
2e2abc9
Added whitespace around operator
jonas-lq Mar 6, 2023
3fbc97a
Catching more cases in the time limit check
jonas-lq Mar 9, 2023
8bf9b1a
Code polishing
jonas-lq Mar 10, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 3 additions & 2 deletions tasks/build.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -269,8 +269,9 @@ def submit_job(job, submitted_jobs, build_env_cfg, ym, pr_id):
cpu_target = '/'.join(job.arch_target.split('/')[1:])

# Add a default time limit of 24h to the command if nothing else is specified by the user
all_opts = " ".join([build_env_cfg[SLURM_PARAMS], job.slurm_opts, '--export=ALL,CPU_TARGET=%s' % cpu_target])
if ("--time" in all_opts) or ("-t" in all_opts):
all_opts_str = " ".join([build_env_cfg[SLURM_PARAMS], job.slurm_opts, '--export=ALL,CPU_TARGET=%s' % cpu_target])
Copy link
Contributor

Choose a reason for hiding this comment

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

I would remove the third argument , '--export=ALL,CPU_TARGET=%' % cpu_target]. This is not a configuration parameter that we expect to include a time limit. In a future version of the bot this code is probably going away anyway.

all_opts_list = all_opts_str.split(" ")
if any([(opt.startswith("--time") or opt.startswith("-t")) for opt in all_opts_list]):
time_limit = ""
else:
time_limit = f"--time={DEFAULT_JOB_TIME_LIMIT}"
Expand Down