Skip to content

ci: remove the self-hosted Docker parity jobs from stage_parity - #248

Merged
mansurjisan merged 1 commit into
developfrom
chore/remove-selfhosted-parity
Oct 19, 2026
Merged

mansurjisan merged 1 commit into
developfrom
chore/remove-selfhosted-parity

Conversation

@mansurjisan

Copy link
Copy Markdown
Collaborator

Description

Removes the four Docker parity (self-hosted) jobs (prep, post, nowcast, forecast) from .github/workflows/stage_parity.yml, before nos-workflow moves to the oceanmodeling org. This was meant to be part of #247 but was pushed after #247 had been merged.

  • Why they go: they needed a self-hosted runner labelled secofs-docker and the repository variable ENABLE_DOCKER_PARITY=1. No runner is registered, so the jobs were always skipped.
  • What stays: the four hosted jobs in the same workflow (manifest schema and diff helpers) are unchanged, and nothing depended on the removed jobs.
  • Header: the comment now describes only the hosted jobs.

hercules-ci.yml is kept.

Type of change

  • Maintenance (refactor, cleanup, CI, comments)

Change characteristics

  • Changes outputs (values, COMOUT files or names, archives)? NO
  • Breaking change to existing functionality? NO (the skipped self-hosted parity checks no longer appear on PRs)
  • Requires a nos-utils submodule bump (nos-workflow) or a paired nos-workflow change (nos-utils)? NO
  • Requires new or regenerated fix files on the machine? NO
  • Requires a re-prep or a new hotstart before the next cycle? NO
  • Machines affected: none

How has this been tested?

  • CI-only change; no code changes. stage_parity.yml still parses as YAML; its jobs are the four hosted *-manifest-and-helpers jobs.

Checklist

  • Dependent PRs merged and published (nos-utils before any submodule bump): ci: remove the r2o-check workflow before the oceanmodeling move #247 merged
  • pytest suite passes (no code changes)
  • Existing operational behavior preserved, or the change is called out above
  • No edits to a shared FIXofs while it is cycling
  • Self-review done; yaml comments and docstrings updated where behavior changed
  • No markdown files added to the repo

…(no runner registered); hosted jobs unchanged
@mansurjisan
mansurjisan merged commit 4d597d0 into develop Oct 19, 2026
4 checks passed
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.

1 participant