-
Notifications
You must be signed in to change notification settings - Fork 528
BIGTOP-3639: Alluxio build fails on ppc64le #861
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
Conversation
| . `dirname $0`/bigtop.bom | ||
|
|
||
| if [ $HOSTTYPE = "powerpc64le" ] ; then | ||
| mvn install:install-file -DgroupId=io.grpc -DartifactId=protoc-gen-grpc-java -Dversion=1.28.1 -Dclassifier=linux-ppcle_64 -Dpackaging=exe -Dfile=/usr/src/grpc-java-1.28.0/compiler/build/exe/java_plugin/protoc-gen-grpc-java |
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.
The binary of grpc 1.28.0 should be installed as io.grpc:protoc-gen-grpc-java:1.28.0 rather than 1.28.1. Then adding -Dgrpc.version=1.28.0 to mvn options could be an option?
If grpc-1.28.0 does not work with Alluxio, you can upgrade the version of grpc added by BIGTOP-3560 (#813). You need to modify bigtop-packages/src/common/zeppelin/patch1-arm64-build.diff too since it refers to the 1.28.0.
|
We're facing this problem during the 3.0.1 build. @afsanjar May I address Masatake's comment based on your work, or could you update this PR? |
|
sure, I will take a look.
…On Wed, Mar 23, 2022 at 2:17 AM Kengo Seki ***@***.***> wrote:
We're facing this problem during the 3.0.1 build.
https://ci.bigtop.apache.org/view/Releases/job/Bigtop-3.0.1/11/
@afsanjar <https://github.com/afsanjar> May I address Masatake's comment
based on your work, or could you update this PR?
—
Reply to this email directly, view it on GitHub
<#861 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AE3DPVFJCGIECW4AK3O3KIDVBLARTANCNFSM5N3HJT6A>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
|
@sekikn, please review the changes. Thanks |
|
Thank you for updating the PR @afsanjar! |
|
No worries at all, I was able to access it. Did you try to ssh to it?
…On Fri, Mar 25, 2022 at 10:23 AM Kengo Seki ***@***.***> wrote:
Thank you for updating the PR @afsanjar <https://github.com/afsanjar>!
I just tried to test it on the ppc64 CI node, but it seems unavailable
now. Would you take a look? (Sorry if it's just a intermittent failure)
—
Reply to this email directly, view it on GitHub
<#861 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AE3DPVEMV4KLDNMOWTNXR2LVBXK7RANCNFSM5N3HJT6A>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
|
+1, tested on ppc64 and it worked. Thanks a lot @afsanjar! |
(cherry picked from commit b296ae6)
|
I found that only this PR leads to a build failure on Debian or Ubuntu. https://ci.bigtop.apache.org/view/Releases/job/Bigtop-3.0.1/12/DISTRO=debian-10,PLATFORM=ppc64le-slave/console I'm going to submit an addendum PR for it. |
Alluxio build fails on ppc64le