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

SEACAS tests 'Not Run' on ATDM mutrino builds #3496

Closed
fryeguy52 opened this issue Sep 25, 2018 · 8 comments
Closed

SEACAS tests 'Not Run' on ATDM mutrino builds #3496

fryeguy52 opened this issue Sep 25, 2018 · 8 comments
Assignees
Labels
ATDM Sev: Nonblocker Problems with Trilinos that should not block ATDM APPs from getting updates client: ATDM Any issue primarily impacting the ATDM project Disabled Tests Issue has been partially addressed by disabling *all* of the failing tests related to the issue PA: Data Services Issues that fall under the Trilinos Data Services Product Area pkg: seacas type: bug The primary issue is a bug in Trilinos code or tests

Comments

@fryeguy52
Copy link
Contributor

fryeguy52 commented Sep 25, 2018

CC: @trilinos/seacas , @kddevin (Trilinos Data Services Product Lead), @bartlettroscoe

Next Action Status

Tests disabled in PR in #3530 merged on 9/29/2018 and these tests went missing in this build on 9/30/2018 as shown here. Next: Fix the tests?

Description

several seacas tests are showing up as "not run" in the ATDM builds on mutrino. As shown here the tests are:

  • SEACASAprepro_aprepro_test_exodus
  • SEACASIoss_exodus32_to_exodus32
  • SEACASIoss_exodus32_to_exodus32_pnetcdf
  • SEACASIoss_exodus32_to_exodus64

are not run in the build:

  • Trilinos-atdm-mutrino-intel-opt-openmp-HSW

the test output on CDash for all 4 of these is:

Unable to find required file: CMND_PATH-NOTFOUND

Steps to Reproduce

One should be able to reproduce this failure on the machine mutrino as described in:

$ cd <some_build_dir>/

$ source $TRILINOS_DIR/cmake/std/atdm/load-env.sh intel-opt-openmp-HSW

$ cmake \
  -DTrilinos_CONFIGURE_OPTIONS_FILE:STRING=cmake/std/atdm/ATDMDevEnv.cmake \
  -DTrilinos_ENABLE_TESTS=ON -DTrilinos_ENABLE_SEACAS=ON \
  $TRILINOS_DIR

$ make -j16

$ salloc -N 1 -p standard -J $JOB_NAME ctest -j16
@fryeguy52 fryeguy52 added type: bug The primary issue is a bug in Trilinos code or tests pkg: seacas client: ATDM Any issue primarily impacting the ATDM project labels Sep 25, 2018
@bartlettroscoe
Copy link
Member

@gsjaardema, any idea what is happening here?

@gsjaardema
Copy link
Contributor

@bartlettroscoe I don’t have any idea. I will see if I can reproduce.

@gsjaardema gsjaardema self-assigned this Sep 27, 2018
@gsjaardema
Copy link
Contributor

When I run this manually on mutrino, I get the following:

4/52 Testing: SEACASIoss_exodus32_to_exodus32
4/52 Test: SEACASIoss_exodus32_to_exodus32
Command: "/projects/netpub/atdm/cmake-3.11.4/bin/cmake" "-DTEST_CONFIG=" "-P" "/lscratch1/gdsjaar/trilinos/mutrino/packages/seacas/libraries/ioss/src/main/SEACASIoss_exodus32_to_exodus32.cmake"
Directory: /lscratch1/gdsjaar/trilinos/mutrino/packages/seacas/libraries/ioss/src/main
"SEACASIoss_exodus32_to_exodus32" start time: 
Output:
----------------------------------------------------------
<end of output>
Test time =   0.00 sec
----------------------------------------------------------
Test Pass Reason:
Required regular expression not found.Regex=[OVERALL FINAL RESULT: TEST PASSED .SEACASIoss_exodus32_to_exodus32.
]
"SEACASIoss_exodus32_to_exodus32" end time: Sep 27 14:49 MDT
"SEACASIoss_exodus32_to_exodus32" time elapsed: 00:00:00
----------------------------------------------------------

Unable to find required file: CMND_PATH-NOTFOUND

It looks like the test runs, but then at the end there is the CMND_PATH-NOTFOUND

If I manually run the Command listed above, it works correctly and the CMND_PATH-NOTFOUND is not printed. I'm not sure where the NOTFOUND is coming from; maybe we need to just disable the tests?

bartlettroscoe added a commit to bartlettroscoe/Trilinos that referenced this issue Sep 28, 2018
…linos#3496)

The commands run fine when run manually but for some reason ctest refuses to
run these and reports a:

  Unable to find required file: CMND_PATH-NOTFOUND

error.

It was requested to disable these tests.
@bartlettroscoe
Copy link
Member

maybe we need to just disable the tests?

@gsjaardema, sure, I can create a PR to disable these tests.

Note that these tests are also showing as "not run" in the KNL build as well as shown here. So I will disable these tests in that build too.

bartlettroscoe added a commit that referenced this issue Sep 29, 2018
…) (#3530)

The commands run fine when run manually but for some reason ctest refuses to
run these and reports a:

  Unable to find required file: CMND_PATH-NOTFOUND

error.

It was requested to disable these tests.
searhein pushed a commit to searhein/Trilinos that referenced this issue Oct 3, 2018
…GDSW-VFEM-Coarse-Spaces

* 'develop' of https://github.com/trilinos/Trilinos: (186 commits)
  Tpetra: initial commit of user's guide (trilinos#3553)
  MueLu: fix master xml list problem
  PyTrilinos: Remove includes of NOX_Version.h
  Xpetra: update threshold type
  Re-enable Belos PCG tests (trilinos#2920 & trilinos#3007) on white/ride
  Phalanx: add debug output support during DAG traversal
  Switch to new devpack/20180521/openmpi/2.1.2/gcc/7.2.0/cuda/9.2.88 env (trilinos#3290)
  MueLu: Fix gold files
  MueLu gold files: Fix bad if statement in rebase scripts
  MueLu: Xpetra: add threshold for zero diag fix
  kokkos-kernels: Patch to fix trilinos#3493
  Tpetra::MultiVector: Improve unit tests (help w/ trilinos#3493)
  running update_params.sh to fix up latex
  Disable four exodus SEACAS tests failing with Not Run on mutrino (trilinos#3496) (trilinos#3530)
  PyTrilinos: Fix case-sensitive include of Ifpack_ConfigDef.h
  Stratimikos Belos adapter: On Belos error, throw Thyra::CatastrophicSolveFailure
  Stratimikos Belos adapter: When "Timer Label" is set, use it in output
  Amesos: SuperLU_DIST version fixes
  put in a new phase 3 aggregation option that avoids singletons at all costs (even groupng vertices with non-neighbors into aggregates)
  Framework: Parameterized CTest build update to enable extra packages to be set by Jenkins parameters. (trilinos#3520)
  ...
@bartlettroscoe
Copy link
Member

Tests disabled in PR in #3530 merged on 9/29/2018 and these tests went missing in this build on 9/30/2018 as shown here.

Adding "Disabled Tests" label and leaving open as per policy shown here.

@bartlettroscoe bartlettroscoe added the Disabled Tests Issue has been partially addressed by disabling *all* of the failing tests related to the issue label Oct 15, 2018
@bartlettroscoe bartlettroscoe added PA: Data Services Issues that fall under the Trilinos Data Services Product Area ATDM Sev: Nonblocker Problems with Trilinos that should not block ATDM APPs from getting updates labels Nov 30, 2018
tjfulle pushed a commit to tjfulle/Trilinos that referenced this issue Dec 6, 2018
…linos#3496) (trilinos#3530)

The commands run fine when run manually but for some reason ctest refuses to
run these and reports a:

  Unable to find required file: CMND_PATH-NOTFOUND

error.

It was requested to disable these tests.
@prwolfe
Copy link
Contributor

prwolfe commented Feb 17, 2021

This issue cropped up on our UVM OFF build over the past several days blocking develop->master promotion. I dug a bit and found a few things.

The tests in question are all structured in the same basic way

if (${CMAKE_PROJECT_NAME}_ENABLE_SEACASExodiff)
   set(SEACAS_EXODIFF_BINARY "../../../../applications/exodiff/exodiff")
endif()

if (SEACAS_EXODIFF_BINARY)
TRIBITS_ADD_ADVANCED_TEST(exodus32_to_exodus32
   TEST_0 EXEC io_shell ARGS ${DECOMP_ARG} ${JOIN_ARG} ${CMAKE_CURRENT_SOURCE_DIR}/test/8-block.g 8-block32.g
     NOEXEPREFIX NOEXESUFFIX
     NUM_MPI_PROCS ${NPROCS}
   TEST_1 CMND ${SEACAS_EXODIFF_BINARY} ARGS -pedantic ${CMAKE_CURRENT_SOURCE_DIR}/test/8-block.g 8-block32.g
  COMM mpi serial
  XHOSTTYPE Windows
  )

The TEST_1 uses CMND even though the exodiff it points at is in the build tree. The macro does not search for EXEC as it assumes it will be built and does for CMND (using FIND_PROGRAM). At configure time that executable will not be there for new builds but may be for re-builds.

I don't understand why we get this on only partial sets, but I expect we need to change both to EXEC and find a way to get that to do what is needed (main difference is mpiexec I think) or just live with the fact that TEST_1 WILL use mpiexec.

@prwolfe
Copy link
Contributor

prwolfe commented Feb 17, 2021

Tagging framework members @jwillenbring and @ZUUL42

@gsjaardema
Copy link
Contributor

I think this has been fixed. If not, please reopen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ATDM Sev: Nonblocker Problems with Trilinos that should not block ATDM APPs from getting updates client: ATDM Any issue primarily impacting the ATDM project Disabled Tests Issue has been partially addressed by disabling *all* of the failing tests related to the issue PA: Data Services Issues that fall under the Trilinos Data Services Product Area pkg: seacas type: bug The primary issue is a bug in Trilinos code or tests
Projects
None yet
Development

No branches or pull requests

4 participants