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

CIME 6.1.9 6.1.10 (current master) failing test_d_create_clone_new_user #4662

Open
jedwards4b opened this issue Aug 12, 2024 · 3 comments
Open

Comments

@jedwards4b
Copy link
Contributor

(test_sys_create_newcase.TestCreateNewcase) ... FAIL
Tested on derecho and github workflow.

@jedwards4b
Copy link
Contributor Author

======================================================================
FAIL: test_d_create_clone_new_user (test_sys_create_newcase.TestCreateNewcase)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/glade/work/jedwards/sandboxes/cesm3_0_x/cime/CIME/tests/test_sys_create_newcase.py", line 264, in test_d_create_clone_new_user
    self.run_cmd_assert_result(
  File "/glade/work/jedwards/sandboxes/cesm3_0_x/cime/CIME/tests/base.py", line 142, in run_cmd_assert_result
    self.assertEqual(stat, expected_stat, msg=msg)
AssertionError: 1 != 0 : 
    COMMAND: ./xmlchange CIME_OUTPUT_ROOT=/glade/derecho/scratch/this_is_not_a_user/scripts_regression_test.20240813_070745/TestCreateNewcase
    FROM_DIR: /glade/derecho/scratch/jedwards/scripts_regression_test.20240813_070745/TestCreateNewcase/testcreatenewcase
    SHOULD HAVE WORKED, INSTEAD GOT STAT 1
    OUTPUT: 
    ERRPUT: Detected diff in locked file 'env_build.xml'
	'CIME_OUTPUT_ROOT' has changed from '/glade/derecho/scratch/jedwards/scripts_regression_test.20240813_070745/TestCreateNewcase' to '/glade/derecho/scratch/this_is_not_a_user/scripts_regression_test.20240813_070745/TestCreateNewcase'
ERROR: For your changes to take effect, run:
./case.build --clean-all
./case.build

@jedwards4b
Copy link
Contributor Author

Passes in cime6.1.1 fails in 6.1.2. @jgfouca Can you have a look?

commit 783f23a5775ca7b11335d460044388b9408ed663 (HEAD, tag: cime6.1.2, feature/github_interface)
Merge: ca6c0cdc7 2fb1de215
Author: James Foucar <[email protected]>
Date:   Thu Jul 11 11:32:57 2024 -0600

    Merge pull request #4650 from ESMCI/fix-xmlchange
    
    Fixes check_lockedfiles call in xmlchange
    
    Fixes check_lockedfiles only being called once when multiple settings
    are changed.

@jgfouca
Copy link
Contributor

jgfouca commented Aug 13, 2024

@jedwards4b , I believe that was @jasonb5 's PR, I just merged it. I'll let him look at it unless he wants me to do it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants