-
Notifications
You must be signed in to change notification settings - Fork 2
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
Add payu dev environment #26
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.
Looks cool.
Would it make sense to use a containerised environment for this? If it slowed this PR down too much then it might make sense to make containerisation a subsequent PR. |
I've been wanting to learn about using singularity containers so I'll be keen to try that |
Add checks if there has been any commits to payu's main branch since last successful workflow run - it was done this way rather than a CD workflow on payu-org/payu to prevent having to store repository secrets of payu-condaenv on payu-org/payu
…nks and remove old environments
Updates: Add a job to check if there's been any new commits to the master branch payu-org/payu repository, since the last successful payu dev deployment workflow run. This check will run periodically. New environment will get deployed to The versions of dev environment Updated modulefiles deployment to generate modulefiles for release and prerelease and sync to respective locations (the common module file and env.sh uses different paths). TODO:
|
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.
Looks good. Approving as the comments aren't necessary changes.
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.
Looks good to me :)
Deploy a payu environment, with the latest changes on payu's main branch to pre-release location on gadi (e.g.
/g/data/vk83/prerelease
).So environment location would be
/g/data/vk83/prerelease/apps/payu/dev
, and modulefile location would be/g/data/vk83/prerelease/modules/payu/dev
.The modulefile would be a symlink to the common modulefile in
/g/data/vk83/modules/payu/.common
, so have extended the common modulefile to check for fullpath of the modulefile being evaluated - to see if its in the prerelease folder.Closes #22