kgiusti commented on pull request #1351: URL: https://github.com/apache/qpid-dispatch/pull/1351#issuecomment-1030390399
@jiridanek: And to your point (re: usefulness): it takes way too long to get feedback from travis for _pull request verification_. Checking the history we're talking ~5 hours plus to get through travis. GHA runs the _same tests_ in under 30 minutes - which gives me enough confidence that my PR is solid enough for review. Where I think travis can give us value is testing alternate platforms and tooling, like ARM and clang, which aren't currently covered in GHA. Also travis seems to have a knack for finding TSAN issues better than GHA - for whatever reason. Let's leverage that strength as well. So here's what this patch proposes: - reduce redundancies with GHA, favoring GHA over travis given the fast turn around time. - reduce the load on travis by trimming the test cases back to those that provide alternative test configurations from what we're already covering in GHA. This results in a quicker test run as well as lessening the burden on ASF resources. The patch does this by reducing the travis test runs to the following: - Coverage (which generates reports on our test effectiveness) - Default Router build using clang - TSAN checking, esp. on non-x86_64 architectures: arm64 and s390x -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
