Skip to content

Keep standalone CIME case output in the Omega build directory - #582

Merged
xylar merged 1 commit into
E3SM-Project:developfrom
xylar:omega/unique-cime-case-name
Sep 29, 2026
Merged

xylar merged 1 commit into
E3SM-Project:developfrom
xylar:omega/unique-cime-case-name

Conversation

@xylar

@xylar xylar commented Sep 29, 2026 •

Copy link
Copy Markdown

Standalone builds now keep the throwaway CIME case's bld and run directories inside their own build directory by passing --output-root to create_newcase. Before this, every standalone build by a user on a machine shared $CIME_OUTPUT_ROOT/e3smcase/{bld,run}, so builds configured at the same time could delete each other's directories.

Fixes #581

Checklist

  • Linting

  • Building

    • CMake build does not produce any new warnings from changes in this PR
  • Testing

    chrysalis, intel, openmpi

    • CTests Pass
  • Provide relevant details in a comment to the PR titled Testing


Posted by Claude Code on @xylar's behalf. The testing, analysis and wording above are AI-authored; please check them accordingly.

🤖 Generated with Claude Code

Pass --output-root to create_newcase so the throwaway e3smcase's EXEROOT
and RUNDIR live under the build directory instead of the shared
$CIME_OUTPUT_ROOT/e3smcase, where concurrent standalone builds deleted
each other's directories.

Fixes E3SM-Project#581

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@xylar xylar added the bug Something isn't working label Sep 29, 2026
@xylar
xylar requested a review from andrewdnolan September 29, 2026 17:45
@xylar

xylar commented Sep 29, 2026

Copy link
Copy Markdown
Author

Testing

CTests pass (50/50) on Chrysalis with Intel and OpenMPI from a clean build of 415422d in:

/gpfs/fs1/home/ac.xylar/e3sm_work/polaris/main/build_omega/build_chrysalis_intel

Two standalone configures started at the same time on Chrysalis both succeeded, each with the case's bld and run inside its own build directory.


Posted by Claude Code on @xylar's behalf. The testing, analysis and wording above are AI-authored; please check them accordingly.

@andrewdnolan andrewdnolan left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems like a very sensible change to me. I haven't had a chance to test myself, but approving based on code inspection and @xylar's testing.

@xylar
xylar requested a review from grnydawn September 29, 2026 18:07
@xylar

xylar commented Sep 29, 2026

Copy link
Copy Markdown
Author

@grnydawn, could you also please give this a quick look?

@grnydawn grnydawn left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This resolves a long-standing issue. I briefly verified that a separate build directory is created for each Omega build. Approved.

@xylar xylar self-assigned this Sep 29, 2026
@xylar

xylar commented Sep 29, 2026

Copy link
Copy Markdown
Author

Thanks @grnydawn!

@xylar xylar added the CMake CMake-related issues label Sep 29, 2026
@xylar
xylar merged commit a244e7e into E3SM-Project:develop Sep 29, 2026
1 check passed
@xylar
xylar deleted the omega/unique-cime-case-name branch September 29, 2026 19:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working CMake CMake-related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Concurrent standalone builds delete each other's CIME case directories

3 participants