Site configurations for NCI-Gadi, Australia.#2013
Site configurations for NCI-Gadi, Australia.#2013Vinod Kumar (vinodk-bom) wants to merge 7 commits intoMetOffice:mainfrom
Conversation
…e the nci-gadi option.
James Frost (jfrost-mo)
left a comment
James Frost (jfrost-mo)
left a comment
There was a problem hiding this comment.
Looks like a sensible site configuration. I've left a few minor comments, but the only thing that needs changing is to move the changes to rose-meta.conf into rose-meta.conf.jinja2, as that is the source from which the .conf file is generated.
There was a problem hiding this comment.
For various reasons we do a bit of templating into this file. The actual source file is rose-meta.conf.jinja2, while this file is regenerated from that. Please make this change in the .jinja2 file for them to be persisted.
|
|
||
| [runtime] | ||
| [[root]] | ||
| platform = gadi_background |
There was a problem hiding this comment.
Does gadi_background have sensible default directives, or do they need setting?
Aside from the baking tasks which you have already got separate definitions for, the other tasks should be resource light. (Single CPU, <1 GiB RAM).
There was a problem hiding this comment.
gadi_background is appropriate for very light tasks, it's running in the background of the users cylc server so no directives needed
| [[validate_environment]] | ||
| platform = gadi_background | ||
|
|
||
| [[parbake_recipes]] | ||
| platform = gadi_background | ||
|
|
||
| [[parbake_aggregation_recipes]] | ||
| platform = gadi_background |
There was a problem hiding this comment.
All of these tasks will inherit from root, so you don't need to specify them explicitly.
Co-authored-by: James Frost <james.frost@metoffice.gov.uk>
Co-authored-by: James Frost <james.frost@metoffice.gov.uk>
…uns on login node.
3 participants
Add a site-specific configuration for NCI-Gadi, as well as an optional workflow configuration that runs at NCI for a deterministic LFRic and UM comparison.
Contribution checklist
Aim to have all relevant checks ticked off before merging. See the developer's guide for more detail.
rose-suite.conf.examplehas been updated if new diagnostic added.