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

[Stabilizer.cpp] enable to change compensation limit : omission of ht… #929

Merged
merged 1 commit into from
Dec 27, 2015

Conversation

eisoku9618
Copy link
Contributor

…tps://github.com//pull/852

#852 のコミット忘れです.

@k-okada
Copy link
Contributor

k-okada commented Dec 26, 2015

Refer to this link for build results (access rights to CI server needed):
http://jenkins.jsk.imi.i.u-tokyo.ac.jp:8080/job/hrpsys-qnx/2508/

Build Log
last 10 lines

[...truncated 3785 lines...]
    at org.jenkinsci.plugins.ghprb.GhprbRepository.createCommitStatus(GhprbRepository.java:122)
    at org.jenkinsci.plugins.ghprb.GhprbBuilds.onCompleted(GhprbBuilds.java:118)
    at org.jenkinsci.plugins.ghprb.GhprbBuildListener.onCompleted(GhprbBuildListener.java:27)
    at org.jenkinsci.plugins.ghprb.GhprbBuildListener.onCompleted(GhprbBuildListener.java:12)
    at hudson.model.listeners.RunListener.fireCompleted(RunListener.java:201)
    at hudson.model.Run.execute(Run.java:1783)
    at hudson.model.FreeStyleBuild.run(FreeStyleBuild.java:43)
    at hudson.model.ResourceController.execute(ResourceController.java:98)
    at hudson.model.Executor.run(Executor.java:410)

Test FAILed.

@snozawa
Copy link
Contributor

snozawa commented Dec 26, 2015

@eisoku9618
Please check build on your PC.
https://travis-ci.org/fkanehiro/hrpsys-base/builds/98835018

@k-okada
Copy link
Contributor

k-okada commented Dec 26, 2015

Refer to this link for build results (access rights to CI server needed):
http://jenkins.jsk.imi.i.u-tokyo.ac.jp:8080/job/hrpsys-qnx/2510/

Build Log
last 10 lines

[...truncated 3779 lines...]
    at org.jenkinsci.plugins.ghprb.GhprbRepository.createCommitStatus(GhprbRepository.java:122)
    at org.jenkinsci.plugins.ghprb.GhprbBuilds.onCompleted(GhprbBuilds.java:118)
    at org.jenkinsci.plugins.ghprb.GhprbBuildListener.onCompleted(GhprbBuildListener.java:27)
    at org.jenkinsci.plugins.ghprb.GhprbBuildListener.onCompleted(GhprbBuildListener.java:12)
    at hudson.model.listeners.RunListener.fireCompleted(RunListener.java:201)
    at hudson.model.Run.execute(Run.java:1783)
    at hudson.model.FreeStyleBuild.run(FreeStyleBuild.java:43)
    at hudson.model.ResourceController.execute(ResourceController.java:98)
    at hudson.model.Executor.run(Executor.java:410)

Test FAILed.

@eisoku9618
Copy link
Contributor Author

Thank you, I've updated and passed travis.
(jenkins failed since there is "No space left on device" @k-okada )

@snozawa
Copy link
Contributor

snozawa commented Dec 27, 2015

LGTM.
travisが通ってることも確認しました。

fkanehiro added a commit that referenced this pull request Dec 27, 2015
[Stabilizer.cpp] enable to change compensation limit : omission of ht…
@fkanehiro fkanehiro merged commit 685f73a into fkanehiro:master Dec 27, 2015
@k-okada
Copy link
Contributor

k-okada commented Dec 27, 2015

http://jenkins.jsk.imi.i.u-tokyo.ac.jp:8080/job/hrpsys-qnx/2517/console
で確認しています.多分通らないと思う

@k-okada
Copy link
Contributor

k-okada commented Dec 27, 2015

#927 (comment) にエラー内容を張りました

@eisoku9618 eisoku9618 deleted the patch-3 branch December 27, 2015 05:00
@eisoku9618
Copy link
Contributor Author

はい, #927 のPRはjenkins通らないのですが,このPRはjenkins通ると思っています.

@k-okada
Copy link
Contributor

k-okada commented Dec 27, 2015

すいません.そうでした.

@garaemon
Copy link
Contributor

garaemon commented Jan 5, 2016

@eisoku9618
この変更が入ってからの実機での動作確認有りますか?

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.

5 participants