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

[ADAM-532] Fix wigFix intermittent test failure #546

Merged
merged 1 commit into from Jan 10, 2015

Conversation

Projects
None yet
3 participants
@laserson
Contributor

laserson commented Jan 10, 2015

When reading the converted output, it's possible that the order of the records is undefined. I added a sort which should make things more reliable.

Fixes #532.

Uri Laserson
[ADAM-532] Fix wigFix intermittent test failure
When reading the converted output, it's possible that the order of the records is undefined.  I added a sort which should make things more reliable.

Fixes #532.
@laserson

This comment has been minimized.

Show comment
Hide comment
@laserson

laserson Jan 10, 2015

Contributor

Pretty sure this should fix the problem.

Contributor

laserson commented Jan 10, 2015

Pretty sure this should fix the problem.

@AmplabJenkins

This comment has been minimized.

Show comment
Hide comment
@AmplabJenkins

AmplabJenkins Jan 10, 2015

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/527/

Build result: FAILURE

GitHub pull request #546 of commit 7056125 automatically merged.[EnvInject] - Loading node environment variables.Building remotely on amp-jenkins-worker-05 (centos) in workspace /home/jenkins/workspace/ADAM-prb > git rev-parse --is-inside-work-tree # timeout=10Fetching changes from the remote Git repository > git config remote.origin.url https://github.com/bigdatagenomics/adam.git # timeout=10Fetching upstream changes from https://github.com/bigdatagenomics/adam.git > git --version # timeout=10 > git fetch --tags --progress https://github.com/bigdatagenomics/adam.git +refs/pull/:refs/remotes/origin/pr/ > git rev-parse origin/pr/546/merge^{commit} # timeout=10Checking out Revision 646efe2 (detached) > git config core.sparsecheckout # timeout=10 > git checkout -f 646efe2 > git rev-list b409bd0004a00c2dac712b5c567963b3e013c8e3 # timeout=10Triggering ADAM-prb ? 2.2.0,centosTriggering ADAM-prb ? 1.0.4,centosTriggering ADAM-prb ? 2.3.0,centosADAM-prb ? 2.2.0,centos completed with result SUCCESSADAM-prb ? 1.0.4,centos completed with result FAILUREADAM-prb ? 2.3.0,centos completed with result SUCCESS
Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/527/

Build result: FAILURE

GitHub pull request #546 of commit 7056125 automatically merged.[EnvInject] - Loading node environment variables.Building remotely on amp-jenkins-worker-05 (centos) in workspace /home/jenkins/workspace/ADAM-prb > git rev-parse --is-inside-work-tree # timeout=10Fetching changes from the remote Git repository > git config remote.origin.url https://github.com/bigdatagenomics/adam.git # timeout=10Fetching upstream changes from https://github.com/bigdatagenomics/adam.git > git --version # timeout=10 > git fetch --tags --progress https://github.com/bigdatagenomics/adam.git +refs/pull/:refs/remotes/origin/pr/ > git rev-parse origin/pr/546/merge^{commit} # timeout=10Checking out Revision 646efe2 (detached) > git config core.sparsecheckout # timeout=10 > git checkout -f 646efe2 > git rev-list b409bd0004a00c2dac712b5c567963b3e013c8e3 # timeout=10Triggering ADAM-prb ? 2.2.0,centosTriggering ADAM-prb ? 1.0.4,centosTriggering ADAM-prb ? 2.3.0,centosADAM-prb ? 2.2.0,centos completed with result SUCCESSADAM-prb ? 1.0.4,centos completed with result FAILUREADAM-prb ? 2.3.0,centos completed with result SUCCESS
Test FAILed.

@laserson

This comment has been minimized.

Show comment
Hide comment
@laserson

laserson Jan 10, 2015

Contributor

Jenkinst, retest this please.

Contributor

laserson commented Jan 10, 2015

Jenkinst, retest this please.

@laserson

This comment has been minimized.

Show comment
Hide comment
@laserson

laserson Jan 10, 2015

Contributor

Jenkins, retest this please.

Contributor

laserson commented Jan 10, 2015

Jenkins, retest this please.

@AmplabJenkins

This comment has been minimized.

Show comment
Hide comment
@AmplabJenkins

AmplabJenkins Jan 10, 2015

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/528/
Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/528/
Test PASSed.

fnothaft added a commit that referenced this pull request Jan 10, 2015

Merge pull request #546 from laserson/ADAM-532-wigfix-test
[ADAM-532] Fix wigFix intermittent test failure

@fnothaft fnothaft merged commit 229efb1 into bigdatagenomics:master Jan 10, 2015

1 check passed

default Merged build finished.
Details
@fnothaft

This comment has been minimized.

Show comment
Hide comment
@fnothaft

fnothaft Jan 10, 2015

Member

Merged! Thanks @laserson.

Member

fnothaft commented Jan 10, 2015

Merged! Thanks @laserson.

@laserson laserson deleted the laserson:ADAM-532-wigfix-test branch Jan 10, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment