Project

General

Profile

Actions
Copy link

Story #9745

closed

[Crunch2] crunch-dispatch-slurm can get API host and token from config.json

Added by Brett Smith almost 7 years ago. Updated almost 7 years ago.

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

100%

Estimated time:
(Total: 0.00 h)
Story points:
0.5

Description

In order to run, crunch-dispatch-slurm requires the usual ARVADOS_API_HOST and ARVADOS_API_TOKEN environment variables set. Additional configuration goes in /etc/arvados/crunch-dispatach-slurm/config.json. This means that the typical install requires writing two files: a configuration file for the init system to set the environment variables, and then config.json for other options.

Teach crunch-dispatch-slurm to read the API hostname and token from config.json [configuration names TBD]. Update the Install Guide to point users to doing all configuration there. This will simplify crunch-dispatch-slurm administration.

Subtasks 1 (0 open — 1 closed)

Task #9754: Review 9745-cds-api-configResolvedTom Clegg08/08/2016

Actions
Actions
Copy link
 #1

Updated by Brett Smith almost 7 years ago

  • Description updated (diff)
Actions
Copy link
 #2

Updated by Tom Clegg almost 7 years ago

9745-cds-api-config

Actions
Copy link
 #3

Updated by Brett Smith almost 7 years ago

  • Status changed from New to In Progress
  • Assigned To set to Tom Clegg
  • Target version set to 2016-08-17 sprint
  • Story points set to 0.5
Actions
Copy link
 #4

Updated by Radhika Chippada almost 7 years ago

LGTM at 608f6a78

Does this need install guide updates?

Actions
Copy link
 #5

Updated by Brett Smith almost 7 years ago

Radhika Chippada wrote:

Does this need install guide updates?

The branch for #9705 is already written as if this branch was merged. So the answer to your question is yes, but the updates are already written. Now we just need #9706 to be done too.

Actions
Copy link
 #6

Updated by Radhika Chippada almost 7 years ago

Brett said:

The branch for #9705 is already written as if this branch was merged. So the answer to your question is yes, but the updates are already written ...

Sounds good. I am merging the branch into master so that you can call #9705 done. Thanks.

Actions
Copy link
 #7

Updated by Radhika Chippada almost 7 years ago

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

Applied in changeset arvados|commit:ac4ee0eb5a2d581b7bb4d26b475f0fbcf64b0630.

Actions
Copy link

Also available in: Atom PDF