Ifpack2 tests broken in standrd CI build starting 8/11/2017
Created by: bartlettroscoe
CC: @trilinos/framework, @trilinos/intrepid2, @kyungjoo-kim
Description:
The package Intrepid2 is now showing a build failure in the CI build with the first breaking CI iteration yesterday shown at:
The build failures shown at:
show:
/scratch/rabartl/Trilinos.base/SEMSCIBuild/Trilinos/packages/intrepid2/refactor/unit-test/Orientation/Serial/test_orientationtools_coeff_matrix.cpp:54:54: fatal error: test_orientationtools_tet_coeff_matrix.hpp: No such file or directory
#include "test_orientationtools_tet_coeff_matrix.hpp"
^
compilation terminated.
Looking at the new commits pulled for this CI iteration at:
we see that the following commits have caused this break:
70c425a: Intrepid2 - tested with triangle hcurl and hdiv
Author: Kyungjoo Kim <kyukim@sandia.gov>
Date: Fri Aug 11 15:18:41 2017 -0600
M packages/intrepid2/refactor/src/Orientation/Intrepid2_OrientationTools.hpp
M packages/intrepid2/refactor/src/Orientation/Intrepid2_OrientationToolsDefMatrixData.hpp
M packages/intrepid2/refactor/unit-test/Orientation/Serial/test_orientationtools_coeff_matrix.cpp
M packages/intrepid2/refactor/unit-test/Orientation/test_orientationtools_hex_coeff_matrix.hpp
M packages/intrepid2/refactor/unit-test/Orientation/test_orientationtools_quad_coeff_matrix.hpp
M packages/intrepid2/refactor/unit-test/Orientation/test_orientationtools_tri_coeff_matrix.hpp
d9897aa: Intrepid2 - hcurl and hdiv of triangle and tet always requires orientation
Author: Kyungjoo Kim <kyukim@sandia.gov>
Date: Fri Aug 11 15:17:24 2017 -0600
M packages/intrepid2/refactor/src/Discretization/Basis/Intrepid2_HCURL_TET_In_FEM.hpp
M packages/intrepid2/refactor/src/Discretization/Basis/Intrepid2_HCURL_TET_In_FEMDef.hpp
M packages/intrepid2/refactor/src/Discretization/Basis/Intrepid2_HCURL_TRI_In_FEM.hpp
M packages/intrepid2/refactor/src/Discretization/Basis/Intrepid2_HDIV_TET_In_FEM.hpp
M packages/intrepid2/refactor/src/Discretization/Basis/Intrepid2_HDIV_TRI_In_FEM.hpp
98561e5: Intrepid2 - triangle hcurl and hdiv orientation setup tested
Author: Kyungjoo Kim <kyukim@sandia.gov>
Date: Fri Aug 11 13:38:24 2017 -0600
M packages/intrepid2/refactor/src/Orientation/Intrepid2_OrientationToolsDefCoeffMatrix_HCURL.hpp
M packages/intrepid2/refactor/src/Orientation/Intrepid2_OrientationToolsDefCoeffMatrix_HDIV.hpp
M packages/intrepid2/refactor/unit-test/Orientation/Serial/test_orientationtools_coeff_matrix.cpp
M packages/intrepid2/refactor/unit-test/Orientation/test_orientationtools_tri_coeff_matrix.hpp
Looking at the push log for Trilinos we see the push:
Fri Aug 11 15:20:17 MDT 2017
commit 70c425ab810664a6470bc02d4e0c58ed15b130c4
Author: Kyungjoo Kim <kyukim@sandia.gov>
AuthorDate: Fri Aug 11 15:18:41 2017 -0600
Commit: Kyungjoo Kim <kyukim@sandia.gov>
CommitDate: Fri Aug 11 15:18:41 2017 -0600
Intrepid2 - tested with triangle hcurl and hdiv
Commits pushed:
70c425a Intrepid2 - tested with triangle hcurl and hdiv
d9897aa Intrepid2 - hcurl and hdiv of triangle and tet always requires orientation
98561e5 Intrepid2 - triangle hcurl and hdiv orientation setup tested
As you can see from that push log, there is no evidence that the checkin-test-sems.sh script was used to test and push this change.
This is just a test failure, and not a library failure. However, anyone trying to use the checkin-test-sems.sh script to push changes to any of the following packages upstream from Ifpack2 will have their push stopped due to this failing test build:
- Kokkos Teuchos KokkosKernels RTOp Sacado Epetra Shards Triutils Tpetra TrilinosSS EpetraExt Thyra Xpetra Galeri Amesos Pamgen
Therefore, this needs to be fixed ASAP or this test needs to be disabled so that it does not stop other people's pushes.