Skip to content

Commit

Permalink
Revert previous commit
Browse files Browse the repository at this point in the history
It doesn't make sense to have the config validation fail just
because a tool doesn't have a destination for the default priority:
the destination should just go to the global default destination,
as the user would likely expect to happen.
  • Loading branch information
Matthew Spelchak committed Feb 9, 2018
1 parent f4a2c55 commit 4c402b3
Showing 1 changed file with 0 additions and 10 deletions.
10 changes: 0 additions & 10 deletions lib/galaxy/jobs/dynamic_tool_destination.py
Original file line number Diff line number Diff line change
Expand Up @@ -949,16 +949,6 @@ def infinite_defaultdict():
if ('priority' in curr['default_destination']
and isinstance(curr['default_destination']['priority'], dict)):

if new_config['default_priority'] not in curr['default_destination']['priority']:
error = ("Default priority '"
+ str(new_config['default_priority'])
+ "' does not appear in"
+ " default_destination for '"
+ str(tool) + "'.")
if verbose:
log.debug(error)
valid_config = False

for priority in curr['default_destination']['priority']:
destination = curr['default_destination']['priority'][priority]
if priority in priority_list:
Expand Down

0 comments on commit 4c402b3

Please sign in to comment.