Skip to content

More fine grained options for --shed_install #1001

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

AndreasSko
Copy link
Contributor

--shed_install uses the default options for installing tools from Ephemeris, resulting in automatic dependency resolution/installation on first run (see: https://github.com/galaxyproject/ephemeris/blob/6a01ac3de3f8e40145365523e3ae8807b9072282/src/ephemeris/shed_tools.py#L103). In some situations, where installing dependencies is not necessary (for example when using containers), disabling this would result in faster execution time.

With this change, the options --install_tool_dependencies, --install_resolver_dependencies, and --install_repository_dependencies are introduced, which enables to define tool installation using --shed_install more fine grained. The default options stay the same, so there should be no breaking change to existing usage.

@AndreasSko
Copy link
Contributor Author

cc @bgruening

@bgruening
Copy link
Member

@AndreasSko a few tests are failing, can you please look at those?

@AndreasSko
Copy link
Contributor Author

Exception [Problem executing commands psql --tuples-only --username postgres -P pager=off --command 'drop database plnmoprof_profile1234;' - (, psql: could not connect to server: No such file or directory
Is the server running locally and accepting connections on Unix domain socket "/var/run/postgresql/.s.PGSQL.5432"?)]

Seems strange..

`--shed_install` uses the default options for installing tools from
Ephemeris, resulting in automatic dependency resolution/installation on
first run
(see: https://github.com/galaxyproject/ephemeris/blob/6a01ac3de3f8e40145365523e3ae8807b9072282/src/ephemeris/shed_tools.py#L103).
In some situations, where installing dependencies is not necessary
(for example when using containers), disabling this would result in
faster execution time.

With this change, the options `--install_tool_dependencies`,
`--install_resolver_dependencies`, and
`--install_repository_dependencies` are introduced, which enables to
define tool installation using `--shed_install` more fine grained.
The default options stay the same, so there should be no breaking
change to existing usage.
@AndreasSko AndreasSko force-pushed the more_shed_install_options branch from 2dff5b3 to d871c03 Compare March 11, 2020 12:29
@AndreasSko
Copy link
Contributor Author

Is it possible that the tests/pipeline itself is broken currently? Tested the master branch on my fork and it failed with the same problem: https://github.com/AndreasSko/planemo/actions/runs/53662366

@jmchilton jmchilton merged commit cd522a6 into galaxyproject:master Mar 13, 2020
@jmchilton
Copy link
Member

Those look like pre-existing problems - thanks for the enhancements! They look great!

@AndreasSko AndreasSko deleted the more_shed_install_options branch March 14, 2020 15:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants