Story #9745

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

Added by Brett Smith over 4 years ago. Updated over 4 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

Task #9754: Review 9745-cds-api-configResolvedTom Clegg

Associated revisions

Revision ac4ee0eb
Added by Radhika Chippada over 4 years ago

closes #9745
Merge branch '9745-cds-api-config'

Revision e18a2167
Added by Tom Clegg over 4 years ago

Merge branch '9706-package-systemd-files'

refs #9706
refs #9745

History

#1 Updated by Brett Smith over 4 years ago

  • Description updated (diff)

#2 Updated by Tom Clegg over 4 years ago

9745-cds-api-config

#3 Updated by Brett Smith over 4 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

#4 Updated by Radhika Chippada over 4 years ago

LGTM at 608f6a78

Does this need install guide updates?

#5 Updated by Brett Smith over 4 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.

#6 Updated by Radhika Chippada over 4 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.

#7 Updated by Radhika Chippada over 4 years ago

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

Applied in changeset arvados|commit:ac4ee0eb5a2d581b7bb4d26b475f0fbcf64b0630.

Also available in: Atom PDF