Skip to content

Draft brainstorming for unified yaml configuration - #204

Draft
ceblanton wants to merge 1 commit into
mainfrom
unified-yaml-config
Draft

Draft brainstorming for unified yaml configuration#204
ceblanton wants to merge 1 commit into
mainfrom
unified-yaml-config

Conversation

@ceblanton

Copy link
Copy Markdown
Contributor

Describe your changes

  • Get rid of rose-suite.conf
  • Imagine using all configuration from the resolved yaml

Issue ticket number and link (if applicable)

Checklist before requesting a review

  • I ran my code
  • I tried to make my code readable
  • I tried to comment my code
  • I wrote a new test, if applicable
  • I wrote new instructions/documentation, if applicable
  • I ran pytest and inspected it's output
  • I ran pylint and attempted to implement some of it's feedback
  • No print statements; all user-facing info uses logging module

Manual Pipeline Run Details

Was the manual pipeline (test_cloud_runner) triggered for this PR?

  • Yes
  • No

Result of manual pipeline run:

(Paste relevant logs, output, or a link to the workflow run here)

How to trigger the manual pipeline:

The test_cloud_runner pipeline is not automatically associated as a required check with the PR; it must be triggered to test changes in a full post-processing run.

To trigger the manual pipeline:

  1. Follow the link to the test_cloud_runner actions tab here

    • you should see "This workflow has a workflow_dispatch event trigger"
  2. Click the dropdown "Run workflow":

    a. If trying to merge from a branch on fre-workflows: choose branch from the first drop down, leave the next 2 inputs blank, and choose the fre-cli branch to test

    b. If trying to merge from a fre-workflows fork: can skip first branch selection, input the fork name (ex: [user]/fre-workflows), input the fork's branch name, and choose the fre-cli branch to test

  3. Click "Run workflow"

Note: you may need to reload the page to see your running workflow.

@singhd789

singhd789 commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

@ceblanton With the cylc pre-configure plugin work, this kind of development can potentially be done or at least implemented to test soo 🤞 Would you want to keep this around and update it when the time comes or should we create a new PR? I think either could work. Just curious! Either way, I moved this to 2026.02

@ceblanton ceblanton left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This is wonderful Dana and exactly what I was hoping for in terms of planning. It brings out the potential known problems too. Like the directories:

config['directories']['history']

Are those the directories only for a specific platform?

@singhd789

Copy link
Copy Markdown
Contributor

This is wonderful Dana and exactly what I was hoping for in terms of planning. It brings out the potential known problems too. Like the directories:

config['directories']['history']

Are those the directories only for a specific platform?

They should be, for sure. That pertains to re-evaluating the settings and platforms yaml though, which is sliiiiightly separate from this capability of just being able to reference the keys in the cylc workflow, imo. (Separate but related)

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants