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

Fix the bisect utility #206

Merged
merged 2 commits into from
Jul 24, 2024
Merged

Fix the bisect utility #206

merged 2 commits into from
Jul 24, 2024

Conversation

xylar
Copy link
Collaborator

@xylar xylar commented Jul 19, 2024

This merge fixes the --copy_exectable flag to polaris setup, which referred to nonexistent config options.

It also fixes the way E3SM submodules are checked out at the beginning of each bisection step. The submodule is cleaned, hard-reset to the HEAD and then submodules are recursively checked out to ensure that every step is starting from a clean state.

Checklist

  • Testing comment in the PR documents testing used to verify the changes

@xylar xylar added bug Something isn't working framework Changes relating to the polaris framework as opposed to individual tests or analysis utility Utility script(s) in the utils directory labels Jul 19, 2024
@xylar xylar self-assigned this Jul 19, 2024
@xylar
Copy link
Collaborator Author

xylar commented Jul 22, 2024

Testing

I was able to use the utility after these fixes to diagnose the issue reported in #205 and E3SM-Project/E3SM#6515

@xylar xylar force-pushed the fix-bisect-util branch from d5431a5 to 477a0f4 Compare July 23, 2024 20:44
@xylar xylar marked this pull request as ready for review July 23, 2024 20:45
@xylar xylar merged commit 51bdb80 into E3SM-Project:main Jul 24, 2024
4 checks passed
@xylar xylar deleted the fix-bisect-util branch July 24, 2024 05:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working framework Changes relating to the polaris framework as opposed to individual tests or analysis utility Utility script(s) in the utils directory
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant