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

Setup CI for alignment pipeline (resolves #239) #251

Merged
merged 5 commits into from
May 4, 2016

Conversation

jvivian
Copy link
Collaborator

@jvivian jvivian commented Apr 25, 2016

No description provided.

@hannes-ucsc
Copy link
Contributor

The title Sneak in commit should describe what it fixes. In the grand scheme of things (when we look at the history one year from now) it's irrelevant that the change was snuck in as part of an unrelated PR.

@jvivian jvivian force-pushed the issues/239-setup-ci-for-alignment-pipeline branch from b7214f6 to eb491a2 Compare April 25, 2016 23:52
@jvivian
Copy link
Collaborator Author

jvivian commented Apr 26, 2016

@hannes-ucsc Jenkins build fails due to timeout of bwa test. On Openstack (15 cores), it runs in 7 minutes but on a 2 core m3.large the BWA algorithm is too cpu intensive to finish within 30m even for a very small input (~160M). I looked on the Jenkins slave and can see by the tmp directory that BWA was almost finished running. We could make the toil-scripts slave a little beefier and it should go through — I'd recommend a c3.large as the best price point at 4-cores.

@hannes-ucsc
Copy link
Contributor

Made instance type c3.large and bumped timeout to 45min.

Jenkins, test this please.

@hannes-ucsc
Copy link
Contributor

In the long run, we should separate out a special slave roles in cgcloud for the toil-scripts and cgl-docker-lib: BD2KGenomics/cgcloud#154

@hannes-ucsc
Copy link
Contributor

Bumped timeout to 1h.

Jenkins, test this please.

@jvivian
Copy link
Collaborator Author

jvivian commented Apr 28, 2016

BWA CI is failing for some reason, unfortunately I don't get a stacktrace or anything from jenkins console output — looking into it now.

@jvivian
Copy link
Collaborator Author

jvivian commented Apr 28, 2016

Just tested this on an m3.large toil-jenkins-slave and it ran in 31 minutes and passed. I made some even smaller inputs (1MB down from 150MB), to see if i can get it to run in a couple minutes, but I'm a bit confused why the last jenkins build marked bwa_alignment as failed...
screen shot 2016-04-28 at 2 26 55 pm

@jvivian
Copy link
Collaborator Author

jvivian commented Apr 28, 2016

@hannes-ucsc — Tests pass now. Using the sample improvements I made here to speed up the RNA-seq CGL pipeline CI tests #254

@hannes-ucsc hannes-ucsc merged commit ab7c2e8 into master May 4, 2016
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