Bug #17384

[arvados-dispatch-cloud] Respect CrunchRunCommand and CrunchRunArgumentsList configs

Added by Tom Clegg 2 months ago. Updated about 2 months ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
Crunch
Target version:
Start date:
02/15/2021
Due date:
% Done:

100%

Estimated time:
(Total: 0.00 h)
Story points:
-
Release relationship:
Auto

Description

(Currently crunch-dispatch-slurm respects these configs, but arvados-dispatch-cloud does not.)

CrunchRunCommand should be used instead of "crunch-run" when DeployRunnerBinary is set to "" (note DeployRunnerBinary defaults to "/proc/self/exe")

CrunchRunArgumentsList should be inserted in the crunch-run command before the uuid argument when starting a container (not with --list or --kill) regardless of CrunchRunCommand / DeployRunnerBinary settings.


Subtasks

Task #17387: Review 17384-crunch-run-configsResolvedTom Clegg


Related issues

Has duplicate Arvados - Bug #17385: [arvados-dispatch-cloud][crunch-run] CrunchRunArgumentsList parameter is not honoredDuplicate

Associated revisions

Revision a999ea55
Added by Tom Clegg 2 months ago

Merge branch '17384-crunch-run-configs'

fixes #17384

Arvados-DCO-1.1-Signed-off-by: Tom Clegg <>

History

#1 Updated by Tom Clegg 2 months ago

  • Has duplicate Bug #17385: [arvados-dispatch-cloud][crunch-run] CrunchRunArgumentsList parameter is not honored added

#3 Updated by Lucas Di Pentima 2 months ago

This LGTM, thanks.

#5 Updated by Tom Clegg 2 months ago

  • % Done changed from 0 to 100
  • Status changed from In Progress to Resolved

#6 Updated by Peter Amstutz about 2 months ago

  • Release set to 37

Also available in: Atom PDF