-
Notifications
You must be signed in to change notification settings - Fork 237
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
Reenable some Kubernetes CI tests #4604
Merged
Merged
Changes from 5 commits
Commits
Show all changes
23 commits
Select commit
Hold shift + click to select a range
4d57bf9
Reenable kubernetes tests that don't require a local cluster, eg CWL on
stxue1 16dbd67
Merge branch 'master' into issues/4601-reenable-nonlocal-kubernetes
stxue1 5cb2810
Disable CWL kubernetes
stxue1 611280e
Merge branch 'issues/4601-reenable-nonlocal-kubernetes' of github.com…
stxue1 7eb5782
Merge branch 'master' into issues/4601-reenable-nonlocal-kubernetes
adamnovak 6e513ab
enable cactus tests
stxue1 a942f41
Merge branch 'master' of github.com:DataBiosphere/toil into issues/46…
stxue1 45ee1fc
Add to scheduled integration tests
stxue1 aacf3e1
Merge branch 'issues/4601-reenable-nonlocal-kubernetes' of github.com…
stxue1 43caf6e
Add forgotten file
stxue1 1f78af3
Remove print statements
stxue1 fe06169
Merge branch 'master' into issues/4601-reenable-nonlocal-kubernetes
adamnovak 2ed0a77
Remove unnecessary env var and move file
stxue1 4698385
Run test when updated
stxue1 2034c18
update gitlab
stxue1 fdf7551
Merge branch 'issues/4601-reenable-nonlocal-kubernetes' of github.com…
stxue1 dc0cb63
Fix typo in path
stxue1 d625a08
Merge branch 'master' into issues/4601-reenable-nonlocal-kubernetes
adamnovak c8af0a8
Add virtualenv and prepare build to gitlab CI to run tests properly
stxue1 6966d11
add gitlab setup scripts
stxue1 2a59345
add gitlab setup scripts
stxue1 5659cdb
Merge branch 'issues/4601-reenable-nonlocal-kubernetes' of github.com…
stxue1 f3deda1
Merge branch 'master' into issues/4601-reenable-nonlocal-kubernetes
stxue1 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This right now is written assuming that the test is running somewhere that has access to a Kubernetes cluster, through a
~/.kube/config
or other source of credentials.For this PR to work, we need to make it more like the CWL on ARM test. There needs to be a
toil launch-cluster
call, and the installation and running ofcactus
has to be at the other end oftoil ssh-cluster
, so it happens on the cluster. You might want a test or script that you could invoke on the cluster to manage the installation and execution of Cactus, so you don't need to make a while bunch oftoil ssh-cluster
calls in a row.