Skip to content

Consolidate disables for all builds on 'mutrino' (#3183)

Created by: bartlettroscoe

CC: @gsjaardema, @fryeguy52

Description

This also adds back the disables for several SEACAS tests that got removed when the file INTEL-RELEASE-OPENMP.cmake got renamed to the file INTEL-RELEASE-OPENMP-HSW.cmake (not clear how that happened).

Motivation and Context

The disables for the build intel-opt-openmp-HSW got removed on the file rename which resulted in these tests failing again in this build (see #3183 (closed)) and this much duplication is bad. We want less duplication.

How Has This Been Tested?

I ran local configures with:

$  ./checkin-test-atdm.sh all --enable-packages=SEACAS --configure

which returned:

PASSED (NOT READY TO PUSH): Trilinos: mutrino

Tue Aug  7 14:33:08 MDT 2018

Enabled Packages: SEACAS

Build test results:
-------------------
0) MPI_RELEASE_DEBUG_SHARED_PT_OPENMP => Test case MPI_RELEASE_DEBUG_SHARED_PT_OPENMP was not run! => Does not affect push readiness! (-1.00 min)
1) intel-debug-openmp-HSW => passed: configure-only passed => Not ready to push! (1.21 min)
2) intel-opt-openmp-HSW => passed: configure-only passed => Not ready to push! (1.23 min)
3) intel-debug-openmp-KNL => passed: configure-only passed => Not ready to push! (1.67 min)
4) intel-opt-openmp-KNL => passed: configure-only passed => Not ready to push! (1.57 min)

For the build intel-opt-openmp-HSW for example, the configure output showed:

-- Including ATDM build treaks file /home/rabartl/Trilinos.base/Trilinos/cmake/std/atdm/mutrino/tweaks/INTEL-RELEASE-OPENMP-HSW.cmake ...
-- Setting default Piro_MatrixFreeDecorator_UnitTests_MPI_4_DISABLE=ON
-- Setting default MueLu_CreateOperatorTpetra_MPI_1_DISABLE=ON
-- Setting default MueLu_CreateOperatorTpetra_MPI_4_DISABLE=ON
-- Setting default MueLu_ParameterListInterpreterTpetra_MPI_1_DISABLE=ON
-- Setting default MueLu_ParameterListInterpreterTpetra_MPI_4_DISABLE=ON
-- Setting default MueLu_ParameterListInterpreterTpetraHeavy_MPI_1_DISABLE=ON
-- Setting default MueLu_ParameterListInterpreterTpetraHeavy_MPI_4_DISABLE=ON
-- Setting default SEACASAprepro_aprepro_array_test_DISABLE=ON
-- Setting default SEACASAprepro_aprepro_command_line_include_test_DISABLE=ON
-- Setting default SEACASAprepro_aprepro_command_line_vars_test_DISABLE=ON
-- Setting default SEACASAprepro_aprepro_test_dump_reread_DISABLE=ON
-- Setting default SEACASAprepro_aprepro_unit_test_DISABLE=ON
-- Setting default SEACASAprepro_lib_aprepro_lib_array_test_DISABLE=ON
-- Setting default SEACASAprepro_lib_aprepro_lib_unit_test_DISABLE=ON
-- Setting default SEACASExodus_exodus_unit_tests_nc5_env_DISABLE=ON
-- Setting default SEACASExodus_exodus_unit_tests_DISABLE=ON

That looks like the right set of disables.

Checklist

  • My commit messages mention the appropriate GitHub issue numbers.

Merge request reports