-
Notifications
You must be signed in to change notification settings - Fork 131
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
Store RTW files in share directory in installed workflow #3600
Store RTW files in share directory in installed workflow #3600
Conversation
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.
Nice work @mo-gill, thank you! 🎉
Once you have responded to my review comments, would you ask Alistair to test the changes to ensure it behaves the way he expects, please? (It looks ok to me, but it would be good to get confirmation from Alistair also) 👍
esmvaltool/utils/recipe_test_workflow/recipe_test_workflow/flow.cylc
Outdated
Show resolved
Hide resolved
esmvaltool/utils/recipe_test_workflow/recipe_test_workflow/flow.cylc
Outdated
Show resolved
Hide resolved
...utils/recipe_test_workflow/recipe_test_workflow/app/install_env_file/bin/install_env_file.sh
Outdated
Show resolved
Hide resolved
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.
Thanks @mo-gill! 🥳
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.
Great work @mo-gill !
This does exactly what I was expecting. Example work task for me:
$ export CYLC_VERSION=8
$ cylc vip -O metoffice
# Then let it run / fail on first cycle. I might then change a file in workflow source, although not required for this demo
$ cylc validate-reinstall -O metoffice recipe_test_workflow
<snip>
send rose-suite.conf
NOTE: Files created by Rose file installation will show as deleted.
They will be re-created during the reinstall process.
TIP: You can "exclude" files/dirs to prevent Cylc from installing or overwriting
them by adding them to the .cylcignore file. See cylc reinstall --help.
Reinstall would make the above changes.
Continue [y/n]: y
REINSTALLED recipe_test_workflow/run12 from /net/data/users/hadtq/esmvaltool/operate_rtw/ESMValTool/esmvaltool/utils/recipe_test_workflow/recipe_test_workflow
Successfully reinstalled.
<snip>
$ cylc gui
# Go to GUI and trigger individual tasks
Without this branch, the validate-reinstall step deletes essential files, and therefore the triggered tasks fail in the final step. With this branch I was able to successfully trigger tasks and they gave the same results as on the first (=clean) run.
Description
This PR should successfully store RTW files in the share directory in installed workflows
share
directory in installed workflow #3434Checklist
It is the responsibility of the author to make sure the pull request is ready to review. The icons indicate whether the item will be subject to the 🛠 Technical or 🧪 Scientific review.
To help with the number of pull requests: