Skip to content

Reorganize Ubuntu CI actions - #6077

Merged
ye-luo merged 3 commits into
QMCPACK:developfrom
prckent:actions_split
Jul 31, 2026
Merged

Reorganize Ubuntu CI actions#6077
ye-luo merged 3 commits into
QMCPACK:developfrom
prckent:actions_split

Conversation

@prckent

@prckent prckent commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Proposed changes

This PR restructures the GitHub actions CI but does not change any of the jobs that are run.

Insteaed of grouping many of the runs under "linux", they are now grouped by OS (ubuntu22, ubuntu24). This was already done for macos and centos10.

The ubuntu24 and centos10 specs are also simiplified -- there should be no need to specify the same container for every job.

Provided this works, a subsequent PR will add some ubuntu26 configurations with appropriate updates to run_step.sh also included.

In order to be merged, we will have to update the branch protections due to the new names. Will do so after addressing any review suggestions and feedback.

What type(s) of changes does this code introduce?

  • Refactoring (no functional changes, no api changes)
  • Testing changes (e.g. new unit/integration/performance tests)

Does this introduce a breaking change?

  • No

What systems has this change been tested on?

None.

Checklist

    • I have read the pull request guidance and develop docs
    • This PR is up to date with the current state of 'develop'
    • Code added or changed in the PR has been clang-formatted
    • This PR adds tests to cover any new code, or to catch a bug that is being fixed
    • Documentation has been added (if appropriate)

@prckent prckent changed the title Reorganize CI actions Reorganize Ubuntu CI actions Jul 31, 2026

@brockdyer03 brockdyer03 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@ye-luo

ye-luo commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Test this please

@ye-luo
ye-luo enabled auto-merge July 31, 2026 16:19
@prckent

prckent commented Jul 31, 2026

Copy link
Copy Markdown
Contributor Author

I will update the branch protections so this can be merged ahead of #6058 and the results verified today.

EDIT: Done. All 14 "linux" rules now point to their ubuntu* equivalents.

@ye-luo
ye-luo merged commit e7237de into QMCPACK:develop Jul 31, 2026
47 checks passed
@prckent
prckent deleted the actions_split branch July 31, 2026 18:24
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