{"action":"closed","issue":{"url":"https://api.github.com/repos/trilinos/Trilinos/issues/1600","repository_url":"https://api.github.com/repos/trilinos/Trilinos","labels_url":"https://api.github.com/repos/trilinos/Trilinos/issues/1600/labels{/name}","comments_url":"https://api.github.com/repos/trilinos/Trilinos/issues/1600/comments","events_url":"https://api.github.com/repos/trilinos/Trilinos/issues/1600/events","html_url":"https://github.com/trilinos/Trilinos/issues/1600","id":249815728,"number":1600,"title":"Ifpack2 tests broken in standrd CI build starting 8/11/2017","user":{"login":"bartlettroscoe","id":1959736,"avatar_url":"https://avatars1.githubusercontent.com/u/1959736?v=4","gravatar_id":"","url":"https://api.github.com/users/bartlettroscoe","html_url":"https://github.com/bartlettroscoe","followers_url":"https://api.github.com/users/bartlettroscoe/followers","following_url":"https://api.github.com/users/bartlettroscoe/following{/other_user}","gists_url":"https://api.github.com/users/bartlettroscoe/gists{/gist_id}","starred_url":"https://api.github.com/users/bartlettroscoe/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bartlettroscoe/subscriptions","organizations_url":"https://api.github.com/users/bartlettroscoe/orgs","repos_url":"https://api.github.com/users/bartlettroscoe/repos","events_url":"https://api.github.com/users/bartlettroscoe/events{/privacy}","received_events_url":"https://api.github.com/users/bartlettroscoe/received_events","type":"User","site_admin":false},"labels":[{"id":310381463,"url":"https://api.github.com/repos/trilinos/Trilinos/labels/Framework","name":"Framework","color":"bfd4f2","default":false},{"id":330784381,"url":"https://api.github.com/repos/trilinos/Trilinos/labels/Intrepid2","name":"Intrepid2","color":"d4c5f9","default":false}],"state":"closed","locked":false,"assignee":{"login":"kyungjoo-kim","id":6992602,"avatar_url":"https://avatars2.githubusercontent.com/u/6992602?v=4","gravatar_id":"","url":"https://api.github.com/users/kyungjoo-kim","html_url":"https://github.com/kyungjoo-kim","followers_url":"https://api.github.com/users/kyungjoo-kim/followers","following_url":"https://api.github.com/users/kyungjoo-kim/following{/other_user}","gists_url":"https://api.github.com/users/kyungjoo-kim/gists{/gist_id}","starred_url":"https://api.github.com/users/kyungjoo-kim/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kyungjoo-kim/subscriptions","organizations_url":"https://api.github.com/users/kyungjoo-kim/orgs","repos_url":"https://api.github.com/users/kyungjoo-kim/repos","events_url":"https://api.github.com/users/kyungjoo-kim/events{/privacy}","received_events_url":"https://api.github.com/users/kyungjoo-kim/received_events","type":"User","site_admin":false},"assignees":[{"login":"kyungjoo-kim","id":6992602,"avatar_url":"https://avatars2.githubusercontent.com/u/6992602?v=4","gravatar_id":"","url":"https://api.github.com/users/kyungjoo-kim","html_url":"https://github.com/kyungjoo-kim","followers_url":"https://api.github.com/users/kyungjoo-kim/followers","following_url":"https://api.github.com/users/kyungjoo-kim/following{/other_user}","gists_url":"https://api.github.com/users/kyungjoo-kim/gists{/gist_id}","starred_url":"https://api.github.com/users/kyungjoo-kim/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kyungjoo-kim/subscriptions","organizations_url":"https://api.github.com/users/kyungjoo-kim/orgs","repos_url":"https://api.github.com/users/kyungjoo-kim/repos","events_url":"https://api.github.com/users/kyungjoo-kim/events{/privacy}","received_events_url":"https://api.github.com/users/kyungjoo-kim/received_events","type":"User","site_admin":false}],"milestone":null,"comments":4,"created_at":"2017-08-12T14:36:39Z","updated_at":"2017-08-14T14:56:23Z","closed_at":"2017-08-14T14:56:23Z","body":"**CC:** @trilinos/framework, @trilinos/intrepid2, @kyungjoo-kim \r\n\r\n**Description:**\r\n\r\nThe package Intrepid2 is now showing a build failure in the CI build with the first breaking CI iteration yesterday shown at:\r\n\r\n* https://testing.sandia.gov/cdash/index.php?project=Trilinos&parentid=3050255\r\n\r\nThe build failures shown at:\r\n\r\n* https://testing.sandia.gov/cdash/viewBuildError.php?buildid=3050256\r\n\r\nshow:\r\n\r\n```\r\n/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\r\n #include \"test_orientationtools_tet_coeff_matrix.hpp\"\r\n                                                      ^\r\ncompilation terminated.\r\n```\r\n\r\nLooking at the new commits pulled for this CI iteration at:\r\n\r\n* https://testing.sandia.gov/cdash/viewNotes.php?buildid=3050256##note0\r\n\r\nwe see that the following commits have caused this break:\r\n\r\n```\r\n70c425a:  Intrepid2 - tested with triangle hcurl and hdiv\r\nAuthor: Kyungjoo Kim <kyukim@sandia.gov>\r\nDate:   Fri Aug 11 15:18:41 2017 -0600\r\n\r\nM\tpackages/intrepid2/refactor/src/Orientation/Intrepid2_OrientationTools.hpp\r\nM\tpackages/intrepid2/refactor/src/Orientation/Intrepid2_OrientationToolsDefMatrixData.hpp\r\nM\tpackages/intrepid2/refactor/unit-test/Orientation/Serial/test_orientationtools_coeff_matrix.cpp\r\nM\tpackages/intrepid2/refactor/unit-test/Orientation/test_orientationtools_hex_coeff_matrix.hpp\r\nM\tpackages/intrepid2/refactor/unit-test/Orientation/test_orientationtools_quad_coeff_matrix.hpp\r\nM\tpackages/intrepid2/refactor/unit-test/Orientation/test_orientationtools_tri_coeff_matrix.hpp\r\n\r\nd9897aa:  Intrepid2 - hcurl and hdiv of triangle and tet always requires orientation\r\nAuthor: Kyungjoo Kim <kyukim@sandia.gov>\r\nDate:   Fri Aug 11 15:17:24 2017 -0600\r\n\r\nM\tpackages/intrepid2/refactor/src/Discretization/Basis/Intrepid2_HCURL_TET_In_FEM.hpp\r\nM\tpackages/intrepid2/refactor/src/Discretization/Basis/Intrepid2_HCURL_TET_In_FEMDef.hpp\r\nM\tpackages/intrepid2/refactor/src/Discretization/Basis/Intrepid2_HCURL_TRI_In_FEM.hpp\r\nM\tpackages/intrepid2/refactor/src/Discretization/Basis/Intrepid2_HDIV_TET_In_FEM.hpp\r\nM\tpackages/intrepid2/refactor/src/Discretization/Basis/Intrepid2_HDIV_TRI_In_FEM.hpp\r\n\r\n98561e5:  Intrepid2 - triangle hcurl and hdiv orientation setup tested\r\nAuthor: Kyungjoo Kim <kyukim@sandia.gov>\r\nDate:   Fri Aug 11 13:38:24 2017 -0600\r\n\r\nM\tpackages/intrepid2/refactor/src/Orientation/Intrepid2_OrientationToolsDefCoeffMatrix_HCURL.hpp\r\nM\tpackages/intrepid2/refactor/src/Orientation/Intrepid2_OrientationToolsDefCoeffMatrix_HDIV.hpp\r\nM\tpackages/intrepid2/refactor/unit-test/Orientation/Serial/test_orientationtools_coeff_matrix.cpp\r\nM\tpackages/intrepid2/refactor/unit-test/Orientation/test_orientationtools_tri_coeff_matrix.hpp\r\n```\r\n\r\nLooking at the push log for Trilinos we see the push:\r\n\r\n```\r\nFri Aug 11 15:20:17 MDT 2017\r\n\r\ncommit 70c425ab810664a6470bc02d4e0c58ed15b130c4\r\nAuthor:     Kyungjoo Kim <kyukim@sandia.gov>\r\nAuthorDate: Fri Aug 11 15:18:41 2017 -0600\r\nCommit:     Kyungjoo Kim <kyukim@sandia.gov>\r\nCommitDate: Fri Aug 11 15:18:41 2017 -0600\r\n\r\n    Intrepid2 - tested with triangle hcurl and hdiv\r\n\r\nCommits pushed:\r\n70c425a Intrepid2 - tested with triangle hcurl and hdiv\r\nd9897aa Intrepid2 - hcurl and hdiv of triangle and tet always requires orientation\r\n98561e5 Intrepid2 - triangle hcurl and hdiv orientation setup tested\r\n```\r\n\r\nAs you can see from that push log, there is no evidence that the [checkin-test-sems.sh](https://github.com/trilinos/Trilinos/wiki/Policies-|-Safe-Checkin-Testing) script was used to test and push this change.\r\n\r\nThis 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:\r\n\r\n* Kokkos Teuchos KokkosKernels RTOp Sacado Epetra Shards Triutils Tpetra TrilinosSS EpetraExt Thyra Xpetra Galeri Amesos Pamgen\r\n\r\nTherefore, this needs to be fixed ASAP or this test needs to be disabled so that it does not stop other people's pushes.\r\n"}}
{"actor":{"display_login":"kyungjoo-kim"}}