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

ToolsAPI: submit command should accept an inline RMS configuration script #449

Closed
AnarManafov opened this issue Jun 3, 2022 · 1 comment
Assignees
Milestone

Comments

@AnarManafov
Copy link
Contributor

AnarManafov commented Jun 3, 2022

ALICE needs a possibility to specify different number of CPU and task slots.
But instead of opening each RMS option one by one, DDS should give a possibility to provide all additional agreements via an inline script.
ToolsAPI's Submit request should implement a string field, which users can use to push their additional RMS arguments in. This inline data will be then injected into the final job script, per plug-in.

@AnarManafov AnarManafov self-assigned this Jun 3, 2022
@AnarManafov AnarManafov added this to the 3.8 milestone Jun 3, 2022
@AnarManafov AnarManafov changed the title Add CPU requirement for submit (command/ToolsAPI) ToolsAPI: submit command should accept an inline RMS configuration script Jun 3, 2022
AnarManafov added a commit to AnarManafov/DDS that referenced this issue Jun 3, 2022
…onfiguration script

dds-tools-api: Added: Users can define additional job RMS configuration via SSubmitRequestData::m_inlineConfig. It will be inlined as is into the final job script. (FairRootGroupGH-449)
dds-submit: Added: The command learned a new argument --inline-config. Content of this string will be added to the RMS job configuration file as is. It can be specified multiple times to add multiline options. (FairRootGroupGH-449)
@AnarManafov
Copy link
Contributor Author

AnarManafov commented Jun 3, 2022

Usage:

ToolsAPI

SSubmitRequest::request_t requestInfo;
requestInfo.m_inlineConfig = <value>

dds-submit command:

dds-submit -r slurm -n 5 --slots=10 --inline-config"#SBATCH --cpus-per-task=124" --inline-config="#SBATCH --test=45"

CC: @rbx

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

1 participant