Project

General

Profile

Actions
Copy link

Bug #17384

closed

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

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

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

100%

Estimated time:
(Total: 0.00 h)
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/2021

Actions

Related issues

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

Actions
Actions
Copy link
 #1

Updated by Tom Clegg over 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 over 3 years ago

This LGTM, thanks.

Actions
Copy link
 #5

Updated by Tom Clegg over 3 years ago

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

Updated by Peter Amstutz over 3 years ago

  • Release set to 37
Actions
Copy link

Also available in: Atom PDF