Skip to content

Include global-workflow ctests in automated weekly-stable-pr#2110

Merged
RussTreadon-NOAA merged 4 commits intodevelopfrom
copilot/include-global-workflow-ctests
Apr 7, 2026
Merged

Include global-workflow ctests in automated weekly-stable-pr#2110
RussTreadon-NOAA merged 4 commits intodevelopfrom
copilot/include-global-workflow-ctests

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 7, 2026

Description

weekly-stable-pr.yaml was creating PRs with a plain-text body, so ci/driver.sh found no [x] checkboxes under "Automated CI tests to run in Global Workflow" and excluded all GW ctests from execution.

Changes to .github/workflows/weekly-stable-pr.yaml:

  • Added a "Generate PR body" step that writes /tmp/pr_body.md with all 9 CI tests pre-checked ([x]), mirroring pull_request_template.md and ci/ci_tests.sh
  • Create PR step: replaced --body "..." with --body-file /tmp/pr_body.md
  • Update existing PR step: added --body-file /tmp/pr_body.md so already-open PRs also get the full checked checklist

Companion PRs

Issues

Resolves #2109

Automated CI tests to run in Global Workflow

  • atm_jjob
  • C96C48_ufs_hybatmDA
  • C96C48_hybatmsnowDA
  • C96_gcafs_cycled
  • C48mx500_3DVarAOWCDA
  • C48mx500_hybAOWCDA
  • C96C48_ufsgsi_hybatmDA
  • C48_ufsenkf_atmDA
  • C96C48_hybatmDA

Agent-Logs-Url: https://github.com/NOAA-EMC/GDASApp/sessions/2a4e9735-c6c5-497c-848c-691862117f57

Co-authored-by: RussTreadon-NOAA <26926959+RussTreadon-NOAA@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix global workflow ctests not included in pull request body Include global-workflow ctests in automated weekly-stable-pr Apr 7, 2026
Copilot AI requested a review from RussTreadon-NOAA April 7, 2026 11:18
Copy link
Copy Markdown
Contributor

@RussTreadon-NOAA RussTreadon-NOAA left a comment

Choose a reason for hiding this comment

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

Looks good. Approve.

@RussTreadon-NOAA RussTreadon-NOAA marked this pull request as ready for review April 7, 2026 11:24
Copy link
Copy Markdown
Contributor

@RussTreadon-NOAA RussTreadon-NOAA left a comment

Choose a reason for hiding this comment

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

Approve.

@RussTreadon-NOAA
Copy link
Copy Markdown
Contributor

@DavidNew-NOAA , @guillaumevernieres , @CoryMartin-NOAA : Would one of you approve this PR so I may merge it into develop? PR #2108 confirms that the github action works. This PR adds g-w tests to the action. I'd like to merge this PR into develop today to ensure that the agent works as intended tomorrow.

@RussTreadon-NOAA
Copy link
Copy Markdown
Contributor

Thank you @CoryMartin-NOAA . Merging now.

@RussTreadon-NOAA RussTreadon-NOAA merged commit a260d3e into develop Apr 7, 2026
6 checks passed
@RussTreadon-NOAA RussTreadon-NOAA deleted the copilot/include-global-workflow-ctests branch April 7, 2026 12:41
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.

Include global-workflow ctests in automated weekly-stable-pr

3 participants