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

Travis fails but stays green #192

Closed
rijobro opened this issue Jan 23, 2019 · 11 comments
Closed

Travis fails but stays green #192

rijobro opened this issue Jan 23, 2019 · 11 comments
Assignees

Comments

@rijobro
Copy link
Contributor

rijobro commented Jan 23, 2019

Travis shows that all builds are green for the Superbuild (e.g., most recent build).

However, if you check one of the OSX builds (e.g., this one), you'll notice:

3: The following tests FAILED:
3: 	  1 - test_all (Failed)
3: Errors while running CTest
3/3 Test #3: Gadgetron_TESTS ..................***Failed   20.27 sec

These tests are also failing on SIRF, but they are actually showing up as having failed (e.g., link).

The reason they should be red is that they are failing the Gagetron unit tests (added in PR #182). This is a problem to be addressed elsewhere, but Travis should be flagging up the fact that it fails.

@rijobro
Copy link
Contributor Author

rijobro commented Jan 23, 2019

I think these are unrelated, as they are for cases that are expected to fail, so we want to return the opposite of the exit code of the script.

In the case of Gadgetron, if the test fails, we want to report that it fails.

Since the ctest fails, but travis is happy, my guess is that the answer lies in the way we've set up the .travis.yml (not sure though).

@KrisThielemans
Copy link
Member

I guess this then needs

ctest -VV || travis_terminate 1

in the .travis.yml

@rijobro
Copy link
Contributor Author

rijobro commented Jan 23, 2019

Ok, waiting to see what travis has to say about that: https://travis-ci.org/CCPPETMR/SIRF-SuperBuild/builds/483358628.

@casperdcl
Copy link
Member

doesn't explain why SIRF fails without needing the travis_terminate

@rijobro
Copy link
Contributor Author

rijobro commented Jan 23, 2019

This seems to fix it (link), but also shows up the docker builds in red. @casperdcl is this normal?

@casperdcl
Copy link
Member

ah I didn't look at the correct .travis.yml - was looking at SIRF, not the superBuild. The PR needs some modification, will do

@rijobro rijobro reopened this Feb 5, 2019
@rijobro
Copy link
Contributor Author

rijobro commented Feb 5, 2019

This has started happening again. Most recent error that should be causing it to be red happened during the cmake of SIRF:

CMake Error at /usr/local/Cellar/cmake/3.11.4/share/cmake/Modules/FindPackageHandleStandardArgs.cmake:137 (message):

  Could NOT find FFTW3 (missing: FFTW3F_LIBRARY FFTW3_INCLUDE_DIR)

Call Stack (most recent call first):

  /usr/local/Cellar/cmake/3.11.4/share/cmake/Modules/FindPackageHandleStandardArgs.cmake:378 (_FPHSA_FAILURE_MESSAGE)

  /Users/travis/build/CCPPETMR/SIRF-SuperBuild/INSTALL/lib/cmake/ISMRMRD/FindFFTW3.cmake:114 (find_package_handle_standard_args)

  src/xGadgetron/CMakeLists.txt:36 (find_package)





-- Configuring incomplete, errors occurred!

See also "/Users/travis/build/CCPPETMR/SIRF-SuperBuild/builds/SIRF/build/CMakeFiles/CMakeOutput.log".

make[2]: *** [builds/SIRF/stamp/SIRF-configure] Error 1

make[1]: *** [CMakeFiles/SIRF.dir/all] Error 2

make[1]: *** Waiting for unfinished jobs....

@rijobro rijobro changed the title Gadgetron tests fail but Travis stays green Travis fails but stays green Feb 5, 2019
@paskino
Copy link
Contributor

paskino commented Feb 5, 2019

on what OS?

@rijobro
Copy link
Contributor Author

rijobro commented Feb 5, 2019

both unfortunately...

@KrisThielemans
Copy link
Member

we won't update Travis things anymore

paskino added a commit to paskino/SIRF-SuperBuild that referenced this issue Oct 25, 2021
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

No branches or pull requests

5 participants