Migrate Tracis CI from GitHub Services to GitHub App

[SergioRAgostinho]

@jspricke GitHub Services are being phased out. We need to start using Travis CI through the GitHub Apps interface.

See more information here
https://developer.github.com/changes/2018-04-25-github-services-deprecation/

[jspricke]

On it

[jspricke]

I changed them to Apps, not sure if they work already..

[SergioRAgostinho]

Probably all is well. I’ll ping you again if needed.

[SergioRAgostinho]

Ok, there's something: I don't see GitHub CI added to the checks section of the repo.

Edit: It was also not triggered with Sergey's recent PR #2532. See https://travis-ci.org/PointCloudLibrary/pcl/pull_requests

[jspricke]

Seems to work now, except the doc job not sure if that should work @taketwo maybe you can have a look?

Also, we have a 100 minute timeout :).

[SergioRAgostinho]

Let me get everything to run green just to confirm and then we disable the old webhook to travis-ci.org and update the badge.

Edit: Just confirmed the 1h40m timeout. It's still not enough to build everything and run the tests.

[jspricke]

Should I ask for a longer timeout? Do you have an example where it failed because of the timeout?

[taketwo]

Here Sergio tried with a single build/test job and it failed with a timeout. After this he split the task into two and they became shorter than 100 minutes

It's hard to estimate how much more time we would need for the single job approach. Probably 30 minutes will be sufficient, but only empirical test can show. But anyway, if they give us longer timeout we will use less of their resources since now a lot of compilation is duplicated between our build and test tasks (despite the name, "test" actually means "build tests + test").

[jspricke]

Thanks, I would propose to leave it slit and fix the failing tests first. One we are sure that compiling everything makes sense, we can still ask for more.

[SergioRAgostinho]

FYI, I just learned Travis CI started rolling out Windows environments. It's not supporting C/C++ yet though.

[taketwo]

Did Anna tell you this? 😉

[SergioRAgostinho]

Reopening because of the issue with the docs job in #2538 .