Skip to content
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

Add ability to specify worker in ramp-test submission #178

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jorisvandenbossche
Copy link
Member

This works with the master branch of ramp-board (only ramp-engine and ramp-utils is needed).

The usage is like:

ramp-test --submission starting_kit_local2 --worker conda_config.yml 
# or
ramp-test --submission starting_kit_local2 --worker aws_config.yml 

where the configuration file then has the information about the worker.

TODO:

  • add tests
  • think about how this needs to interact with the other options to ramp-test, because currently they are not passed through.

@jorisvandenbossche
Copy link
Member Author

Need to update this for the latest changes in ramp-board

@kegl
Copy link
Contributor

kegl commented Mar 20, 2019

Looks great! Yes, can't we just pass the other options to the ramp-test command that the worker executes?

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.

2 participants