Bug #5285

[SDKs] arv-run-pipeline-instance does not use 'settings.conf' file like other 'arv' tools

Added by Abram Connelly almost 6 years ago. Updated over 5 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
SDKs
Target version:
Start date:
02/20/2015
Due date:
% Done:

100%

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

Description

Having the file

$HOME/.config/arvados/settings.conf
with the proper variables does not get used by 'arv-run-pipeline-instance'. It would be nice to have this tool be consistent with the rest of the 'arv' command line tools.


Subtasks

Task #5504: Review 5285-arpi-use-settingsResolvedPeter Amstutz


Related issues

Related to Arvados - Task #2822: [SDKs] Use Ruby SDK instead of google-api-client in arv-run-pipeline-instanceNew

Associated revisions

Revision 4925686f
Added by Peter Amstutz over 5 years ago

Merge branch '5285-arpi-use-settings' closes #5285

History

#1 Updated by Brett Smith almost 6 years ago

  • Subject changed from arv-run-pipeline-instance does not use 'settings.conf' file like other 'arv' tools to [SDKs] arv-run-pipeline-instance does not use 'settings.conf' file like other 'arv' tools
  • Category set to SDKs

This tool already uses the Arvados class from the Ruby SDK, which reads from the configuration file. However, it also has separate checks early on for the ARVADOS environment variables, and bails if they're not set. Fixing this might be as simple as refactoring to get configuration from Arvados as needed.

#2 Updated by Brett Smith almost 6 years ago

  • Target version set to Bug Triage

#3 Updated by Peter Amstutz over 5 years ago

  • Target version changed from Bug Triage to 2015-04-01 sprint

#4 Updated by Peter Amstutz over 5 years ago

  • Assigned To set to Peter Amstutz

#5 Updated by Peter Amstutz over 5 years ago

  • Status changed from New to In Progress

#6 Updated by Brett Smith over 5 years ago

a578a05 is good to merge. Thanks.

#7 Updated by Peter Amstutz over 5 years ago

  • Status changed from In Progress to Resolved

Applied in changeset arvados|commit:4925686f6aa7214568ebd60be3acaa49dbf9dd1a.

Also available in: Atom PDF