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

Support ppc64le architecture build #2272

Closed
svghadi opened this issue Jun 3, 2020 · 4 comments · Fixed by #2293
Closed

Support ppc64le architecture build #2272

svghadi opened this issue Jun 3, 2020 · 4 comments · Fixed by #2293

Comments

@svghadi
Copy link
Contributor

svghadi commented Jun 3, 2020

Adding a travis-ci job which runs on ppc64le arch. Any thoughts on this?

@ghost ghost added the needs-triage label Jun 3, 2020
@pavolloffay
Copy link
Member

There have been already some attempts to do this https://github.com/jaegertracing/jaeger/search?q=ppc64le&type=Issues.

Now the makefile is refactored which should make it easy to add new architecture builds. Would you like to submit a PR?

@pavolloffay pavolloffay changed the title Support ppc64le architecture on travis-ci ? Support ppc64le architecture build Jun 3, 2020
@svghadi
Copy link
Contributor Author

svghadi commented Jun 3, 2020

Sure. I will start working on it.

@svghadi
Copy link
Contributor Author

svghadi commented Jun 11, 2020

Hi @pavolloffay,

Could community release a multi-arch jaegertracing/protobuf:0.1.0 image which is required for PROTO_GEN_TEST? Currently the image is only available for amd64 and is resulting in exec format error on ppc64le architecture. Log

Or should we only run unit test on ppc64le like mentioned in #1982 (comment)

  1. the extra time required to perform a build. Instead of running the whole build in a new arch, only the fast unit-tests should be executed

@pavolloffay
Copy link
Member

Could you please open an issue or a PR to that repository with changes required to the dockerfile and build infra(if any)?

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 a pull request may close this issue.

2 participants