Skip to content

Derive Slurm account args from allocation project - #94

Merged
yoshifuminakamura merged 1 commit into
developfrom
feature/scheduler-allocation-project-args
Aug 7, 2026
Merged

Derive Slurm account args from allocation project#94
yoshifuminakamura merged 1 commit into
developfrom
feature/scheduler-allocation-project-args

Conversation

@yoshifuminakamura

Copy link
Copy Markdown
Collaborator

Summary

Derive scheduler account arguments from the Portal allocation project ID only where the system submit syntax is known to require it.

Details

  • Keep Portal execution profiles scheduler-agnostic: profiles store the semantic allocation_project_id, not scheduler command-line fragments.
  • Translate BK_ALLOCATION_PROJECT_ID to Slurm --account=<id> for RIKYU.
  • Leave RC cloud systems without derived --account; they can still use explicit site-local BK_SCHEDULER_EXTRA_ARGS* overrides if needed.
  • Allow approved Portal profiles to leave allocation_project_id empty.
  • Add shell coverage for derived RIKYU args, RC no-op behavior, and explicit RC override behavior.

Validation

  • bash scripts/tests/test_scheduler_extra_args.sh
  • shellcheck -S error scripts/job_functions.sh scripts/test_submit.sh scripts/test_submit_build.sh scripts/tests/test_scheduler_extra_args.sh scripts/tests/test_result_profile_data.sh scripts/tests/test_process_and_send_results.sh
  • git diff --check
  • /home/nakamura/fugakunext/venv/bin/python result_server/tests/run_result_server_tests.py -q
  • BK_ALLOCATION_PROJECT_ID=test bash scripts/matrix_generate.sh

Signed-off-by: yoshifuminakamura <nakamura@riken.jp>
@yoshifuminakamura
yoshifuminakamura merged commit 9609ab8 into develop Aug 7, 2026
9 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