Skip to content
This repository was archived by the owner on Sep 30, 2022. It is now read-only.

Conversation

rhc54
Copy link

@rhc54 rhc54 commented Nov 6, 2014

... provided. This is a 1.8 series change only, so no master commit is cited.

@hjelmn please review

@mellanox-github
Copy link

Merged build triggered.

@mellanox-github
Copy link

Merged build started.

@mellanox-github
Copy link

Merged build finished. Test PASSed.

@mellanox-github
Copy link

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
http://bgate.mellanox.com/jenkins/job/gh-ompi-release-v1.8-pr/10/
Test PASSed.

@mellanox-github
Copy link

Merged build triggered.

@mellanox-github
Copy link

Merged build started.

@mellanox-github
Copy link

Merged build finished. Test FAILed.

@mellanox-github
Copy link

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
http://bgate.mellanox.com/jenkins/job/gh-ompi-master-pr/1/

Build Log
last 50 lines

[...truncated 6 lines...]
 > /usr/bin/git init /var/lib/jenkins/jobs/gh-ompi-master-pr/workspace # timeout=60
Fetching upstream changes from https://github.com/open-mpi/ompi
 > /usr/bin/git --version # timeout=60
using .gitcredentials to set credentials
 > /usr/bin/git config --local credential.helper store --file=/tmp/git1418481543298245604.credentials # timeout=60
 > /usr/bin/git fetch --tags --progress https://github.com/open-mpi/ompi +refs/heads/*:refs/remotes/origin/* # timeout=60
 > /usr/bin/git config --local --remove-section credential # timeout=60
 > /usr/bin/git config remote.origin.url https://github.com/open-mpi/ompi # timeout=60
 > /usr/bin/git config remote.origin.fetch +refs/heads/*:refs/remotes/origin/* # timeout=60
 > /usr/bin/git config remote.origin.url https://github.com/open-mpi/ompi # timeout=60
Pruning obsolete local branches
Fetching upstream changes from https://github.com/open-mpi/ompi
using .gitcredentials to set credentials
 > /usr/bin/git config --local credential.helper store --file=/tmp/git830811408821228796.credentials # timeout=60
 > /usr/bin/git fetch --tags --progress https://github.com/open-mpi/ompi +refs/pull/*:refs/remotes/origin/pr/* --prune # timeout=60
 > /usr/bin/git config --local --remove-section credential # timeout=60
 > /usr/bin/git rev-parse origin/pr/79/merge^{commit} # timeout=60
FATAL: Command "/usr/bin/git rev-parse origin/pr/79/merge^{commit}" returned status code 128:
stdout: origin/pr/79/merge^{commit}

stderr: fatal: ambiguous argument 'origin/pr/79/merge^{commit}': unknown revision or path not in the working tree.
Use '--' to separate paths from revisions, like this:
'git <command> [<revision>...] -- [<file>...]'

hudson.plugins.git.GitException: Command "/usr/bin/git rev-parse origin/pr/79/merge^{commit}" returned status code 128:
stdout: origin/pr/79/merge^{commit}

stderr: fatal: ambiguous argument 'origin/pr/79/merge^{commit}': unknown revision or path not in the working tree.
Use '--' to separate paths from revisions, like this:
'git <command> [<revision>...] -- [<file>...]'

    at org.jenkinsci.plugins.gitclient.CliGitAPIImpl.launchCommandIn(CliGitAPIImpl.java:1435)
    at org.jenkinsci.plugins.gitclient.CliGitAPIImpl.launchCommandIn(CliGitAPIImpl.java:1411)
    at org.jenkinsci.plugins.gitclient.CliGitAPIImpl.launchCommandIn(CliGitAPIImpl.java:1407)
    at org.jenkinsci.plugins.gitclient.CliGitAPIImpl.launchCommand(CliGitAPIImpl.java:1110)
    at org.jenkinsci.plugins.gitclient.CliGitAPIImpl.launchCommand(CliGitAPIImpl.java:1120)
    at org.jenkinsci.plugins.gitclient.CliGitAPIImpl.revParse(CliGitAPIImpl.java:516)
    at hudson.plugins.git.GitAPI.revParse(GitAPI.java:257)
    at hudson.plugins.git.RevisionParameterAction.toRevision(RevisionParameterAction.java:83)
    at hudson.plugins.git.GitSCM.determineRevisionToBuild(GitSCM.java:815)
    at hudson.plugins.git.GitSCM.checkout(GitSCM.java:915)
    at hudson.model.AbstractProject.checkout(AbstractProject.java:1258)
    at hudson.model.AbstractBuild$AbstractBuildExecution.defaultCheckout(AbstractBuild.java:622)
    at jenkins.scm.SCMCheckoutStrategy.checkout(SCMCheckoutStrategy.java:86)
    at hudson.model.AbstractBuild$AbstractBuildExecution.run(AbstractBuild.java:528)
    at hudson.model.Run.execute(Run.java:1759)
    at hudson.model.FreeStyleBuild.run(FreeStyleBuild.java:43)
    at hudson.model.ResourceController.execute(ResourceController.java:89)
    at hudson.model.Executor.run(Executor.java:240)

Test FAILed.

@jsquyres
Copy link
Member

jsquyres commented Nov 6, 2014

@miked-mellanox That's a false jenkins failure, right?

@mike-dubman
Copy link
Member

yep, i just added jenkins job for "master" but forgot to change url.
sorry.

@jladd-mlnx
Copy link
Member

@rhc54 @hppritchard I think this should be signed off on by Howard before it's merged. It should also be added to the 1.8.4 release notes.

@hjelmn
Copy link
Member

hjelmn commented Nov 6, 2014

Looks good to me.

@hjelmn
Copy link
Member

hjelmn commented Nov 6, 2014

@jladd-mlnx this was discussed already at lanl and howard and I agreed this was the way to go for now

@rhc54
Copy link
Author

rhc54 commented Nov 6, 2014

I will be sure it gets into the NEWS as well.

@jladd-mlnx
Copy link
Member

@rhc54 thanks!

@hjelmn
Copy link
Member

hjelmn commented Nov 6, 2014

@jladd-mlnx I should mention the fix for message coalescing can be found here: hjelmn/ompi@db84aa9

@rhc54 rhc54 added this to the v1.8.4 milestone Nov 7, 2014
rhc54 pushed a commit that referenced this pull request Nov 7, 2014
Turn off message coalescing to avoid segfaults until a better fix can be...
@rhc54 rhc54 merged commit 5fec9ab into open-mpi:v1.8 Nov 7, 2014
@rhc54 rhc54 deleted the cmr/turnoff-coalesc branch December 3, 2014 15:12
mike-dubman added a commit to mike-dubman/ompi-release that referenced this pull request Apr 24, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants