-
Notifications
You must be signed in to change notification settings - Fork 15
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
Automate zppy tests #520
base: main
Are you sure you want to change the base?
Automate zppy tests #520
Conversation
tests/scripts/test_dev.bash
Outdated
exit 1 | ||
fi | ||
|
||
exit 7 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Exiting here for now, since the subsequent steps are difficult to automate.
@chengzhuzhang This is an initial attempt at automating the steps from https://e3sm-project.github.io/zppy/_build/html/main/dev_guide/release_testing.html -- specifically just the There's a number of issues in fully automating this workflow. See the #TODO in-line comments. |
tests/integration/utils.py
Outdated
@@ -276,4 +278,6 @@ def generate_cfgs(unified_testing=False, dry_run=False): | |||
|
|||
|
|||
if __name__ == "__main__": | |||
generate_cfgs(unified_testing=False) | |||
generate_cfgs( | |||
unified_testing=(sys.argv[1] == "True"), diags_environment_commands=sys.argv[2] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be args 2 and 3
Made some adjustments post-zppy/zppy-interfaces split. Can't run these changes yet because need to get around 1) issues with conda activation in a script and 2) automating the waiting for multiple jobs to finish. |
Automate zppy tests. Resolves #516.