Adding multi-arch support for linux/ppc64le in ml-metadata #171
Conversation
|
Note that this PR is part of a greater effort to enable Kubeflow and all its dependencies for multiple processor architectures (we start with ppc64le): kubeflow/community#781 |
|
Hi @lehrig, I have transferred to another team. Please reach out to @ryanpeters-google as he is the owner for MLMD OSS now. |
|
Alright, so @ryanpeters-google, are we good to proceed here? |
|
Due to an internal issue unrelated to the content of this PR, I am unable to test your change at this time. I am working to resolve the issue internally. Targeting Q2 for resolution. |
|
@ryanpeters-google: Any updates on this request? |
XinranTang
left a comment
XinranTang
left a comment
There was a problem hiding this comment.
Overall LGTM :) Could you provide some end-to-end results to show how you are able to build MLMD on ppc64le platforms?
| unzip \ | ||
| software-properties-common \ | ||
| git \ | ||
| openjdk-8-jdk |
There was a problem hiding this comment.
| openjdk-8-jdk | |
| openjdk-8-jdk \ |
There was a problem hiding this comment.
Why this dependency is needed?
There was a problem hiding this comment.
we need this dependency because bazel binary for linux/ppc64le needs java.
There was a problem hiding this comment.
@XinranTang - does that clarify your question?
|
I believe my PR #188 (for ARM64) could be extended with PPC64 support once merged. |
7 participants
No description provided.