Project

General

Profile

Actions
Copy link

Bug #17384

closed

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

Added by Tom Clegg about 3 years ago. Updated about 3 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Tom Clegg
Category:
Crunch
Target version:
2021-02-17 sprint
Story points:
-
Release:
Arvados 2.1.2
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 1 (0 open — 1 closed)

Task #17387: Review 17384-crunch-run-configsResolvedTom Clegg02/15/2021Actions

Related issues

Has duplicate Arvados - Bug #17385: [arvados-dispatch-cloud][crunch-run] CrunchRunArgumentsList parameter is not honoredDuplicateTom CleggActions
Actions
Copy link
 #1

Updated by Tom Clegg about 3 years ago

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

Updated by Lucas Di Pentima about 3 years ago

This LGTM, thanks.

Actions
Copy link
 #5

Updated by Tom Clegg about 3 years ago

  • % Done changed from 0 to 100
  • Status changed from In Progress to Resolved
Actions
Copy link
 #6

Updated by Peter Amstutz about 3 years ago

  • Release set to 37
Actions
Copy link

Also available in: Atom PDF