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

Adding power support to Travis builds #1566

Merged
merged 1 commit into from
Jan 15, 2020

Conversation

ghatwala
Copy link
Contributor

As we all know Travis officially supports multiple arch's now - https://blog.travis-ci.com/2019-11-12-multi-cpu-architecture-ibm-power-ibm-z .
Adding support for "ppc64le" arch for this repo with below changes ..

  1. Installing moonjit in place of luajit for power
  2. Using gcc-7, g++-7 compiler as default during builds for power
  3. Avoiding installing gcc-4.8 for power
  4. Adding power support in travis.yml

sysdig-CLA-1.0-contributing-entity: IBM
sysdig-CLA-1.0-signed-off-by: Amit Ghatwal ghatwala@us.ibm.com

As we all know Travis officially supports multiple arch's now - https://blog.travis-ci.com/2019-11-12-multi-cpu-architecture-ibm-power-ibm-z .
Adding support for "ppc64le" arch for this repo with below changes ..

1. Installing moonjit in place of luajit for power
2. Using gcc-7, g++-7 compiler as default during builds for power
3. Avoiding installing gcc-4.8 for power
4. Adding power support in travis.yml

sysdig-CLA-1.0-contributing-entity: IBM
sysdig-CLA-1.0-signed-off-by: Amit Ghatwal ghatwala@us.ibm.com
@nathan-b
Copy link
Contributor

Hi @ghatwala . When I look at the travis results for this PR, I don't see the ppc64le run. Where can I check to verify that it's running correctly?

@edmondsw
Copy link

@nathan-b I see ppc64le in the Travis results. If you click on the Details link for the Travic CI - Pull Request check, you should see 6 jobs were run (2 for x86 linux, 2 for osx, and 2 for ppc64le linux) instead of just 4. The last two of those are the new ppc64le jobs. E.g. https://travis-ci.com/draios/sysdig/jobs/268086746

@nathan-b
Copy link
Contributor

@edmondsw thanks, I thought I clicked through all the Linux builds and saw AMD64, but I must have clicked on the same ones twice. Looks good, thanks!

@nathan-b nathan-b merged commit a064440 into draios:dev Jan 15, 2020
@ghatwala
Copy link
Contributor Author

Thanks @nathan-b and @edmondsw !!!

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.

None yet

3 participants