Skip to content

Update CI self hosted runner setting - #290

Merged
Chris Thomas (ctgh) merged 2 commits into
developfrom
bugfix/ci-runs-on
Aug 14, 2026
Merged

Update CI self hosted runner setting#290
Chris Thomas (ctgh) merged 2 commits into
developfrom
bugfix/ci-runs-on

Conversation

@matthewrmshin

Copy link
Copy Markdown
Collaborator

More fine grain group and labels.

See also:

  • MetOffice/ops-um-jedi#283

More fine grain group and labels.

@ctgh Chris Thomas (ctgh) left a comment

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.

I don't understand the yaml syntax enough to comment on the change, so can you please explain roughly what it is doing?

@matthewrmshin

Copy link
Copy Markdown
Collaborator Author

The syntax is recommended by a member of the Software Asset Management team to allow our action to pick up our runner using its runner group name as well as a label, so the path to chosing a runner should be less ambiguous.

@ctgh Chris Thomas (ctgh) left a comment

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.

Thanks!

@ctgh
Chris Thomas (ctgh) merged commit c67eb68 into develop Aug 14, 2026
6 checks passed
@ctgh
Chris Thomas (ctgh) deleted the bugfix/ci-runs-on branch August 14, 2026 08:18
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