{"action":"created","issue":{"url":"https://api.github.com/repos/galaxyproject/galaxy/issues/782","labels_url":"https://api.github.com/repos/galaxyproject/galaxy/issues/782/labels{/name}","comments_url":"https://api.github.com/repos/galaxyproject/galaxy/issues/782/comments","events_url":"https://api.github.com/repos/galaxyproject/galaxy/issues/782/events","html_url":"https://github.com/galaxyproject/galaxy/pull/782","id":107948119,"number":782,"title":"Unify the docker command, command_inject, and command_wrapper","user":{"login":"erasche","id":458683,"avatar_url":"https://avatars.githubusercontent.com/u/458683?v=3","gravatar_id":"","url":"https://api.github.com/users/erasche","html_url":"https://github.com/erasche","followers_url":"https://api.github.com/users/erasche/followers","following_url":"https://api.github.com/users/erasche/following{/other_user}","gists_url":"https://api.github.com/users/erasche/gists{/gist_id}","starred_url":"https://api.github.com/users/erasche/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/erasche/subscriptions","organizations_url":"https://api.github.com/users/erasche/orgs","repos_url":"https://api.github.com/users/erasche/repos","events_url":"https://api.github.com/users/erasche/events{/privacy}","received_events_url":"https://api.github.com/users/erasche/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/galaxyproject/galaxy/labels/enhancement","name":"enhancement","color":"84b6eb"}],"state":"open","locked":false,"assignee":{"login":"bgruening","id":469983,"avatar_url":"https://avatars.githubusercontent.com/u/469983?v=3","gravatar_id":"","url":"https://api.github.com/users/bgruening","html_url":"https://github.com/bgruening","followers_url":"https://api.github.com/users/bgruening/followers","following_url":"https://api.github.com/users/bgruening/following{/other_user}","gists_url":"https://api.github.com/users/bgruening/gists{/gist_id}","starred_url":"https://api.github.com/users/bgruening/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bgruening/subscriptions","organizations_url":"https://api.github.com/users/bgruening/orgs","repos_url":"https://api.github.com/users/bgruening/repos","events_url":"https://api.github.com/users/bgruening/events{/privacy}","received_events_url":"https://api.github.com/users/bgruening/received_events","type":"User","site_admin":false},"milestone":null,"comments":2,"created_at":"2015-09-23T15:39:28Z","updated_at":"2015-09-23T16:00:00Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/galaxyproject/galaxy/pulls/782","html_url":"https://github.com/galaxyproject/galaxy/pull/782","diff_url":"https://github.com/galaxyproject/galaxy/pull/782.diff","patch_url":"https://github.com/galaxyproject/galaxy/pull/782.patch"},"body":"Assigned to Björn to ensure he's happy with the change as well. CC @scholtalbers can you test this with your `sg` setup please?\r\n\r\nInstead of three separate options: `command`, `command_inject`, and `command_wrapper`, they've been unified into the `command` option. This makes life easier and it *looks* more like the docker command you run on the command line, so a little less magic for the admins."},"comment":{"url":"https://api.github.com/repos/galaxyproject/galaxy/issues/comments/142647826","html_url":"https://github.com/galaxyproject/galaxy/pull/782#issuecomment-142647826","issue_url":"https://api.github.com/repos/galaxyproject/galaxy/issues/782","id":142647826,"user":{"login":"jmchilton","id":216771,"avatar_url":"https://avatars.githubusercontent.com/u/216771?v=3","gravatar_id":"","url":"https://api.github.com/users/jmchilton","html_url":"https://github.com/jmchilton","followers_url":"https://api.github.com/users/jmchilton/followers","following_url":"https://api.github.com/users/jmchilton/following{/other_user}","gists_url":"https://api.github.com/users/jmchilton/gists{/gist_id}","starred_url":"https://api.github.com/users/jmchilton/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jmchilton/subscriptions","organizations_url":"https://api.github.com/users/jmchilton/orgs","repos_url":"https://api.github.com/users/jmchilton/repos","events_url":"https://api.github.com/users/jmchilton/events{/privacy}","received_events_url":"https://api.github.com/users/jmchilton/received_events","type":"User","site_admin":false},"created_at":"2015-09-23T16:00:00Z","updated_at":"2015-09-23T16:00:00Z","body":"The config file should have all the options commented out and the default displayed. The default should be hard coded in the Python and not read from the file. That way if the default behavior changes, these config files should not have to change. You should only have to update the config files you explicitly modify. This is the pattern we try to use with galaxy.ini and I think is pretty common."}}