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.

Sign up for GitHub

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

Jump to bottom

Some cime_developer tests fail during phase SHAREDLIB_BUILD #3473

Closed
dqwu opened this issue Apr 3, 2020 · 1 comment · Fixed by #3475
Closed

Some cime_developer tests fail during phase SHAREDLIB_BUILD #3473

dqwu opened this issue Apr 3, 2020 · 1 comment · Fixed by #3475
Assignees
@jgfouca
Labels
ty: help wanted

Comments

@dqwu
Copy link
Contributor

dqwu commented Apr 3, 2020

This issue is reproducible on both anlworkstation and summit.
It is caused by PR #3449: Improve case.setup clean (happens for --clean or --reset)

Steps to reproduce:

git clone https://github.com/ESMCI/cime.git
cd cime/scripts
./create_test DAE.ww3a.ADWAV --no-run

Errors are:

        ...
        WARNING: Test case setup failed. Case2 has been removed, but the main case may be in an inconsistent state. If you want to rerun this test, you should create a new test rather than trying to rerun this one.
        ERROR: Cannot move past core phase 'SETUP', it didn't pass:

CaseStatus shows:

case.setup error 
ERROR: FakeCase does not support getting value of 'SMP_PRESENT'

Another way to reproduce this issue:

git clone https://github.com/ESMCI/cime.git
cd cime/scripts
./create_test DAE.ww3a.ADWAV --no-build
cd to Case dir
./case.setup --reset

Output messages:

Successfully cleaned .case.test
Successfully cleaned env_mach_specific.xml
Successfully cleaned Macros.make
Successfully cleaned Macros.cmake
ERROR: FakeCase does not support getting value of 'SMP_PRESENT'
@dqwu
Copy link
Contributor Author

dqwu commented Apr 3, 2020

Not reproducible on Cori.

@jgfouca jgfouca mentioned this issue Apr 4, 2020
Merged
jgfouca pushed a commit that referenced this issue Apr 7, 2020
@ndkeen
Merge branch E3SM-Project/ndk/machinefiles/nersc-project-e3sm (PR #3473)
d6b691f 
For NERSC, change the PROJECT (repo name) from "acme" to "e3sm".

[bfb]
jgfouca added a commit that referenced this issue Apr 9, 2020
@jgfouca
Merge pull request #3475 from ESMCI/jgfouca/fix_case_reset
79a8e11 
case.setup: Fix case reset on some machines

Removing env_mach_specific.xml triggered the FakeCase path in
BuildConf.configure. This path doesn't work if the machine is using
advanced attribute selectors (e.g. SMP_PRESENT) in their environment
block in config_machines.xml.

case_setup only needs BuildConf.configure to recreate the env_mach_specific.xml
file and the Macro files.

Test suite: scripts_regression_tests
Test baseline:
Test namelist changes:
Test status: bit for bit

Fixes #3473

User interface changes?:

Update gh-pages html (Y/N)?:

Code review: @jedwards4b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jgfouca jgfouca

Labels
ty: help wanted
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

case.setup: Fix case reset on some machines ESMCI/cime
2 participants
@jgfouca @dqwu