Skip to content

Update unified test script to run e2e tests#284

Merged
mark-a-potts merged 2 commits into
ufs-community:release/public-v2from
jessemcfarland:feature/cicd_we2e
May 31, 2022
Merged

Update unified test script to run e2e tests#284
mark-a-potts merged 2 commits into
ufs-community:release/public-v2from
jessemcfarland:feature/cicd_we2e

Conversation

@jessemcfarland
Copy link
Copy Markdown
Collaborator

DESCRIPTION OF CHANGES:

Execute the end-to-end test workflow during the test stage and archive
the test results.

TESTS CONDUCTED:

Add the run_ci label to this PR to execute the end-to-end test workflow on all platforms. Jenkins will report back a passed or failed status based on the results of all platform/compiler combinations. Individual results can be retrieved from the test_results-<platform>-<compiler>.txt artifact file on Jenkins.

Jesse McFarland and others added 2 commits May 25, 2022 13:14
The slash in branch names such as release/.* and feature/.* is
interpreted as a directory separator. This change replaces the "/"
character with a "_".
Execute the end-to-end test workflow during the test stage and archive
the test results.

Co-authored-by: Bruce Kropp <bruce.kropp@raytheon.com>
Co-authored-by: Jesse McFarland <jesse@mcfarland.sh>
@mark-a-potts mark-a-potts merged commit ede42b3 into ufs-community:release/public-v2 May 31, 2022
@jessemcfarland jessemcfarland deleted the feature/cicd_we2e branch May 31, 2022 16:26
Comment thread .cicd/scripts/srw_test.sh
workspace="$(cd -- "${script_dir}/../.." && pwd)"
fi

we2e_experiment_base_dir="${workspace}/experiments"
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@jessemcfarland The script get_expts_status.sh in ufs-srweather-app/regional_workflow/tests/WE2E does things similar to this script (but not exactly the same). You can get help for that by using get_expts_status.sh --help. It would be good to combine the two so the automated tests and any manual tests the user runs use the same set of scripts.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

@gsketefian thanks! I'll check it out and try to incorporate it here.

Copy link
Copy Markdown
Collaborator

@danielabdi-noaa danielabdi-noaa May 31, 2022

Choose a reason for hiding this comment

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

There is also setup_WE2E_tests.sh under regional_workflow/tests/WE2E that is used to run 10 regression tests in machine_suites on hera/jet using ci-hera/jet-* labels. I think modifying that for other platforms should avoid duplicate work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

run_ci Launches CI/CD pipeline via jenkins

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants